[U-Boot] [PATCH v1 0/4] spi: Split CONFIG_DM_SPI* to CONFIG_{SPL_TPL}DM_SPI*

Lukasz Majewski
2019-08-12T14:02:47Z

Hi Adam,

> On Mon, Aug 12, 2019 at 2:08 AM Lukasz Majewski <lukma@denx.de> wrote:
> >
> > This patch series introduces new SPL and TPL specific Kconfig
> > entries for DM_SPI* options. Such change allows using the spi
> > driver in SPL/TPL or U-Boot proper.
> >
> > First two patches - related to ls10{42}* NXP soc fix some issues
> > with defining the DM_SPI* defines in <board>.h file instead of
> > Kconfig.
> >
> > This series doesn't introduce build breaks, but board maintainers
> > are kindly asked to check if their boards still boots.
> >
>
> FYI, when I pulled the latest u-boot/master, this patch didn't apply
> cleanly, so I ended up having to skip a few items that wouldn't impact
> the da850-evm which boots from SPI flash.

I see. I've applied it on the newest master:

SHA1: feb5a02f869d5678190dfc915ef6c2781b4f7a6c

>
> From what I can tell, this seems to work just fine on the da850-evm
>
> Tested-by: Adam Ford <aford173@gmail.com> #da850-evm
>

Thanks for testing :-)

> >
> >
> > Lukasz Majewski (4):
> > kconfig: Update comment regarding CONFIG_IS_ENABLED(FOO) for TPL
> > spi: Move DM_SPI_FLASH to Kconfig (for NXP's ls1043a)
> > spi: Move DM_SPI_FLASH and SPI_FLASH_DATAFLASH to Kconfig (for
> > ls1021aXXX)
> > spi: Convert CONFIG_DM_SPI* to CONFIG_$(SPL_TPL_)DM_SPI*
> >
> > arch/arm/Kconfig | 23
> > +++++++++++++++++++++-- board/l+g/vinco/vinco.c
> > | 4 ++-- cmd/sf.c | 4 ++--
> > cmd/spi.c | 6 +++---
> > common/spl/Kconfig | 28
> > ++++++++++++++++++++++++++++ configs/am57xx_evm_defconfig
> > | 2 ++ configs/am57xx_hs_evm_defconfig | 2 ++
> > configs/am57xx_hs_evm_usb_defconfig | 2 ++
> > configs/axm_defconfig | 2 ++
> > configs/chromebook_link64_defconfig | 2 ++
> > configs/chromebook_samus_tpl_defconfig | 4 ++++
> > configs/dra7xx_evm_defconfig | 2 ++
> > configs/dra7xx_hs_evm_defconfig | 2 ++
> > configs/dra7xx_hs_evm_usb_defconfig | 2 ++
> > configs/imx28_xea_defconfig | 2 ++
> > configs/j721e_evm_a72_defconfig | 2 ++
> > configs/j721e_evm_r5_defconfig | 2 ++
> > configs/ls1021aiot_qspi_defconfig | 2 ++
> > configs/ls1021aiot_sdcard_defconfig | 2 ++
> > configs/ls1021aqds_qspi_defconfig | 1 +
> > configs/ls1021aqds_sdcard_qspi_defconfig | 1 +
> > configs/ls1021atwr_qspi_defconfig | 1 +
> > configs/sama5d2_xplained_spiflash_defconfig | 2 ++
> > configs/sama5d3xek_spiflash_defconfig | 7 ++++---
> > configs/sama5d4_xplained_spiflash_defconfig | 2 ++
> > configs/sama5d4ek_spiflash_defconfig | 2 ++
> > configs/stm32mp15_basic_defconfig | 2 ++
> > configs/taurus_defconfig | 2 ++
> > drivers/mtd/spi/Makefile | 4 ++--
> > drivers/mtd/spi/sf_probe.c | 8 ++++----
> > drivers/net/fm/fm.c | 4 ++--
> > drivers/spi/Makefile | 2 +-
> > drivers/spi/atmel_spi.c | 4 ++--
> > drivers/spi/davinci_spi.c | 6 +++---
> > drivers/spi/fsl_dspi.c | 5 +++--
> > drivers/spi/kirkwood_spi.c | 2 +-
> > drivers/spi/mxc_spi.c | 6 +++---
> > drivers/spi/omap3_spi.c | 4 ++--
> > drivers/spi/sh_qspi.c | 4 ++--
> > env/sf.c | 2 +-
> > include/configs/ls1021aiot.h | 6 ------
> > include/configs/ls1021aqds.h | 8 --------
> > include/configs/ls1021atwr.h | 5 -----
> > include/configs/ls1043a_common.h | 2 --
> > include/linux/kconfig.h | 2 ++
> > include/spi.h | 8 ++++----
> > include/spi_flash.h | 2 +-
> > test/dm/spi.c | 2 +-
> > 48 files changed, 137 insertions(+), 64 deletions(-)
> >
> > --
> > 2.11.0
> >

Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190812/4dc4a99e/attachment.sig>