public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v8 05/13] kconfig: switch to Kconfig
Date: Thu, 31 Jul 2014 14:55:28 -0600	[thread overview]
Message-ID: <53DAAD40.8010204@wwwdotorg.org> (raw)
In-Reply-To: <20140731203453.GV19374@bill-the-cat>

On 07/31/2014 02:34 PM, Tom Rini wrote:
> On Wed, Jul 30, 2014 at 08:08:02PM -0600, Stephen Warren wrote:
>> On 07/30/2014 07:56 PM, Masahiro Yamada wrote:
>>> Hi Stephen,
>>>
>>>
>>> On Wed, 30 Jul 2014 17:05:21 -0600
>>> Stephen Warren <swarren@wwwdotorg.org> wrote:
>>>
>>>> On 07/29/2014 11:08 PM, Masahiro Yamada wrote:
>>>>> This commit enables Kconfig.
>>>>> Going forward, we use Kconfig for the board configuration.
>>>>> mkconfig will never be used. Nor will include/config.mk be generated.
>>>>>
>>>>> Kconfig must be adjusted for U-Boot because our situation is
>>>>> a little more complicated than Linux Kernel.
>>>>> We have to generate multiple boot images (Normal, SPL, TPL)
>>>>> from one source tree.
>>>>> Each image needs its own configuration input.
>>>>>
>>>>> Usage:
>>>>>
>>>>> Run "make <board>_defconfig" to do the board configuration.
>>>>
>>>> This is quite unfortunate; it breaks any scripts that were building U-Boot via "make <board>_config; make". Can't we add another rule to allow the old build commands to work?
>>>
>>>
>>> Technically, yes. I think we can.
>>>
>>> But I do not like having it permanently.
>>>
>>>
>>> So, we support both *_defconfig and *_config for a while (maybe 6 months or so?)
>>> and then remove *_config.
>>>
>>> Deal?
>>
>> If the old command-line is ever going to be removed, there's no point
>> supporting both at all; I'd have to hack my scripts to support both
>> sometime, so I may as well do it now rather than wait.
>>
>>>> Otherwise, I guess I'll have to hack my scripts to check whether e.g. scripts/multiconfig.py (which was added in this commit) is present in the tree, and execute different build commands based on that...
>>>
>>>
>>> Do you mean, you need to build some different versions of U-boot ?
>>
>> Yes. I own some scripts that build U-Boot, and they need to work on any
>> reasonable version of U-Boot that anyone might want to build. For
>> example, they build 2014.07 just fine, and there's no reason they should
>> ever stop being able to do that. I obviously also want my scripts to be
>> able to build any future version of U-Boot.
>
> So long as we have MAKEALL (and we'll have the discussion about moving
> to buildman sometime soon) this just becomes:
> if [ -x tools/genboardscfg.py ]; then
>     tools/genboardscfg.py
> fi
>
> MAKEALL machine-name

There's now a large disadvantage to MAKEALL; it takes longer to run that 
to build U-Boot itself, since it must auto-generate boards.cfg. Perhaps 
that's only done once, or when the data changes.

Also, is MAKEALL an/the official way to build U-Boot, or just a useful 
tool? I have the impression that running "make" is the official way to 
build U-Boot, but MAKEALL/buildman are useful wrapper tools if you want 
to do batch builds etc.

  reply	other threads:[~2014-07-31 20:55 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-30  5:08 [U-Boot] [PATCH v8 0/13] Kconfig for U-Boot Masahiro Yamada
2014-07-30  5:08 ` [U-Boot] [PATCH v8 01/13] kconfig: import Kconfig files from Linux 3.16-rc7 Masahiro Yamada
2014-07-30 19:14   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 02/13] kconfig: add board Kconfig and defconfig files Masahiro Yamada
2014-07-30 19:14   ` Tom Rini
2014-08-06 18:24   ` York Sun
2014-08-07  6:59     ` Masahiro Yamada
2014-08-07 16:49       ` York Sun
2014-07-30  5:08 ` [U-Boot] [PATCH v8 03/13] kconfig: add basic Kconfig files Masahiro Yamada
2014-07-30 19:14   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 04/13] include: define CONFIG_SPL and CONFIG_TPL as 1 Masahiro Yamada
2014-07-30 19:14   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 05/13] kconfig: switch to Kconfig Masahiro Yamada
2014-07-30 19:14   ` Tom Rini
2014-07-30 23:05   ` Stephen Warren
2014-07-31  1:56     ` Masahiro Yamada
2014-07-31  2:08       ` Stephen Warren
2014-07-31 20:34         ` Tom Rini
2014-07-31 20:55           ` Stephen Warren [this message]
2014-07-31 21:06             ` Jeroen Hofstee
2014-07-30  5:08 ` [U-Boot] [PATCH v8 06/13] Add board MAINTAINERS files Masahiro Yamada
2014-07-30 19:15   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 07/13] tools: add genboardscfg.py Masahiro Yamada
2014-07-30 14:07   ` Tom Rini
2014-07-30 15:12     ` Simon Glass
2014-07-30 15:22       ` Tom Rini
2014-07-30 15:38         ` Simon Glass
2014-07-31  2:22     ` Masahiro Yamada
2014-07-31 12:17       ` Tom Rini
2014-07-30 19:15   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 08/13] kconfig: remove mkconfig and boards.cfg Masahiro Yamada
2014-07-30 19:15   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 09/13] MAKEALL: adjust for Kconfig Masahiro Yamada
2014-07-30 19:15   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 10/13] buildman: " Masahiro Yamada
2014-07-30 19:15   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 11/13] kconfig: delete redundant CONFIG_${ARCH} definition Masahiro Yamada
2014-07-30 19:15   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 12/13] powerpc: remove redundant CPU definition Masahiro Yamada
2014-07-30  5:34   ` Stefan Roese
2014-07-30 19:15   ` Tom Rini
2014-07-30  5:08 ` [U-Boot] [PATCH v8 13/13] include: remove CONFIG_SPL/CONFIG_TPL definition in config headers Masahiro Yamada
2014-07-30 19:15   ` Tom Rini
2014-07-30 19:16 ` [U-Boot] [PATCH v8 0/13] Kconfig for U-Boot Tom Rini
2014-07-31 22:19   ` 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=53DAAD40.8010204@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --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