[U-Boot] [RFC PATCH 18/20] sandbox: Remove unused variable warnings

Simon Glass sjg at chromium.org
Fri Sep 23 18:01:46 CEST 2011


Hi Mike,

On Sat, Sep 17, 2011 at 5:25 PM, Mike Frysinger <vapier at gentoo.org> wrote:
> On Saturday, September 17, 2011 12:48:57 Simon Glass wrote:
>> There are a few variables set but not used - this marks these as unused
>> for the compiler.
>
> summary says "sandbox:" but i dont think this has anything to do with sandbox.
> you're just using a newer compiler.
>
>> --- a/common/cmd_mem.c
>> +++ b/common/cmd_mem.c
>>
>> -     ulong   addr, length, i, junk;
>> +     ulong   addr, length, i, junk __attribute__((unused));
>
> this one is tricky as we just want to read the value from the volatile
> pointer.  i'm guessing do "*longp++;" just adds a different warning ?

Isn't that a nop?

Well I can just assign it to a different variable - I will use 'size'
in my new patch set.

>
>> --- a/common/cmd_nvedit.c
>> +++ b/common/cmd_nvedit.c
>>
>> -     int len;
>> +     int len __attribute((unused));
>
> this file though looks like we should just delete "len"

Yes, done. I don't use lint anyway.

Regards,
Simon

> -mike
>


More information about the U-Boot mailing list