On Mon, 2012-01-23 at 21:47 -0400, Konrad Rzeszutek Wilk wrote:

On Fri, Jan 20, 2012 at 04:01:12PM -0800, Joe Perches wrote:

quoted

It's equivalent to __printf, so prefer __scanf.

So ... looking at this patch it just seems to macro-fy the
__printf and __scanf attributes.

It's just for __scanf.  The __printf change is just
a neatening/spacing change.

Is this required to make
cleanpatch.pl work easier?

No.  It's a trivial symmetry patch added to
make fewer uses of __attribute__((format(...)
similar to the __printf commit from awhile ago.

commit b9075fa968a0a4347aef35e235e2995c0e57dddd

And there is also some checkpatch.pl features. Should that part
be in a seperate patch?

I think it's OK to do the whole thing at once.