On Sat 20 July 2013 14:14:25 Larry Finger wrote: > On 07/20/2013 01:37 PM, Thomas Fjellstrom wrote: > > Hi, > > > > Recently while trying to figure out a separate issue, I did a > > make allmodconfig and a make oldconfig with debian's default config on > > the 3.10.1 kernel, and debian's config with 3.10.0. > > > > It failed in the wil6210 driver. > > > > The system it failed on is running a fresh install of Debian Sid, with gcc > > 4.8.1. > > > > This is the error I got: > > CC [M] drivers/net/wireless/ath/wil6210/debugfs.o > > > > drivers/net/wireless/ath/wil6210/debugfs.c: In function âwil_print_ringâ: > > drivers/net/wireless/ath/wil6210/debugfs.c:163:11: error: pointer targets > > in passing argument 5 of âhex_dump_to_bufferâ differ in signedness > > [-Werror=pointer-sign]> > > false); > > ^ > > > > In file included from include/linux/kernel.h:13:0, > > > > from include/linux/cache.h:4, > > from include/linux/time.h:4, > > from include/linux/stat.h:18, > > from include/linux/module.h:10, > > > > from drivers/net/wireless/ath/wil6210/debugfs.c:17: > > include/linux/printk.h:361:13: note: expected âchar *â but argument is of > > type âunsigned char *â> > > extern void hex_dump_to_buffer(const void *buf, size_t len, > > > > ^ > > > > drivers/net/wireless/ath/wil6210/debugfs.c: In function > > âwil_txdesc_debugfs_showâ: > > drivers/net/wireless/ath/wil6210/debugfs.c:429:10: error: pointer targets > > in passing argument 5 of âhex_dump_to_bufferâ differ in signedness > > [-Werror=pointer-sign]> > > sizeof(printbuf), false); > > ^ > > > > In file included from include/linux/kernel.h:13:0, > > > > from include/linux/cache.h:4, > > from include/linux/time.h:4, > > from include/linux/stat.h:18, > > from include/linux/module.h:10, > > > > from drivers/net/wireless/ath/wil6210/debugfs.c:17: > > include/linux/printk.h:361:13: note: expected âchar *â but argument is of > > type âunsigned char *â> > > extern void hex_dump_to_buffer(const void *buf, size_t len, > > > > ^ > > > > cc1: all warnings being treated as errors > > make[5]: *** [drivers/net/wireless/ath/wil6210/debugfs.o] Error 1 > > make[4]: *** [drivers/net/wireless/ath/wil6210] Error 2 > > make[3]: *** [drivers/net/wireless/ath] Error 2 > > make[2]: *** [drivers/net/wireless] Error 2 > > make[1]: *** [drivers/net] Error 2 > > make: *** [drivers] Error 2 > > > > please excuse the mangled quotes, fresh install and I negelected to set up > > the locale properly. It's fixed now, but it seems I need to restart X for > > it to notice. > > This error should be fixed by the attached patch. Please test. > > Note that my system did not issue a warning for this "problem". It seems we > have different warnings turned on in the compiler. The warnings have gone away, and the first error, but the second one is still there. > Larry -- Thomas Fjellstrom thomas@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
