On Thursday 13 October 2011 13:25:51 Joe Hershberger wrote: > On Wed, Oct 12, 2011 at 10:27 PM, Mike Frysinger <[email protected]> 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 -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
