Re: [PATCH] checkpatch.pl: Add warning for new __packed additions

From: Joe Perches
Date: Wed Feb 26 2014 - 17:04:30 EST


On Tue, 2014-02-25 at 07:30 -0500, Tom Rini wrote:
> The only intentional differences we have today are adding debug/printf
> to the list of log functions

That seems fine and trivial to keep current.

> and pointing people at boards.cfg not
> CHECKPATCH in MAINTAINERS,

That seems wrong.

The idea of that message is to fix defects in checkpatch,
not any other code.

Maybe that could be reworded to
"See the CHECKPATCH entry in the MAINTAINERS file of linux git"
or some such.

> so I can re-sync us no problem as, yup, we're
> about a year out of date there. Oh and I see now someone snuck in a
> check for whitespace before semi-colon to catch 'return 0 ;' which I'll
> check real quick isn't something already caught and if not post that as
> well.

Does anyone know of other projects that use checkpatch?


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/