2008-10-21 17:42:13

by Randy Dunlap

[permalink] [raw]
Subject: -git10 staging build warnings

i386:

drivers/staging/sxg/sxg.c:462: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
drivers/staging/sxg/sxg.c:479: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
drivers/staging/sxg/sxg.c:540: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
drivers/staging/sxg/sxg.c:571: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'

x86_64:

drivers/staging/winbond/./linux/wbusb .c:109: warning: 'struct ieee80211_tx_control' declared inside parameter list drivers/staging/winbond/./linux/wbusb.c:109: warning: its scope is only this definition or declaration, which is proba bly not what you want
drivers/staging/winbond/./linux/wbusb .c:172: warning: initialization from incompatible pointer type
drivers/staging/winbond/./linux/wbusb .c:174: warning: initialization from incompatible pointer type
drivers/staging/winbond/./linux/wbusb .c:180: warning: initialization from incompatible pointer type
drivers/staging/winbond/./linux/wbusb .c:181: warning: initialization from incompatible pointer type
drivers/staging/winbond/./linux/wbusb .c:299: warning: ISO C90 forbids mixed declarations and code
drivers/staging/usbip/vhci_rx.c:204: warning: format '%d' expects type 'int', but argument 7 has type 'long unsigned int'
drivers/staging/winbond/sme_api.c:13: warning: no return statement in function returning non-void


2008-10-21 18:11:25

by Greg KH

[permalink] [raw]
Subject: Re: -git10 staging build warnings

On Tue, Oct 21, 2008 at 10:41:45AM -0700, Randy Dunlap wrote:
> i386:
>
> drivers/staging/sxg/sxg.c:462: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
> drivers/staging/sxg/sxg.c:479: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
> drivers/staging/sxg/sxg.c:540: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
> drivers/staging/sxg/sxg.c:571: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
>
> x86_64:
>
> drivers/staging/winbond/./linux/wbusb .c:109: warning: 'struct ieee80211_tx_control' declared inside parameter list drivers/staging/winbond/./linux/wbusb.c:109: warning: its scope is only this definition or declaration, which is proba bly not what you want
> drivers/staging/winbond/./linux/wbusb .c:172: warning: initialization from incompatible pointer type
> drivers/staging/winbond/./linux/wbusb .c:174: warning: initialization from incompatible pointer type
> drivers/staging/winbond/./linux/wbusb .c:180: warning: initialization from incompatible pointer type
> drivers/staging/winbond/./linux/wbusb .c:181: warning: initialization from incompatible pointer type
> drivers/staging/winbond/./linux/wbusb .c:299: warning: ISO C90 forbids mixed declarations and code
> drivers/staging/usbip/vhci_rx.c:204: warning: format '%d' expects type 'int', but argument 7 has type 'long unsigned int'
> drivers/staging/winbond/sme_api.c:13: warning: no return statement in function returning non-void

Thanks, I have quite a few -staging patches queued up, need to ensure
these get fixed as well.

greg k-h

2008-10-21 18:15:14

by J.R. Mauro

[permalink] [raw]
Subject: Re: -git10 staging build warnings

On Tue, Oct 21, 2008 at 2:07 PM, Greg KH <[email protected]> wrote:
> On Tue, Oct 21, 2008 at 10:41:45AM -0700, Randy Dunlap wrote:
>> i386:
>>
>> drivers/staging/sxg/sxg.c:462: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
>> drivers/staging/sxg/sxg.c:479: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
>> drivers/staging/sxg/sxg.c:540: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
>> drivers/staging/sxg/sxg.c:571: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'

Greg, does this have anything to do with the fix-gcc-warnings patch I
sent you a while back?

>>
>> x86_64:
>>
>> drivers/staging/winbond/./linux/wbusb .c:109: warning: 'struct ieee80211_tx_control' declared inside parameter list drivers/staging/winbond/./linux/wbusb.c:109: warning: its scope is only this definition or declaration, which is proba bly not what you want
>> drivers/staging/winbond/./linux/wbusb .c:172: warning: initialization from incompatible pointer type
>> drivers/staging/winbond/./linux/wbusb .c:174: warning: initialization from incompatible pointer type
>> drivers/staging/winbond/./linux/wbusb .c:180: warning: initialization from incompatible pointer type
>> drivers/staging/winbond/./linux/wbusb .c:181: warning: initialization from incompatible pointer type
>> drivers/staging/winbond/./linux/wbusb .c:299: warning: ISO C90 forbids mixed declarations and code
>> drivers/staging/usbip/vhci_rx.c:204: warning: format '%d' expects type 'int', but argument 7 has type 'long unsigned int'
>> drivers/staging/winbond/sme_api.c:13: warning: no return statement in function returning non-void
>
> Thanks, I have quite a few -staging patches queued up, need to ensure
> these get fixed as well.
>
> greg k-h
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>

2008-10-21 18:36:27

by Greg KH

[permalink] [raw]
Subject: Re: -git10 staging build warnings

On Tue, Oct 21, 2008 at 02:14:58PM -0400, J.R. Mauro wrote:
> On Tue, Oct 21, 2008 at 2:07 PM, Greg KH <[email protected]> wrote:
> > On Tue, Oct 21, 2008 at 10:41:45AM -0700, Randy Dunlap wrote:
> >> i386:
> >>
> >> drivers/staging/sxg/sxg.c:462: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
> >> drivers/staging/sxg/sxg.c:479: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
> >> drivers/staging/sxg/sxg.c:540: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
> >> drivers/staging/sxg/sxg.c:571: warning: format '%lx' expects type 'long unsigned int', but argument 3 has type 'unsigned int'
>
> Greg, does this have anything to do with the fix-gcc-warnings patch I
> sent you a while back?

I don't know, I don't see that one in my "to-apply" queue, so either I
missed it, or it's already applied :)

I've now fixed this in my tree, it was a simple change.

thanks,

greg k-h

2008-10-21 19:04:04

by Greg KH

[permalink] [raw]
Subject: Re: -git10 staging build warnings

On Tue, Oct 21, 2008 at 10:41:45AM -0700, Randy Dunlap wrote:
> x86_64:
>
> drivers/staging/winbond/./linux/wbusb .c:109: warning: 'struct ieee80211_tx_control' declared inside parameter list drivers/staging/winbond/./linux/wbusb.c:109: warning: its scope is only this definition or declaration, which is proba bly not what you want
> drivers/staging/winbond/./linux/wbusb .c:172: warning: initialization from incompatible pointer type
> drivers/staging/winbond/./linux/wbusb .c:174: warning: initialization from incompatible pointer type
> drivers/staging/winbond/./linux/wbusb .c:180: warning: initialization from incompatible pointer type
> drivers/staging/winbond/./linux/wbusb .c:181: warning: initialization from incompatible pointer type
> drivers/staging/winbond/./linux/wbusb .c:299: warning: ISO C90 forbids mixed declarations and code
> drivers/staging/usbip/vhci_rx.c:204: warning: format '%d' expects type 'int', but argument 7 has type 'long unsigned int'
> drivers/staging/winbond/sme_api.c:13: warning: no return statement in function returning non-void

Thanks, I've now fixed these.

greg k-h