Hi Jacopo,
Thanks for your work.
On 2021-11-03 21:46:54 +0100, Jacopo Mondi wrote:
> Valid pixel clock detection is performed by spinning on a register read,
> which if repeated too frequently might fail. As the error is not fatal
> ignore it instead of bailing out to continue spinning until the timeout
> completion.
>
> Also relax the time between bus transactions and slightly increase the
> wait interval to mitigate the failure risk.
>
> Signed-off-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx>
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> ---
> drivers/media/i2c/max9271.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/media/i2c/max9271.c b/drivers/media/i2c/max9271.c
> index aa9ab6831574..5c4248de64c2 100644
> --- a/drivers/media/i2c/max9271.c
> +++ b/drivers/media/i2c/max9271.c
> @@ -55,7 +55,7 @@ static int max9271_write(struct max9271_device *dev, u8 reg, u8 val)
> /*
> * max9271_pclk_detect() - Detect valid pixel clock from image sensor
> *
> - * Wait up to 10ms for a valid pixel clock.
> + * Wait up to 15ms for a valid pixel clock.
> *
> * Returns 0 for success, < 0 for pixel clock not properly detected
> */
> @@ -64,15 +64,15 @@ static int max9271_pclk_detect(struct max9271_device *dev)
> unsigned int i;
> int ret;
>
> - for (i = 0; i < 100; i++) {
> + for (i = 0; i < 10; i++) {
> ret = max9271_read(dev, 0x15);
> if (ret < 0)
> - return ret;
> + continue;
>
> if (ret & MAX9271_PCLKDET)
> return 0;
>
> - usleep_range(50, 100);
> + usleep_range(1000, 1500);
> }
>
> dev_err(&dev->client->dev, "Unable to detect valid pixel clock\n");
> --
> 2.33.1
>
--
Regards,
Niklas Söderlund
