Re: [U-Boot] [PATCH] cmd: sf: Mark it default if DM_SPI_FLASH enabled

2019-10-16 Thread Vignesh Raghavendra
Hi Jagan,

On 16/10/19 9:25 PM, Jagan Teki wrote:
> If DM_SPI_FLASH enabled that means it is using sf command
> for flash interface to access.
> 
> SPI_FLASH can be used via sf command and board/driver
> functions to call spi flash ops, so mark it default only
> for DM_SPI_FLASH.
> 
> This would prevent explicit adding of CONFIG_CMD_SF when
> DM_SPI_FLASH being enabled.
> 
> Cc: Tom Rini 
> Cc: Vignesh Raghavendra 
> Signed-off-by: Jagan Teki 
> ---

Acked-by: Vignesh Raghavendra 

>  cmd/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/cmd/Kconfig b/cmd/Kconfig
> index 07060c63a7..c45286cc20 100644
> --- a/cmd/Kconfig
> +++ b/cmd/Kconfig
> @@ -1168,6 +1168,7 @@ config CMD_SDRAM
>  config CMD_SF
>   bool "sf"
>   depends on DM_SPI_FLASH || SPI_FLASH
> + default y if DM_SPI_FLASH
>   help
> SPI Flash support
>  
> 

-- 
Regards
Vignesh
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH] cmd: sf: Mark it default if DM_SPI_FLASH enabled

2019-10-16 Thread Jagan Teki
If DM_SPI_FLASH enabled that means it is using sf command
for flash interface to access.

SPI_FLASH can be used via sf command and board/driver
functions to call spi flash ops, so mark it default only
for DM_SPI_FLASH.

This would prevent explicit adding of CONFIG_CMD_SF when
DM_SPI_FLASH being enabled.

Cc: Tom Rini 
Cc: Vignesh Raghavendra 
Signed-off-by: Jagan Teki 
---
 cmd/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/cmd/Kconfig b/cmd/Kconfig
index 07060c63a7..c45286cc20 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -1168,6 +1168,7 @@ config CMD_SDRAM
 config CMD_SF
bool "sf"
depends on DM_SPI_FLASH || SPI_FLASH
+   default y if DM_SPI_FLASH
help
  SPI Flash support
 
-- 
2.18.0.321.gffc6fa0e3

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