Hi Tom,
On 2017/8/2 23:31, Tom Rini wrote:
On Wed, Aug 02, 2017 at 03:47:58PM +0800, Wenyou Yang wrote:
The i2c_eeprom isn't always necessary when building for SPL, add the condition on build i2c_eeprom.
Signed-off-by: Wenyou Yang wenyou.yang@microchip.com
drivers/misc/Makefile | 6 ++++++ 1 file changed, 6 insertions(+)
diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile index 10265c8fb4..ccc84c38fc 100644 --- a/drivers/misc/Makefile +++ b/drivers/misc/Makefile @@ -20,7 +20,13 @@ obj-$(CONFIG_CROS_EC_SPI) += cros_ec_spi.o endif obj-$(CONFIG_FSL_IIM) += fsl_iim.o obj-$(CONFIG_LED_STATUS_GPIO) += gpio_led.o +ifdef CONFIG_SPL_BUILD +ifneq ($(CONFIG_SPL_I2C_SUPPORT),) +obj-$(CONFIG_I2C_EEPROM) += i2c_eeprom.o +endif +else obj-$(CONFIG_I2C_EEPROM) += i2c_eeprom.o +endif obj-$(CONFIG_FSL_MC9SDZ60) += mc9sdz60.o obj-$(CONFIG_MXC_OCOTP) += mxc_ocotp.o obj-$(CONFIG_MXS_OCOTP) += mxs_ocotp.o
What's wrong with building this when not required? Build failure? If so, we should add SPL_I2C_EEPROM and then the above becomes obj-$(CONFIG_$(SPL_)I2C_EEPROM) += i2c_eeprom.o or so. Thanks!
Yes, build failure. if CONFIG_SPL_I2C_SUPPORT isn't enabled, it failed to build u-boot-spl with the following error. ---8<--- drivers/built-in.o: In function `i2c_eeprom_std_read': /home/wyang/work/linux4sam/u-boot-at91/drivers/misc/i2c_eeprom.c:36: undefined reference to `dm_i2c_read' scripts/Makefile.spl:333: recipe for target 'spl/u-boot-spl' failed --->8---
Add SPL_I2C_EEPROM is a good idea, I will send a patch.
Thank you.
Best Regards, Wenyou Yang