Search Linux Wireless

Re: [PATCH 08/18] staging: wilc1000: remove declaration of WILC_WFI_frame_register

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

 





On 2015년 09월 21일 11:01, Greg KH wrote:
On Sun, Sep 20, 2015 at 03:51:15PM +0900, Chaehyun Lim wrote:
This patch removes function declaration of WILC_WFI_frame_register in
linux_wlan.c file, then adds it in wilc_wfi_cfgoperations.h file.

The compilation warning occurs because it is assigned to incorrect
pointer type of second parameter of WILC_WFI_frame_register.
It is assigned with struct wireless_dev pointer type.
How was this code ever even working properly if the wrong pointer was
being passed into it?

Tony, what is going on here, is this code just never called?  If so, can
we just delete the whole function?

thanks,

greg k-h

This file with linux_wlan.c file is being fully changed, but at this time, the second parameter which passed as
net_device pointer instead of wireless_dev incorrectly is never being used inside the WILC_WFI_frame_register().
That's why no problem is found yet.

As of the patch which Chaehyun Lim made, the second parameter, nic->wilc_netdev should be replaced by nic->wilc_netdev->ieee80211_ptr.
Thanks,
Tony.


--
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 Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux