[PATCH v2 6/7] Holly DTS
Segher Boessenkool
segher at kernel.crashing.org
Mon May 14 22:33:13 EST 2007
>>> easiest is to
>>> replace it with a configurable poison value. When using asm output,
>>> or with a map file, it might be possible to generate out-of-band
>>> information which we can use to check that the right things are
>>> filled
>>> in.
>>>
>>> It's not a simply a matter of making the kernel parser recognize the
>>> uninitialized info, because there's no way to encode that in the dtb
>>> itself.
>>
>> There can be, with a new DTB version -- perhaps simply
>> define proplen == -1 to mean "undefined" or similar.
>
> That doesn't quite do it, because I'd prefer ? to cover cases where
> the property's length is known, just not its value. In particular
> that could be useful for device trees where *only* non-moving edits
> are required by the bootloader, letting us use the asm output mode as
> intended.
[First things first -- it is generally wrong to
require properties to have a certain length, newer
(versions of) bindings can add extra fields to the
end of a property. It should be okay for this ??
stuff though, if a bootwrapper/firmware fills in
the property it better know its whole semantics,
including the new part.]
Okay, so use negative lengths to represent undefined
properties of a certain length, and use -inf to mean
"completely undefined yet". Or some other encoding,
there are many ways that would work.
My point is that you need a new DTB version to properly
represent ??.
Segher
More information about the Linuxppc-dev
mailing list