On Tue, May 15, 2012 at 8:45 PM, Greg Ungerer <g...@snapgear.com> wrote:
> On 16/05/12 01:57, Mike Frysinger wrote:
>> On Tue, May 15, 2012 at 12:08 AM,<g...@snapgear.com>  wrote:
>>>
>>> . make the struct uclinux_ram_map static
>>
>>
>> NAK: this breaks Blackfin systems.  we specifically don't want this to
>> be static.
>>
>> it should probably get a comment added above it saying as much.
>
> A comment won't fix the sparse warning. You need a proper declaration.

perhaps, but marking it static to fix a warning that people rarely see
whilst simultaneously knowingly breaking an arch doesn't sound like
the correct trade off.
-mike
_______________________________________________
uClinux-dev mailing list
uClinux-dev@uclinux.org
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by uclinux-dev@uclinux.org
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev

Reply via email to