2009-01-12 17:03:52

by Jaswinder Singh

Subject: scripts/checkpatch.pl: WARNING: patch prefix 'arch' exists, appears to be a -p0 patch

Hello Andy,

How to get rid of this warning:

#./scripts/checkpatch.pl --file arch/x86/kernel/module_32.c
WARNING: patch prefix 'arch' exists, appears to be a -p0 patch

total: 0 errors, 1 warnings, 152 lines checked

arch/x86/kernel/module_32.c has style problems, please review. If any
of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

Thanks
--
JSR

2009-01-15 15:56:57

by Jaswinder Singh

Subject: Re: scripts/checkpatch.pl: WARNING: patch prefix 'arch' exists, appears to be a -p0 patch

Hello Andy,

On Mon, Jan 12, 2009 at 10:33 PM, Jaswinder Singh Rajput
<[email protected]> wrote:
>
> #./scripts/checkpatch.pl --file arch/x86/kernel/module_32.c
> WARNING: patch prefix 'arch' exists, appears to be a -p0 patch
>
> total: 0 errors, 1 warnings, 152 lines checked
>

The problem is on my side or this is a BUG ?

Thanks
--
JSR

2009-01-15 18:50:29

by Andy Whitcroft

Subject: Re: scripts/checkpatch.pl: WARNING: patch prefix 'arch' exists, appears to be a -p0 patch

On Mon, Jan 12, 2009 at 10:33:40PM +0530, Jaswinder Singh Rajput wrote:
> Hello Andy,
>
> How to get rid of this warning:
>
> #./scripts/checkpatch.pl --file arch/x86/kernel/module_32.c
> WARNING: patch prefix 'arch' exists, appears to be a -p0 patch
>
> total: 0 errors, 1 warnings, 152 lines checked
>
> arch/x86/kernel/module_32.c has style problems, please review. If any
> of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.

That looks wrong to me. Will have a look at it.

Ahh yes, thats just wrong. Should be fixed in the tip of my tree, and
will be in the next block of updates. You can check against the version
at:

http://www.kernel.org/pub/linux/kernel/people/apw/checkpatch/checkpatch.pl-testing

-apw

2009-01-16 16:34:20

by Jaswinder Singh

Subject: Re: scripts/checkpatch.pl: WARNING: patch prefix 'arch' exists, appears to be a -p0 patch

On Fri, Jan 16, 2009 at 12:20 AM, Andy Whitcroft <[email protected]> wrote:
> On Mon, Jan 12, 2009 at 10:33:40PM +0530, Jaswinder Singh Rajput wrote:
>> Hello Andy,
>>
>> How to get rid of this warning:
>>
>> #./scripts/checkpatch.pl --file arch/x86/kernel/module_32.c
>> WARNING: patch prefix 'arch' exists, appears to be a -p0 patch
>>
>> total: 0 errors, 1 warnings, 152 lines checked
>>
>> arch/x86/kernel/module_32.c has style problems, please review. If any
>> of these errors
>> are false positives report them to the maintainer, see
>> CHECKPATCH in MAINTAINERS.
>
> That looks wrong to me. Will have a look at it.
>
> Ahh yes, thats just wrong. Should be fixed in the tip of my tree, and
> will be in the next block of updates. You can check against the version
> at:
>
> http://www.kernel.org/pub/linux/kernel/people/apw/checkpatch/checkpatch.pl-testing
>

Fixed, thanks :-)

--
JSR