From: Jeroen Hofstee <dasuboot@myspectrum.nl>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v8 05/13] kconfig: switch to Kconfig
Date: Thu, 31 Jul 2014 23:06:19 +0200 [thread overview]
Message-ID: <53DAAFCB.5000103@myspectrum.nl> (raw)
In-Reply-To: <53DAAD40.8010204@wwwdotorg.org>
On 31-07-14 22:55, Stephen Warren wrote:
> 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.
>
there is, it is _terribly_ slow when dealing with warnings in the build.
MAKEALL is much
better for that. Replacing the _config rule with _defconfig is just a
bad idea afaic.
Regards,
Jeroen
next prev parent reply other threads:[~2014-07-31 21:06 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
2014-07-31 21:06 ` Jeroen Hofstee [this message]
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=53DAAFCB.5000103@myspectrum.nl \
--to=dasuboot@myspectrum.nl \
--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