Commit 6d04d7a0 authored Dec 21, 2015 by Leo Kim Committed by Greg Kroah-Hartman Dec 21, 2015

staging: wilc1000: wilc_deinit(): fixes inconsistent returns



This patch fixes the warning reported by smatch.
 - wilc_deinit() warn: inconsistent returns 'sem:&hif_drv->sem_cfg_values'

This semaphore protect a cfg_values variable but cfg_values variables was not
used here. So, just remove this line.

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>

parent cb3b05c3

drivers/staging/wilc1000/host_interface.c

+0 −2

Original line number	Diff line number	Diff line
		@@ -3940,8 +3940,6 @@ s32 wilc_deinit(struct host_if_drv *hif_drv)
		wilc_mq_destroy(&hif_msg_q);
		}

		down(&hif_drv->sem_cfg_values);

		ret = remove_handler_in_list(hif_drv);
		if (ret)
		result = -ENOENT;