[U-Boot] [PATCH 01/12] dm: spi_flash: fix wrong dependency

[Simon Glass]

On 30 March 2015 at 21:47, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> CONFIG_SPI does not exist in Kconfig in the first place, so the
> dependency "depends on DM && SPI" is never met, i.e., DM_SPI_FLASH
> can never be enabled (unless you ignore the dependency in an illegal
> way.  See below.)
>
> Actually, some defconfigs such as socfpga_*_defconfig define
> CONFIG_DM_SPI_FLASH=y, but it never appears in the .config file
> because of this wrong dependency.
>
> On the other hand, all the Tegra boards enable DM_SPI_FLASH because
>
>   config DM_SPI_FLASH
>           default y
>
> silently ignores the dependency.
> Unfortunately, this style of CONFIG definition is abused everywhere
> in U-Boot, so we easily miss such a wrong dependency.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> ---
>
>  drivers/mtd/spi/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/spi/Kconfig b/drivers/mtd/spi/Kconfig
> index 2dc46b4..fd2d7ac 100644
> --- a/drivers/mtd/spi/Kconfig
> +++ b/drivers/mtd/spi/Kconfig
> @@ -1,6 +1,6 @@
>  config DM_SPI_FLASH
>         bool "Enable Driver Model for SPI flash"
> -       depends on DM && SPI
> +       depends on DM && DM_SPI
>         help
>           Enable driver model for SPI flash. This SPI flash interface
>           (spi_flash_probe(), spi_flash_write(), etc.) is then
> --
> 1.9.1
>

Acked-by: Simon Glass <sjg at chromium.org>