From: Wolfgang Grandegger <wg@grandegger•com>
To: Kumar Gala <galak@kernel•crashing.org>,
Wolfgang Grandegger <wg@grandegger•com>,
linuxppc-dev@ozlabs•org
Subject: Re: [PATCH v2 1/4] [POWERPC] 85xx: add board support for the TQM8548 modules
Date: Fri, 06 Jun 2008 12:09:59 +0200 [thread overview]
Message-ID: <48490CF7.6050205@grandegger.com> (raw)
In-Reply-To: <20080605225215.GF30980@yookeroo.seuss>
David Gibson wrote:
> On Thu, Jun 05, 2008 at 08:43:51AM -0500, Kumar Gala wrote:
>> On Jun 5, 2008, at 4:05 AM, Wolfgang Grandegger wrote:
> [snip]
>>> + timebase-frequency = <0>; // from U-Boot
>>> + bus-frequency = <0>; // from U-Boot
>>> + clock-frequency = <0>; // from U-Boot
>> u-boot will add this for us so no need for them.
>
> I think it's worth including them (with the comments) for
> documentation purposes though. Plus allowing the bootloader and
> bootwrapper to just replace property values without resizing or
> inserting is somewhat worthwhile of itself.
I understand your point and while testing the new blob, I realized that
the DTB image created with
$ make ARCH=powerpc tqm8548.dtb
does not work because work space is missing:
## Flattened Device Tree blob at 00900000
Booting using the fdt blob at 0x900000
Loading Device Tree to 007fe000, end 007ff8e0 ... OK
WARNING: could not create /chosen FDT_ERR_NOSPACE.
ERROR: /chosen node create failed - must RESET the board to recover.
As I see it, I have to define DTS_FLAGS manually
$ export DTS_FLAGS="-R4 -S0x3000"
$ make ARCH=powerpc tqm8548.dtb
to get a working blob. Is that the intended behavior?
Wolfgang.
next prev parent reply other threads:[~2008-06-06 10:10 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-05 9:05 [PATCH v2 1/4] [POWERPC] 85xx: add board support for the TQM8548 modules Wolfgang Grandegger
2008-06-05 13:43 ` Kumar Gala
2008-06-05 17:19 ` Wolfgang Grandegger
2008-06-05 22:52 ` David Gibson
2008-06-06 10:09 ` Wolfgang Grandegger [this message]
2008-06-06 13:59 ` Kumar Gala
2008-06-06 14:33 ` Wolfgang Denk
2008-06-06 14:51 ` Default flats for running dtc from kernel build Kumar Gala
2008-06-06 15:31 ` Jon Loeliger
2008-06-06 15:42 ` Kumar Gala
2008-06-06 15:44 ` Grant Likely
2008-06-06 16:08 ` Jon Loeliger
2008-06-06 21:39 ` Segher Boessenkool
2008-06-09 9:42 ` Peter Czanik
2008-06-16 3:00 ` Jerry Van Baren
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=48490CF7.6050205@grandegger.com \
--to=wg@grandegger$(echo .)com \
--cc=galak@kernel$(echo .)crashing.org \
--cc=linuxppc-dev@ozlabs$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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