All of lore.kernel.org
 help / color / mirror / Atom feed
From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 00/44] Kconfig: Move CONFIG_SPL_..._SUPPORT to Kconfig
Date: Tue, 30 Aug 2016 07:28:27 +0200	[thread overview]
Message-ID: <57C5197B.10900@denx.de> (raw)
In-Reply-To: <1472516512-26412-1-git-send-email-sjg@chromium.org>

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

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...

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

  parent reply	other threads:[~2016-08-30  5:28 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-30  0:21 [U-Boot] [PATCH v2 00/44] Kconfig: Move CONFIG_SPL_..._SUPPORT to Kconfig Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 01/44] Correct defconfigs using savedefconfig Simon Glass
2016-09-05  2:47   ` Masahiro Yamada
2016-09-06 18:01     ` Tom Rini
2016-09-06 18:08       ` Simon Glass
2016-09-07  4:03       ` Masahiro Yamada
2016-09-07 13:05         ` Tom Rini
2016-09-08  9:24           ` Masahiro Yamada
2016-08-30  0:21 ` [U-Boot] [PATCH v2 02/44] moveconfig: Add an option to skip prompts Simon Glass
2016-08-30  3:41   ` Masahiro Yamada
2016-08-30  0:21 ` [U-Boot] [PATCH v2 03/44] moveconfig: Add an option to commit changes Simon Glass
2016-08-30  4:00   ` Masahiro Yamada
2016-08-30  0:21 ` [U-Boot] [PATCH v2 04/44] Kconfig: Move SPL settings into their own file Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 05/44] arm: fsl: Adjust ordering of #ifndef CONFIG_SPL_BUILD Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 06/44] Drop CONFIG_SPL_RAM_SUPPORT Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 07/44] Use separate options for TPL support Simon Glass
2016-09-06 18:06   ` Tom Rini
2016-09-06 18:12     ` Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 08/44] Kconfig: spl: Add SPL support options to Kconfig Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 09/44] Kconfig: tpl: Add some TPL " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 10/44] Move existing use of CONFIG_SPL_DM " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 11/44] Move existing use of CONFIG_SPL_RSA " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 12/44] spear: Use upper case for CONFIG options Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 13/44] Convert CONFIG_SPL_CRYPTO_SUPPORT to Kconfig Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 14/44] Convert CONFIG_SPL_HASH_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 15/44] Convert CONFIG_SPL_DMA_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 16/44] Convert CONFIG_SPL_DRIVERS_MISC_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 17/44] Convert CONFIG_SPL_ENV_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 18/44] Convert CONFIG_SPL_ETH_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 19/44] Convert CONFIG_SPL_EXT_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 20/44] Convert CONFIG_SPL_FAT_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 21/44] Convert CONFIG_SPL_GPIO_SUPPORT " Simon Glass
2016-08-30  3:25   ` Masahiro Yamada
2016-08-30 18:33     ` Simon Glass
2016-09-02 14:35       ` Tom Rini
2016-09-05  2:40         ` Masahiro Yamada
2016-09-06  1:04           ` Simon Glass
2016-09-06 15:54             ` Tom Rini
2016-09-07  4:55               ` Masahiro Yamada
2016-09-12  4:16               ` Simon Glass
2016-09-12  4:32                 ` Masahiro Yamada
2016-09-12 14:19                   ` Tom Rini
2016-08-30  0:21 ` [U-Boot] [PATCH v2 22/44] Convert CONFIG_SPL_I2C_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 23/44] Convert CONFIG_SPL_LIBCOMMON_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 24/44] Convert CONFIG_SPL_LIBDISK_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 25/44] Convert CONFIG_SPL_LIBGENERIC_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 26/44] Convert CONFIG_SPL_MMC_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 27/44] Convert CONFIG_SPL_MPC8XXX_INIT_DDR_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 28/44] Convert CONFIG_SPL_MTD_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 29/44] Convert CONFIG_SPL_MUSB_NEW_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 30/44] Convert CONFIG_SPL_NAND_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 31/44] Convert CONFIG_SPL_NET_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 32/44] Convert CONFIG_SPL_NOR_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 33/44] Convert CONFIG_SPL_ONENAND_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 34/44] Convert CONFIG_SPL_PINCTRL_SUPPORT " Simon Glass
2016-08-30  3:31   ` Masahiro Yamada
2016-08-30  0:21 ` [U-Boot] [PATCH v2 35/44] Convert CONFIG_SPL_POWER_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 36/44] Convert CONFIG_SPL_SATA_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 37/44] Convert CONFIG_SPL_SERIAL_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 38/44] Convert CONFIG_SPL_SPI_FLASH_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 39/44] Convert CONFIG_SPL_SPI_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 40/44] Convert CONFIG_SPL_USBETH_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 41/44] Convert CONFIG_SPL_USB_HOST_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 42/44] Convert CONFIG_SPL_USB_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 43/44] Convert CONFIG_SPL_WATCHDOG_SUPPORT " Simon Glass
2016-08-30  0:21 ` [U-Boot] [PATCH v2 44/44] Convert CONFIG_SPL_YMODEM_SUPPORT " Simon Glass
2016-08-30  5:28 ` Heiko Schocher [this message]
2016-09-06  1:04   ` [U-Boot] [PATCH v2 00/44] Kconfig: Move CONFIG_SPL_..._SUPPORT " Simon Glass

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=57C5197B.10900@denx.de \
    --to=hs@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.