Re: [PATCH] checkpatch: fix spurious vendor compatible warnings

2014-02-27 Thread Rob Herring
On Thu, Feb 27, 2014 at 5:53 PM, Joe Perches wrote: > On Thu, 2014-02-27 at 21:24 +0100, Florian Vaussard wrote: >> On 02/27/2014 09:10 PM, Joe Perches wrote: >> > On Thu, 2014-02-27 at 20:56 +0100, Florian Vaussard wrote: >> >> With a compatible string like >> >> compatible = "foo"; >> >> checkpa

Re: [PATCH] checkpatch: fix spurious vendor compatible warnings

2014-02-27 Thread Joe Perches
On Thu, 2014-02-27 at 21:24 +0100, Florian Vaussard wrote: > On 02/27/2014 09:10 PM, Joe Perches wrote: > > On Thu, 2014-02-27 at 20:56 +0100, Florian Vaussard wrote: > >> With a compatible string like > >> compatible = "foo"; > >> checkpatch will currently try to find "foo" in vendor-prefixes.txt

Re: [PATCH] checkpatch: fix spurious vendor compatible warnings

2014-02-27 Thread Florian Vaussard
On 02/27/2014 09:10 PM, Joe Perches wrote: > On Thu, 2014-02-27 at 20:56 +0100, Florian Vaussard wrote: >> With a compatible string like >> >> compatible = "foo"; >> >> checkpatch will currently try to find "foo" in vendor-prefixes.txt, >> which is wrong since the vendor prefix is empty in this

Re: [PATCH] checkpatch: fix spurious vendor compatible warnings

2014-02-27 Thread Joe Perches
On Thu, 2014-02-27 at 20:56 +0100, Florian Vaussard wrote: > With a compatible string like > > compatible = "foo"; > > checkpatch will currently try to find "foo" in vendor-prefixes.txt, > which is wrong since the vendor prefix is empty in this specific case. > > Skip the vendor test if the com

[PATCH] checkpatch: fix spurious vendor compatible warnings

2014-02-27 Thread Florian Vaussard
With a compatible string like compatible = "foo"; checkpatch will currently try to find "foo" in vendor-prefixes.txt, which is wrong since the vendor prefix is empty in this specific case. Skip the vendor test if the compatible is not like compatible = "vendor,something"; Signed-off-by: Flori