[U-Boot] [PATCH v2 00/44] Kconfig: Move CONFIG_SPL_..._SUPPORT to Kconfig

Simon Glass sjg at chromium.org
Tue Sep 6 03:04:17 CEST 2016


Hi Heiko,

On 29 August 2016 at 23:28, Heiko Schocher <hs at denx.de> wrote:
> Hello Simon,
>
>
> Am 30.08.2016 um 02:21 schrieb Simon Glass:
>>
>> This series moves all the CONFIG_SPL_..._SUPPORT options to Kconfig and
>> fixes up existing boards to continue to build.
>>
>> It also adds a few small but useful features to moveconfig.
>>
>> There is existing work going on in this area, so some of these patches may
>> be superseded. It has taken me a while to get this building cleanly. But I
>> have run out of time so want to get this out.
>>
>> As mentioned on a recent thread [1] there is some confusion about whether
>> an
>> option means enabling driver support or media support. Andrew's recent
>> series seems like a good vehicle to tidy that up. But I hope this series
>> will make it easier.
>>
>> NOTE: in the v2 series I have tried to use common things in Kconfig to
>> reduce the diffs in the defconfig files. This has helped a fair bit. But
>> it
>> is very error-prone and time consuming. Also I have had to add some
>> exceptions (disabling an option in specific board configs). Overall it was
>> not a pleasant experience :-(
>>
>> There are a few strange features of this conversion. The main difficulty
>> is
>> that some PowerPC boards do things like this in their board config file:
>>
>>   #ifdef CONFIG_TPL_BUILD
>>   #define CONFIG_SPL_SERIAL_SUPPORT
>>   #define CONFIG_SPL_ENV_SUPPORT
>>   #else
>>   #define CONFIG_SPL_SERIAL_SUPPORT
>>   #endif
>>
>> This means that TPL reuses the SPL options. We can't support this in
>> Kconfig
>> so I have added a small number of CONFIG_TPL_xxx_SUPPORT options to cope
>> with this. This made the conversion more painful than it should have been.
>>
>> A related issue is boards using a common header file and setting options
>> only
>> for SPL:
>>
>>   #ifdef CONFIG_SPL_BUILD
>>   #define CONFIG_SPL_SERIAL_SUPPORT
>>   #endif
>>
>> This is not noticed by moveconfig so we have to clean it up manually. Also
>> there are a few incorrect things where Kconfig options are set with
>> #define:
>>
>>   #ifdef CONFIG_SPL_BUILD
>>   #define CONFIG_SPL_DM
>>   #endif
>>
>> Finally, many defconfig files are not ordered correctly, resulting in
>> larger
>> patches than we might like. It would be great to have a solution for this,
>> perhaps with buildman providing a warning. But it might slow down
>> development.
>>
>> The series is fully build-tested (for bisectability) and causes no
>> failures
>> for the boards that already pass. The following boards fail for me at
>> present on mainline (which I have not yet looked at):
>>
>>    openrisc:  +   openrisc-generic
>>       sparc:  +   grsim grsim_leon2 gr_cpci_ax2000 gr_xc3s_1500 gr_ep2s60
>>    blackfin:  +   bf609-ezkit
>>
>> This build is also config tested (with buildman -K). The changes are with
>> spear (to rename options) and am335x_evm_usbspl (which I think is
>> correct).
>>
>> boards.cfg is up to date. Nothing to do.
>> Summary of 49 commits for 1185 boards (32 threads, 1 job per thread)
>> 01: cfg
>>    openrisc:  +   openrisc-generic
>>       sparc:  +   grsim grsim_leon2 gr_cpci_ax2000 gr_xc3s_1500 gr_ep2s60
>>    blackfin:  +   bf609-ezkit
>> 02: move
>> 03: buildman
>> 04: Correct defconfigs using savedefconfig
>> 05: moveconfig: Add an option to skip prompts
>> 06: moveconfig: Add an option to commit changes
>> 07: Kconfig: Move SPL settings into their own file
>> 08: arm: fsl: Adjust ordering of #ifndef CONFIG_SPL_BUILD
>> 09: Drop CONFIG_SPL_RAM_SUPPORT
>> 10: Use separate options for TPL support
>> 11: Kconfig: spl: Add SPL support options to Kconfig
>> 12: Kconfig: tpl: Add some TPL support options to Kconfig
>> 13: Move existing use of CONFIG_SPL_DM to Kconfig
>> 14: Move existing use of CONFIG_SPL_RSA to Kconfig
>> 15: spear: Use upper case for CONFIG options
>> arm:
>>     + u-boot.cfg: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1 CONFIG_STV0991=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000 CONFIG_USBTTY=1
>>     - u-boot.cfg: CONFIG_DW_UDC=1 CONFIG_ENV_IS_IN_NAND=1
>> CONFIG_ENV_OFFSET=0x60000 CONFIG_ENV_RANGE=0x10000 CONFIG_SPEAR_USBTTY=1
>> CONFIG_USBD_HS=1 CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_nand=1 CONFIG_spear300=1 CONFIG_spear310=1 CONFIG_spear320=1
>> CONFIG_spear600=1 CONFIG_stv0991=1 CONFIG_usbtty=1
>>     + all: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1 CONFIG_STV0991=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000 CONFIG_USBTTY=1
>>     - all: CONFIG_DW_UDC=1 CONFIG_ENV_IS_IN_NAND=1
>> CONFIG_ENV_OFFSET=0x60000 CONFIG_ENV_RANGE=0x10000 CONFIG_SPEAR_USBTTY=1
>> CONFIG_USBD_HS=1 CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_nand=1 CONFIG_spear300=1 CONFIG_spear310=1 CONFIG_spear320=1
>> CONFIG_spear600=1 CONFIG_stv0991=1 CONFIG_usbtty=1
>> spear600_nand :
>>     + u-boot.cfg: CONFIG_NAND=1
>>     - u-boot.cfg: CONFIG_nand=1 CONFIG_spear600=1
>>     + all: CONFIG_NAND=1
>>     - all: CONFIG_nand=1 CONFIG_spear600=1
>> spear300_nand :
>>     + u-boot.cfg: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000
>>     - u-boot.cfg: CONFIG_ENV_IS_IN_NAND=1 CONFIG_ENV_OFFSET=0x60000
>> CONFIG_ENV_RANGE=0x10000 CONFIG_nand=1 CONFIG_spear300=1
>>     + all: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000
>>     - all: CONFIG_ENV_IS_IN_NAND=1 CONFIG_ENV_OFFSET=0x60000
>> CONFIG_ENV_RANGE=0x10000 CONFIG_nand=1 CONFIG_spear300=1
>> spear300_usbtty :
>>     + u-boot.cfg: CONFIG_USBTTY=1
>>     - u-boot.cfg: CONFIG_DW_UDC=1 CONFIG_SPEAR_USBTTY=1 CONFIG_USBD_HS=1
>> CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_spear300=1 CONFIG_usbtty=1
>>     + all: CONFIG_USBTTY=1
>>     - all: CONFIG_DW_UDC=1 CONFIG_SPEAR_USBTTY=1 CONFIG_USBD_HS=1
>> CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_spear300=1 CONFIG_usbtty=1
>> spear320_usbtty_nand :
>>     + u-boot.cfg: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000 CONFIG_USBTTY=1
>>     - u-boot.cfg: CONFIG_DW_UDC=1 CONFIG_ENV_IS_IN_NAND=1
>> CONFIG_ENV_OFFSET=0x60000 CONFIG_ENV_RANGE=0x10000 CONFIG_SPEAR_USBTTY=1
>> CONFIG_USBD_HS=1 CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_nand=1 CONFIG_spear320=1 CONFIG_usbtty=1
>>     + all: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000 CONFIG_USBTTY=1
>>     - all: CONFIG_DW_UDC=1 CONFIG_ENV_IS_IN_NAND=1
>> CONFIG_ENV_OFFSET=0x60000 CONFIG_ENV_RANGE=0x10000 CONFIG_SPEAR_USBTTY=1
>> CONFIG_USBD_HS=1 CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_nand=1 CONFIG_spear320=1 CONFIG_usbtty=1
>> spear310_usbtty_nand :
>>     + u-boot.cfg: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000 CONFIG_USBTTY=1
>>     - u-boot.cfg: CONFIG_DW_UDC=1 CONFIG_ENV_IS_IN_NAND=1
>> CONFIG_ENV_OFFSET=0x60000 CONFIG_ENV_RANGE=0x10000 CONFIG_SPEAR_USBTTY=1
>> CONFIG_USBD_HS=1 CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_nand=1 CONFIG_spear310=1 CONFIG_usbtty=1
>>     + all: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000 CONFIG_USBTTY=1
>>     - all: CONFIG_DW_UDC=1 CONFIG_ENV_IS_IN_NAND=1
>> CONFIG_ENV_OFFSET=0x60000 CONFIG_ENV_RANGE=0x10000 CONFIG_SPEAR_USBTTY=1
>> CONFIG_USBD_HS=1 CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_nand=1 CONFIG_spear310=1 CONFIG_usbtty=1
>> spear310_usbtty spear310_usbtty_pnor :
>>     + u-boot.cfg: CONFIG_USBTTY=1
>>     - u-boot.cfg: CONFIG_DW_UDC=1 CONFIG_SPEAR_USBTTY=1 CONFIG_USBD_HS=1
>> CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_spear310=1 CONFIG_usbtty=1
>>     + all: CONFIG_USBTTY=1
>>     - all: CONFIG_DW_UDC=1 CONFIG_SPEAR_USBTTY=1 CONFIG_USBD_HS=1
>> CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_spear310=1 CONFIG_usbtty=1
>> spear300 :
>>     - u-boot.cfg: CONFIG_spear300=1
>>     - all: CONFIG_spear300=1
>> spear600_usbtty :
>>     + u-boot.cfg: CONFIG_USBTTY=1
>>     - u-boot.cfg: CONFIG_spear600=1 CONFIG_usbtty=1
>>     + all: CONFIG_USBTTY=1
>>     - all: CONFIG_spear600=1 CONFIG_usbtty=1
>> spear320_nand :
>>     + u-boot.cfg: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000
>>     - u-boot.cfg: CONFIG_ENV_IS_IN_NAND=1 CONFIG_ENV_OFFSET=0x60000
>> CONFIG_ENV_RANGE=0x10000 CONFIG_nand=1 CONFIG_spear320=1
>>     + all: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000
>>     - all: CONFIG_ENV_IS_IN_NAND=1 CONFIG_ENV_OFFSET=0x60000
>> CONFIG_ENV_RANGE=0x10000 CONFIG_nand=1 CONFIG_spear320=1
>> stv0991 :
>>     + u-boot.cfg: CONFIG_STV0991=1
>>     - u-boot.cfg: CONFIG_stv0991=1
>>     + all: CONFIG_STV0991=1
>>     - all: CONFIG_stv0991=1
>> spear320 spear320_pnor :
>>     - u-boot.cfg: CONFIG_spear320=1
>>     - all: CONFIG_spear320=1
>> spear310_nand :
>>     + u-boot.cfg: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000
>>     - u-boot.cfg: CONFIG_ENV_IS_IN_NAND=1 CONFIG_ENV_OFFSET=0x60000
>> CONFIG_ENV_RANGE=0x10000 CONFIG_nand=1 CONFIG_spear310=1
>>     + all: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000
>>     - all: CONFIG_ENV_IS_IN_NAND=1 CONFIG_ENV_OFFSET=0x60000
>> CONFIG_ENV_RANGE=0x10000 CONFIG_nand=1 CONFIG_spear310=1
>> spear320_usbtty spear320_usbtty_pnor :
>>     + u-boot.cfg: CONFIG_USBTTY=1
>>     - u-boot.cfg: CONFIG_DW_UDC=1 CONFIG_SPEAR_USBTTY=1 CONFIG_USBD_HS=1
>> CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_spear320=1 CONFIG_usbtty=1
>>     + all: CONFIG_USBTTY=1
>>     - all: CONFIG_DW_UDC=1 CONFIG_SPEAR_USBTTY=1 CONFIG_USBD_HS=1
>> CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_spear320=1 CONFIG_usbtty=1
>> spear600 :
>>     - u-boot.cfg: CONFIG_spear600=1
>>     - all: CONFIG_spear600=1
>> spear300_usbtty_nand :
>>     + u-boot.cfg: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000 CONFIG_USBTTY=1
>>     - u-boot.cfg: CONFIG_DW_UDC=1 CONFIG_ENV_IS_IN_NAND=1
>> CONFIG_ENV_OFFSET=0x60000 CONFIG_ENV_RANGE=0x10000 CONFIG_SPEAR_USBTTY=1
>> CONFIG_USBD_HS=1 CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_nand=1 CONFIG_spear300=1 CONFIG_usbtty=1
>>     + all: CONFIG_ENV_ADDR=(CONFIG_SYS_FLASH_BASE +
>> CONFIG_SYS_MONITOR_LEN) CONFIG_ENV_IS_IN_FLASH=1
>> CONFIG_ENV_SECT_SIZE=0x00010000 CONFIG_NAND=1
>> CONFIG_SYS_MONITOR_LEN=0x00040000 CONFIG_USBTTY=1
>>     - all: CONFIG_DW_UDC=1 CONFIG_ENV_IS_IN_NAND=1
>> CONFIG_ENV_OFFSET=0x60000 CONFIG_ENV_RANGE=0x10000 CONFIG_SPEAR_USBTTY=1
>> CONFIG_USBD_HS=1 CONFIG_USBD_MANUFACTURER="ST Microelectronics"
>> CONFIG_USBD_PRODUCT_NAME="SPEAr SoC" CONFIG_USB_DEVICE=1 CONFIG_USB_TTY=1
>> CONFIG_nand=1 CONFIG_spear300=1 CONFIG_usbtty=1
>> spear310 spear310_pnor :
>>     - u-boot.cfg: CONFIG_spear310=1
>>     - all: CONFIG_spear310=1
>> spear600_usbtty_nand :
>>     + u-boot.cfg: CONFIG_NAND=1 CONFIG_USBTTY=1
>>     - u-boot.cfg: CONFIG_nand=1 CONFIG_spear600=1 CONFIG_usbtty=1
>>     + all: CONFIG_NAND=1 CONFIG_USBTTY=1
>>     - all: CONFIG_nand=1 CONFIG_spear600=1 CONFIG_usbtty=1
>> 16: Convert CONFIG_SPL_CRYPTO_SUPPORT to Kconfig
>> 17: Convert CONFIG_SPL_HASH_SUPPORT to Kconfig
>> 18: Convert CONFIG_SPL_DMA_SUPPORT to Kconfig
>> 19: Convert CONFIG_SPL_DRIVERS_MISC_SUPPORT to Kconfig
>> 20: Convert CONFIG_SPL_ENV_SUPPORT to Kconfig
>> 21: Convert CONFIG_SPL_ETH_SUPPORT to Kconfig
>> 22: Convert CONFIG_SPL_EXT_SUPPORT to Kconfig
>> 23: Convert CONFIG_SPL_FAT_SUPPORT to Kconfig
>> 24: Convert CONFIG_SPL_GPIO_SUPPORT to Kconfig
>> 25: Convert CONFIG_SPL_I2C_SUPPORT to Kconfig
>> 26: Convert CONFIG_SPL_LIBCOMMON_SUPPORT to Kconfig
>> 27: Convert CONFIG_SPL_LIBDISK_SUPPORT to Kconfig
>> 28: Convert CONFIG_SPL_LIBGENERIC_SUPPORT to Kconfig
>> 29: Convert CONFIG_SPL_MMC_SUPPORT to Kconfig
>> 30: Convert CONFIG_SPL_MPC8XXX_INIT_DDR_SUPPORT to Kconfig
>> 31: Convert CONFIG_SPL_MTD_SUPPORT to Kconfig
>> 32: Convert CONFIG_SPL_MUSB_NEW_SUPPORT to Kconfig
>> 33: Convert CONFIG_SPL_NAND_SUPPORT to Kconfig
>> 34: Convert CONFIG_SPL_NET_SUPPORT to Kconfig
>> 35: Convert CONFIG_SPL_NOR_SUPPORT to Kconfig
>> 36: Convert CONFIG_SPL_ONENAND_SUPPORT to Kconfig
>> 37: Convert CONFIG_SPL_PINCTRL_SUPPORT to Kconfig
>> 38: Convert CONFIG_SPL_POWER_SUPPORT to Kconfig
>> 39: Convert CONFIG_SPL_SATA_SUPPORT to Kconfig
>> 40: Convert CONFIG_SPL_SERIAL_SUPPORT to Kconfig
>> 41: Convert CONFIG_SPL_SPI_FLASH_SUPPORT to Kconfig
>> 42: Convert CONFIG_SPL_SPI_SUPPORT to Kconfig
>> 43: Convert CONFIG_SPL_USBETH_SUPPORT to Kconfig
>> arm:
>>     + u-boot.cfg: CONFIG_SPL_ENV_SUPPORT=1 CONFIG_SPL_NET_SUPPORT=1
>>     + u-boot-spl.cfg: CONFIG_SPL_MMC_SUPPORT=1 CONFIG_SPL_NAND_SUPPORT=1
>>     + all: CONFIG_SPL_ENV_SUPPORT=1 CONFIG_SPL_MMC_SUPPORT=1
>> CONFIG_SPL_NAND_SUPPORT=1 CONFIG_SPL_NET_SUPPORT=1
>> am335x_evm_usbspl :
>>     + u-boot.cfg: CONFIG_SPL_ENV_SUPPORT=1 CONFIG_SPL_NET_SUPPORT=1
>>     + u-boot-spl.cfg: CONFIG_SPL_MMC_SUPPORT=1 CONFIG_SPL_NAND_SUPPORT=1
>>     + all: CONFIG_SPL_ENV_SUPPORT=1 CONFIG_SPL_MMC_SUPPORT=1
>> CONFIG_SPL_NAND_SUPPORT=1 CONFIG_SPL_NET_SUPPORT=1
>> 44: Convert CONFIG_SPL_USB_HOST_SUPPORT to Kconfig
>> 45: Convert CONFIG_SPL_USB_SUPPORT to Kconfig
>> 46: Convert CONFIG_SPL_WATCHDOG_SUPPORT to Kconfig
>> 47: Convert CONFIG_SPL_YMODEM_SUPPORT to Kconfig
>> 48: Kconfig: Add a whitelist of ad-hoc CONFIG options
>> 49: Makefile: Give a build error if ad-hoc CONFIG options are added
>
>
> Your patchserie has 44 patches .. here buildman test 49 ...
> It seems to me the first 3 patches and the last 2 are missing in
> your series ...
>
> Do you have a public git tree with this patches?
> Ah, may:
> http://git.denx.de/?p=u-boot/u-boot-dm.git;a=shortlog;h=refs/heads/kconfig-working3

Yes. I always push a working branch before sending patches, but don't
always mention it :-(

>
> Ok, I started tbot tc:
> https://github.com/hsdenx/tbot/blob/master/src/tc/uboot/tc_uboot_check_kconfig.py
> to check, if your series does not change u-boot.bin and spl/uboot-spl.bin
>
> first outputs:
>
> hs at localhost:tbot  [master] $ python2.7 src/common/tbot.py -c
> config/tbot_uboot_kconfig_check.cfg -t tc_uboot_check_kconfig.py -l
> log/tbot_checkconfig.log
> **** option cfg: config/tbot_uboot_kconfig_check.cfg log:
> log/tbot_checkconfig.log tc: tc_uboot_check_kconfig.py v 0
> ('CUR WORK PATH: ', '/home/hs/data/Entwicklung/tbot')
> ('CFGFILE ', 'config/tbot_uboot_kconfig_check.cfg')
> ('LOGFILE ', '/home/hs/data/Entwicklung/tbot/log/tbot_checkconfig.log')
> get u-boot source
> testing board 10m50 1 / 1189
> testing board 3c120 2 / 1189
> testing board A10-OLinuXino-Lime 3 / 1189
> A10-OLinuXino-Lime u-boot spl bin diff 6443bde88d3f4dd2e4a03ea0b5b23d40 !=
> ccf133653c402c471644492201854d31
> testing board A10s-OLinuXino-M 4 / 1189
> A10s-OLinuXino-M u-boot spl bin diff 620fe3f1f87e6a4d283e088b20eaaabb !=
> eba0ced2452df73c8c832217bce4e255
> testing board A13-OLinuXino 5 / 1189
> A13-OLinuXino u-boot spl bin diff 9524cb79bda360baa2e59982e792b62d !=
> e65c6fa3842baf6b2964c29c3d92afcd
> testing board A13-OLinuXinoM 6 / 1189
> A13-OLinuXinoM u-boot spl bin diff accae01a1f32f4ab8d5fe178b347fa19 !=
> 153716412d66dc33790293d340c6fdbd
> testing board A20-Olimex-SOM-EVB 7 / 1189
> A20-Olimex-SOM-EVB u-boot spl bin diff ccb7e2e6515af8511220af50886bc539 !=
> 530e7d17fda711516b733a899f70b353
> testing board A20-OLinuXino-Lime2 8 / 1189
> A20-OLinuXino-Lime2 u-boot spl bin diff 75b6f0b647be28c085f78adef374799d !=
> 7ca441f27790846d8ffad383de1a60c1
> testing board A20-OLinuXino-Lime 9 / 1189
> A20-OLinuXino-Lime u-boot spl bin diff 7086f5ffd5faf8ea144e7a2abb288c47 !=
> 6a2805806c25aad683c8e6de71041133
> testing board A20-OLinuXino_MICRO 10 / 1189
> A20-OLinuXino_MICRO u-boot spl bin diff c0669e033d627abb96d04209abaa8eb8 !=
> 76663a41fc8b33d748d2de2c4c18749c
> testing board a3m071 11 / 1189
> a3m071 u-boot spl bin diff 770b372a12ed027051933f00a87e123b !=
> e5e1f8335c1ef3faf66ebba031c8d296
> testing board a4m072 12 / 1189
> testing board a4m2k 13 / 1189
> a4m2k u-boot spl bin diff ead2adaf8f425857ebaea243af5768bc !=
> cb0afb3ef300c953255f7f4d2a644373
> testing board ac14xx 14 / 1189
> testing board acadia 15 / 1189
> testing board adp-ag101p 16 / 1189
> testing board Ainol_AW1 17 / 1189
> Ainol_AW1 u-boot spl bin diff 9b22f5439fdef333932be5948db0a0c1 !=
> d9bdcfb6af5100c9a1830b5a3ace992f
> testing board alt 18 / 1189
> testing board am335x_baltos 19 / 1189
> am335x_baltos u-boot spl bin diff 103f8a788ff657ec0dbc21be1e89789e !=
> dbd4ca792ccb091698d09756d6703f09
> testing board am335x_boneblack 20 / 1189
> testing board am335x_boneblack second compile failed
>
> log here says:
> 2016-08-30 07:24:26,280:CON    :tbotlib   # tb_ctrl2: common/spl/spl_nand.c:
> In function 'spl_nand_load_image':^M
> common/spl/spl_nand.c:95:23: error: 'CONFIG_CMD_SPL_NAND_OFS' undeclared
> (first use in this function)^M
> common/spl/spl_nand.c:95:23: note: each undeclared identifier is reported
> only once for each function it appears in^M
> common/spl/spl_nand.c:96:4: error: 'CONFIG_CMD_SPL_WRITE_SIZE' undeclared
> (first use in this function)^M
> common/spl/spl_nand.c:108:23: error: 'CONFIG_SYS_NAND_SPL_KERNEL_OFFS'
> undeclared (first use in this function)^M
> common/spl/spl_nand.c:136:30: error: 'CONFIG_SYS_NAND_U_BOOT_OFFS'
> undeclared (first use in this function)^M
> scripts/Makefile.build:280: recipe for target 'spl/common/spl/spl_nand.o'
> failed^M
> make[2]: *** [spl/common/spl/spl_nand.o] Error 1^M
> make[2]: *** Waiting for unfinished jobs....^M
> scripts/Makefile.spl:300: recipe for target 'spl/common/spl' failed^M
> make[1]: *** [spl/common/spl] Error 2^M
> make[1]: *** Waiting for unfinished jobs....^M
> Makefile:1334: recipe for target 'spl/u-boot-spl' failed^M
> make: *** [spl/u-boot-spl] Error 2^M
> make: *** Waiting for unfinished jobs....^M
> hs at pollux [ 7:24:26] ttbott>
>
> testing board am335x_boneblack_vboot 21 / 1189
>
> all boards takes a while ... but it seems there is an issue with SPL
> builds...

I'm not sure what is going on here. My build was clean.

I'll try again with v3.

>
> bye,
> Heiko
> --
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Regards,
Simon


More information about the U-Boot mailing list