summaryrefslogtreecommitdiffstats
path: root/drivers/usb/storage
diff options
context:
space:
mode:
authorLee Jones <lee.jones@linaro.org>2020-07-03 18:41:37 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2020-07-09 16:46:59 +0200
commit7565fce17d19d9f0ab9062cb30653608d5d0cfad (patch)
tree92406ba3f2d6432652c58c7255919d667be24928 /drivers/usb/storage
parent142c8bb6c516a4caed569a68589489d4d91377e6 (diff)
downloadlinux-7565fce17d19d9f0ab9062cb30653608d5d0cfad.tar.gz
linux-7565fce17d19d9f0ab9062cb30653608d5d0cfad.tar.bz2
linux-7565fce17d19d9f0ab9062cb30653608d5d0cfad.zip
usb: storage: alauda: Remove set but unchecked variable
The return value of alauda_get_media_status() hasn't been checked since the driver's inception back in 2005. If nothing have gone wrong/been detected until this point, it's probably safe to just remove the variable. Fixes the following W=1 kernel build warning(s): drivers/usb/storage/alauda.c: In function ‘alauda_check_media’: drivers/usb/storage/alauda.c:456:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable] 456 | int rc; | ^~ Cc: Alan Stern <stern@rowland.harvard.edu> Cc: Daniel Drake <dsd@gentoo.org> Cc: usb-storage@lists.one-eyed-alien.net Signed-off-by: Lee Jones <lee.jones@linaro.org> Link: https://lore.kernel.org/r/20200703174148.2749969-20-lee.jones@linaro.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/storage')
-rw-r--r--drivers/usb/storage/alauda.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/usb/storage/alauda.c b/drivers/usb/storage/alauda.c
index ddab2cd3d2e7..20b857e97e60 100644
--- a/drivers/usb/storage/alauda.c
+++ b/drivers/usb/storage/alauda.c
@@ -453,9 +453,8 @@ static int alauda_check_media(struct us_data *us)
{
struct alauda_info *info = (struct alauda_info *) us->extra;
unsigned char status[2];
- int rc;
- rc = alauda_get_media_status(us, status);
+ alauda_get_media_status(us, status);
/* Check for no media or door open */
if ((status[0] & 0x80) || ((status[0] & 0x1F) == 0x10)