public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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