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

[Marek Vasut]

On Wednesday, April 22, 2015 at 11:19:53 AM, Stefan Roese wrote:
> Hi Tom,
> 
> On 08.04.2015 16:01, Simon Glass wrote:
> > On 7 April 2015 at 20:50, Simon Glass <sjg at chromium.org> wrote:
> >> 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>
> > 
> > Applied to u-boot-dm/next, thanks!
> 
> Do you plan to release a v2015.04.1 stable release at some time? Then
> please add this patch to this version. As it fixes SPI NOR flash on
> SoCFPGA.

Maybe we should consider u-boot-stable ...

Best regards,
Marek Vasut