From: Lothar Felten <lothar.felten@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] package/dtc: fix include guards for older kernel/u-boot
Date: Mon, 08 Oct 2018 16:35:00 +0200 [thread overview]
Message-ID: <1539009300.1603.8.camel@gmail.com> (raw)
In-Reply-To: <CANQCQpYouTTsJGm6VjpCgdZ54agG0jktaNPwB9my88g_dL_Bsw@mail.gmail.com>
Hello Matthew,
> Lothar,
>
>
> On Mon, Oct 8, 2018 at 6:52 AM Lothar Felten <lothar.felten@gmail.com
> > wrote:
> >
> > Patch dtc to accept libfdt include guards with and without leading
> > underscore.
> > A compiler warning is added to notify the user.
> >
> > Signed-off-by: Lothar Felten <lothar.felten@gmail.com>
> > ---
> > ?...de-guards-for-older-kernel-u-boot-sources.patch | 42
> > ++++++++++++++++++++++
> > ?1 file changed, 42 insertions(+)
> > ?create mode 100644 package/dtc/0001-Fix-include-guards-for-older-
> > kernel-u-boot-sources.patch
> >
> > diff --git a/package/dtc/0001-Fix-include-guards-for-older-kernel-
> > u-boot-sources.patch b/package/dtc/0001-Fix-include-guards-for-
> > older-kernel-u-boot-sources.patch
> > new file mode 100644
> > index 0000000000..40cb0bdb2a
> > --- /dev/null
> > +++ b/package/dtc/0001-Fix-include-guards-for-older-kernel-u-boot-
> > sources.patch
> > @@ -0,0 +1,42 @@
> > +From b1f8b84489c96465b63485b884238b61d31ca84d Mon Sep 17 00:00:00
> > 2001
> > +From: Lothar Felten <lothar.felten@gmail.com>
> > +Date: Mon, 8 Oct 2018 13:29:44 +0200
> > +Subject: [PATCH 1/1] Fix include guards for older kernel/u-boot
> > sources
> > +
> > +Linux kernels before 4.17 and U-Boot versions before 2018.07 use
> > libfdt include guards with leading underscores.
> > +Those have been removed in dtc-1.4.7.
>
> Is there an reference that could be listed to either
> 1) A upstream mailinglist/patchwork link that proposes adding this
> patch
> 2) If the patch below can't be upstreamed, a reference to the commit
> that is reverting this support you're adding back in and maybe a
> summary of why the backwards compatibility isn't' accepted.
This patch is a temporary fix for this issue:
https://git.buildroot.org/buildroot/tree/CHANGES#n35
It does not fix any bug, just an incompatibility of a new dtc and older
Linux kernels / u-boot loaders.
Once all Linux kernel versions are >=4.17 and u-boot versions are
>=2018.07 this patch is no longer required: they will all use the
include guard without underscore.
I could also patch the beaglebone kernel, but other targets have the
same issue (e.g. bananapi_m1).
>
> Matt
Lothar
next prev parent reply other threads:[~2018-10-08 14:35 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-10-08 11:51 [Buildroot] [PATCH 1/1] package/dtc: fix include guards for older kernel/u-boot Lothar Felten
2018-10-08 13:10 ` Matthew Weber
2018-10-08 14:35 ` Lothar Felten [this message]
2018-10-08 15:54 ` Arnout Vandecappelle
2018-10-08 15:58 ` Yann E. MORIN
2018-10-08 16:02 ` Thomas Petazzoni
2018-10-08 16:32 ` Yann E. MORIN
2018-10-08 19:21 ` Arnout Vandecappelle
2018-10-08 19:31 ` Yann E. MORIN
2018-10-09 8:22 ` Arnout Vandecappelle
2018-10-09 8:07 ` Lothar Felten
2018-10-20 11:37 ` Arnout Vandecappelle
2018-10-24 12:17 ` Peter Korsgaard
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=1539009300.1603.8.camel@gmail.com \
--to=lothar.felten@gmail.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.