summaryrefslogtreecommitdiffstats
path: root/drivers/staging/csr
Commit message (Collapse)AuthorAgeFilesLines
* staging/csr: Use kmemdup rather than duplicating its implementationPeter Huewe2013-01-291-3/+3
| | | | | | | | | Found with coccicheck. The semantic patch that makes this change is available in scripts/coccinelle/api/memdup.cocci. Signed-off-by: Peter Huewe <peterhuewe@gmx.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging/csr: Fix dereference before checkPeter Huewe2013-01-071-15/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | Smatch complains about some pointers that are dereferenced before being checked: drivers/staging/csr/sme_sys.c:285 CsrWifiRouterCtrlHipReqHandler() warn: variable dereferenced before check 'priv' (see line 283) drivers/staging/csr/sme_sys.c:1503 CsrWifiRouterMaPacketReqHandler() warn: variable dereferenced before check 'priv' (see line 1501) drivers/staging/csr/sme_sys.c:2062 CsrWifiRouterCtrlPeerDelReqHandler() warn: variable dereferenced before check 'priv' (see line 2059) drivers/staging/csr/sme_sys.c:2477 CsrWifiRouterCtrlPeerAddReqHandler() warn: variable dereferenced before check 'priv' (see line 2474) drivers/staging/csr/sme_sys.c:3045 CsrWifiRouterCtrlWapiRxPktReqHandler() warn: variable dereferenced before check 'priv' (see line 3039) We put the check before the dereferencing and prevent an oops and fix the warning. Signed-off-by: Peter Huewe <peterhuewe@gmx.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove unneeded call to memset().Cyril Roelandt2013-01-071-1/+0
| | | | | | | | In uf_send_pkt_to_encrypt(), the memory area zeroed by this call to memset() is overwritten by a call to memcpy() a few instructions later, so it is not needed. Signed-off-by: Cyril Roelandt <tipecaml@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove CONFIG_HOTPLUG ifdefsBill Pemberton2012-11-211-6/+0
| | | | | | | | Remove conditional code based on CONFIG_HOTPLUG being false. It's always on now in preparation of it going away as an option. Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: csr_wifi_hip_unifi_signal_names: Fix pointer positionHojung Youn2012-11-061-1/+1
| | | | | | | | Fixed pointers' positions in the csr_wifi_hip_unifi_signal_names.c file that were identifed by checkpatch.pl tool. Signed-off-by: Hojung Youn <amoc.yn@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: csr_wifi_hip_unifi_signal_names: fix whitespaced paddingsHojung Youn2012-11-061-11/+11
| | | | | | | | Fixed whitespace paddings in the csr_wifi_hip_unifi_signal_names.c file that were, though, not identified by checkpatch.pl tool. Signed-off-by: Hojung Youn <amoc.yn@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: csr_wifi_hip_unifi_signal_names: fix bracketsHojung Youn2012-11-061-6/+1
| | | | | | | | | Fixed brackets' positions in the csr_wifi_hip_unifi_signal_names.c file that were identified by checkpatch.pl tool. Some brackets are removed which are not needed by the rule of CodingStyle documentation. Signed-off-by: Hojung Youn <amoc.yn@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: csr_wifi_hip_unifi_signal_names: fix indentHojung Youn2012-11-061-22/+22
| | | | | | | | | | Fixed indent style in the csr_wifi_hip_unifi_signal_names.c file that were identified by checkpatch.pl tool. All whitespaced indents are converted into tab characters, except ones in the topmost commentation. Signed-off-by: Hojung Youn <amoc.yn@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: csr_wifi_nme_task.h: remove function prototypesDevendra Naga2012-11-011-3/+0
| | | | | | | | there are function definitions missing for these prototypes, so remove all 3 function protos. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: csr_wifi_nme_ap_lib.h: remove the function prototypesDevendra Naga2012-11-011-18/+0
| | | | | | | | these function prototypes are missing the function definition, remove them all. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: csr_wifi_nme_lib.h: remove function prototypesDevendra Naga2012-11-011-54/+0
| | | | | | | | the function definitions are missing for these prototypes, so remove all these. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: sme_blocking.c: add missing check for return of sme_wait_for_replyDevendra Naga2012-11-011-0/+2
| | | | | | | | | sme_ap_config misses a check of the return of the sme_wait_for_reply as all other functions do here, for the return of sme_wait_for_reply add the check and fail if it returns other than 0. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: sme_blocking.c: remove braces around single statement blocksDevendra Naga2012-11-011-145/+74
| | | | | | | remove the braces around single statement if blocks Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: sme_blocking.c: replace spaces with the tabs in sme_mgt_tspecDevendra Naga2012-11-011-19/+19
| | | | | | | replace spaces at start of line with tabs in sme_mgt_tspec Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: sme_blocking.c: replace spaces with tabsDevendra Naga2012-11-011-24/+24
| | | | | | | replace spaces at the beginning of the line with tabs in sme_mgt_packet_filter_set Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: inet.c: replaces spaces with tabsDevendra Naga2012-11-011-4/+4
| | | | | | | replace spaces with tabs at the start of line Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: inet.c: single statement if blocks doesn't need bracesDevendra Naga2012-11-011-4/+2
| | | | | | | remove braces to single statement if blocks Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: csr_log_configure.h: remove a bunch of function protosDevendra Naga2012-10-301-87/+0
| | | | | | | | all protos that are removed does not have a function definition so remove all of them together Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove csr_lib.hDevendra Naga2012-10-302-181/+0
| | | | | | | | remove this header file as no struct or no function is defined else where in csr. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: os.c: remove braces around single statement blocksDevendra Naga2012-10-301-8/+4
| | | | | | | remove all braces around single statement if blocks Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: os.c: remove col variableDevendra Naga2012-10-301-4/+2
| | | | | | | | | | | theres no point in checking the col variable, its always zero, and wont print a new line, actually its better to have a new line after a set of characters printed, instead of confusing with the concatenated characters when called multiple times at a time. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: drv.c: replace spaces with tabs in uf_destroy_device_nodesDevendra Naga2012-10-301-4/+4
| | | | | | | replace spaces at the start of line with tabs Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: drv.c: replace spaces with tabsDevendra Naga2012-10-301-27/+26
| | | | | | | replace spaces at the start of the line with tabs Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: io.c: replace spaces with tabsDevendra Naga2012-10-301-43/+43
| | | | | | | replace the spaces with the tabs at the start of line Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: Remove struct CsrEventSeongJae Park2012-10-301-14/+0
| | | | | | | Nobody use struct CsrEvent. So, remove it. Signed-off-by: SeongJae Park <sj38.park@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging : csr: Fix typo in csr/netdev.cMasanari Iida2012-10-291-7/+7
| | | | | | | Correct spelling typos in csr/netdev.c Signed-off-by: Masanari Iida <standby24x7@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove func_exit_r macroDevendra Naga2012-10-297-109/+0
| | | | | | | | | | | | | | | | this macro is used for debugging purposes, it actually defined as if (unifi_debug >= 5) { printk("unifi: <= %s %d\n", __FUNCTION__, (int)rc); } which produces too many of those prints if the unifi_debug is >=5. remove these calls and the macro itself altogether Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove func_exit macroDevendra Naga2012-10-2918-200/+0
| | | | | | | | | | | | | | | | this macro is used for debugging purposes, it actually defined as if (unifi_debug >= 5) { printk("unifi: <= %s\n", __FUNCTION__); } which produces too many of those prints if the unifi_debug is >=5. remove these calls and the macro itself altogether Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove func_enter macroDevendra Naga2012-10-2917-214/+0
| | | | | | | | | | | | | | | | this macro is used for debugging purposes, it actually defined as if (unifi_debug >= 5) { printk("unifi => %s\n", __FUNCTION__); } which produces too many of those prints if the unifi_debug is >=5. remove these calls and the macro itself altogether Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: csr: remove CSRMIN() macroGreg Kroah-Hartman2012-10-262-3/+1
| | | | | | Use the in-kernel min_t() macro for the one place it was being used. Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: csr: csr_macro.h: remove unused macrosGreg Kroah-Hartman2012-10-261-65/+0
| | | | | | Remove a bunch of unused #defines. Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: csr: remove __cplusplus nonsense from the .h filesGreg Kroah-Hartman2012-10-2666-611/+0
| | | | | | In the kernel, we use C. Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove unused macros and prototypesDevendra Naga2012-10-261-73/+0
| | | | | | | | this file includes prototypes that doesnt have a function for them, and some macros which are never used, remove them instead Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove unused macros/ prototypes in csr_sched.hDevendra Naga2012-10-261-114/+0
| | | | | | | | there are macros and functions that are not used, or their function is missing, so remove them. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove csrPanic and CSR_LOG_ASSERT and CSR_LOG_ASSERT_ENABLEDevendra Naga2012-10-267-91/+1
| | | | | | | | these are not called at anypoint or enabled at anyplace in the code, remove them Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove all prototypes of not defined functionsDevendra Naga2012-10-261-82/+0
| | | | | | | | these prototypes doesn't have function definitions in any of the c files Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove CsrMutexCreate functionDevendra Naga2012-10-262-42/+0
| | | | | | | nobody are calling this function in here, so remove this Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove CsrMutexDestroy functionDevendra Naga2012-10-262-28/+0
| | | | | | | this function never gets called from anywhere, so remove this too. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove CsrMutexUnlock functionDevendra Naga2012-10-262-55/+0
| | | | | | | Nobody is using this function inside the csr directory, so remove this function also Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove CsrMutexLock functionDevendra Naga2012-10-262-43/+0
| | | | | | | nobody inside the csr directory is calling this function, so remove this Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: csr_log.h: Fix coding styleSeongJae Park2012-10-251-85/+80
| | | | | | | Fix coding style for csr_log.h Signed-off-by: SeongJae Park <sj38.park@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: csr: bh.c: fixing spaces coding style issuesChihau Chau2012-10-251-10/+9
| | | | | | | | Fixed some coding style issues replacing spaces for tab at the beginning of some lines Signed-off-by: Chihau Chau <chihau@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove panic at locking the mutexDevendra Naga2012-10-251-1/+0
| | | | | | | | | when down_interruptible fail, means a signal occur, or any other failure we are panicing, and it seems that we should not panic, instead we would have done a spinlock, but currently removing the panic call. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove unused function prototypesDevendra Naga2012-10-241-85/+0
| | | | | | | | remove some of the function prototypes , they dont have a definition of the function Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: remove the CsrTime typedef altogetherDevendra Naga2012-10-2410-48/+35
| | | | | | | remove CsrTime typedef and replace all the users with the u32 Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: Remove CsrThreadHandleSeongJae Park2012-10-232-51/+0
| | | | | | | | | CsrThreadHandle is typedef of struct CsrThread. Some functions in csr_framework_ext.h use it as parameter. But, nobody call them. Signed-off-by: SeongJae Park <sj38.park@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: csr: Remove struct CsrThreadSeongJae Park2012-10-231-5/+0
| | | | | | | Nobody use struct CsrThread. So, remove it. Signed-off-by: SeongJae Park <sj38.park@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: csr: fix code indent coding style issues in csr_wifi_hip_signals.hSunhee Hwang2012-10-221-4/+4
| | | | | | | | This is a patch that fixes up code indent coding style warnings in the csr_wifi_hip_signals.h found by checkpatch.pl tool. Signed-off-by: Sunhee Hwang <sunheehwang821@gmail.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: csr: csr_wifi_router_ctrl_sef.c: fix line over 80 charactersJugwan Eom2012-10-221-32/+34
| | | | | | | | This patch fixes line over 80 characters problem found by checkpatch.pl tool. Signed-off-by: Jugwan Eom <zugwan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: csr: csr_wifi_router_ctrl_sef.c: fix tab messJugwan Eom2012-10-221-31/+31
| | | | | | | This patch converts spaces to tabs. Signed-off-by: Jugwan Eom <zugwan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
OpenPOWER on IntegriCloud