Re: [U-Boot] [PATCH 2/3] mkconfig: change CONFIG_MK_ prefix into plain CONFIG_

2010-10-12 Thread Wolfgang Denk
Dear Mike Frysinger,

In message <201010111517.18063.vap...@gentoo.org> you wrote:
>
> On Monday, October 11, 2010 05:45:09 Wolfgang Denk wrote:
> >  board/bf527-ezkit/video.c |   10 +-
> >  include/configs/bf527-ezkit.h |2 +-
>
> these look fine ... were you going to update boards.cfg too ?  this was the
> only reason the bf527-ezkit-v2 target exists in the top level Makefile ...

I still need to find out how to correctly process the make config
options (still waiting for feedback, none received so far :-( ).

Then I will convert all PPC boards.  I hope others pick up for the
other architectures.  I'm growing tired of producing series of >1000
lines patches ;-)

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
The price one pays for pursuing any profession,  or  calling,  is  an
intimate knowledge of its ugly side.  - James Baldwin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/3] mkconfig: change CONFIG_MK_ prefix into plain CONFIG_

2010-10-11 Thread Mike Frysinger
On Monday, October 11, 2010 05:45:09 Wolfgang Denk wrote:
>  board/bf527-ezkit/video.c |   10 +-
>  include/configs/bf527-ezkit.h |2 +-

these look fine ... were you going to update boards.cfg too ?  this was the 
only reason the bf527-ezkit-v2 target exists in the top level Makefile ...
-mike


signature.asc
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 2/3] mkconfig: change CONFIG_MK_ prefix into plain CONFIG_

2010-10-11 Thread Wolfgang Denk
When planning for more generalization and Makefile cleanup it became
obvious that the introduction of a separate CONFIG_MK_ name space for
config options that were set through scripting in the Makefile was
not a good idea.

Originally the idea was to provide a script-free approach to supply
configuration options - there was no real need for a separate name
space. But when we now convert the existing Makefile entries to make
use of this approach, it would mean that we have to touch a large
number of board config files and add #ifdef / #define sequences to
"convert" from the CONFIG_MK_ to the CONFIG_ name space.

It seems much cleaner to get rid of this somewhat arbitrary _MK
string now for the few boards that actually use it.

Signed-off-by: Wolfgang Denk 
---
 board/bf527-ezkit/video.c |   10 +-
 board/freescale/mpc8315erdb/config.mk |2 +-
 board/freescale/mpc8536ds/config.mk   |6 +++---
 board/freescale/mpc8569mds/config.mk  |2 +-
 board/freescale/p1_p2_rdb/config.mk   |6 +++---
 include/configs/MPC8315ERDB.h |2 +-
 include/configs/MPC8536DS.h   |8 
 include/configs/MPC8569MDS.h  |4 ++--
 include/configs/MPC8572DS.h   |2 +-
 include/configs/P1_P2_RDB.h   |   14 +++---
 include/configs/P2020DS.h |4 ++--
 include/configs/SBC8540.h |2 +-
 include/configs/astro_mcf5373l.h  |4 ++--
 include/configs/bf527-ezkit.h |2 +-
 include/configs/edb93xx.h |   16 
 include/configs/sbc8349.h |8 ++--
 include/configs/sbc8548.h |9 -
 include/configs/sbc8560.h |2 +-
 include/configs/spear3xx.h|6 +++---
 include/configs/vme8349.h |2 +-
 mkconfig  |2 +-
 21 files changed, 54 insertions(+), 59 deletions(-)

diff --git a/board/bf527-ezkit/video.c b/board/bf527-ezkit/video.c
index 87e7658..ca5e9b0 100644
--- a/board/bf527-ezkit/video.c
+++ b/board/bf527-ezkit/video.c
@@ -24,7 +24,7 @@
 #define LCD_Y_RES  240 /* Vertical Resolution */
 #define DMA_BUS_SIZE   16
 
-#ifdef CONFIG_MK_BF527_EZKIT_REV_2_1 /* lq035q1 */
+#ifdef CONFIG_BF527_EZKIT_REV_2_1 /* lq035q1 */
 
 #if !defined(CONFIG_LQ035Q1_USE_RGB888_8_BIT_PPI) && \
 !defined(CONFIG_LQ035Q1_USE_RGB565_8_BIT_PPI)
@@ -125,7 +125,7 @@
 #define PPI_PACK_EN0x80
 #define PPI_POLS_1 0x8000
 
-#ifdef CONFIG_MK_BF527_EZKIT_REV_2_1
+#ifdef CONFIG_BF527_EZKIT_REV_2_1
 static struct spi_slave *slave;
 static int lq035q1_control(unsigned char reg, unsigned short value)
 {
@@ -305,7 +305,7 @@ void EnableTIMER12(void)
 int video_init(void *dst)
 {
 
-#ifdef CONFIG_MK_BF527_EZKIT_REV_2_1
+#ifdef CONFIG_BF527_EZKIT_REV_2_1
lq035q1_control(LQ035_SHUT_CTL, LQ035_ON);
lq035q1_control(LQ035_DRIVER_OUTPUT_CTL, (CONFIG_LQ035Q1_LCD_MODE &
LQ035_DRIVER_OUTPUT_MASK) | LQ035_DRIVER_OUTPUT_DEFAULT);
@@ -318,7 +318,7 @@ int video_init(void *dst)
Init_PPI();
EnablePPI();
 
-#ifdef CONFIG_MK_BF527_EZKIT_REV_2_1
+#ifdef CONFIG_BF527_EZKIT_REV_2_1
EnableTIMER12();
 #else
/* Frame sync 2 (VS) needs to start at least one PPI clk earlier */
@@ -388,7 +388,7 @@ void video_stop(void)
DisableDMA();
DisableTIMER0();
DisableTIMER1();
-#ifdef CONFIG_MK_BF527_EZKIT_REV_2_1
+#ifdef CONFIG_BF527_EZKIT_REV_2_1
lq035q1_control(LQ035_SHUT_CTL, LQ035_SHUT);
 #endif
 }
diff --git a/board/freescale/mpc8315erdb/config.mk 
b/board/freescale/mpc8315erdb/config.mk
index bf972fb..b601601 100644
--- a/board/freescale/mpc8315erdb/config.mk
+++ b/board/freescale/mpc8315erdb/config.mk
@@ -1,5 +1,5 @@
 ifndef NAND_SPL
-ifeq ($(CONFIG_MK_NAND), y)
+ifeq ($(CONFIG_NAND), y)
 TEXT_BASE = $(CONFIG_RAMBOOT_TEXT_BASE)
 endif
 endif
diff --git a/board/freescale/mpc8536ds/config.mk 
b/board/freescale/mpc8536ds/config.mk
index 3f5447a..ac15ff3 100644
--- a/board/freescale/mpc8536ds/config.mk
+++ b/board/freescale/mpc8536ds/config.mk
@@ -24,18 +24,18 @@
 # mpc8536ds board
 #
 ifndef NAND_SPL
-ifeq ($(CONFIG_MK_NAND), y)
+ifeq ($(CONFIG_NAND), y)
 TEXT_BASE = $(CONFIG_RAMBOOT_TEXT_BASE)
 LDSCRIPT := $(TOPDIR)/$(CPUDIR)/u-boot-nand.lds
 endif
 endif
 
-ifeq ($(CONFIG_MK_SDCARD), y)
+ifeq ($(CONFIG_SDCARD), y)
 TEXT_BASE = $(CONFIG_RAMBOOT_TEXT_BASE)
 RESET_VECTOR_ADDRESS = 0xf8fc
 endif
 
-ifeq ($(CONFIG_MK_SPIFLASH), y)
+ifeq ($(CONFIG_SPIFLASH), y)
 TEXT_BASE = $(CONFIG_RAMBOOT_TEXT_BASE)
 RESET_VECTOR_ADDRESS = 0xf8fc
 endif
diff --git a/board/freescale/mpc8569mds/config.mk 
b/board/freescale/mpc8569mds/config.mk
index 86f138c..4f7e18d 100644
--- a/board/freescale/mpc8569mds/config.mk
+++ b/board/freescale/mpc8569mds/config.mk
@@ -24,7 +24,7 @@
 # mpc8569mds board
 #
 ifndef NAND_SPL
-ifeq ($(CONFIG_MK_NAND), y)
+ifeq ($(CONFIG_NAND), y)
 TEXT_BASE = $(CONFIG_RAMBOOT_TEXT_BASE)
 LDSCRIPT