Re: [PATCH V3 linux-next] checkpatch: don't ask for asm/file.h to linux/file.h unconditionally

2015-03-31 Thread Joe Perches
On Tue, 2015-03-31 at 21:56 +0200, Fabian Frederick wrote: > Currently checkpatch warns when asm/file.h is included and linux/file.h > exists. That conversion can be made when linux/file.h includes asm/file.h > which is not always the case.(See signal.h) OK by me. I would have done it directly

[PATCH V3 linux-next] checkpatch: don't ask for asm/file.h to linux/file.h unconditionally

2015-03-31 Thread Fabian Frederick
Currently checkpatch warns when asm/file.h is included and linux/file.h exists. That conversion can be made when linux/file.h includes asm/file.h which is not always the case.(See signal.h) Signed-off-by: Fabian Frederick --- V3: Only grep when $root/$checkfile exists (suggested by Joe

[PATCH V3 linux-next] checkpatch: don't ask for asm/file.h to linux/file.h unconditionally

2015-03-31 Thread Fabian Frederick
Currently checkpatch warns when asm/file.h is included and linux/file.h exists. That conversion can be made when linux/file.h includes asm/file.h which is not always the case.(See signal.h) Signed-off-by: Fabian Frederick f...@skynet.be --- V3: Only grep when $root/$checkfile exists

Re: [PATCH V3 linux-next] checkpatch: don't ask for asm/file.h to linux/file.h unconditionally

2015-03-31 Thread Joe Perches
On Tue, 2015-03-31 at 21:56 +0200, Fabian Frederick wrote: Currently checkpatch warns when asm/file.h is included and linux/file.h exists. That conversion can be made when linux/file.h includes asm/file.h which is not always the case.(See signal.h) OK by me. I would have done it directly in