public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 00/45] Kconfig: Move CONFIG_SPL_..._SUPPORT to Kconfig
Date: Tue, 13 Sep 2016 08:02:16 +0200	[thread overview]
Message-ID: <57D79668.5090906@denx.de> (raw)
In-Reply-To: <1473743943-15003-1-git-send-email-sjg@chromium.org>

Hello Simon,

Am 13.09.2016 um 07:18 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:
>
> 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:
>
> 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:
>
> 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):
>
> 01: buildman
>    blackfin:  +   cm-bf527 bf609-ezkit bf537-stamp
>       sparc:  +   grsim grsim_leon2 gr_cpci_ax2000 gr_xc3s_1500 gr_ep2s60
>       nios2:  +   10m50 3c120
> microblaze:  +   microblaze-generic
>    openrisc:  +   openrisc-generic
>
> [1] https://patchwork.ozlabs.org/patch/661511/
>
> Changes in v3:
> - Move SPL_NET_VCI_STRING into the SPL Kconfig file also
> - Rebase on master
>
> Changes in v2:
> - Add some notes on this option to moveconfig.py
> - Improve the commit message and add one for resyncing with savedefconfig
> - Add some notes on this option to moveconfig.py
> - Add new patch to convert CONFIG_SPL_NET_VCI_STRING
> - Drop CONFIG_SPL_PINCTRL_SUPPORT
> - Lots of work to make use of common values across multiple boards
> - Added a patch to change 'spear' CONFIGs to upper case
>
> Simon Glass (45):
>    Correct defconfigs using savedefconfig
>    moveconfig: Add an option to skip prompts
>    moveconfig: Add an option to commit changes
>    Kconfig: Move SPL settings into their own file
>    arm: fsl: Adjust ordering of #ifndef CONFIG_SPL_BUILD
>    Drop CONFIG_SPL_RAM_SUPPORT
>    Use separate options for TPL support
>    Kconfig: spl: Add SPL support options to Kconfig
>    Kconfig: tpl: Add some TPL support options to Kconfig
>    Move existing use of CONFIG_SPL_DM to Kconfig
>    Move existing use of CONFIG_SPL_RSA to Kconfig
>    spear: Use upper case for CONFIG options
>    Convert CONFIG_SPL_CRYPTO_SUPPORT to Kconfig
>    Convert CONFIG_SPL_HASH_SUPPORT to Kconfig
>    Convert CONFIG_SPL_DMA_SUPPORT to Kconfig
>    Convert CONFIG_SPL_DRIVERS_MISC_SUPPORT to Kconfig
>    Convert CONFIG_SPL_ENV_SUPPORT to Kconfig
>    Convert CONFIG_SPL_ETH_SUPPORT to Kconfig
>    Convert CONFIG_SPL_EXT_SUPPORT to Kconfig
>    Convert CONFIG_SPL_FAT_SUPPORT to Kconfig
>    Convert CONFIG_SPL_GPIO_SUPPORT to Kconfig
>    Convert CONFIG_SPL_I2C_SUPPORT to Kconfig
>    Convert CONFIG_SPL_LIBCOMMON_SUPPORT to Kconfig
>    Convert CONFIG_SPL_LIBDISK_SUPPORT to Kconfig
>    Convert CONFIG_SPL_LIBGENERIC_SUPPORT to Kconfig
>    Convert CONFIG_SPL_MMC_SUPPORT to Kconfig
>    Convert CONFIG_SPL_MPC8XXX_INIT_DDR_SUPPORT to Kconfig
>    Convert CONFIG_SPL_MTD_SUPPORT to Kconfig
>    Convert CONFIG_SPL_MUSB_NEW_SUPPORT to Kconfig
>    Convert CONFIG_SPL_NAND_SUPPORT to Kconfig
>    Convert CONFIG_SPL_NET_VCI_STRING to Kconfig
>    Convert CONFIG_SPL_NET_SUPPORT to Kconfig
>    Convert CONFIG_SPL_NOR_SUPPORT to Kconfig
>    Convert CONFIG_SPL_ONENAND_SUPPORT to Kconfig
>    Remove CONFIG_SPL_PINCTRL_SUPPORT
>    Convert CONFIG_SPL_POWER_SUPPORT to Kconfig
>    Convert CONFIG_SPL_SATA_SUPPORT to Kconfig
>    Convert CONFIG_SPL_SERIAL_SUPPORT to Kconfig
>    Convert CONFIG_SPL_SPI_FLASH_SUPPORT to Kconfig
>    Convert CONFIG_SPL_SPI_SUPPORT to Kconfig
>    Convert CONFIG_SPL_USBETH_SUPPORT to Kconfig
>    Convert CONFIG_SPL_USB_HOST_SUPPORT to Kconfig
>    Convert CONFIG_SPL_USB_SUPPORT to Kconfig
>    Convert CONFIG_SPL_WATCHDOG_SUPPORT to Kconfig
>    Convert CONFIG_SPL_YMODEM_SUPPORT to Kconfig

Many thanks for this big work!

Do you have this patches in a git repo?
I want to test this patches with tbot testcase:
https://github.com/hsdenx/tbot/blob/master/src/tc/uboot/tc_uboot_check_kconfig.py

if this series does not break boards ...

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-09-13  6:02 UTC|newest]

Thread overview: 97+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-13  5:18 [U-Boot] [PATCH v3 00/45] Kconfig: Move CONFIG_SPL_..._SUPPORT to Kconfig Simon Glass
2016-09-13  5:18 ` [U-Boot] [PATCH v3 01/45] Correct defconfigs using savedefconfig Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 02/45] moveconfig: Add an option to skip prompts Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 03/45] moveconfig: Add an option to commit changes Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 04/45] Kconfig: Move SPL settings into their own file Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 05/45] arm: fsl: Adjust ordering of #ifndef CONFIG_SPL_BUILD Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 06/45] Drop CONFIG_SPL_RAM_SUPPORT Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot,v3,06/45] " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 07/45] Use separate options for TPL support Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 08/45] Kconfig: spl: Add SPL support options to Kconfig Simon Glass
2016-09-13  5:18 ` [U-Boot] [PATCH v3 09/45] Kconfig: tpl: Add some TPL " Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 10/45] Move existing use of CONFIG_SPL_DM " Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 11/45] Move existing use of CONFIG_SPL_RSA " Simon Glass
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 12/45] spear: Use upper case for CONFIG options Simon Glass
2016-09-16 20:38   ` Tom Rini
2016-09-18 15:58   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 13/45] Convert CONFIG_SPL_CRYPTO_SUPPORT to Kconfig Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 14/45] Convert CONFIG_SPL_HASH_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 15/45] Convert CONFIG_SPL_DMA_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 16/45] Convert CONFIG_SPL_DRIVERS_MISC_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 17/45] Convert CONFIG_SPL_ENV_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 18/45] Convert CONFIG_SPL_ETH_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 19/45] Convert CONFIG_SPL_EXT_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 20/45] Convert CONFIG_SPL_FAT_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 21/45] Convert CONFIG_SPL_GPIO_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 22/45] Convert CONFIG_SPL_I2C_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 23/45] Convert CONFIG_SPL_LIBCOMMON_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 24/45] Convert CONFIG_SPL_LIBDISK_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 25/45] Convert CONFIG_SPL_LIBGENERIC_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 26/45] Convert CONFIG_SPL_MMC_SUPPORT " Simon Glass
2016-09-18 15:59   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 27/45] Convert CONFIG_SPL_MPC8XXX_INIT_DDR_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 28/45] Convert CONFIG_SPL_MTD_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 29/45] Convert CONFIG_SPL_MUSB_NEW_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 30/45] Convert CONFIG_SPL_NAND_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 31/45] Convert CONFIG_SPL_NET_VCI_STRING " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 32/45] Convert CONFIG_SPL_NET_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 33/45] Convert CONFIG_SPL_NOR_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 34/45] Convert CONFIG_SPL_ONENAND_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 35/45] Remove CONFIG_SPL_PINCTRL_SUPPORT Simon Glass
2016-09-13  5:28   ` Masahiro Yamada
2016-09-13 13:09     ` Simon Glass
2016-09-13  5:18 ` [U-Boot] [PATCH v3 36/45] Convert CONFIG_SPL_POWER_SUPPORT to Kconfig Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 37/45] Convert CONFIG_SPL_SATA_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 38/45] Convert CONFIG_SPL_SERIAL_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 39/45] Convert CONFIG_SPL_SPI_FLASH_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 40/45] Convert CONFIG_SPL_SPI_SUPPORT " Simon Glass
2016-09-18 16:00   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:18 ` [U-Boot] [PATCH v3 41/45] Convert CONFIG_SPL_USBETH_SUPPORT " Simon Glass
2016-09-18 16:01   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:19 ` [U-Boot] [PATCH v3 42/45] Convert CONFIG_SPL_USB_HOST_SUPPORT " Simon Glass
2016-09-18 16:01   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:19 ` [U-Boot] [PATCH v3 43/45] Convert CONFIG_SPL_USB_SUPPORT " Simon Glass
2016-09-18 16:01   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:19 ` [U-Boot] [PATCH v3 44/45] Convert CONFIG_SPL_WATCHDOG_SUPPORT " Simon Glass
2016-09-18 16:01   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  5:19 ` [U-Boot] [PATCH v3 45/45] Convert CONFIG_SPL_YMODEM_SUPPORT " Simon Glass
2016-09-18 16:01   ` [U-Boot] [U-Boot, v3, " Tom Rini
2016-09-13  6:02 ` Heiko Schocher [this message]
2016-09-13 13:08   ` [U-Boot] [PATCH v3 00/45] Kconfig: Move CONFIG_SPL_..._SUPPORT " Simon Glass
2016-09-14  5:59     ` Heiko Schocher
2016-09-18 15:58 ` Tom Rini
2016-09-19  0:56   ` 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=57D79668.5090906@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox