On 16/07/19 1:06 AM, Simon Goldschmidt wrote: > This adds separate kconfig options for drivers/sysreset and > drivers/firmware. > > Up to now, CONFIG_SPL_DRIVERS_MISC_SUPPORT added drivers/misc to SPL > build but also added drivers/firmware and drivers/sysreset at the same > time. > > Since that is confusing, this patch uses CONFIG_SPL_SYSRESET for > drivers/sysreset and adds CONFIG_SPL_FIRMWARE for > drivers/firmware (and accordingly for the TPL options). > > CONFIG_SPL_DRIVERS_MISC_SUPPORT stays for including drivers/misc into > the SPL build (and accordingly for TPL) since there are boards using > non-DM (non UCLASS_MISC) files from drivers/misc. Such boards don't > have CONFIG_SPL_MISC enabled, so cannot use this to include > drivers/misc into the SPL build. > > Signed-off-by: Simon Goldschmidt <simon.k.r.goldschm...@gmail.com> > --- > > Changes in v3: > - Revert to keeping CONFIG_SPL_DRIVERS_MISC_SUPPORT as there are boards > that need to include drivers/misc without UCLASS_MISC being enabled > - Only CONFIG_SPL_FIRMWARE is added as new config symbol. To ensure no > arch Kconfig files need to be changed, add dependencies to ensure this > is enabled where required. > > Changes in v2: > - adapt config names to match the non-SPL config options: > - changed CONFIG_SPL_SYSRESET_SUPPORT to CONFIG_SPL_SYSRESET > - changed CONFIG_SPL_DRIVERS_FIRMWARE_SUPPORT to CONFIG_SPL_FIRMWARE > - changed CONFIG_SPL_DRIVERS_MISC_SUPPORT to CONFIG_SPL_MISC > > configs/evb-rk3036_defconfig | 1 + > configs/kylin-rk3036_defconfig | 1 + > configs/sandbox_spl_defconfig | 1 + > drivers/Makefile | 7 ++++++- > drivers/firmware/Kconfig | 7 ++++++- > drivers/sysreset/Kconfig | 1 + > 6 files changed, 16 insertions(+), 2 deletions(-) > > diff --git a/configs/evb-rk3036_defconfig b/configs/evb-rk3036_defconfig > index 5e6bb54551..0eb7384340 100644 > --- a/configs/evb-rk3036_defconfig > +++ b/configs/evb-rk3036_defconfig > @@ -47,6 +47,7 @@ CONFIG_SPI_FLASH=y > CONFIG_SF_DEFAULT_SPEED=20000000 > CONFIG_PINCTRL=y > # CONFIG_SPL_DM_SERIAL is not set > +# CONFIG_SPL_SYSRESET is not set > CONFIG_DEBUG_UART_SHIFT=2 > CONFIG_SYSRESET=y > CONFIG_USB=y > diff --git a/configs/kylin-rk3036_defconfig b/configs/kylin-rk3036_defconfig > index 921af0d476..a76ae9d326 100644 > --- a/configs/kylin-rk3036_defconfig > +++ b/configs/kylin-rk3036_defconfig > @@ -49,6 +49,7 @@ CONFIG_SF_DEFAULT_SPEED=20000000 > CONFIG_PINCTRL=y > CONFIG_DM_REGULATOR_FIXED=y > # CONFIG_SPL_DM_SERIAL is not set > +# CONFIG_SPL_SYSRESET is not set > CONFIG_DEBUG_UART_SHIFT=2 > CONFIG_SYSRESET=y > CONFIG_USB=y > diff --git a/configs/sandbox_spl_defconfig b/configs/sandbox_spl_defconfig > index 27034cfaa9..5b7dae9be6 100644 > --- a/configs/sandbox_spl_defconfig > +++ b/configs/sandbox_spl_defconfig > @@ -3,6 +3,7 @@ CONFIG_SPL_LIBCOMMON_SUPPORT=y > CONFIG_SPL_LIBGENERIC_SUPPORT=y > CONFIG_SPL_SERIAL_SUPPORT=y > CONFIG_SPL_DRIVERS_MISC_SUPPORT=y > +CONFIG_SPL_FIRMWARE=y > CONFIG_NR_DRAM_BANKS=1 > CONFIG_SPL=y > CONFIG_BOOTSTAGE_STASH_ADDR=0x0 > diff --git a/drivers/Makefile b/drivers/Makefile > index 603aa98590..1f225bc2ca 100644 > --- a/drivers/Makefile > +++ b/drivers/Makefile > @@ -4,7 +4,6 @@ obj-$(CONFIG_$(SPL_TPL_)CLK) += clk/ > obj-$(CONFIG_$(SPL_TPL_)DM) += core/ > obj-$(CONFIG_$(SPL_TPL_)DFU) += dfu/ > obj-$(CONFIG_$(SPL_TPL_)GPIO_SUPPORT) += gpio/ > -obj-$(CONFIG_$(SPL_TPL_)DRIVERS_MISC_SUPPORT) += misc/ sysreset/ firmware/
why can't the following be added instead of adding SPL and TPL options separately? obj-$(CONFIG_$(SPL_TPL_)FIRMWARE) += firmware/ obj-$(CONFIG_$(SPL_TPL_)SYSRESET) += sysreset/ Thanks and regards, Lokesh > obj-$(CONFIG_$(SPL_TPL_)I2C_SUPPORT) += i2c/ > obj-$(CONFIG_$(SPL_TPL_)INPUT) += input/ > obj-$(CONFIG_$(SPL_TPL_)LED) += led/ > @@ -58,6 +57,9 @@ obj-$(CONFIG_SPL_SATA_SUPPORT) += ata/ scsi/ > obj-$(CONFIG_HAVE_BLOCK_DEVICE) += block/ > obj-$(CONFIG_SPL_FPGA_SUPPORT) += fpga/ > obj-$(CONFIG_SPL_THERMAL) += thermal/ > +obj-$(CONFIG_SPL_DRIVERS_MISC_SUPPORT) += misc/ > +obj-$(CONFIG_SPL_SYSRESET) += sysreset/ > +obj-$(CONFIG_SPL_FIRMWARE) +=firmware/ > > endif > endif > @@ -65,6 +67,9 @@ endif > ifdef CONFIG_TPL_BUILD > > obj-$(CONFIG_TPL_MPC8XXX_INIT_DDR_SUPPORT) += ddr/fsl/ > +obj-$(CONFIG_TPL_DRIVERS_MISC_SUPPORT) += misc/ > +obj-$(CONFIG_TPL_SYSRESET) += sysreset/ > +obj-$(CONFIG_TPL_FIRMWARE) += firmware/ > > endif > > diff --git a/drivers/firmware/Kconfig b/drivers/firmware/Kconfig > index 7d8f161b26..873bc8c796 100644 > --- a/drivers/firmware/Kconfig > +++ b/drivers/firmware/Kconfig > @@ -1,9 +1,13 @@ > config FIRMWARE > bool "Enable Firmware driver support" > > +config SPL_FIRMWARE > + bool "Enable Firmware driver support in SPL" > + depends on FIRMWARE > + > config SPL_ARM_PSCI_FW > bool > - select FIRMWARE > + select SPL_FIRMWARE > > config ARM_PSCI_FW > bool > @@ -13,6 +17,7 @@ config TI_SCI_PROTOCOL > tristate "TI System Control Interface (TISCI) Message Protocol" > depends on K3_SEC_PROXY > select FIRMWARE > + select SPL_FIRMWARE if SPL > help > TI System Control Interface (TISCI) Message Protocol is used to manage > compute systems such as ARM, DSP etc with the system controller in > diff --git a/drivers/sysreset/Kconfig b/drivers/sysreset/Kconfig > index 30aed2c4c1..a69b74cee2 100644 > --- a/drivers/sysreset/Kconfig > +++ b/drivers/sysreset/Kconfig > @@ -50,6 +50,7 @@ config SYSRESET_MICROBLAZE > config SYSRESET_PSCI > bool "Enable support for PSCI System Reset" > depends on ARM_PSCI_FW > + select SPL_ARM_PSCI_FW if SPL > help > Enable PSCI SYSTEM_RESET function call. To use this, PSCI firmware > must be running on your system. > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
