Re: [U-Boot] [PATCH 2/3] mtd: spi: Kconfig: Update CONFIG_SPI_FLASH
On Tue, Feb 12, 2019 at 1:31 PM Vignesh R wrote: > > Hi, > > On 09/02/19 5:45 PM, Jagan Teki wrote: > > 1) CONFIG_SPI_FLASH is not just a legacy code, but it has common > >core code which handle both dm and non-dm spi flash code. So > >fix the info text to make it clear globally. > > > > 2) Since it's flash core it shouldn't depends on legacy SPI, > >so remove the 'depends on SPI' > > > > Cc: Vignesh R > > Signed-off-by: Jagan Teki > > --- > > drivers/mtd/spi/Kconfig | 5 ++--- > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/mtd/spi/Kconfig b/drivers/mtd/spi/Kconfig > > index e3b40fc157..605f60c713 100644 > > --- a/drivers/mtd/spi/Kconfig > > +++ b/drivers/mtd/spi/Kconfig > > @@ -26,11 +26,10 @@ config SPI_FLASH_SANDBOX > > stored in a file on the host filesystem. > > > > config SPI_FLASH > > - bool "Legacy SPI Flash Interface support" > > - depends on SPI > > + bool "SPI Flash Core Interface support" > > Nit, since we have SPI NAND flash as well, should description be updated > to say SPI NOR Flash instead of SPI Flash? True, I have bundle of changes even for configs to rename till those are in lets go with as we are. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] mtd: spi: Kconfig: Update CONFIG_SPI_FLASH
Hi, On 09/02/19 5:45 PM, Jagan Teki wrote: > 1) CONFIG_SPI_FLASH is not just a legacy code, but it has common >core code which handle both dm and non-dm spi flash code. So >fix the info text to make it clear globally. > > 2) Since it's flash core it shouldn't depends on legacy SPI, >so remove the 'depends on SPI' > > Cc: Vignesh R > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi/Kconfig | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/mtd/spi/Kconfig b/drivers/mtd/spi/Kconfig > index e3b40fc157..605f60c713 100644 > --- a/drivers/mtd/spi/Kconfig > +++ b/drivers/mtd/spi/Kconfig > @@ -26,11 +26,10 @@ config SPI_FLASH_SANDBOX > stored in a file on the host filesystem. > > config SPI_FLASH > - bool "Legacy SPI Flash Interface support" > - depends on SPI > + bool "SPI Flash Core Interface support" Nit, since we have SPI NAND flash as well, should description be updated to say SPI NOR Flash instead of SPI Flash? > select SPI_MEM > help > - Enable the legacy SPI flash support. This will include basic > + Enable the SPI flash Core support. This will include basic > standard support for things like probing, read / write, and > erasing through cmd_sf interface. > > -- Regards Vignesh ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH 2/3] mtd: spi: Kconfig: Update CONFIG_SPI_FLASH
1) CONFIG_SPI_FLASH is not just a legacy code, but it has common core code which handle both dm and non-dm spi flash code. So fix the info text to make it clear globally. 2) Since it's flash core it shouldn't depends on legacy SPI, so remove the 'depends on SPI' Cc: Vignesh R Signed-off-by: Jagan Teki --- drivers/mtd/spi/Kconfig | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/mtd/spi/Kconfig b/drivers/mtd/spi/Kconfig index e3b40fc157..605f60c713 100644 --- a/drivers/mtd/spi/Kconfig +++ b/drivers/mtd/spi/Kconfig @@ -26,11 +26,10 @@ config SPI_FLASH_SANDBOX stored in a file on the host filesystem. config SPI_FLASH - bool "Legacy SPI Flash Interface support" - depends on SPI + bool "SPI Flash Core Interface support" select SPI_MEM help - Enable the legacy SPI flash support. This will include basic + Enable the SPI flash Core support. This will include basic standard support for things like probing, read / write, and erasing through cmd_sf interface. -- 2.18.0.321.gffc6fa0e3 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot