Hi Boris,
Boris Brezillon boris.brezillon@bootlin.com wrote on Thu, 29 Nov 2018 00:46:23 +0100:
On Thu, 29 Nov 2018 00:07:37 +0100 Miquel Raynal miquel.raynal@bootlin.com wrote:
--- a/drivers/Makefile +++ b/drivers/Makefile @@ -6,7 +6,7 @@ obj-$(CONFIG_$(SPL_TPL_)DRIVERS_MISC_SUPPORT) += misc/ sysreset/ firmware/ obj-$(CONFIG_$(SPL_TPL_)I2C_SUPPORT) += i2c/ obj-$(CONFIG_$(SPL_TPL_)LED) += led/ obj-$(CONFIG_$(SPL_TPL_)MMC_SUPPORT) += mmc/ -obj-$(CONFIG_$(SPL_TPL_)NAND_SUPPORT) += mtd/nand/raw/ +obj-$(CONFIG_$(SPL_TPL_)NAND_SUPPORT) += mtd/nand/
Can't we have += mtd/
instead?
As you can see, an SPL build just compiles a few files and does not parse the whole "Makefile tree". To avoid compiling the world in the SPL, when I later change the Makefiles to have this "tree" logic I also enclosed some lines into a "#ifndef CONFIG_SPL_BUILD" conditional to avoid enlarging the SPL without reason. There is probably a way to simplify this but for now I don't want to do it :)
Thanks, Miquèl