public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 3/4] tegra: config: Enable FIT and device tree for all boards
Date: Mon, 01 Dec 2014 11:41:48 -0700	[thread overview]
Message-ID: <547CB66C.9070707@wwwdotorg.org> (raw)
In-Reply-To: <CAPnjgZ3+vnqbXA1S-LFOhv24wgGtWUo=Taf+Q+ZXsM4LZjff1Q@mail.gmail.com>

On 11/25/2014 10:44 AM, Simon Glass wrote:
> Hi Stephen,
>
> On 25 November 2014 at 09:23, Stephen Warren <swarren@wwwdotorg.org> wrote:
>> On 11/24/2014 04:49 PM, Simon Glass wrote:
>>>
>>> Hi Stephen,
>>>
>>> On 24 November 2014 at 10:11, Stephen Warren <swarren@wwwdotorg.org>
>>> wrote:
>>>>
>>>> On 11/23/2014 09:12 AM, Simon Glass wrote:
>>>>>
>>>>>
>>>>> Modern kernels require a device tree to boot.
>>>>
>>>> True.
>>>>
>>>>> Enable FIT support to permit
>>>>> booting these images, rather than just legacy images.
>>>>
>>>> I don't understand this? Modern kernels boot perfectly well without FIT
>>>> support. U-Boot supports the kernel's standard separate DTB and zImage
>>>> file formats just fine.
>>>>
>>>> To be honest, I'd strongly prefer not to enable support for non-universal
>>>> (bootloader-specific) formats such as FIT.
>>>
>>> In U-Boot? FIT is U-Boot's standard format
>>
>> That's rather my point: FIT is *U-Boot's* standard format, not a global
>> standard format.
>>
>> I want to strongly guide anyone using Tegra towards globally standard
>> formats, not intimate that they should be using bootloader-specific formats.
>>
>> zImage (without appended DTB) is a standard Linux format that all
>> booatloaders should support.
>>
>> Raw DTB in a separate file (or perhaps provided by earlier firmware directly
>> in RAM/ROM) is a standard Linux format that all bootloaders should support.
>>
>> extlinux.conf is something that all bootloaders should support.
>>
>> Linux distros that install binaries or config files in those standard
>> formats should expect them to work with any bootloader, on any board. This
>> way, distros won't have to write explicit support for any board; they'll
>> just install standard files and everything will just work anywhere.
>
> Just so I am clear, on ARM what is the list of bootloaders that you
> are concerned with? FIT is actually not a difficult thing to add to a
> boot loader. Presumably they all support libfdt, so it is just a case
> of plumbing in the FIT access stuff.

I believe that Barebox supports extlinux.conf too.

> I'm really not keen on this lowest-common-denominator approach, it's
> just a sad situation. Also I don't see why extlinux.conf should
> preclude people using FIT if they want to.

It doesn't. My point is that it's unlikely people will want FIT support 
except in certain specific cases (such as ChromeOS compatibility), and 
we shouldn't enable it except where there's a demonstrable use-case, so 
we don't confuse people with non-standard options and accidentally lead 
them down the wrong path.

...
>> Example /boot/extlinux.conf (for media-based booting) or
>> pxelinux.cfg/default (for network booting via syslinux command):
...
> How does U-Boot select which device tree to pass to the kernel with
> the scheme above? We shouldn't rely on the user, right? With FIT we
> use CONFIG_FIT_BEST_MATCH.

extlinux.conf can specify a particular DTB filename if it wants. 
Alternatively, it can specify a directory containing a set of DTBs, and 
the bootloader must select which DTB to load.

If $fdtfile is set in U-Boot's environment, that file is used.

Otherwise, the U-Boot syslinux code uses some other environment 
variables to calculate a default DTB filename; ${soc}-${board}.dtb.

  reply	other threads:[~2014-12-01 18:41 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-23 16:12 [U-Boot] [PATCH v3 1/4] dts: Bring in Chrome OS keyboard device tree definition Simon Glass
2014-11-23 16:12 ` [U-Boot] [PATCH v3 2/4] tegra: dts: Sync tegra124.dtsi with linux kernel Simon Glass
2014-11-24 17:09   ` Stephen Warren
2014-11-24 17:12     ` Simon Glass
2014-11-23 16:12 ` [U-Boot] [PATCH v3 3/4] tegra: config: Enable FIT and device tree for all boards Simon Glass
2014-11-24 17:11   ` Stephen Warren
2014-11-24 23:49     ` Simon Glass
2014-11-25 16:23       ` Stephen Warren
2014-11-25 17:44         ` Simon Glass
2014-12-01 18:41           ` Stephen Warren [this message]
2014-12-01 20:17             ` Simon Glass
2014-11-23 16:12 ` [U-Boot] [PATCH v3 4/4] ARM: tegra: Add support for nyan board Simon Glass
2014-11-23 17:02   ` Simon Glass
2014-11-24 17:21     ` Stephen Warren
2014-11-24 17:25     ` Stephen Warren
2014-11-24 17:28   ` Stephen Warren
2014-11-24 23:48     ` Simon Glass
2014-11-25 16:14       ` Stephen Warren
2014-11-25 16:18         ` 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=547CB66C.9070707@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --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