Re: checkpatch induced patches...

2015-02-12 Thread Sudip Mukherjee
On Thu, Feb 12, 2015 at 01:43:00AM +0300, Dan Carpenter wrote: On Wed, Feb 11, 2015 at 12:43:03PM -0800, Joe Perches wrote: Maybe some help/warning text like: --forceWithout --force, checkpatch will not scan files using -f or --file

Re: checkpatch induced patches...

2015-02-12 Thread Kalle Valo
Joe Perches j...@perches.com writes: On Wed, 2015-02-11 at 21:02 +0100, Richard Weinberger wrote: On Wed, Feb 11, 2015 at 7:36 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Wed, Feb 11, 2015 at 10:00:29AM -0800, Joe Perches wrote: I'm half tempted to submit some patch like this to

Re: checkpatch induced patches...

2015-02-11 Thread Dan Carpenter
On Wed, Feb 11, 2015 at 10:00:29AM -0800, Joe Perches wrote: I'm half tempted to submit some patch like this to make it difficult to use checkpatch on files outside of drivers/staging. o Only allow checkpatch to be used with the -f/--file option for drivers/staging/ o Add an undocumented

checkpatch induced patches...

2015-02-11 Thread Joe Perches
On Wed, 2015-02-11 at 13:51 +0300, Dan Carpenter wrote: On Wed, Feb 11, 2015 at 01:40:37AM -0800, Joe Perches wrote: On Wed, 2015-02-11 at 11:33 +0300, Dan Carpenter wrote: You can't fight checkpatch.pl. Sure you can, Ignore it whenever appropriate. People will just keep sending

Re: checkpatch induced patches...

2015-02-11 Thread Joe Perches
On Wed, 2015-02-11 at 21:02 +0100, Richard Weinberger wrote: On Wed, Feb 11, 2015 at 7:36 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Wed, Feb 11, 2015 at 10:00:29AM -0800, Joe Perches wrote: I'm half tempted to submit some patch like this to make it difficult to use checkpatch

Re: checkpatch induced patches...

2015-02-11 Thread Richard Weinberger
On Wed, Feb 11, 2015 at 7:36 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Wed, Feb 11, 2015 at 10:00:29AM -0800, Joe Perches wrote: I'm half tempted to submit some patch like this to make it difficult to use checkpatch on files outside of drivers/staging. o Only allow checkpatch to

Re: checkpatch induced patches...

2015-02-11 Thread Pavel Machek
On Wed 2015-02-11 12:20:25, Joe Perches wrote: On Wed, 2015-02-11 at 21:02 +0100, Richard Weinberger wrote: On Wed, Feb 11, 2015 at 7:36 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Wed, Feb 11, 2015 at 10:00:29AM -0800, Joe Perches wrote: I'm half tempted to submit some patch

Re: checkpatch induced patches...

2015-02-11 Thread Joe Perches
On Wed, 2015-02-11 at 21:24 +0100, Pavel Machek wrote: On Wed 2015-02-11 12:20:25, Joe Perches wrote: On Wed, 2015-02-11 at 21:02 +0100, Richard Weinberger wrote: On Wed, Feb 11, 2015 at 7:36 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Wed, Feb 11, 2015 at 10:00:29AM -0800,

Re: checkpatch induced patches...

2015-02-11 Thread Joe Perches
On Thu, 2015-02-12 at 01:43 +0300, Dan Carpenter wrote: On Wed, Feb 11, 2015 at 12:43:03PM -0800, Joe Perches wrote: Maybe some help/warning text like: --forceWithout --force, checkpatch will not scan files using -f or --file outside of

Re: checkpatch induced patches...

2015-02-11 Thread Richard Weinberger
Am 11.02.2015 um 23:43 schrieb Dan Carpenter: On Wed, Feb 11, 2015 at 12:43:03PM -0800, Joe Perches wrote: Maybe some help/warning text like: --forceWithout --force, checkpatch will not scan files using -f or --file outside of

Re: checkpatch induced patches...

2015-02-11 Thread Dan Carpenter
On Wed, Feb 11, 2015 at 12:43:03PM -0800, Joe Perches wrote: Maybe some help/warning text like: --forceWithout --force, checkpatch will not scan files using -f or --file outside of drivers/staging/... Do not use