Re: [PATCH 1/1] media:dvb: check the return value of mb86a16_read()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Yuanjun Gong,

Please add the name of the driver to the subject line:

"media: dvb: mb86a16: check the return value of mb86a16_read()"

On 15/07/2023 11:45, Yuanjun Gong wrote:
> return an error code if mb86a16_read() gets an unexpected return
> value.
> 
> Signed-off-by: Yuanjun Gong <ruc_gongyuanjun@xxxxxxx>
> ---
>  drivers/media/dvb-frontends/mb86a16.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/media/dvb-frontends/mb86a16.c b/drivers/media/dvb-frontends/mb86a16.c
> index d3e29937cf4c..f04188a72e7a 100644
> --- a/drivers/media/dvb-frontends/mb86a16.c
> +++ b/drivers/media/dvb-frontends/mb86a16.c
> @@ -1487,9 +1487,12 @@ static int mb86a16_set_fe(struct mb86a16_state *state)
>  		}
>  	}
>  
> -	mb86a16_read(state, 0x15, &agcval);
> -	mb86a16_read(state, 0x26, &cnmval);
> -	dprintk(verbose, MB86A16_INFO, 1, "AGC = %02x CNM = %02x", agcval, cnmval);
> +	if (mb86a16_read(state, 0x15, &agcval) != 2 ||	mb86a16_read(state, 0x26, &cnmval) != 2){
> +		dprintk(verbose, MB86A16_ERROR, 1, "I2C transfer error");
> +		ret = -EREMOTEIO;
> +	}
> +	else
> +		dprintk(verbose, MB86A16_INFO, 1, "AGC = %02x CNM = %02x", agcval, cnmval);

And also run 'checkpatch.pl --strict' over your patch and fix the issues it finds.

Thank you!

	Hans

>  
>  	return ret;
>  }




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux