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