Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] configs: beaglebone: update kernel to 4.1.13 and u-boot to 2015.10
Date: Fri, 15 Jan 2016 09:21:51 +0100	[thread overview]
Message-ID: <20160115092151.5452f93c@free-electrons.com> (raw)
In-Reply-To: <56987FDD.5060508@gmail.com>

Zoltan,

On Fri, 15 Jan 2016 06:13:01 +0100, Zoltan Gyarmati wrote:

> I looked into this a bit more:
> * The u-boot doesn't need the host DTC (even though the
> BR2_TARGET_UBOOT_NEEDS_DTC is set in the patch i sent, something i'll
> need to fix in the next iteration)
> 
> * As the shipped .dtb files are built with the dtc version included in
> the linux tree from https://github.com/beagleboard/linux.git, which
> has the support for the overlays, so for merely getting a booting
> system, no need to hack the dtc package
> 
> * As the kernel build also installs this dtc version as
> 'output/host/usr/bin/linux-dtc', if one needs to compile overlays on
> the build host, s/he can use that one (although this need to be
> documented at least in the board's README)

So in fact, for the purpose of the beaglebone defconfig itself, we
don't need to package an overlay-capable DTC, right?

If that's the case, then I would suggest that we separate the problem
in two parts: first get an updated beaglebone defconfig merged, then
work on packaging an overlay-capable DTC.

> So as far i can see the options are (and none of them is ideal):
> 
> 1. Implementing a similar solution as Antoine with an additional
> dtc-overlay package with a custom upstream repo.
> 
> 2. Stick to the current package and upstream, but introduce a config
> variable like "BR2_PACKAGE_DTC_SUPPORT_OVERLAY", and if it's set, the
> needed patchset is applied (or just using an other upstream)
> 
> 3. Create a virtual package and 2 implementations as dtc-kernel-org
> and a dtc-overlay
> 
> 4. Only add the needed patches on the boards where needed and patch
> dtc by setting BR2_GLOBAL_PATCH_DIR
> 
> 5. Introducing custom repository/custom patch dir config options for
> the dtc package like for u-boot and linux (maybe also with an option
> to use the dtc source coming with the kernel tree?)

Do we really need to package the two variants of DTC ? The
overlay-capable version is perfectly backward compatible with the
mainline DTC. So why don't we simply include in package/dtc/ the
patches needed to make DTC overlay-capable ?

Yes, this is moving away from our general rule of not including patches
that add new features. But general rules are meant to have exceptions,
and in this specific case, the upstream DTC project is so slow at
integrating major changes such as overlays, and this feature is now
being used so often (BeagleBone, Raspberry Pi, CHIP, etc.) that it
makes sense to support it.

Arnout ? Antoine ?

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2016-01-15  8:21 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-13  2:52 [Buildroot] [PATCH 1/1] configs: beaglebone: update kernel to 4.1.13 and u-boot to 2015.10 Zoltan Gyarmati
2016-01-13  8:51 ` Thomas Petazzoni
2016-01-14 21:41   ` Arnout Vandecappelle
2016-01-15  5:13     ` Zoltan Gyarmati
2016-01-15  8:21       ` Thomas Petazzoni [this message]
2016-01-16 13:36         ` Zoltan Gyarmati
2016-01-16 14:45           ` Chris LaRocque
2016-01-15  1:53   ` Zoltan Gyarmati
2016-03-08 16:36 ` Thomas Petazzoni
2016-05-25 12:35 ` Thomas Petazzoni

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=20160115092151.5452f93c@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox