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