From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] u-boot: 2015.07 - fix creation of .config
Date: Mon, 17 Aug 2015 11:31:30 +0200 [thread overview]
Message-ID: <20150817093130.GA3988@free.fr> (raw)
In-Reply-To: <1439793449.3667.7.camel@synopsys.com>
Alexey, All,
On 2015-08-17 06:37 +0000, Alexey Brodkin spake thusly:
> On Sun, 2015-08-16 at 23:43 +0200, J?rg Krause wrote:
> > Hi Thomas, Alexey,
> >
> > On Mi, 2015-08-12 at 10:23 +0200, Thomas Petazzoni wrote:
> > > Dear Alexey Brodkin,
> > >
> > > On Tue, 11 Aug 2015 12:20:01 +0000, Alexey Brodkin wrote:
> > >
> > > > Any chance for this one to be applied?
> > > > Note without this change or anything similar U-Boot from
> > > > snsp_axs10x_defconfig fails to config.
> > >
> > > I haven't had the time to look into this in detail. It would be great
> > > if Yann, or J?rg could have a look. I remain unhappy with solving
> > > such
> > > a problem by using a patch against U-Boot 2015.07, because it means
> > > anyone specifying a slightly different U-Boot version (but based on
> > > 2015.07) will still have the problem.
> >
> > It looks like upstream [1] does not revert this patch but fixes the
> > problem with this patch [2]. Alexey, does this patch solves your
> > problem?
> >
> > Best regards
> > J?rg Krause
> >
> >
> > [1]
> > https://www.mail-archive.com/u-boot at lists.denx.de/msg179681.html
> >
> > [2]
> > http://patchwork.ozlabs.org/patch/502793/
>
> The thing is [2] solves problem only for ARM, while other arches are still
> affected. And that means ARC is not cured by that patch.
>
> Indeed proper fix is to patch all arches in the same way,
> i.e. select default option for each and every architecture in U-Boot.
>
> But speaking about Buildroot that approach has at least 2 downsides:
> 1) Not all arches are fixed upstream so in Buildroot we'll have patches
> that are not there in upstream yet.
>
> 2) We'll have quite a few patches in Buildroot for U-boot each fixing
> a separate arch.
>
> If (1) and (2) are OK for Buildroot I'll try to come up with those patches ASAP.
>
> My initial approach was IMHO the cleanest and simplest. But if it doesn't work
> we'll do another fix :)
OK, chiming in...
I guess removing the "optional" keywork would be the simplest solution,
indeed.
But then I'm afraid that defconfig files bundled with U-Boot will no
longer work. If they still work, then I'm OK for having a local patch
that reverts upstream's (until upstream fixes all of their archs).
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
next prev parent reply other threads:[~2015-08-17 9:31 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-31 14:04 [Buildroot] [PATCH] u-boot: 2015.07 - fix creation of .config Alexey Brodkin
2015-07-31 14:18 ` Thomas Petazzoni
2015-07-31 15:14 ` Alexey Brodkin
2015-08-03 11:16 ` Alexey Brodkin
2015-08-11 12:20 ` Alexey Brodkin
2015-08-12 8:23 ` Thomas Petazzoni
2015-08-16 21:43 ` Jörg Krause
2015-08-17 6:37 ` Alexey Brodkin
2015-08-17 9:31 ` Yann E. MORIN [this message]
2015-08-17 9:58 ` Alexey Brodkin
2015-08-22 22:57 ` Arnout Vandecappelle
2015-08-28 9:36 ` Alexey Brodkin
2015-08-30 20:41 ` Peter Korsgaard
2015-08-30 20:51 ` Peter Korsgaard
2015-08-31 16:43 ` Alexey Brodkin
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=20150817093130.GA3988@free.fr \
--to=yann.morin.1998@free.fr \
--cc=buildroot@busybox.net \
/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