public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Bruens <stefan.bruens@rwth-aachen.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/9] Switch bcm283x platform to use OF_CONTROL
Date: Fri, 7 Oct 2016 23:59:02 +0200	[thread overview]
Message-ID: <4387203.cBMEeMVPVY@pebbles.site> (raw)
In-Reply-To: <20161007204240.GK4884@bill-the-cat>

On Freitag, 7. Oktober 2016 16:42:40 CEST Tom Rini wrote:
> On Thu, Oct 06, 2016 at 09:42:52PM -0700, Eric Anholt wrote:
> > Stefan Bruens <stefan.bruens@rwth-aachen.de> writes:
> > > On Donnerstag, 6. Oktober 2016 12:32:12 CEST Eric Anholt wrote:
> > >> Alexander Graf <agraf@suse.de> writes:
> > >> >> Am 05.10.2016 um 18:48 schrieb Fabian Vogt <fvogt@suse.com>:
> > >> >> 
> > >> >> Hi,
> > >> >> 
> > >> >> Am Mittwoch, 5. Oktober 2016, 09:54:46 CEST schrieb Stephen Warren:
> > >> >>> On 09/26/2016 06:26 AM, Fabian Vogt wrote:
> > >> >>>> This patch series modifies the used drivers to work with
> > >> >>>> OF_CONTROL
> > >> >>>> and switches the board code and configs to use it.
> > >> >>>> The added device trees are directly from the linux kernel tree
> > >> >>>> and can thus be used for booting the (upstream) kernel.
> > >> >>> 
> > >> >>> Is there a user-visible or developer-visible benefit to this
> > >> >>> change? In
> > >> >>> general, converting to use DT to instantiate devices simply ends up
> > >> >>> using more code (and hence complexity and time) to get to the exact
> > >> >>> same
> > >> >>> state afterwards.
> > >> >> 
> > >> >> There are various reasons, like:
> > >> >> 
> > >> >> - The device tree describes the platform, so it can also be used by
> > >> >> the
> > >> >> 
> > >> >>  linux kernel for configuration (no separate dtb needed)
> > >> > 
> > >> > With a bit of lobbying, we might even be able to get a working dt
> > >> > from
> > >> > the rpi firmware. That again would enable awesome things like hat
> > >> > support in u-boot :).
> > >> 
> > >> I can't imagine the firmware handing any DT off to the kernel other
> > >> than
> > >> the one that's being shipped from whatever kernel is being loaded.
> > >> Being able to update the DT in lockstep with the kernel is very much
> > >> part of their process.  (This makes linux insisting that DT is ABI that
> > >> must maintain backwards compat quite painful for upstreaming)
> > >> 
> > >> Given that the firmware already hands the kernel's DT back to the
> > >> kernel, it seems like we should be able to load the kernel's DT too if
> > >> we wanted that.
> > > 
> > > You mean the DT modified by the firmware according to config.txt, at
> > > least
> > > applying the specified DT overlays?
> > 
> > Yeah, that.
> 
> That would also be a good thing, yes.

My primary concern here was the bcm2708-rpi-b.dtb vs bcm2835-rpi-b.dtb 
devicetrees, i.e. the one distributed by the RPi foundation vs the upstream 
kernel one.

This concern may no longer be valid, https://www.raspberrypi.org/
documentation/configuration/device-tree.md#part3 says:
---
The loader now supports builds using bcm2835_defconfig, which selects the 
upstreamed BCM2835 support. This configuration will cause bcm2835-rpi-b.dtb 
and bcm2835-rpi-b-plus.dtb to be built. If these files are copied with the 
kernel, and if the kernel has been tagged by a recent mkknlimg, then the 
loader will attempt to load one of those DTBs by default.
---

Open points:
1. is u-boot.bin tagged in an appropriate way (i.e. as done by mkknlimg)?
2. what about the other RPis, i.e. RPi2, RPI3, RPi1-CM?

Kind regards,

Stefan

-- 
Stefan Br?ns  /  Bergstra?e 21  /  52062 Aachen
home: +49 241 53809034     mobile: +49 151 50412019
work: +49 2405 49936-424

  reply	other threads:[~2016-10-07 21:59 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-26 12:26 [U-Boot] [PATCH 0/9] Switch bcm283x platform to use OF_CONTROL Fabian Vogt
2016-09-26 12:26 ` [U-Boot] [PATCH 1/9] gpio: bcm2835: add device tree support Fabian Vogt
2016-09-27  0:32   ` Simon Glass
2016-11-29 17:33   ` [U-Boot] [U-Boot,1/9] " Tom Rini
2016-09-26 12:26 ` [U-Boot] [PATCH 2/9] serial: bcm283x_mu: " Fabian Vogt
2016-09-27  0:34   ` Simon Glass
2016-11-29 17:34   ` [U-Boot] [U-Boot, " Tom Rini
2016-09-26 12:26 ` [U-Boot] [PATCH 3/9] serial: pl01x: expose skip_init platdata option in DT Fabian Vogt
2016-09-27  0:34   ` Simon Glass
2016-11-29 17:34   ` [U-Boot] [U-Boot, " Tom Rini
2016-09-26 12:26 ` [U-Boot] [PATCH 4/9] fdt: add dt-bindings for bcm2835 Fabian Vogt
2016-09-27  0:34   ` Simon Glass
2016-11-29 17:34   ` [U-Boot] [U-Boot,4/9] " Tom Rini
2016-09-26 12:26 ` [U-Boot] [PATCH 5/9] fdt: import bcm283x device tree sources from the linux kernel tree Fabian Vogt
2016-09-27  0:34   ` Simon Glass
2016-11-29 17:35   ` [U-Boot] [U-Boot, " Tom Rini
2016-09-26 12:26 ` [U-Boot] [PATCH 6/9] fdt: adjust bcm283x device tree for u-boot Fabian Vogt
2016-09-27  0:35   ` Simon Glass
2016-11-29 17:35   ` [U-Boot] [U-Boot, " Tom Rini
2016-09-26 12:26 ` [U-Boot] [PATCH 7/9] serial: bcm283x_mu: support disabling after initialization Fabian Vogt
2016-09-27  0:35   ` Simon Glass
2016-11-29 17:36   ` [U-Boot] [U-Boot, " Tom Rini
2016-09-26 12:26 ` [U-Boot] [PATCH 8/9] board: rpi: move uart deactivation to board_init Fabian Vogt
2016-09-27  0:35   ` Simon Glass
2016-11-29 17:37   ` [U-Boot] [U-Boot, " Tom Rini
2016-09-26 12:26 ` [U-Boot] [PATCH 9/9] ARM: bcm283x: use OF_CONTROL for bcm283x Fabian Vogt
2016-09-27  0:35   ` Simon Glass
2016-11-29 17:37   ` [U-Boot] [U-Boot,9/9] " Tom Rini
2016-10-05 15:54 ` [U-Boot] [PATCH 0/9] Switch bcm283x platform to use OF_CONTROL Stephen Warren
2016-10-05 16:48   ` Fabian Vogt
2016-10-05 16:51     ` Simon Glass
2016-10-05 20:28     ` Alexander Graf
2016-10-06 19:32       ` Eric Anholt
2016-10-06 23:22         ` Stefan Bruens
2016-10-07  4:42           ` Eric Anholt
2016-10-07 20:42             ` Tom Rini
2016-10-07 21:59               ` Stefan Bruens [this message]
2016-11-16 13:43 ` Fabian Vogt
2016-11-25 19:45   ` Simon Glass

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=4387203.cBMEeMVPVY@pebbles.site \
    --to=stefan.bruens@rwth-aachen.de \
    --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