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
next prev parent 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