mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Alexander Dahl <ada@thorsis.com>
To: ptxdist@pengutronix.de
Cc: Michael Olbrich <m.olbrich@pengutronix.de>,
	Ahmad Fatoum <a.fatoum@pengutronix.de>
Subject: Re: [ptxdist] [PATCH 2/7] u-boot: add support for oldconfig/menuconfig
Date: Tue, 16 Oct 2018 10:02:45 +0200	[thread overview]
Message-ID: <5482787.aF19sAydvr@ada> (raw)
In-Reply-To: <20181016075255.hsgbolfjjmyo5lcn@pengutronix.de>

Hei hei,

Am Dienstag, 16. Oktober 2018, 09:52:55 CEST schrieb Michael Olbrich:
> On Wed, Oct 10, 2018 at 02:03:33PM +0200, Ahmad Fatoum wrote:
> > while at it, remove U_BOOT_PATH, because it's PATH=$(CROSS_PATH) already
> > by default.
> > 
> > Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> > ---
> > 
> >  rules/u-boot.make | 42 ++++++++++++++++++++++++++++++++++--------
> >  1 file changed, 34 insertions(+), 8 deletions(-)
> > 
> > diff --git a/rules/u-boot.make b/rules/u-boot.make
> > index aa7a01ca8fea..46cfc192825a 100644
> > --- a/rules/u-boot.make
> > +++ b/rules/u-boot.make
> > @@ -26,6 +26,9 @@ U_BOOT_SOURCE	:= $(SRCDIR)/$(U_BOOT).$(U_BOOT_SUFFIX)
> > 
> >  U_BOOT_DIR	:= $(BUILDDIR)/$(U_BOOT)
> >  U_BOOT_LICENSE	:= GPL-2.0-only
> > 
> > +U_BOOT_CONFIG	:= $(call ptx/in-platformconfigdir, \
> > +		$(call remove_quotes, $(PTXCONF_U_BOOT_CONFIG)))
> 
> So, I'm pretty sure that the u-boot 'config' was simply a make target in
> the past, not a config file.
> Has this changed for all target or do we need an option to support both?

You may need both. U-Boot has started introducing KConfig some time ago and is 
in the (long) process of migrating all options to it. What you provide as make 
target to recent U-Boot versions is actually a KConfig defconfig which 
overwrites your .config and which is somehow magically merged with 
preprocessor defines in includes/configs/yourboardname.h until the migration 
process is done. (Correct me, if I understood that wrong.)

What I currently do is on a u-boot clone apart from ptxdist: menuconfig, then 
savedefconfig, copy that over, commit in my u-boot tree, export patches and 
use it as a defconfig make target in ptxdist.

Greets
Alex


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

  reply	other threads:[~2018-10-16  8:02 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-10 12:03 [ptxdist] [PATCH 1/7] u-boot: specify GPL-2.0-only as LICENSE Ahmad Fatoum
2018-10-10 12:03 ` [ptxdist] [PATCH 2/7] u-boot: add support for oldconfig/menuconfig Ahmad Fatoum
2018-10-16  7:52   ` Michael Olbrich
2018-10-16  8:02     ` Alexander Dahl [this message]
2018-10-16  8:25   ` Michael Olbrich
2018-10-10 12:03 ` [ptxdist] [PATCH 3/7] u-boot: add V=$(PTXDIST_VERBOSE) to make options Ahmad Fatoum
2018-10-11  6:52   ` Alexander Dahl
2018-10-10 12:03 ` [ptxdist] [PATCH 4/7] u-boot: run install(1)/rm(1) with -v Ahmad Fatoum
2018-10-10 12:03 ` [ptxdist] [PATCH 5/7] u-boot: add u-boot{-dtb, -with-spl-pbl}.bin image installation options Ahmad Fatoum
2019-02-05  7:30   ` Alexander Dahl
2019-02-05  9:29     ` Ahmad Fatoum
2018-10-10 12:03 ` [ptxdist] [PATCH 6/7] u-boot: allow generation of custom environment binary Ahmad Fatoum
2018-10-16  8:13   ` Michael Olbrich
2018-10-16 10:58   ` Denis OSTERLAND
2018-10-16 11:27     ` Alexander Dahl
2018-10-16 12:26       ` Michael Olbrich
2018-10-16 13:13         ` Alexander Dahl
2019-04-04 14:17         ` Alexander Dahl
2018-10-16 11:20   ` Alexander Dahl
2018-10-10 12:03 ` [ptxdist] [PATCH 7/7] u-boot: allow generation of boot script Ahmad Fatoum
2018-10-11  6:44 ` [ptxdist] [PATCH 1/7] u-boot: specify GPL-2.0-only as LICENSE Alexander Dahl
2018-10-11  9:05   ` Ahmad Fatoum

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=5482787.aF19sAydvr@ada \
    --to=ada@thorsis.com \
    --cc=a.fatoum@pengutronix.de \
    --cc=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.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