Re: checkpatch induced patches...

2015-02-12 Thread Kalle Valo
Joe Perches writes: > On Wed, 2015-02-11 at 21:02 +0100, Richard Weinberger wrote: >> On Wed, Feb 11, 2015 at 7:36 PM, Dan Carpenter >> 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 us

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 --f

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 out

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 >> dri

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

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 > > > wrote: > > > > On Wed, Feb 11, 2015 at 10:00:29AM -0800, Joe Perches

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 > 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

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 > > 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 Richard Weinberger
On Wed, Feb 11, 2015 at 7:36 PM, Dan Carpenter 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 be used with th

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 undocu

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