2013-07-20 16:00:45

by Thomas Fjellstrom

[permalink] [raw]
Subject: compile error on 3.10.1

Hi, just trying to compile a vanilla kernel and building using
allmodconfig and using oldconfig with the debian 3.10.1 config as a base.

With the later config, I get the following:

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

There are also a bunch of warnings, most of the ones I could catch are
signedness related.

--
Thomas Fjellstrom
[email protected]


2013-07-20 18:02:06

by Greg KH

[permalink] [raw]
Subject: Re: compile error on 3.10.1

On Sat, Jul 20, 2013 at 09:59:57AM -0600, Thomas Fjellstrom wrote:
> Hi, just trying to compile a vanilla kernel and building using
> allmodconfig and using oldconfig with the debian 3.10.1 config as a base.
>
> With the later config, I get the following:

Is this new to 3.10.1 and works fine for 3.10.0?

> 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
>
> There are also a bunch of warnings, most of the ones I could catch are
> signedness related.

Care to send this to the atheros wireless developers? I'm sure they
would like to know about it.

greg k-h

2013-07-20 18:07:52

by Thomas Fjellstrom

[permalink] [raw]
Subject: Re: compile error on 3.10.1

On Sat 20 July 2013 11:02:10 you wrote:
> On Sat, Jul 20, 2013 at 09:59:57AM -0600, Thomas Fjellstrom wrote:
> > Hi, just trying to compile a vanilla kernel and building using
> > allmodconfig and using oldconfig with the debian 3.10.1 config as a base.
> >
> > With the later config, I get the following:
>
> Is this new to 3.10.1 and works fine for 3.10.0?

I'll check.

All I've tested so far is debian's 2.6.32, 3.9.8, 3.10.1, and a vanilla 3.10.1
(with allmodconfig and oldconfig+debian config).

> > 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
> >
> > There are also a bunch of warnings, most of the ones I could catch are
> > signedness related.
>
> Care to send this to the atheros wireless developers? I'm sure they
> would like to know about it.

Can do.

> greg k-h
--
Thomas Fjellstrom
[email protected]

2013-07-20 18:40:27

by Thomas Fjellstrom

[permalink] [raw]
Subject: Re: compile error on 3.10.1

On Sat 20 July 2013 12:07:03 Thomas Fjellstrom wrote:
> On Sat 20 July 2013 11:02:10 you wrote:
> > On Sat, Jul 20, 2013 at 09:59:57AM -0600, Thomas Fjellstrom wrote:
> > > Hi, just trying to compile a vanilla kernel and building using
> > > allmodconfig and using oldconfig with the debian 3.10.1 config as a
> > > base.
> >
> > > With the later config, I get the following:
> > Is this new to 3.10.1 and works fine for 3.10.0?
>
> I'll check.
>
> All I've tested so far is debian's 2.6.32, 3.9.8, 3.10.1, and a vanilla
> 3.10.1 (with allmodconfig and oldconfig+debian config).

I meant to say I only tried 3.10.1... I'm getting this and my last email
confused a little.

Just tested the v3.10 tag from the stable repo and it exhibits the same
problem.

> > > 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
> > >
> > > There are also a bunch of warnings, most of the ones I could catch are
> > > signedness related.
> >
> > Care to send this to the atheros wireless developers? I'm sure they
> > would like to know about it.
>
> Can do.

Done.

> > greg k-h
--
Thomas Fjellstrom
[email protected]