This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: drivers:usb:remove unneeded variable
Author:  chiminghao <[email protected]>
Date:    Thu Dec 9 02:50:09 2021 +0100

return value form directly instead of
taking this in another redundant variable.

Link: 
https://lore.kernel.org/linux-media/[email protected]
Reported-by: Zeal Robot <[email protected]>
Signed-off-by: chiminghao <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/usb/gspca/m5602/m5602_s5k83a.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

---

diff --git a/drivers/media/usb/gspca/m5602/m5602_s5k83a.c 
b/drivers/media/usb/gspca/m5602/m5602_s5k83a.c
index 4504d615b1e4..8ef010a87445 100644
--- a/drivers/media/usb/gspca/m5602/m5602_s5k83a.c
+++ b/drivers/media/usb/gspca/m5602/m5602_s5k83a.c
@@ -407,25 +407,21 @@ static int s5k83a_set_gain(struct gspca_dev *gspca_dev, 
__s32 val)
 
 static int s5k83a_set_brightness(struct gspca_dev *gspca_dev, __s32 val)
 {
-       int err;
        u8 data[1];
        struct sd *sd = (struct sd *) gspca_dev;
 
        data[0] = val;
-       err = m5602_write_sensor(sd, S5K83A_BRIGHTNESS, data, 1);
-       return err;
+       return m5602_write_sensor(sd, S5K83A_BRIGHTNESS, data, 1);
 }
 
 static int s5k83a_set_exposure(struct gspca_dev *gspca_dev, __s32 val)
 {
-       int err;
        u8 data[2];
        struct sd *sd = (struct sd *) gspca_dev;
 
        data[0] = 0;
        data[1] = val;
-       err = m5602_write_sensor(sd, S5K83A_EXPOSURE, data, 2);
-       return err;
+       return m5602_write_sensor(sd, S5K83A_EXPOSURE, data, 2);
 }
 
 static int s5k83a_set_flip_real(struct gspca_dev *gspca_dev,

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to