media: usb: dvb-usb-v2: af9015.c: return 0 instead of 'ret'.
authorHans Verkuil <hverkuil-cisco@xs4all.nl>
Thu, 26 Jan 2023 12:26:19 +0000 (13:26 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Wed, 8 Feb 2023 07:31:05 +0000 (08:31 +0100)
Since 'ret' is known to be 0, just return '0'. This fixes two smatch warnings:

af9015.c:1168 af9015_rc_query() warn: missing error code? 'ret'
af9015.c:1177 af9015_rc_query() warn: missing error code? 'ret'

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Cc: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/usb/dvb-usb-v2/af9015.c

index d33514acc2b5d48df70ee5d8baef8366a3e600e8..4014f7d07330f65f7645a6c3a32b17373f0d7cd2 100644 (file)
@@ -1165,7 +1165,7 @@ static int af9015_rc_query(struct dvb_usb_device *d)
        /* If any of these are non-zero, assume invalid data */
        if (buf[1] || buf[2] || buf[3]) {
                dev_dbg(&intf->dev, "invalid data\n");
-               return ret;
+               return 0;
        }
 
        /* Check for repeat of previous code */
@@ -1174,7 +1174,7 @@ static int af9015_rc_query(struct dvb_usb_device *d)
                dev_dbg(&intf->dev, "key repeated\n");
                rc_repeat(d->rc_dev);
                state->rc_repeat = buf[6];
-               return ret;
+               return 0;
        }
 
        /* Only process key if canary killed */