Re: default enable sparse __CHECK_ENDIAN__ (was: Re: [PATCH v7 2/3] net: Add Keystone NetCP ethernet driver)

2014-12-11 Thread Marcel Holtmann
Hi Joe, 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:

default enable sparse __CHECK_ENDIAN__ (was: Re: [PATCH v7 2/3] net: Add Keystone NetCP ethernet driver)

2014-12-10 Thread Joe Perches
On Wed, 2014-12-10 at 20:41 -0500, David Miller wrote: From: Murali Karicheri m-kariche...@ti.com 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