Hi Heiko, On 29 August 2016 at 23:28, Heiko Schocher <h...@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@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@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
