Skip to content

Commit

Permalink
staging: wilc1000: Remove 'if' statement, which is always false
Browse files Browse the repository at this point in the history
- 'if' which is always false is removed
- variable associated with this 'if' is deleted

Signed-off-by: Chandra S Gorentla <csgorentla@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Chandra S Gorentla authored and Greg Kroah-Hartman committed Dec 18, 2015
1 parent cc28e4b commit ff5d40a
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions drivers/staging/wilc1000/wilc_msgqueue.c
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,6 @@ int wilc_mq_recv(WILC_MsgQueueHandle *pHandle,
u32 *pu32ReceivedLength)
{
Message *pstrMessage;
int result = 0;
unsigned long flags;

if ((!pHandle) || (u32RecvBufferSize == 0)
Expand All @@ -135,12 +134,6 @@ int wilc_mq_recv(WILC_MsgQueueHandle *pHandle,

down(&pHandle->hSem);

/* other non-timeout scenarios */
if (result) {
PRINT_ER("Non-timeout\n");
return result;
}

if (pHandle->bExiting) {
PRINT_ER("pHandle fail\n");
return -EFAULT;
Expand Down Expand Up @@ -174,5 +167,5 @@ int wilc_mq_recv(WILC_MsgQueueHandle *pHandle,

spin_unlock_irqrestore(&pHandle->strCriticalSection, flags);

return result;
return 0;
}

0 comments on commit ff5d40a

Please sign in to comment.