[U-Boot] [PATCH 2/2] config.h: clean unused CONFIG_ENV_SPI_* if using driver model

[york sun]

On 07/20/2016 03:51 AM, Gong Qianyu wrote:
> When using SPI driver model, it will get the values from DT. So
> there is no need to set CONFIG_ENV_SPI_MAX_HZ and
> CONFIG_ENV_SPI_MODE any more.
>

You indicate these macros are not needed _if_ using driver model. You 
presume the driver model is always used. You have CONFIG_DM_SPI_FLASH in 
defconfig, but you don't have it selected in Kconfig for those 
platforms. This can leave a possible configuration if one runs "make 
menuconfig" and deselect DM_SPI_FLASH.

York


> Signed-off-by: Gong Qianyu <Qianyu.Gong at nxp.com>
> ---
>  include/configs/ls1012a_common.h | 2 --
>  include/configs/ls1043a_common.h | 2 --
>  2 files changed, 4 deletions(-)
>
> diff --git a/include/configs/ls1012a_common.h b/include/configs/ls1012a_common.h
> index fba2fac..1602f09 100644
> --- a/include/configs/ls1012a_common.h
> +++ b/include/configs/ls1012a_common.h
> @@ -52,8 +52,6 @@
>  #define CONFIG_SYS_FMAN_FW_ADDR		0x400d0000
>  #define CONFIG_ENV_SPI_BUS		0
>  #define CONFIG_ENV_SPI_CS		0
> -#define CONFIG_ENV_SPI_MAX_HZ		1000000
> -#define CONFIG_ENV_SPI_MODE		0x03
>  #define CONFIG_SPI_FLASH_SPANSION
>  #define CONFIG_FSL_SPI_INTERFACE
>  #define CONFIG_SF_DATAFLASH
> diff --git a/include/configs/ls1043a_common.h b/include/configs/ls1043a_common.h
> index b0d4a8d..028f7d9 100644
> --- a/include/configs/ls1043a_common.h
> +++ b/include/configs/ls1043a_common.h
> @@ -222,8 +222,6 @@
>  #define CONFIG_SYS_FMAN_FW_ADDR		0x400d0000
>  #define CONFIG_ENV_SPI_BUS		0
>  #define CONFIG_ENV_SPI_CS		0
> -#define CONFIG_ENV_SPI_MAX_HZ		1000000
> -#define CONFIG_ENV_SPI_MODE		0x03
>  #else
>  #define CONFIG_SYS_QE_FMAN_FW_IN_NOR
>  /* FMan fireware Pre-load address */
>