- 08 Nov, 2011 40 commits
-
-
Arik Nemtsov authored
Some cards can generate CCMP IVs in HW, but require the space for the IV to be pre-allocated in the frame at the correct offset. Add a key flag that allows us to achieve this. Signed-off-by:
Arik Nemtsov <arik@wizery.com> Signed-off-by:
John W. Linville <linville@tuxdriver.com>
-
Johannes Berg authored
genlmsg_end() cannot fail, it just returns the length of the message. Thus, error handling for it is useless. While removing it, I also noticed a useless variable and removed this it as well. Signed-off-by:
Johannes Berg <johannes.berg@intel.com> Signed-off-by:
-
Arend van Spriel authored
The buffer endianess conversion functions in srom.c had a size argument giving number of bytes but the function converts words. Providing the number of words to the function is more sensible so that is done in this patch. Reported-by:
Pavel Roskin <proski@gnu.org> Reported-by:
Larry Finger <Larry.Finger@lwfinger.net> Reviewed-by:
Pieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by:
Alwin Beukers <alwin@broadcom.com> Signed-off-by:
Arend van Spriel <arend@broadcom.com> Signed-off-by:
-
Franky Lin authored
The usage of state decrease readability. Optimize the code flow to get rid of it Reviewed-by:
Roland Vossen <rvossen@broadcom.com> Reviewed-by:
-
Franky Lin authored
Make a proper use of private data area of net device by storing interface related data structure instead of generic driver data Reported-by:
Johannes Berg <johannes@sipsolutions.net> Reviewed-by:
Franky (Zhenhui) Lin <frankyl@broadcom.com> Signed-off-by:
-
Franky Lin authored
Use brcmf_add_if for primary and virtual net device interfaces. This is part of the net device interface clean up for fullmac. Reviewed-by:
-
Arend van Spriel authored
In the receive path the frames are obtained from the dma using multiple sk_buff that were linked using the skb next pointer. This has been changed and it now used sk_buff lists and skb_queue functions instead. Reported-by:
-
Arend van Spriel authored
The functions ltoh16_buf() and htol16_buf() have been renamed to le16_to_cpu_buf() and cpu_to_le16_buf() for more clarity what it does. Reported-by:
Joe Perches <joe@perches.com> Reported-by:
-
Roland Vossen authored
The driver currently only supports STA operation. However, in brcms_ops_add_interface() also AP and SSID mode were accepted. Reviewed-by:
-
Roland Vossen authored
Monitor mode functionality (not functional yet) requires transmit to be muted after ops_start() is called, transmit is unmuted when the first interface is added. Reviewed-by:
-
Roland Vossen authored
Upon ops_start(), a Mac80211 driver should enable receive functionality to support monitor mode. Also, upon ops_stop(), it should disable rx. Driver did not follow this rule so code has been changed. Reported-by:
-
Roland Vossen authored
Transmit was muted in two ways: full mute and a partial mute called 'pre ism cac time' mute. But, this 'pre ism cac time' mute was done at one place in the code (when tx_mute == false), and overridden later on in another place in code. To fix this, the 'pre ism cac time' mute has been replaced by a non mute. Reviewed-by:
-
Roland Vossen authored
Transmit can only be muted if the mac core is enabled. When brcms_b_init() is called, the mac core is suspended. Brcms_b_init() calls a transmit mute function that requires an enabled mac core. This code path is never taken, but would have been taken in subsequent patches. Reviewed-by:
-
Roland Vossen authored
Softmac would bring its interface down on an RF kill switch condition, without Mac80211 intervention. Because Mac80211 should be the only party initiating interfaces going up and down, this functionality has been removed. Reviewed-by:
-
Roland Vossen authored
Softmac would bring its interface down on a certain Minimum Power Save related condition, without Mac80211 intervention. Because Mac80211 should be the only party initiating interfaces going up and down, this functionality has been removed. All notions of 'MPC' have been removed in the code as well. Reviewed-by:
-
Roland Vossen authored
Several member variables were never read. Reviewed-by:
-
Roland Vossen authored
The chip init sequence enables MPC (Minimum Power Consumption), but the driver disables it after that. As there are no interfaces to enable this mode the related code is unused (member variable wlc->mpc is false). Reviewed-by:
-
Franky Lin authored
Use brcmf_del_if for primary and virtual net device interfaces. This is part of the net device interface clean up for fullmac. Reviewed-by:
-
Franky Lin authored
Watchdog timer should be able to be stopped even firmware is not loaded. Reviewed-by:
-
Johannes Berg authored
If there are no cooked monitor interfaces, there's no point in building the radiotap RX header for the frame and iterating the interface list. Signed-off-by:
-
Arend van Spriel authored
The functions in the receive patch of the fullmac now use sk_buff list and skb_queue_xx() functions instead of dealing with list pointers in the sk_buff directly. Reported-by:
-
Larry Finger authored
The latest vendor (non-mac80211) driver of 9/22/2011 shows some new device IDs for rtl8192cu. In addition, some typos in the table are fixed and one duplicate is removed. Signed-off-by:
-
Arend van Spriel authored
The function brcmu_format_hex() filled a string buffer with byte values from a data buffer. The calling function used this string buffer in a printk. Now the calling function uses the kernel function print_hex_dump_bytes(). Reported-by:
-
Arend van Spriel authored
The function brcmu_pktfrombuf was only used in the brcmfmac source and has been moved there. It has been refactored to match its use. Reported-by:
-
Arend van Spriel authored
Obviously the phy api should be used to interface with the phy. In rate.h a table within phy was accessed directly by declaring the table extern in rate.h itself. This patch fixes this using the provided api function to obtain the table reference. This bypass was found by a sparse warning on the table not being defined static. Reviewed-by:
-
Arend van Spriel authored
In main.c a couple of functions were not static although they were only locally used. Sparse gave warnings on them and these functions have been made static. Reviewed-by:
-
Arend van Spriel authored
The data from the sprom consists of u16 values stored in little endian notation over which a crc8 was determined. To validate this the buffer needed to be converted for big-endian systems. Reading the sprom data is now done per byte so conversion is only done after a successful crc8 check. Reviewed-by:
-
Arend van Spriel authored
The n-phy requested some srom variables that are no longer needed and consequently not present in the srom revision 8 and higher that this driver support. This code has been removed from the n-phy. Reviewed-by:
-
Roland Vossen authored
In case the hardware crashes, a reinitialization internal to the driver was performed. Since Mac80211 must be in the know of such an event as well, ieee80211_restart_hw() is now called. Reviewed-by:
-
Roland Vossen authored
struct brcmf_scan_results contained a 1 element array, but in reality the number of scan results can be 0 or more, as indicated by the count field in the same struct. Array has be redefined to be 0 elements length to indicate the array is purely for reference. Reported-by:
-
Roland Vossen authored
Moved function to where it is called and made it more readable. Reviewed-by:
-
Roland Vossen authored
Made code more readable. Reviewed-by:
-
Roland Vossen authored
Supported chips contain SROM rev 8 and upwards. Reported-by:
Hauke Mehrtens <hauke@hauke-m.de> Reviewed-by:
-
Arend van Spriel authored
The brcmsmac driver registered with mac80211 with HT capability info set to 40MHz intolerant. This cause any other station on the channel to be forced to use 20MHz. This flag has been removed. Reported-by:
-
Alwin Beukers authored
Removed defines from aiutils.h also present in soc.h. Reported-by:
-
Alwin Beukers authored
Signed-off-by:
-
Johannes Berg authored
There's an extra pair of parentheses here that is simply confusing because it implies a nesting that doesn't actually exist. Just remove it. Signed-off-by:
-
Dan Carpenter authored
My previous patch added a check to get_key() but missed a couple other places which need range checks. The problem here is that wifi drivers have different numbers of keys. The lower levels assume that they can have up to 4 default keys and 2 management keys but this driver only has the default keys so we could go past the end of the ->keys[] array. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Acked-by:
Samuel Ortiz <sameo@linux.intel.com> Signed-off-by:
-
Dan Carpenter authored
Wifi drivers can have up to 6 keys but the rndis_wlan only has 4 so it needs to have its own checks to make sure we don't go out of bounds. The add_key() function already checks but I added some checks to del_key() and set_default_key(). Signed-off-by:
Jussi Kivilinna <jussi.kivilinna@mbnet.fi> Signed-off-by:
-
Dan Carpenter authored
The code here treats very large values of "limit" as less than MAX_POWER_RATE because of the cast to int. We should do the compare as u32 instead. Signed-off-by:
-
