[U-Boot] [PATCH] tools: checkpatch.pl from Linux added to tools

Joe Hershberger joe.hershberger at gmail.com
Thu Oct 13 20:30:12 CEST 2011


On Thu, Oct 13, 2011 at 1:20 PM, Mike Frysinger <vapier at gentoo.org> wrote:
> On Thursday 13 October 2011 13:25:51 Joe Hershberger wrote:
>> On Wed, Oct 12, 2011 at 10:27 PM, Mike Frysinger <vapier at gentoo.org> wrote:
>> > On Tuesday 11 October 2011 21:41:19 Joe Hershberger wrote:
>> >> --- /dev/null
>> >> +++ b/.checkpatch.conf
>> >>
>> >> +# Not Linux, so don't expect a Linux tree.
>> >> +--no-tree
>>
>> Why depend on or have to specify where a linux tree is for building u-boot?
>
> that's not how it works.  use the u-boot tree as the "linux" tree.
>
>> > the only things checkpatch.pl uses $tree for, i think we want.  like
>> > #include asm/foo.h instead of linux/foo.h.  why do you want this ?
>> >
>> >> +# For Linux includes
>> >> +--ignore ARCH_INCLUDE_LINUX
>> >> +--ignore INCLUDE_LINUX
>> >
>> > i think these checks are good actually.  we import headers from Linux all
>> > the time ...
>>
>> These seem to be irrelevant if --no-tree is specified.  It doesn't
>> seem like checkpatch would make good decisions about this anyway given
>> that an include file that exists in the referred-to Linux tree may not
>> exist in the u-boot tree.
>
> use the u-boot tree, then it works fine

With the latest version of checkpatch that does not work... checkpatch
looks for the following files/folders to identify the
top_of_kernel_tree:

                "COPYING", "CREDITS", "Kbuild", "MAINTAINERS", "Makefile",
                "README", "Documentation", "arch", "include", "drivers",
                "fs", "init", "ipc", "kernel", "lib", "scripts",

u-boot has no Kbuild, Documentation, init, ipc, kernel, or scripts.

-Joe


More information about the U-Boot mailing list