* [PATCH 0/2] libfdt: problems with real life blobs
@ 2007-02-19 18:00 Jerry Van Baren
2007-02-19 23:46 ` David Gibson
0 siblings, 1 reply; 3+ messages in thread
From: Jerry Van Baren @ 2007-02-19 18:00 UTC (permalink / raw)
To: David Gibson; +Cc: linuxppc-dev
Hi David,
I've been trying to use your libfdt in u-boot and my first step, get a
value from the blob, failed terminally. After poking about a bit, it
appears that your libfdt and Jon Loeliger's dtc (-V 16) disagree with
respect to the format of the blob - libfdt won't traverse the path.
I've created two patches:
1) Make the libfdt tests use "fdt endian" so that dtc can be used.
2) Create a minimal test tree and compile it with dtc.
The first patch is clean, the second patch is a bit of a hack job, I did
just enough to check this out and confirm/deny my suspicions that libfdt
doesn't like the dtc format. Running the tests on the dtc-compiled blob
shows the same problems with traversing paths.
I think I created the correct tree in test_tree1.dts (I didn't do the
truncated node, but that is immaterial for my primary objective) but I
could be wrong...
Supporting using dtc in the long run as well as the assembly-generated
blob would be nice so that regression tests can be done on different
versions of the blob format (both supported and unsupported) and to make
sure libfdt and dtc stay in sync.
Best regards,
gvb
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/2] libfdt: problems with real life blobs
2007-02-19 18:00 [PATCH 0/2] libfdt: problems with real life blobs Jerry Van Baren
@ 2007-02-19 23:46 ` David Gibson
2007-02-20 13:02 ` Jerry Van Baren
0 siblings, 1 reply; 3+ messages in thread
From: David Gibson @ 2007-02-19 23:46 UTC (permalink / raw)
To: Jerry Van Baren; +Cc: linuxppc-dev
On Mon, Feb 19, 2007 at 01:00:57PM -0500, Jerry Van Baren wrote:
> Hi David,
>
> I've been trying to use your libfdt in u-boot and my first step, get a
> value from the blob, failed terminally. After poking about a bit, it
> appears that your libfdt and Jon Loeliger's dtc (-V 16) disagree with
> respect to the format of the blob - libfdt won't traverse the path.
That's certainly bad.
> I've created two patches:
Ug. Damn. Trouble is, for IBM paranoid procedural reasons, I'm not
really in a position to accept patches for libfdt from outside at
present. I'm already working on fixing that, but there being a
ponderous bureaucracy involved, it's going to take a little while.
Oh, plus I want to relicense libfdt (so it can be used in non-GPL
firmware), so for that reason also I can't accept random patches at
present.
> 1) Make the libfdt tests use "fdt endian" so that dtc can be used.
That sounds sensible, I was always pretty dubious about using a
different endianness for the framing and content in the tests.
> 2) Create a minimal test tree and compile it with dtc.
>
> The first patch is clean, the second patch is a bit of a hack job, I did
> just enough to check this out and confirm/deny my suspicions that libfdt
> doesn't like the dtc format. Running the tests on the dtc-compiled blob
> shows the same problems with traversing paths.
>
> I think I created the correct tree in test_tree1.dts (I didn't do the
> truncated node, but that is immaterial for my primary objective) but I
> could be wrong...
>
> Supporting using dtc in the long run as well as the assembly-generated
> blob would be nice so that regression tests can be done on different
> versions of the blob format (both supported and unsupported) and to make
> sure libfdt and dtc stay in sync.
Yes, absolutely. I want to merge libfdt and dtc into a single
package, so they can be used to test each other. Again, planning to
do that just as soon as I get the bureaucratic hurdles out of the way.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/2] libfdt: problems with real life blobs
2007-02-19 23:46 ` David Gibson
@ 2007-02-20 13:02 ` Jerry Van Baren
0 siblings, 0 replies; 3+ messages in thread
From: Jerry Van Baren @ 2007-02-20 13:02 UTC (permalink / raw)
To: David Gibson, linuxppc-dev
David Gibson wrote:
> On Mon, Feb 19, 2007 at 01:00:57PM -0500, Jerry Van Baren wrote:
>> Hi David,
>>
>> I've been trying to use your libfdt in u-boot and my first step, get a
>> value from the blob, failed terminally. After poking about a bit, it
>> appears that your libfdt and Jon Loeliger's dtc (-V 16) disagree with
>> respect to the format of the blob - libfdt won't traverse the path.
>
> That's certainly bad.
>
>> I've created two patches:
>
> Ug. Damn. Trouble is, for IBM paranoid procedural reasons, I'm not
> really in a position to accept patches for libfdt from outside at
> present. I'm already working on fixing that, but there being a
> ponderous bureaucracy involved, it's going to take a little while.
That's OK, just call it "the illustration of advice formerly known as a
patch." I just hope I didn't poison the well. ;-)
Seriously, I don't claim copyright on the changes, that would be
inappropriate their trivial nature. I do understand IBM's paranoia,
however. }:-(
gvb
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-02-20 13:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-02-19 18:00 [PATCH 0/2] libfdt: problems with real life blobs Jerry Van Baren
2007-02-19 23:46 ` David Gibson
2007-02-20 13:02 ` Jerry Van Baren
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.