[PATCH] some more CONFIG_ prefixes

[Patrick Georgi]

Hi,

attached patch renames some preprocessor symbols. I have no idea why 
those symbols were left alone before, after this, they're somewhat more 
in line with the rest of the tree.

Signed-off-by: Patrick Georgi

Index: src/southbridge/intel/esb6300/esb6300_lpc.c
===================================================================
--- src/southbridge/intel/esb6300/esb6300_lpc.c	(revision 4441)
+++ src/southbridge/intel/esb6300/esb6300_lpc.c	(working copy)
@@ -18,8 +18,8 @@
 #define MAINBOARD_POWER_OFF 0
 #define MAINBOARD_POWER_ON  1

-#ifndef MAINBOARD_POWER_ON_AFTER_FAIL
-#define MAINBOARD_POWER_ON_AFTER_FAIL MAINBOARD_POWER_ON
+#ifndef CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL
+#define CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL MAINBOARD_POWER_ON
 #endif

#define ALL		(0xff << 24)
@@ -277,7 +277,7 @@
 {
    uint8_t byte;
    uint32_t value;
-	int pwr_on=MAINBOARD_POWER_ON_AFTER_FAIL;
+	int pwr_on=CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL;

/* sata settings */
    pci_write_config32(dev, 0x58, 0x00001181);
Index: src/southbridge/intel/i82801gx/i82801gx_lpc.c
===================================================================
--- src/southbridge/intel/i82801gx/i82801gx_lpc.c	(revision 4441)
+++ src/southbridge/intel/i82801gx/i82801gx_lpc.c	(working copy)
@@ -33,8 +33,8 @@
 #define MAINBOARD_POWER_OFF 0
 #define MAINBOARD_POWER_ON  1

-#ifndef MAINBOARD_POWER_ON_AFTER_FAIL
-#define MAINBOARD_POWER_ON_AFTER_FAIL MAINBOARD_POWER_ON
+#ifndef CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL
+#define CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL MAINBOARD_POWER_ON
 #endif

#define NMI_OFF 0
Index: src/include/x86emu/fpu_regs.h
===================================================================
--- src/include/x86emu/fpu_regs.h	(revision 4441)
+++ src/include/x86emu/fpu_regs.h	(working copy)
@@ -40,7 +40,7 @@
 #define __X86EMU_FPU_REGS_H

-#if defined(CONFIG_DEBUG) && (DEBUG == 0)
+#if defined(CONFIG_DEBUG) && (CONFIG_DEBUG == 0)
 #undef CONFIG_DEBUG
 #endif

[Myles Watson]

> attached patch renames some preprocessor symbols. I have no idea why
> those symbols were left alone before, after this, they're somewhat more
> in line with the rest of the tree.

@@ -18,8 +18,8 @@
 #define MAINBOARD_POWER_OFF 0
 #define MAINBOARD_POWER_ON  1

-#ifndef MAINBOARD_POWER_ON_AFTER_FAIL
-#define MAINBOARD_POWER_ON_AFTER_FAIL MAINBOARD_POWER_ON
+#ifndef CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL
+#define CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL MAINBOARD_POWER_ON
 #endif

Since they're #ifndef, that means they weren't part of the old config
system, or there is a bug.  Those symbols are always defined, and are tested
based on their values.  Is there a uses statement for them somewhere?

Thanks,
Myles

[Stefan Reinauer]

On 20.07.2009 15:57 Uhr, Myles Watson wrote:
> > attached patch renames some preprocessor symbols. I have no idea why
> > those symbols were left alone before, after this, they're somewhat more
> > in line with the rest of the tree.
>
> @@ -18,8 +18,8 @@
>  #define MAINBOARD_POWER_OFF 0
>  #define MAINBOARD_POWER_ON  1
>
> -#ifndef MAINBOARD_POWER_ON_AFTER_FAIL
> -#define MAINBOARD_POWER_ON_AFTER_FAIL MAINBOARD_POWER_ON
> +#ifndef CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL
> +#define CONFIG_MAINBOARD_POWER_ON_AFTER_POWER_FAIL MAINBOARD_POWER_ON
>  #endif
>
> Since they're #ifndef, that means they weren't part of the old config
> system, or there is a bug.  Those symbols are always defined, and are tested
> based on their values.  Is there a uses statement for them somewhere?

The old config tool has an flag to determine whether an option should be
exported always or only if it is used.

Stefan

[Myles Watson]

On Mon, Jul 20, 2009 at 9:45 AM, Stefan Reinauer wrote:
> On 20.07.2009 15:57 Uhr, Myles Watson wrote:
>
> Since they're #ifndef, that means they weren't part of the old config
> system, or there is a bug.  Those symbols are always defined, and are tested
> based on their values.  Is there a uses statement for them somewhere?
>
> The old config tool has an flag to determine whether an option should be
> exported always or only if it is used.

You're right.  I'd forgotten about that.

Thanks,
Myles

[Stefan Reinauer]

On 20.07.2009 13:08 Uhr, Patrick Georgi wrote:
> Hi,
>
> attached patch renames some preprocessor symbols. I have no idea why
> those symbols were left alone before, after this, they're somewhat
> more in line with the rest of the tree.
>
> Signed-off-by: Patrick Georgi

Acked-by: Stefan Reinauer