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] FDT: Question about embedded fdt enable
Date: Wed, 30 May 2012 10:28:37 -0600	[thread overview]
Message-ID: <4FC64AB5.7050103@wwwdotorg.org> (raw)
In-Reply-To: <CAPnjgZ0xrsLhxJNFKOUF9JB2a5Ftuk-tnN-c_aHyL14mf1+zWQ@mail.gmail.com>

On 05/30/2012 08:26 AM, Simon Glass wrote:
> Hi Donghwa,
> 
> On Wed, May 30, 2012 at 12:37 AM, Donghwa Lee <dh09.lee@samsung.com> wrote:
...
>> And I have one more question about passing *.dtb to kernel.
>> I had make a .dtb binary by using CONFIG_OF_SEPARATE and confirmed
>> normally working in u-boot.
>> When kernel is booting, how could kernel access .dtb binary?
>> Could kernel access dtb binary by setting /memreserve/ in u-boot .dts file
>> or by using "fdtaddrcontrol"
>> environment variable in u-boot board file?
>> But, by my result, dt_phys address in setup_machine_fdt() in
>> arch/arm/devtree.c had always indicated only
>> one point regardless of /memreserve/ or fdtcontroladdr.
>>
> 
> Please be very careful - this is a completely separate FDT. The use of
> CONFIG_OF_CONTROL is for U-Boot's internal configuration only. The same FDT
> should not be passed to Linux in the normal case. Please continue to use
> the existing FDT support for passing a .dtb file to the kernel.
> 
> So you have an FDT for U-Boot which is available via CONFIG_OF_SEPARATE,
> and an FDT which U-Boot passes to the kernel, which is read using the
> normal uImage/Fit image method.

I echo this sentiment.

There is a very strong desire in the kernel community that the .dtb file
and the kernel image be stored in similar ways - e.g. they both be
stored as files in a /boot filesystem, or both be stored as specific
partitions in some flash. The reason for this is so that both the kernel
and .dtb image can be updated in the exact same way at the same time,
and without changing the bootloader/firmware/... in any way at all. This
allows kernel upgrades that rely on .dtb upgrades, and furthermore, it
allows that without updating the firmware/bootloader, which may be risky
in some cases.

      reply	other threads:[~2012-05-30 16:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-23  7:39 [U-Boot] FDT: Question about embedded fdt enable Donghwa Lee
2012-05-30  6:13 ` Simon Glass
2012-05-30  7:37   ` Donghwa Lee
2012-05-30 14:26     ` Simon Glass
2012-05-30 16:28       ` Stephen Warren [this message]

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=4FC64AB5.7050103@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