- 28 Jun, 2011 39 commits
-
-
Tim Retout authored
Use consistent spacing in array indexing, and a whitespace fix. Signed-off-by:
Tim Retout <tim@retout.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Joe Perches authored
Signed-off-by:
Joe Perches <joe@perches.com> Signed-off-by:
-
Ralf Thielow authored
Fixed some whitespace coding style issues. Signed-off-by:
Ralf Thielow <ralf.thielow@googlemail.com> Signed-off-by:
-
Ralf Thielow authored
Fixed a declaration coding style issue. Signed-off-by:
-
Jesper Juhl authored
It was pointed out by 'make versioncheck' that include of linux/version.h is not needed in drivers/staging/mei/main.c . This patch removes it. Signed-off-by:
Jesper Juhl <jj@chaosbits.net> Signed-off-by:
-
Tomas Winkler authored
Fix warning: reported in http://marc.info/?l=linux-kernel&m=130812960402606&w=2 drivers/staging/mei/interrupt.c: warning: 'buffer' may be used uninitialized in this function: => 198 It is a real issue and wrong path in execution is taken when list is empty or (cl && _mei_irq_thread_state_ok(cl, mei_hdr)) evaluates to false Signed-off-by:
Tomas Winkler <tomas.winkler@intel.com> Signed-off-by:
-
Tomas Winkler authored
1. remove list used for loop. There were only 2 loops used in non time critical places so we can safely unroll them 2. normalize functions names operating on io_list to mei_io_list_<op> 3. rename mei_fe_same_id to mei_cl_cmp_id used for comparing list elements containing struct mei_cl 4. group together io_list functions in the header file Signed-off-by:
Oren Weil <oren.jer.weil@intel.com> Signed-off-by:
-
Tomas Winkler authored
Signed-off-by:
-
Tomas Winkler authored
remove write only/unsed variables mei_dev.write_hang and mei_io_lis.tdevice_extension Signed-off-by:
-
Tomas Winkler authored
use type bool for boolean variables in struct mei_dev this should save some space providing boolean is 8 bits Signed-off-by:
-
Tomas Winkler authored
All global functions must start with mei_ to reduce the risk of name colisions Signed-off-by:
-
Tomas Winkler authored
Signed-off-by:
-
Jouni Malinen authored
Implement cfg80211_ops set_pmksa, del_pmksa, and flush_pmksa to enable PMKSA caching support. kvalo: backported from ath6kl-cleanup tree Signed-off-by:
Jouni Malinen <jouni@qca.qualcomm.com> Signed-off-by:
Kalle Valo <kvalo@qca.qualcomm.com> Signed-off-by:
-
Kalle Valo authored
Add testmode support for running low level hardware tests. The testmode is enabled by setting testmode module parameter to 1. For now only a simple command passing is supported. More advanced support will be implemented later. Signed-off-by:
-
Kalle Valo authored
cfg80211 exports wiphy->fw_version to user space via ethtool interface. The obligatory screenshot: $ sudo ethtool -i wlan0 driver: ath6kl_hifdev version: 2.6.39-rc4+ firmware-version: 3:1:1:149 bus-info: mmc0:0001:1 $ Signed-off-by:
-
Jesper Juhl authored
It was pointed out by 'make versioncheck' that some includes of linux/version.h are not needed in drivers/staging/ath6kl/. This patch removes them. Signed-off-by:
-
Kalle Valo authored
Drivers should not request firmware during resume. Fix ath6kl to cache the firmware instead. Signed-off-by:
-
Kalle Valo authored
When ath6kl module was resumed while a scan was ongoing, for example during suspend, the driver would crash in ar6k_cfg80211_scanComplete_event(): [26581.586440] Call Trace: [26581.586440] [<f99ffeda>] ? ar6k_cfg80211_scanComplete_event+0xaa/0xaa [ath6kl] [26581.586440] [<f9a0a020>] wmi_iterate_nodes+0xb/0xd [ath6kl] [26581.586440] [<f99ffe78>] ar6k_cfg80211_scanComplete_event+0x48/0xaa [ath6kl] [26581.586440] [<f9a038ae>] ar6000_close+0x77/0x7e [ath6kl] [26581.586440] [<c139c25d>] __dev_close_many+0x87/0xab [26581.586440] [<c139c30a>] dev_close_many+0x54/0xab [26581.586440] [<c139c437>] rollback_registered_many+0xa5/0x19e [26581.586440] [<c139c595>] rollback_registered+0x23/0x2f [26581.586440] [<c139c5ed>] unregister_netdevice_queue+0x4c/0x69 [26581.586440] [<c139c6b2>] unregister_netdev+0x18/0x1f [26581.586440] [<f9a00d4c>] ar6000_destroy+0xf8/0x115 [ath6kl] [26581.586440] [<f9a0c765>] ar6k_cleanup_module+0x20/0x29 [ath6kl] [26581.586440] [<c1062843>] sys_delete_module+0x181/0x1d9 [26581.586440] [<c105876b>] ? lock_release_holdtime+0x2b/0xcd [26581.586440] [<c10b55dc>] ? sys_munmap+0x3b/0x42 [26581.586440] [<c14a99dc>] ? restore_all+0xf/0xf [26581.586440] [<c14aeb6c>] sysenter_do_call+0x12/0x32 [26581.586440] Code: 89 53 6c 75 07 89 d8 e8 c0 ff ff ff 89 f0 e8 2c f2 a9 c7 5b 5e 5d c3 55 89 e5 57 56 53 89 c3 83 ec 08 89 55 f0 8d 78 04 89 4d ec <8b> b0 b8 00 00 00 46 89 b0 b8 00 00 00 89 f8 e8 ae ed a9 c7 8b Fix the function not to iterate nodes when the scan is aborted. The nodes are already freed when the module is being unloaded. Patch "ath6kl: Fix a kernel panic furing suspend/resume" tried to fix this already but it wasn't enough as a pointer was still used even after the null check. This patch removes the null check entirely as the wmi structure is not accessed anymore during module unload. Also fix a bug where the status was checked as a bitfield with '&' operator. But it's not a bitfield, just a regular (enum like) value. Signed-off-by:
-
Greg Dietsche authored
Compile tested. remove unnecessary code that matches this coccinelle pattern if (...) return ret; return ret; Signed-off-by:
Greg Dietsche <Gregory.Dietsche@cuw.edu> Acked-by:
Franky Lin <frankyl@broadcom.com> Signed-off-by:
-
Roland Vossen authored
Change wl_cfg80211_suspend in fullmac to meet cfg80211 interface update. Signed-off-by:
Roland Vossen <rvossen@broadcom.com> Signed-off-by:
-
Arend van Spriel authored
Several source files in brcmsmac driver have checkpatch errors that have been removed (see below). ERROR: spaces required around that ':' (ctx:VxV) + (u8)NPHY_RSSI_SEL_TSSI_5G:(u8)NPHY_RSSI_SEL_TSSI_2G; -- ERROR: code indent should use tabs where possible + ^Ierr = initvars_table(base, vp, vars, count);$ -- Signed-off-by:
Arend van Spriel <arend@broadcom.com> Reviewed-by:
-
Arend van Spriel authored
The softmac source file main.c was passed through checkpatch to see what issues against coding style are still present. This patch removes the checkpatch errors listed below. ERROR: trailing whitespace +^Imemcpy(wlc->default_bss->rateset.rates, rs.rates, $ -- ERROR: spaces required around that ':' (ctx:VxV) + pval = arg ? (int *)arg:NULL; -- Signed-off-by:
-
Arend van Spriel authored
A checkpatch run was done over the complete source files to determine how much coding style issues there are in the driver. This patch fixes the checkpatch errors in fullmac driver brcmfmac listed below. ERROR: do not initialise globals to 0 or NULL +uint dhd_poll = false; -- ERROR: do not use assignment in if condition + if (!ifp && !(ifp = kmalloc(sizeof(dhd_if_t), GFP_ATOMIC))) { -- ERROR: do not use assignment in if condition + if (unlikely(!(wl_cfg80211_dev && \ -- ERROR: do not initialise statics to 0 or NULL + static bool iscan_pass_abort = false; -- Signed-off-by: -
Roland Vossen authored
Code cleanup. Function renaming. Triggered by feedback comment from John Linville, just like the other function renames that preceded this patch. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Renamed functions. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Signed-off-by:
Julian Calaby <julian.calaby@gmail.com> Reviewed-by:
-
Roland Vossen authored
Code cleanup. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Signed-off-by:
-
Sukesh Srikakula authored
Issue with disconnect at the time of suspend is fixed by properly setting connection status bits maintained by brcmfmac driver. Signed-off-by:
Sukesh Srikakula <sukeshs@broadcom.com> Reviewed-by:
-
Roland Vossen authored
Code cleanup. Removed one redundant include and reshuffled #include order as to include Linux header files first. Reported-by:
julie Sullivan <kernelmail.jms@gmail.com> Signed-off-by:
-
Arend van Spriel authored
Two fields were not used in the driver code so they have been removed from struct si_info definition. Signed-off-by:
-
Arend van Spriel authored
The device id is passed to ai_attach() and subsequent functions without actually being used. It can be removed. Signed-off-by:
-
Arend van Spriel authored
The function wlc_tbtt() has variable regs as parameter, but it is not used. Parameter has been removed from the function. Signed-off-by:
-
Arend van Spriel authored
This variable was declared and initialized but never used so it has been removed. Signed-off-by:
-
Arend van Spriel authored
The function ai_ispcie() is a static function that is currently being called from the c sources in the driver. Side-effect of the keyword is that warning will suppressed when this function is no longer used. As there is no assembly code using this function, the keyword is removed. Signed-off-by:
-
Franky Lin authored
Remove this debug only variable in fullmace driver to avoid compile warning when BCMDBG is off Signed-off-by:
-
Roland Vossen authored
Code cleanup. Number of includes has been reduced by only including what is necessary. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Moved softmac specific definitions into brcmsmac/types.h Removed unused definitions. Signed-off-by:
-
Roland Vossen authored
Code cleanup. Moved the typedefs to brcmsmac/types.h for source files that only require opaque pointer definitions. Next patches will decrease the number of #includes for each file. Signed-off-by:
-
- 09 Jun, 2011 1 commit
-
-
Greg Kroah-Hartman authored
This was done to resolve the conflicts that were in a number of files due to changes done upstream with others done in the staging-next branch. Signed-off-by:
-
