All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.