On Tue, Dec 20, 2011 at 01:01:36PM -0600, Larry Finger wrote: > On 12/20/2011 12:46 PM, Luis R. Rodriguez wrote: > >From: Luis R. Rodriguez<mcgrof@xxxxxxxxxxxxxxxx> > > > >Here's a slew of sparse warning fixes for all Atheros wireless drivers. > >These have been itching me for a while now. I ask all patch submitters > >and driver maintainers to please run their patches with these two tests: > > > >make C=1 M=drivers/net/wireless/ath/ > > > >And then if all is happy try: > > > >make C=2 CF="-D__CHECK_ENDIAN__" M=drivers/net/wireless/ath/ > > Why not follow a suggestion that Johannes made to me? Add the line > > ccflags-y += -D__CHECK_ENDIAN__ > > To the bottom of the make file(s). That way they only need one run. > Then enforce the rule by not ACKing any patches with too many of the > endian warnings, and do not allow any of the others. Awesome idea. Luis -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
