public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [ANN] U-Boot v2017.07-rc2 released
Date: Sat, 8 Jul 2017 08:21:40 -0400	[thread overview]
Message-ID: <20170708122140.GT22707@bill-the-cat> (raw)
In-Reply-To: <CALeDE9MigvOeytqAJJ=PbeUwKP_Mz61gEG6FJF2-dPoi452WEA@mail.gmail.com>

On Sat, Jul 08, 2017 at 09:27:59AM +0100, Peter Robinson wrote:
> On Wed, Jul 5, 2017 at 8:37 AM, Peter Robinson <pbrobinson@gmail.com> wrote:
> >> On 21 June 2017 at 01:07, Peter Robinson <pbrobinson@gmail.com> wrote:
> >>>>>>> Simon, do you have some suggestions on what to do here?  Thanks!
> >>>>>>>
> >>>>>>> --
> >>>>>>> Tom
> >>>>>>
> >>>>>> My guess is that there is already a libfdt.py in the system. Someone
> >>>>>> else reported this too.
> >>>>>>
> >>>>>> We could perhaps change the ordering in PYTHONPATH so that our one is first.
> >>>>>
> >>>>> No, I'm not sure that's completely the case because I first saw a
> >>>>> related issue before my dtc had the python patch set added to it, I
> >>>>> would actually prefer to build with the distro dtc rather than a fork
> >>>>> of upstream like we use to.
> >>>>
> >>>> OK I think I see what is happening then. It seems to be picking up
> >>>> _libfdt.so from your system and libfdy.py from U-Boot. If so that
> >>>> seems like a bad idea at the best of times.
> >>>>
> >>>> Despite upstreaming efforts we still have local libfdt changes in
> >>>> U-Boot. The main one is fdtgrep. I did try to upstream it a while back
> >>>> but failed. I've been thinking of trying again but have not mustered
> >>>> the energy.
> >>>>
> >>>> This particular error could probably be worked around in the short
> >>>> term by dropping FDT_ERR_TOODEEP. But do we really want to allow this
> >>>> sort of thing? I think we should either use one libfdt module or the
> >>>> other, not a mixture of the two
> >>>
> >>> I suspect your right but I don't want to get into a situation where
> >>> something might work in the kernel and and not in u-boot or
> >>> vice-versa, and as things like overlays come into play where they
> >>> could be applied to either the possible differences get greater and
> >>> from a distro PoV that increased the requirements of support and debug
> >>> and believe me people will do weird shit that they expect you to
> >>> magically fix with little information hence my reluctance to diverge.
> >>
> >> I'm not sure what to do about this other than what I suggested. I
> >> wonder it if is possible to detect the case where there is a mismatch
> >> with the installation?
> >
> > Was that dropping FDT_ERR_TOODEEP? Why do we diverge from upstream on
> > this, what does it do? Maybe provide an option to specify whether to
> > use external dtc or bundled?
> 
> So dropping dtc from our deps to try and get anything on 2017.07 built
> I get for a bunch of devices I get this:
> 
> /builddir/build/BUILD/u-boot-2017.07-rc3/scripts/dtc-version.sh: line
> 18: dtc: command not found
> rm -f .tmp_quiet_recordmcount
> *** Your dtc is too old, please upgrade to dtc 1.4 or newer
> 
> Can we please have some level of resolution for 2017.07 GA?

Can we short term do the thing where I guess it was PYTHONPATH gets
modified so that you only pick up U-Boot provided parts here?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170708/0203bfc5/attachment.sig>

  reply	other threads:[~2017-07-08 12:21 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-20  0:47 [U-Boot] [ANN] U-Boot v2017.07-rc2 released Tom Rini
2017-06-20 10:15 ` Peter Robinson
2017-06-20 11:19   ` Tom Rini
2017-06-20 18:26     ` Simon Glass
2017-06-20 23:02       ` Peter Robinson
2017-06-21  1:27         ` Simon Glass
2017-06-21  7:07           ` Peter Robinson
2017-07-04 19:33             ` Simon Glass
2017-07-05  7:37               ` Peter Robinson
2017-07-08  8:27                 ` Peter Robinson
2017-07-08 12:21                   ` Tom Rini [this message]
2017-07-09  9:45                     ` Peter Robinson
2017-07-09 12:49                       ` Tom Rini
2017-07-09 18:38                         ` Simon Glass
2017-07-09 22:36                           ` Tom Rini
2017-07-10  3:31                             ` Simon Glass
2017-07-10  8:26                               ` Peter Robinson
2017-07-10 16:38                                 ` Simon Glass
2017-06-23 16:19 ` Andreas Färber
2017-07-04 19:33   ` Simon Glass
  -- strict thread matches above, loose matches on Subject: below --
2017-06-25  9:13 Kostya Porotchkin

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=20170708122140.GT22707@bill-the-cat \
    --to=trini@konsulko.com \
    --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