diff mbox series

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

Message ID 20191016155550.9281-1-jagan@amarulasolutions.com
State Accepted
Commit c2af7fb1ff6f8d6489bdba4216128cbb08ae4243
Delegated to: Jagannadha Sutradharudu Teki
Headers show
Series [U-Boot] cmd: sf: Mark it default if DM_SPI_FLASH enabled | expand

Commit Message

Jagan Teki Oct. 16, 2019, 3:55 p.m. UTC
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 <trini@konsulko.com>
Cc: Vignesh Raghavendra <vigneshr@ti.com>
Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 cmd/Kconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Raghavendra, Vignesh Oct. 17, 2019, 5:08 a.m. UTC | #1
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 <trini@konsulko.com>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> ---

Acked-by: Vignesh Raghavendra <vigneshr@ti.com>

>  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
>  
>
diff mbox series

Patch

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