On Jun 6, 2008, at 5:09 AM, Wolfgang Grandegger wrote:
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?
this is normal. Andy Fleming has posted a patch to u-boot to help
improve the situation. I'm not sure if WD has picked it up for 1.3.4
yet.
- k
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
