From: Leo Kim <leo.kim@xxxxxxxxx> This patch renames variable hSemHostIntDeinit to hif_sema_deinit to avoid CamelCase naming convention. Add to static because follow warning reported by kbuild system. - drivers/staging/wilc1000/host_interface.c:247:18: sparse: symbol 'hif_sema_deinit' was not declared. Should it be static? Signed-off-by: Leo Kim <leo.kim@xxxxxxxxx> Signed-off-by: Tony Cho <tony.cho@xxxxxxxxx> --- drivers/staging/wilc1000/host_interface.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index 9346d42..5900972 100644 --- a/drivers/staging/wilc1000/host_interface.c +++ b/drivers/staging/wilc1000/host_interface.c @@ -244,7 +244,7 @@ static WILC_MsgQueueHandle hif_msg_q; static struct semaphore hif_sema_thread; static struct semaphore hif_sema_driver; static struct semaphore hif_sema_wait_response; -struct semaphore hSemHostIntDeinit; +static struct semaphore hif_sema_deinit; struct timer_list g_hPeriodicRSSI; @@ -4249,7 +4249,7 @@ s32 host_int_init(struct host_if_drv **hif_drv_handler) if (clients_count == 0) { sema_init(&hif_sema_thread, 0); sema_init(&hif_sema_driver, 0); - sema_init(&hSemHostIntDeinit, 1); + sema_init(&hif_sema_deinit, 1); } sema_init(&hif_drv->hSemTestKeyBlock, 0); @@ -4333,7 +4333,7 @@ s32 host_int_deinit(struct host_if_drv *hif_drv) return 0; } - down(&hSemHostIntDeinit); + down(&hif_sema_deinit); terminated_handle = hif_drv; PRINT_D(HOSTINF_DBG, "De-initializing host interface for client %d\n", clients_count); @@ -4395,7 +4395,7 @@ s32 host_int_deinit(struct host_if_drv *hif_drv) clients_count--; terminated_handle = NULL; - up(&hSemHostIntDeinit); + up(&hif_sema_deinit); return s32Error; } @@ -4438,7 +4438,7 @@ void GnrlAsyncInfoReceived(u8 *pu8Buffer, u32 u32Length) int id; struct host_if_drv *hif_drv = NULL; - down(&hSemHostIntDeinit); + down(&hif_sema_deinit); id = ((pu8Buffer[u32Length - 4]) | (pu8Buffer[u32Length - 3] << 8) | (pu8Buffer[u32Length - 2] << 16) | (pu8Buffer[u32Length - 1] << 24)); hif_drv = get_handler_from_id(id); @@ -4447,13 +4447,13 @@ void GnrlAsyncInfoReceived(u8 *pu8Buffer, u32 u32Length) if (!hif_drv || hif_drv == terminated_handle) { PRINT_D(HOSTINF_DBG, "Wifi driver handler is equal to NULL\n"); - up(&hSemHostIntDeinit); + up(&hif_sema_deinit); return; } if (!hif_drv->strWILC_UsrConnReq.pfUserConnectResult) { PRINT_ER("Received mac status is not needed when there is no current Connect Reques\n"); - up(&hSemHostIntDeinit); + up(&hif_sema_deinit); return; } @@ -4471,7 +4471,7 @@ void GnrlAsyncInfoReceived(u8 *pu8Buffer, u32 u32Length) if (s32Error) PRINT_ER("Error in sending message queue asynchronous message info: Error(%d)\n", s32Error); - up(&hSemHostIntDeinit); + up(&hif_sema_deinit); } void host_int_ScanCompleteReceived(u8 *pu8Buffer, u32 u32Length) -- 1.9.1 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel