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 <[email protected]> (maintainer:ARM/Ux500 ARM AR)
Linus Walleij <[email protected]> (maintainer:ARM/Ux500 ARM ARC...)
Samuel Ortiz <[email protected]> (supporter:MULTIFUNCTION DEV...)
[email protected] (moderated list:ARM/Ux500 ARM ARC...)
[email protected] (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
[email protected]
[email protected]
[email protected]


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
[email protected]
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to