* [PATCH] media: dvb: redundant assignment to variable tmp
@ 2019-08-27 11:15 Colin King
2019-08-27 13:10 ` Dan Carpenter
0 siblings, 1 reply; 2+ messages in thread
From: Colin King @ 2019-08-27 11:15 UTC (permalink / raw)
To: Mauro Carvalho Chehab, linux-media; +Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
Variable tmp is being assigned a value that is never read and tmp
is being re-assigned a little later on. The assignment is redundant
and hence can be removed.
Addresses-Coverity: ("Ununsed value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
drivers/media/dvb-frontends/sp8870.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/media/dvb-frontends/sp8870.c b/drivers/media/dvb-frontends/sp8870.c
index 655db8272268..f6793c9c2dc3 100644
--- a/drivers/media/dvb-frontends/sp8870.c
+++ b/drivers/media/dvb-frontends/sp8870.c
@@ -378,8 +378,6 @@ static int sp8870_read_ber (struct dvb_frontend* fe, u32 * ber)
if (ret < 0)
return -EIO;
- tmp = ret & 0x3F;
-
ret = sp8870_readreg(state, 0xC07);
if (ret < 0)
return -EIO;
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] media: dvb: redundant assignment to variable tmp
2019-08-27 11:15 [PATCH] media: dvb: redundant assignment to variable tmp Colin King
@ 2019-08-27 13:10 ` Dan Carpenter
0 siblings, 0 replies; 2+ messages in thread
From: Dan Carpenter @ 2019-08-27 13:10 UTC (permalink / raw)
To: Colin King
Cc: Mauro Carvalho Chehab, linux-media, kernel-janitors, linux-kernel
On Tue, Aug 27, 2019 at 12:15:27PM +0100, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Variable tmp is being assigned a value that is never read and tmp
> is being re-assigned a little later on. The assignment is redundant
> and hence can be removed.
>
> Addresses-Coverity: ("Ununsed value")
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
> drivers/media/dvb-frontends/sp8870.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/media/dvb-frontends/sp8870.c b/drivers/media/dvb-frontends/sp8870.c
> index 655db8272268..f6793c9c2dc3 100644
> --- a/drivers/media/dvb-frontends/sp8870.c
> +++ b/drivers/media/dvb-frontends/sp8870.c
> @@ -378,8 +378,6 @@ static int sp8870_read_ber (struct dvb_frontend* fe, u32 * ber)
> if (ret < 0)
> return -EIO;
>
> - tmp = ret & 0x3F;
> -
This is pre git code. It's not clear if the tmp is supposed to be used
or if we can remove the sp8870_readreg() call also... The problem is
that we're disabling the warning without necessarily writing the best
fix. It's better to leave the warning there until we are more sure of
the correct fix. The warning has useful information and it's not
hurting anyone.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-08-27 13:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-27 11:15 [PATCH] media: dvb: redundant assignment to variable tmp Colin King
2019-08-27 13:10 ` Dan Carpenter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).