diff options
| author | ye xingchen <ye.xingchen@zte.com.cn> | 2022-08-30 10:30:27 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@kernel.org> | 2022-09-24 07:50:12 +0200 |
| commit | 2dfe2c4f1720b6b0860d36d25107ffa57f0bbc63 (patch) | |
| tree | 9c5c9b1a0ac30f3b37d0a0606e0e092e65d31eaf /drivers/media/rc/imon.c | |
| parent | 41fd1cb6151439b205ac7611883d85ae14250172 (diff) | |
| download | linux-2dfe2c4f1720b6b0860d36d25107ffa57f0bbc63.tar.bz2 | |
media: imon: Remove the unneeded result variable
Return the value send_packet() directly instead of storing it in another
redundant variable.
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: ye xingchen <ye.xingchen@zte.com.cn>
Signed-off-by: Sean Young <sean@mess.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Diffstat (limited to 'drivers/media/rc/imon.c')
| -rw-r--r-- | drivers/media/rc/imon.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/media/rc/imon.c b/drivers/media/rc/imon.c index 735b925da998..5edfd8a9e849 100644 --- a/drivers/media/rc/imon.c +++ b/drivers/media/rc/imon.c @@ -684,7 +684,6 @@ static int send_packet(struct imon_context *ictx) */ static int send_associate_24g(struct imon_context *ictx) { - int retval; const unsigned char packet[8] = { 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x20 }; @@ -699,9 +698,8 @@ static int send_associate_24g(struct imon_context *ictx) } memcpy(ictx->usb_tx_buf, packet, sizeof(packet)); - retval = send_packet(ictx); - return retval; + return send_packet(ictx); } /* |