diff options
| author | Leo Kim <leo.kim@atmel.com> | 2015-12-21 14:18:25 +0900 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-12-21 13:20:04 -0800 |
| commit | cb3b05c3845815e1fdeff94fcd5e8a57ceff702e (patch) | |
| tree | 3467df0f934047334a98e7fe0c214470931d7ce8 | |
| parent | 653bb46301617e39e720e60fa2e0a01f946c9a6e (diff) | |
| download | linux-cb3b05c3845815e1fdeff94fcd5e8a57ceff702e.tar.gz linux-cb3b05c3845815e1fdeff94fcd5e8a57ceff702e.zip | |
staging: wilc1000: wilc_init(): fixes inconsistent returns
This patch fixes the warning reported by smatch.
- wilc_init() warn: inconsistent returns 'sem:&hif_drv->sem_cfg_values'
No need to up the sema here since down was not called before get here.
Signed-off-by: Leo Kim <leo.kim@atmel.com>
Signed-off-by: Glen Lee <glen.lee@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/wilc1000/host_interface.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index 2289ba344300..6dd3076a78e1 100644 --- a/drivers/staging/wilc1000/host_interface.c +++ b/drivers/staging/wilc1000/host_interface.c @@ -3873,7 +3873,6 @@ s32 wilc_init(struct net_device *dev, struct host_if_drv **hif_drv_handler) return result; _fail_timer_2: - up(&hif_drv->sem_cfg_values); del_timer_sync(&hif_drv->connect_timer); del_timer_sync(&hif_drv->scan_timer); kthread_stop(hif_thread_handler); |