[OpenBIOS] [PATCH v5 3/5] Pretty-print reg property

[Andreas Färber]

Am 12.11.2010 um 23:26 schrieb Segher Boessenkool:

>> +    " #size-cells" rot get-package-property if 1 exit then
>> +    \ we don't have to support more than 4
>> +    decode-int nip nip 0 max
>
> It's not a signed number, so it should be umax, not max.
> But "0 umax" is a noop.  Just remove it, use the value
> as-is.

Your loop uses them as signed numbers though...

Your quote also shows a stray comment that I'll remove.

Andreas

> A sanity check somewhere (probably not here though) might be
> useful; but mangling the data if deemed out-of-range is hiding
> the problem, at best.
>
>
> Segher
>
>
> -- 
> OpenBIOS                 http://openbios.org/
> Mailinglist:  http://lists.openbios.org/mailman/listinfo
> Free your System - May the Forth be with you