On 04/11/2013 11:28 AM, Naresh Kamboju wrote:
Hi All,

Greetings from Linaro QA services.
We have setup the process of validating Linux on ARM Kernel builds.
One of those ARM targets is Snowball. Snowball build broken log can be
found here

Build Error:

04:48:08   CC      drivers/mfd/ab8500-debugfs.o
04:48:09
/mnt/ci_build/workspace/linux-next/hwpack/snowball/label/precise_hwpack_cloud/drivers/mfd/ab8500-debugfs.c:95:23:
fatal error: mach/irqs.h: No such file or directory
04:48:09 compilation terminated.
04:48:09 make[3]: *** [drivers/mfd/ab8500-debugfs.o] Error 1
04:48:09 make[2]: *** [drivers/mfd] Error 2
04:48:09 make[1]: *** [drivers] Error 2

Code snippet:
------------------
drivers/mfd/ab8500-debugfs.c

  89 #ifdef CONFIG_DEBUG_FS
   90 #include <linux/string.h>
   91 #include <linux/ctype.h>
   92 #endif
   93
   94 /* TODO: this file should not reference IRQ_DB8500_AB8500! */
   95 #include <mach/irqs.h>
   96
   97 static u32 debug_bank;
   98 static u32 debug_address;
   99

First of all, this is very cool output to see! A few thoughts..


It might also be good to get the maintainers of the affected file on cc.

This can be done automatically via:
$ ./scripts/get_maintainer.pl -f drivers/mfd/ab8500-debugfs.c
Srinidhi Kasagar <srinidhi.kasa...@stericsson.com> (maintainer:ARM/Ux500 ARM AR)
Linus Walleij <linus.wall...@linaro.org> (maintainer:ARM/Ux500 ARM ARC...)
Samuel Ortiz <sa...@linux.intel.com> (supporter:MULTIFUNCTION DEV...)
linux-arm-ker...@lists.infradead.org (moderated list:ARM/Ux500 ARM ARC...)
linux-ker...@vger.kernel.org (open list)


You might also be able to pull recent patch authors into the cc as well with something like:

$ git log --pretty=%aE drivers/mfda/ab8500-debugfs.c | head -n 3
linus.wall...@linaro.org
philippe.langl...@linaro.org
asho...@stericsson.com


I am a little curious how these automated messages will be received on lkml (I don't think I've seen the kbuild test robot output directly on lkml). Would cc'ing the linux-next mailing list be better when testing linux-next?

Also, I'm sure you see this error caused two almost identical messages. Not sure why the messages were slightly different? Regardless, we'll want to avoid sending dups to mailing lists.

Overall, great to see this work moving forward!

thanks
-john





_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to