Re: [PATCH] checkpatch.pl: Remove --file option

From: Pavel Machek
Date: Mon Aug 04 2014 - 10:30:36 EST


On Tue 2014-07-22 08:27:22, Kalle Valo wrote:
> Borislav Petkov <bp@xxxxxxxxx> writes:
>
> > No, checkpatch should only check patches and not whole file. It is
> > wrong. Fullstop.
>
> Please don't remove --file, I use it all the time when maintaining
> ath6kl and ath10k. It's a lot easier in my workflow to test the whole
> driver in one go than start testing individual patches.

Exactly. Also, with --file, you have nice place to write warning
"please don't do cleanups just because of this"...
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
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/