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

Simon Glass sjg at chromium.org
Wed Apr 8 16:01:36 CEST 2015


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!


More information about the U-Boot mailing list