Re: [PATCH V2] dtc: Basic integer expressions

[Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]

On 03/31/2012 08:24 AM, Jon Loeliger wrote:
>> Written by David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>. Additions by me:
>> * Ported to ToT dtc.
>> * Renamed cell to integer throughout.
>> * Implemented value range checks.
>> * Allow L/UL/LL/ULL suffix on literals.
>> * Enabled the commented test.
>>
>> Signed-off-by: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
>> ---
>> v2:
>> * s/cell/integer/ throughout.
>> * Allow signed-extended values to pass the overall cell range check.
>> * Allow L/UL/LL/ULL suffix on literals. This is purely for compatibility
>>   with C, and has no effect on dtc's processing.
>> * Enabled the 3 disabled tests.
>>
>> I'm not sure if the literal suffix handling is hacky or not...
> 
> I get this too:
> 
>  CC tests/integer-expressions.o
> cc1: warnings being treated as errors
> tests/integer-expressions.c: In function 'main'
> tests/integer-expressions.c:105: error: format  tests/integer-expressions.o
> cc1: warnings being treated as errors
> tests/integer-expressions.c:105: error: format '%ld' expects type 'long int', but argument 3 has type 'unsigned int'
> make: *** [tests/integer-expressions.o] Error 1
> 
> $ gcc --version
> gcc (Debian 4.4.5-8) 4.4.5

I assume this is a 32-bit system? I guess I can use <inttypes.h> to
solve this if needed.