diff options
author | Javier Martinez Canillas <javier@osg.samsung.com> | 2015-10-02 14:14:26 -0400 |
---|---|---|
committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2015-10-02 14:44:17 -0400 |
commit | 5ccd9abd508644cefee8641ccd6157e8b55a6ee3 (patch) | |
tree | 62fe4f516bf6e709d7c1ddd6e60b532c671f3be1 /drivers/input/misc | |
parent | 5702222c9a7af4a207066d54aa95cfe31f34f2f8 (diff) |
Input: kxtj9 - remove unneeded retval variable
The retval variable isn't needed since isn't used in the function. Remove
the variable and just return 0 instead.
This also fixes the following make coccicheck warning:
drivers/input/misc/kxtj9.c:638:5-11: Unneeded variable: "retval". Return "0" on line 646
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Diffstat (limited to 'drivers/input/misc')
-rw-r--r-- | drivers/input/misc/kxtj9.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/input/misc/kxtj9.c b/drivers/input/misc/kxtj9.c index e058d711256a..efaffcc57e36 100644 --- a/drivers/input/misc/kxtj9.c +++ b/drivers/input/misc/kxtj9.c | |||
@@ -635,7 +635,6 @@ static int __maybe_unused kxtj9_resume(struct device *dev) | |||
635 | struct i2c_client *client = to_i2c_client(dev); | 635 | struct i2c_client *client = to_i2c_client(dev); |
636 | struct kxtj9_data *tj9 = i2c_get_clientdata(client); | 636 | struct kxtj9_data *tj9 = i2c_get_clientdata(client); |
637 | struct input_dev *input_dev = tj9->input_dev; | 637 | struct input_dev *input_dev = tj9->input_dev; |
638 | int retval = 0; | ||
639 | 638 | ||
640 | mutex_lock(&input_dev->mutex); | 639 | mutex_lock(&input_dev->mutex); |
641 | 640 | ||
@@ -643,7 +642,7 @@ static int __maybe_unused kxtj9_resume(struct device *dev) | |||
643 | kxtj9_enable(tj9); | 642 | kxtj9_enable(tj9); |
644 | 643 | ||
645 | mutex_unlock(&input_dev->mutex); | 644 | mutex_unlock(&input_dev->mutex); |
646 | return retval; | 645 | return 0; |
647 | } | 646 | } |
648 | 647 | ||
649 | static SIMPLE_DEV_PM_OPS(kxtj9_pm_ops, kxtj9_suspend, kxtj9_resume); | 648 | static SIMPLE_DEV_PM_OPS(kxtj9_pm_ops, kxtj9_suspend, kxtj9_resume); |