Re: [U-Boot] [PATCH v3 2/7] include/configs: move definition of CONFIG_CMD_BLOB

2016-01-27 Thread york sun
On 01/22/2016 03:10 AM, Aneesh Bansal wrote:
> CONFIG_CMD_BLOB must be defined in case of Secure Boot.
> It was earlier defined in all config files. The definition
> has been moved to a common file which is included by all configs.
> 
> Signed-off-by: Aneesh Bansal 
> ---
> Changes in v3:
> None
> 
> Changes in v2:
> None (Changed the Sign-Off with New E-Mail ID)
> 
>  arch/arm/include/asm/fsl_secure_boot.h | 1 +
>  arch/powerpc/include/asm/fsl_secure_boot.h | 1 +
>  include/configs/B4860QDS.h | 4 
>  include/configs/BSC9132QDS.h   | 4 
>  include/configs/C29XPCIE.h | 4 
>  include/configs/P1010RDB.h | 4 
>  include/configs/P2041RDB.h | 4 
>  include/configs/T102xQDS.h | 4 
>  include/configs/T102xRDB.h | 4 
>  include/configs/T1040QDS.h | 4 
>  include/configs/T104xRDB.h | 4 
>  include/configs/T208xQDS.h | 5 -
>  include/configs/T208xRDB.h | 5 -
>  include/configs/T4240QDS.h | 4 
>  include/configs/T4240RDB.h | 9 -
>  include/configs/corenet_ds.h   | 4 
>  include/configs/ls1021aqds.h   | 4 
>  include/configs/ls1021atwr.h   | 4 
>  18 files changed, 2 insertions(+), 71 deletions(-)

Prefix subject with "secure_boot:". Slightly reformat commit message.
Applied to u-boot-fsl-qoriq master. Awaiting upstream.

Thanks.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3 2/7] include/configs: move definition of CONFIG_CMD_BLOB

2016-01-27 Thread Ruchika Gupta


> -Original Message-
> From: Aneesh Bansal [mailto:aneesh.ban...@nxp.com]
> Sent: Friday, January 22, 2016 4:37 PM
> To: u-boot@lists.denx.de
> Cc: york sun ; Ruchika Gupta
> ; Prabhakar Kushwaha
> ; Aneesh Bansal
> 
> Subject: [PATCH v3 2/7] include/configs: move definition of
> CONFIG_CMD_BLOB
> 
> CONFIG_CMD_BLOB must be defined in case of Secure Boot.
> It was earlier defined in all config files. The definition has been moved to a
> common file which is included by all configs.
> 
> Signed-off-by: Aneesh Bansal 
> ---
> Changes in v3:
> None
> 
> Changes in v2:
> None (Changed the Sign-Off with New E-Mail ID)
> 
>  arch/arm/include/asm/fsl_secure_boot.h | 1 +
>  arch/powerpc/include/asm/fsl_secure_boot.h | 1 +
>  include/configs/B4860QDS.h | 4 
>  include/configs/BSC9132QDS.h   | 4 
>  include/configs/C29XPCIE.h | 4 
>  include/configs/P1010RDB.h | 4 
>  include/configs/P2041RDB.h | 4 
>  include/configs/T102xQDS.h | 4 
>  include/configs/T102xRDB.h | 4 
>  include/configs/T1040QDS.h | 4 
>  include/configs/T104xRDB.h | 4 
>  include/configs/T208xQDS.h | 5 -
>  include/configs/T208xRDB.h | 5 -
>  include/configs/T4240QDS.h | 4 
>  include/configs/T4240RDB.h | 9 -
>  include/configs/corenet_ds.h   | 4 
>  include/configs/ls1021aqds.h   | 4 
>  include/configs/ls1021atwr.h   | 4 
>  18 files changed, 2 insertions(+), 71 deletions(-)
> 
> diff --git a/arch/arm/include/asm/fsl_secure_boot.h
> b/arch/arm/include/asm/fsl_secure_boot.h
> index b29e71c..8491a72 100644
> --- a/arch/arm/include/asm/fsl_secure_boot.h
> +++ b/arch/arm/include/asm/fsl_secure_boot.h
> @@ -9,6 +9,7 @@
> 
>  #ifdef CONFIG_SECURE_BOOT
>  #define CONFIG_CMD_ESBC_VALIDATE
> +#define CONFIG_CMD_BLOB
>  #define CONFIG_FSL_SEC_MON
>  #define CONFIG_SHA_PROG_HW_ACCEL
>  #define CONFIG_RSA
> diff --git a/arch/powerpc/include/asm/fsl_secure_boot.h
> b/arch/powerpc/include/asm/fsl_secure_boot.h
> index 87415b1..7d217a6 100644
> --- a/arch/powerpc/include/asm/fsl_secure_boot.h
> +++ b/arch/powerpc/include/asm/fsl_secure_boot.h
> @@ -10,6 +10,7 @@
> 
>  #ifdef CONFIG_SECURE_BOOT
>  #define CONFIG_CMD_ESBC_VALIDATE
> +#define CONFIG_CMD_BLOB
>  #define CONFIG_FSL_SEC_MON
>  #define CONFIG_SHA_PROG_HW_ACCEL
>  #define CONFIG_DM
> diff --git a/include/configs/B4860QDS.h b/include/configs/B4860QDS.h index
> 9fb5cee..bcbae50 100644
> --- a/include/configs/B4860QDS.h
> +++ b/include/configs/B4860QDS.h
> @@ -924,8 +924,4 @@ unsigned long get_board_ddr_clk(void);
> 
>  #include 
> 
> -#ifdef CONFIG_SECURE_BOOT
> -#define CONFIG_CMD_BLOB
> -#endif
> -
>  #endif   /* __CONFIG_H */
> diff --git a/include/configs/BSC9132QDS.h b/include/configs/BSC9132QDS.h
> index d0e5a25..89907dc 100644
> --- a/include/configs/BSC9132QDS.h
> +++ b/include/configs/BSC9132QDS.h
> @@ -722,8 +722,4 @@ combinations. this should be removed later
> 
>  #include 
> 
> -#ifdef CONFIG_SECURE_BOOT
> -#define CONFIG_CMD_BLOB
> -#endif
> -
>  #endif   /* __CONFIG_H */
> diff --git a/include/configs/C29XPCIE.h b/include/configs/C29XPCIE.h index
> 890dcbb..16920c6 100644
> --- a/include/configs/C29XPCIE.h
> +++ b/include/configs/C29XPCIE.h
> @@ -567,8 +567,4 @@
> 
>  #include 
> 
> -#ifdef CONFIG_SECURE_BOOT
> -#define CONFIG_CMD_BLOB
> -#endif
> -
>  #endif   /* __CONFIG_H */
> diff --git a/include/configs/P1010RDB.h b/include/configs/P1010RDB.h index
> f9776c0..3c0faca 100644
> --- a/include/configs/P1010RDB.h
> +++ b/include/configs/P1010RDB.h
> @@ -952,8 +952,4 @@ extern unsigned long get_sdram_size(void);
> 
>  #include 
> 
> -#ifdef CONFIG_SECURE_BOOT
> -#define CONFIG_CMD_BLOB
> -#endif
> -
>  #endif   /* __CONFIG_H */
> diff --git a/include/configs/P2041RDB.h b/include/configs/P2041RDB.h index
> b2e51b5..f250e7f 100644
> --- a/include/configs/P2041RDB.h
> +++ b/include/configs/P2041RDB.h
> @@ -756,8 +756,4 @@ unsigned long get_board_sys_clk(unsigned long
> dummy);
> 
>  #include 
> 
> -#ifdef CONFIG_SECURE_BOOT
> -#define CONFIG_CMD_BLOB
> -#endif
> -
>  #endif   /* __CONFIG_H */
> diff --git a/include/configs/T102xQDS.h b/include/configs/T102xQDS.h index
> fb41a7d..e5df784 100644
> --- a/include/configs/T102xQDS.h
> +++ b/include/configs/T102xQDS.h
> @@ -946,8 +946,4 @@ unsigned long get_board_ddr_clk(void);
> 
>  #include 
> 
> -#ifdef CONFIG_SECURE_BOOT
> -#define CONFIG_CMD_BLOB
> -#endif
> -
>  #endif   /* __T1024QDS_H */
> diff --git a/include/configs/T102xRDB.h b/include/configs/T102xRDB.h index
> 113df37..3cda3b1 100644
> --- a/include/configs/T102xRDB.h
> +++ b/include/configs/T102xRDB.h
> @@ -958,8 +958,4 @@ unsigned long get_board_ddr_clk(void);
> 
>  #include 
> 
> -#ifdef CONFIG_SECURE_BOOT
> -#define CONFIG_CMD_BLOB
> -#endif
>