Search Linux Wireless

Re: [RFC] prism54: fix compilation warning on wrqu.data.pointer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2011-11-29 at 12:41 -0500, Luis R. Rodriguez wrote:
> This fixes this compilation warning:
> 
>   CHECK   drivers/net/wireless/prism54/isl_ioctl.c
> drivers/net/wireless/prism54/isl_ioctl.c:2052:27: warning: incorrect type in assignment (different address spaces)
> drivers/net/wireless/prism54/isl_ioctl.c:2052:27:    expected void [noderef] <asn:1>*pointer
> drivers/net/wireless/prism54/isl_ioctl.c:2052:27:    got char *[assigned] memptr
> drivers/net/wireless/prism54/isl_ioctl.c:2071:27: warning: incorrect type in assignment (different address spaces)
> drivers/net/wireless/prism54/isl_ioctl.c:2071:27:    expected void [noderef] <asn:1>*pointer
> drivers/net/wireless/prism54/isl_ioctl.c:2071:27:    got char *[assigned] memptr
> 
> The wrqu.data.pointer shouldn't be assigned by the driver, addiotionally
> custom wext driver events would simply be dropped anyway.
> 
> Cc: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>
> Signed-off-by: Luis R. Rodriguez <mcgrof@xxxxxxxxxxxxxxxx>
> ---
> 
> Johannes, it was unclear to me whether or not wireless_send_event()
> no longer supports custom driver events if so then not only was this
> bogus but also all the private driver event crap could likely be
> removed from prism54 as well.

Looking at cfg80211_michael_mic_failure() in net/wireless/mlme.c, this
should work.

johannes

--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux