On Fri, 2010-04-16 at 16:55 -0400, Pavel Roskin wrote: > On Fri, 2010-04-16 at 09:28 +0800, Zhu Yi wrote: > > From: Samuel Ortiz <sameo@xxxxxxxxxxxxxxx> > > > > Add -D__CHECK_ENDIAN__ to driver ccflags so that sparse will > > always check endianness by default. > > I believe it's wrong. I would be fine with enabling __CHECK_ENDIAN__ > for the whole kernel by default (perhaps with an option to disable it). > But there is no reason why some particular drivers need that check more > than the rest of the kernel. We've done it on other drivers -- we can't do it for all of the kernel because that would drown people in warnings, but for those drivers that _should_ be clean it ought to be fine to add it by default. johannes -- 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
