On Mon, Aug 15, 2011 at 12:04:41PM -0400, Pavel Roskin wrote: > Checks enabled by __CHECK_ENDIAN__ produce less than a half of the > sparse warnings. For instance, the allyesconfig configuration on > linux-next (tag next-20110812) produces on x86_64: > > no __CHECK_ENDIAN__ with __CHECK_ENDIAN__ > stderr bytes: 1155037 1992954 (73% increase) > stderr lines: 10716 18657 (74% increase) > affected files: 1191 1458 (22% increase) > > It is no longer true that __CHECK_ENDIAN__ produces so many warnings > that other sparse warnings drown in the noise. > > Janitors going through the whole tree are already prepared to search for > the warnings they want to fix. > > The main point of this patch is to make sure that developers of a > particular driver who use sparse rarely get all the warnings for their > code even if they forget to specify __CHECK_ENDIAN__ or mistype it. > > Some developers actually added __CHECK_ENDIAN__ to the driver makefiles > so they don't forget to use it. Remove them now. > > Remove all references to __CHECK_ENDIAN__ from the documentation except > the Chinese version, which would need to be updated separately. > > Signed-off-by: Pavel Roskin <proski@xxxxxxx> Acked-by: Sam Ravnborg <sam@xxxxxxxxxxxx> Maybe ask Michal to merge it? Sam -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
