summaryrefslogtreecommitdiffstats
path: root/drivers/staging/wilc1000
Commit message (Collapse)AuthorAgeFilesLines
...
| * staging: wilc1000: rename pu8Buffer in handle_scanChaehyun Lim2016-04-281-6/+6
| | | | | | | | | | | | | | This patch renames pu8Buffer to buffer to avoid camelcase. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: rename u32WidsCount in handle_scanChaehyun Lim2016-04-281-29/+29
| | | | | | | | | | | | | | This patch renames u32WidsCount to index to avoid camelcase. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: rename strWIDList in handle_scanChaehyun Lim2016-04-281-24/+24
| | | | | | | | | | | | | | This patch renames strWIDList to wid_list to avoid camelcase. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: rename pstrHostIFscanAttr in handle_scanChaehyun Lim2016-04-281-28/+27
| | | | | | | | | | | | | | This patch renames pstrHostIFscanAttr to scan_info to avoid camelcase. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: rename Handle_ScanChaehyun Lim2016-04-281-2/+2
| | | | | | | | | | | | | | This patch renames Handle_Scan to handle_scan to avoid camelcase. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: remove block scope braces and fix indentationChaehyun Lim2016-04-281-7/+5
| | | | | | | | | | | | | | | | This patch removes unnecessary block scope braces and fix indentation of the codes. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: move Handle_ScanDone function declarationChaehyun Lim2016-04-281-3/+1
| | | | | | | | | | | | | | | | Handle_ScanDone function declaration is moved to front of host_interface.c file. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: rename goto label in handle_cfg_paramChaehyun Lim2016-04-281-19/+19
| | | | | | | | | | | | | | | | This patch changes goto label from ERRORHANDLER to unlock. unlock is a more meaningful name than ERRORHANDLER. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: adds a check routine when performed del_key()Leo Kim2016-04-281-3/+6
| | | | | | | | | | | | | | | | | | | | | | This patch is adds a check routine when performed del_key(). We was find this situation while test the 'rmmod sdio'. That is received inform a .del_key cmd from cfg80211. If is not exist wep keys, must be ignore a wilc_remove_wep_key() function. Thus, adds a check routine that key length of wep. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: adds a cfg80211_disconnected() functionLeo Kim2016-04-281-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | This patch is adds a cfg80211_disconnected() when connection is lost already. We was find this situation while test the 'rmmod sdio'. SDIO remove function are include both remove mac_close and unregister net_device. That is received one more a disconnect cmd from cfg80211. Driver was already performed disconnect. If wilc->close value was set to true, adds a call to cfg80211_disconnected(). Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: add set to null after release firmwareLeo Kim2016-04-281-1/+3
| | | | | | | | | | | | | | This patch add set to null after release firmware. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes unnecessary test codeLeo Kim2016-04-281-9/+0
| | | | | | | | | | | | | | | | This patch removes unnecessary test code that mac address hardcoding setting. This test code is support to old firmware. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: txq_thread_started: replaces a semaphore with a completionLeo Kim2016-04-282-7/+6
| | | | | | | | | | | | | | | | | | This patch replaces a semaphore(txq_thread_started) with a completion. Struct semaphore txq_thread_started is used to signal completion of txq thread. It is better to use completion than semaphore for this case. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes duplicate del_timer_syncLeo Kim2016-04-281-1/+0
| | | | | | | | | | | | | | | | This patch removes del_timer_sync(&periodic_rssi) which was already performed this action. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: changes logic when happened mq_send failLeo Kim2016-04-281-15/+18
| | | | | | | | | | | | | | | | | | | | | | This patch changes logic when happened mq_send fail. The check as a result value that mq_send fail. But, regardless of the result value then perform a semaphore up. That is an occur that semaphore locking. Add to 'else' routine and move to 'else' position a semaphore up. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: use mutex instead of struct semaphore hSemScanReqChaehyun Lim2016-03-292-6/+6
| | | | | | | | | | | | | | | | This patch replaces struct semaphore hSemScanReq with struct mutex scan_req_lock. It is better to use mutex than semaphore. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: remove unused struct semaphore SemHandleUpdateStatsChaehyun Lim2016-03-292-2/+0
| | | | | | | | | | | | | | | | struct semaphore SemHandleUpdateStats is defined but never used in this driver, so just remove it. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: change data type of wid argument in wilc_wlan_cfg_get_valChaehyun Lim2016-03-292-3/+3
| | | | | | | | | | | | | | | | | | | | This patch changes data type of wid argument in wilc_wlan_cfg_get_val from u32 to u16. It is better to change data type of wid because wid has one of enum WID_T that is data type of u16. And, there is no need to use u16 type casting when calling wilc_wlan_cfg_get_wid_value function. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: change data type of wid argument in wilc_wlan_cfg_getChaehyun Lim2016-03-292-4/+3
| | | | | | | | | | | | | | | | | | | | This patch changes data type of wid argument in wilc_wlan_cfg_get from u32 to u16. It is better to change data type of wid because wid has one of enum WID_T that is data type of u16. And, there is no need to use u16 type casting when calling wilc_wlan_cfg_get_wid function. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: change data type of wid argument in wilc_wlan_cfg_setChaehyun Lim2016-03-292-3/+3
| | | | | | | | | | | | | | | | | | | | This patch changes data type of wid argument in wilc_wlan_cfg_set from u32 to u16. It is better to change data type of wid because wid has one of enum WID_T that is data type of u16. And, there is no need to use u16 type casting when calling wilc_wlan_cfg_set_wid function. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes unused local variableLeo Kim2016-03-281-18/+0
| | | | | | | | | | | | | | | | | | | | | | This patch removes unused local variable. This variable is operation definition that back off from sending packets for some time. However, that has been deleted operation code. That is removes all relative code. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes define USE_TX_BACKOFF_DELAY_IF_NO_BUFFERSLeo Kim2016-03-281-8/+0
| | | | | | | | | | | | | | | | This patch removes define USE_TX_BACKOFF_DELAY_IF_NO_BUFFERS and use it's feature codes. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes unused defineLeo Kim2016-03-282-11/+0
| | | | | | | | | | | | | | This patch removes unused define. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: replaces memcmp with ether_addr_equal_unalignedLeo Kim2016-03-281-2/+4
| | | | | | | | | | | | | | | | | | This patch replaces memcmp with ether_addr_equal_unaligned. Warning reported by checkpatch.pl - Prefer ether_addr_equal() or ether_addr_equal_unaligned() over memcmp() Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes unused debug flagsLeo Kim2016-03-281-12/+0
| | | | | | | | | | | | | | This patch removes unused debug flags. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes unused dead codesLeo Kim2016-03-281-8/+0
| | | | | | | | | | | | | | This patch removes unused dead codes that define custom feature. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: replaces frame_type with type of struct frame_regLeo Kim2016-03-283-7/+7
| | | | | | | | | | | | | | This patch replaces frame_type with type of struct frame_reg. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: replaces g_struct_frame_reg with frame_regLeo Kim2016-03-283-11/+11
| | | | | | | | | | | | | | This patch replaces g_struct_frame_reg with frame_reg. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes typedef of struct struct_frame_regLeo Kim2016-03-281-4/+3
| | | | | | | | | | | | | | | | This patch removes typedef of struct struct_frame_reg. Renames the struct_frame_reg to frame_reg as well. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: wilc_frame_register: removes unused hif_drvLeo Kim2016-03-281-6/+0
| | | | | | | | | | | | | | | | This patch removes unused hif_drv of wilc_frame_register function. It's perform an unnecessary null check and debug print log. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes WIRELESS_EXTLeo Kim2016-03-281-1/+0
| | | | | | | | | | | | | | | | This patch removes WIRELESS_EXT. Does not used the WIRELESS_EXT define. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: use completion instead of struct semaphore hif_sema_driverChaehyun Lim2016-03-281-5/+5
| | | | | | | | | | | | | | | | | | | | struct semaphore hif_sema_driver is used to signal completion of host interface message. This patch replaces struct semaphore hif_sema_driver with struct completion hif_driver_comp. It is better to use completion than semaphore for this case. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: use completion instead of struct semaphore hif_sema_threadChaehyun Lim2016-03-281-4/+4
| | | | | | | | | | | | | | | | | | | | struct semaphore hif_sema_thread is used to signal completion of host interface thread. This patch replaces struct semaphore hif_sema_thread with struct completion hif_thread_comp. It is better to use completion than semaphore for this case. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: Remove camel case in variable names.Anchal Jain2016-03-281-3/+3
| | | | | | | | | | | | | | | | Remove a problem detect by checkpatch.pl CHECK: Avoid CamelCase: <srcAdd> Signed-off-by: Anchal Jain <anchalj109@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: use mutex instead of struct semaphore hif_sema_deinitChaehyun Lim2016-03-281-9/+9
| | | | | | | | | | | | | | | | | | This patch replaces struct semaphore hif_sema_deinit with struct mutex hif_deinit_lock. It is better to use mutex because mutex gives better performance than semaphore. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: Replaced comparison to NULL statementsRoger H. Newell2016-03-282-3/+3
| | | | | | | | | | | | | | | | | | This patch corrects checks generated by checkpatch.pl by replacing comparison to null statements with equivalent statements in the form "x" or "!x" Signed-off-by: Roger H. Newell <newell.roger@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: Removed braces from single block statementsRoger H. Newell2016-03-282-21/+13
| | | | | | | | | | | | | | | | | | This patch corrects warnings generated by checkpatch.pl by removing braces from single block statements. Signed-ff-by: Roger H. Newell <newell.roger@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: replace sem_test_key_block with a completionAlison Schofield2016-03-282-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | Semaphore sem_test_key_block is used to signal completion of its host interface message. Since the thread locking this semaphore will have to wait, completions are the preferred mechanism and will offer a performance improvement. Signed-off-by: Alison Schofield <amsfield22@gmail.com> Reviewed-by: Arnd Bergmann <arnd@arndb.de> Tested-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: replace sem_test_disconn_block with a completionAlison Schofield2016-03-282-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | Semaphore sem_test_disconn_block is used to signal completion of its host interface message. Since the thread locking this semaphore will have to wait, completions are the preferred mechanism and will offer a performance improvement. Signed-off-by: Alison Schofield <amsfield22@gmail.com> Reviewed-by: Arnd Bergmann <arnd@arndb.de> Tested-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: replace semaphore sem_get_rssi with a completionAlison Schofield2016-03-282-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | Semaphore sem_get_rssi is used to signal completion of its host interface message. Since the thread locking this semaphore will have to wait, completions are the preferred mechanism and will offer a performance improvement. Signed-off-by: Alison Schofield <amsfield22@gmail.com> Reviewed-by: Arnd Bergmann <arnd@arndb.de> Tested-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: replace semaphore sem_inactive_time with a completionAlison Schofield2016-03-282-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | Semaphore sem_inactive_time is used to signal completion of its host interface message. Since the thread locking this semaphore will have to wait, completions are the preferred mechanism and will offer a performance improvement. Signed-off-by: Alison Schofield <amsfield22@gmail.com> Reviewed-by: Arnd Bergmann <arnd@arndb.de> Tested-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: Fix lines over 80 charactersAnchal Jain2016-03-281-7/+11
| | | | | | | | | | | | | | | | Break lines so that they do not exceed 80 characters. Problem found using checkpatch. Signed-off-by: Anchal Jain <anchalj109@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: else is not generally useful after a break or returnAnchal Jain2016-03-281-11/+8
| | | | | | | | | | | | | | | | Remove else after a break. Because else is generally not useful after a break or return. Signed-off-by: Anchal Jain <anchalj109@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: use completion instead of struct semaphore ↵Chaehyun Lim2016-03-281-10/+10
| | | | | | | | | | | | | | | | | | | | | | hif_sema_wait_response This patch replaces struct semaphore hif_sema_wait_response with struct completion hif_wait_response. In case of struct hif_sema_wait_response, it better to use completion than semaphore. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes an unnecessary if-conditionLeo Kim2016-03-281-2/+0
| | | | | | | | | | | | | | | | This patch removes an unnecessary if-condition. Regardless of an if-condition is performed unconditionally '_end_' statement. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes goto definitions from wilc_wlan_firmware_downloadLeo Kim2016-03-281-6/+2
| | | | | | | | | | | | | | | | | | This patch removes goto definitions from wilc_wlan_firmware_download function. Goto '_fail_1' feature is error return. It returns error type directly without result variable replacement as well. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: changes an ambiguous debug messagesLeo Kim2016-03-281-1/+1
| | | | | | | | | | | | | | | | This patches changes an ambiguous debug messages. The device types are both SDIO or SPI. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes duplicate wilc variable settingLeo Kim2016-03-281-6/+4
| | | | | | | | | | | | | | | | | | This patches removes duplicate wilc variable setting. This value has already been set to wl variable previously. Replace wilc with wl as well. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: removes duplicate vif variable settingLeo Kim2016-03-281-1/+0
| | | | | | | | | | | | | | | | This patches removes duplicate vif variable setting. This value has already been set previously. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: wilc1000: wilc_spi.c: removes debug print logLeo Kim2016-03-281-3/+0
| | | | | | | | | | | | | | This patches removes unnecessary debug print logs. Signed-off-by: Leo Kim <leo.kim@atmel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
OpenPOWER on IntegriCloud