default enable sparse __CHECK_ENDIAN__ (was: Re: [PATCH v7 2/3] net: Add Keystone NetCP ethernet driver)
flat view
From: joe@perches.com (Joe Perches)
Date: 2014-12-11 03:05:06
Also in:
linux-devicetree, lkml, netdev
On Wed, 2014-12-10 at 20:41 -0500, David Miller wrote:
From: Murali Karicheri <redacted>quoted
Are you referring to the static code analyser sparse that is invoked through?You have to explicitly enable endian checking, it's not on by default.
There don't seem to be thousands of warnings anymore.
Maybe it's time to default enable it when using C=?
from: Documentation/sparse.txt:
The optional make variable CF can be used to pass arguments to sparse. The
build system passes -Wbitwise to sparse automatically. To perform endianness
checks, you may define __CHECK_ENDIAN__:
make C=2 CF="-D__CHECK_ENDIAN__"
These checks are disabled by default as they generate a host of warnings.