Commit 48c80ccc authored by Nishka Dasgupta's avatar Nishka Dasgupta Committed by Greg Kroah-Hartman

staging: pi433: Remove unnecessary variable

The variable retval is assigned constant values twice, and can therefore
be replaced by its values.
Signed-off-by: default avatarNishka Dasgupta <nishkadg.linux@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent dcb76ad1
...@@ -871,7 +871,6 @@ pi433_write(struct file *filp, const char __user *buf, ...@@ -871,7 +871,6 @@ pi433_write(struct file *filp, const char __user *buf,
static long static long
pi433_ioctl(struct file *filp, unsigned int cmd, unsigned long arg) pi433_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
{ {
int retval = 0;
struct pi433_instance *instance; struct pi433_instance *instance;
struct pi433_device *device; struct pi433_device *device;
struct pi433_tx_cfg tx_cfg; struct pi433_tx_cfg tx_cfg;
...@@ -923,10 +922,10 @@ pi433_ioctl(struct file *filp, unsigned int cmd, unsigned long arg) ...@@ -923,10 +922,10 @@ pi433_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
mutex_unlock(&device->rx_lock); mutex_unlock(&device->rx_lock);
break; break;
default: default:
retval = -EINVAL; return -EINVAL;
} }
return retval; return 0;
} }
#ifdef CONFIG_COMPAT #ifdef CONFIG_COMPAT
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment