Patchwork [U-Boot,v3,1/8] arm/km: remove CONFIG_SYS_KWD_CONFIG from keymile-common.h

login
register
mail settings
Submitter Valentin Longchamp
Date May 4, 2011, 3:53 p.m.
Message ID <3cd703323c1273b13554794ece39c79d61410a8a.1304523385.git.valentin.longchamp@keymile.com>
Download mbox | patch
Permalink /patch/94078/
State Changes Requested
Delegated to: Albert ARIBAUD
Headers show

Comments

Valentin Longchamp - May 4, 2011, 3:53 p.m.
This define is marvell specific, so it should be present in km_arm.
It is however not needed there either, since we set it to the default
value that is already set in include/asm/arch-kirkwood/config.h

Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
cc: Holger Brunck <holger.brunck@keymile.com>
cc: Prafulla Wadaskar <prafulla@marvell.com>
cc: Heiko Schocher <hs@denx.de>
cc: Wolfgang Denk <wd@denx.de>
cc: Detlev Zundel <dzu@denx.de>
---
changes for v3:
	- this patch is new in the series

 include/configs/keymile-common.h |    9 ---------
 1 files changed, 0 insertions(+), 9 deletions(-)
Prafulla Wadaskar - May 12, 2011, 10:45 a.m.
> -----Original Message-----
> From: Valentin Longchamp [mailto:valentin.longchamp@keymile.com]
> Sent: Wednesday, May 04, 2011 9:24 PM
> To: u-boot@lists.denx.de
> Cc: holger.brunck@keymile.com; Prafulla Wadaskar; Valentin Longchamp;
> Heiko Schocher; Wolfgang Denk; Detlev Zundel
> Subject: [PATCH v3 1/8] arm/km: remove CONFIG_SYS_KWD_CONFIG from
> keymile-common.h
> 
> This define is marvell specific, so it should be present in km_arm.
> It is however not needed there either, since we set it to the default
> value that is already set in include/asm/arch-kirkwood/config.h
> 
> Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
> cc: Holger Brunck <holger.brunck@keymile.com>
> cc: Prafulla Wadaskar <prafulla@marvell.com>
> cc: Heiko Schocher <hs@denx.de>
> cc: Wolfgang Denk <wd@denx.de>
> cc: Detlev Zundel <dzu@denx.de>
> ---
> changes for v3:
> 	- this patch is new in the series
> 
>  include/configs/keymile-common.h |    9 ---------
>  1 files changed, 0 insertions(+), 9 deletions(-)
> 
> diff --git a/include/configs/keymile-common.h b/include/configs/keymile-
> common.h
> index cb6d0fb..7115724 100644
> --- a/include/configs/keymile-common.h
> +++ b/include/configs/keymile-common.h
> @@ -31,15 +31,6 @@
>  #define CONFIG_BOOTCOUNT_LIMIT
> 
>  /*
> - * By default kwbimage.cfg from board specific folder is used
> - * If for some board, different configuration file need to be used,
> - * CONFIG_SYS_KWD_CONFIG should be defined in board specific header
> file
> - */
> -#ifndef CONFIG_SYS_KWD_CONFIG
> -#define	CONFIG_SYS_KWD_CONFIG	$(SRCTREE)/$(CONFIG_BOARDDIR)/kwbimage.cfg
> -#endif /* CONFIG_SYS_KWD_CONFIG */
> -
> -/*
>   * Command line configuration.
>   */
>  #include <config_cmd_default.h>

Acked-by: Prafulla Wadaskar <prafulla@marvell.com>

Regards..
Prafulla . .

Patch

diff --git a/include/configs/keymile-common.h b/include/configs/keymile-common.h
index cb6d0fb..7115724 100644
--- a/include/configs/keymile-common.h
+++ b/include/configs/keymile-common.h
@@ -31,15 +31,6 @@ 
 #define CONFIG_BOOTCOUNT_LIMIT
 
 /*
- * By default kwbimage.cfg from board specific folder is used
- * If for some board, different configuration file need to be used,
- * CONFIG_SYS_KWD_CONFIG should be defined in board specific header file
- */
-#ifndef CONFIG_SYS_KWD_CONFIG
-#define	CONFIG_SYS_KWD_CONFIG	$(SRCTREE)/$(CONFIG_BOARDDIR)/kwbimage.cfg
-#endif /* CONFIG_SYS_KWD_CONFIG */
-
-/*
  * Command line configuration.
  */
 #include <config_cmd_default.h>