diff mbox series

[v4,08/17] media: i2c: rdacm21: Add dealy after OV490 reset

Message ID 20210412093451.14198-9-jacopo+renesas@jmondi.org
State Superseded
Headers show
Series media: gmsl: Reliability improvements | expand

Commit Message

Jacopo Mondi April 12, 2021, 9:34 a.m. UTC
Add a delay after the OV490 chip is put in reset state. The reset
signal shall be held for at least 250 useconds.

Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
---
 drivers/media/i2c/rdacm21.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Kieran Bingham April 14, 2021, 4:49 p.m. UTC | #1
Hi Jacopo,

There's still a s/dealy/delay/ in $SUBJECT

On 12/04/2021 10:34, Jacopo Mondi wrote:
> Add a delay after the OV490 chip is put in reset state. The reset

> signal shall be held for at least 250 useconds.

> 

> Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>


I added this on v3...

Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>


> ---

>  drivers/media/i2c/rdacm21.c | 6 +++++-

>  1 file changed, 5 insertions(+), 1 deletion(-)

> 

> diff --git a/drivers/media/i2c/rdacm21.c b/drivers/media/i2c/rdacm21.c

> index 553e3f03752b..6be8ce130e78 100644

> --- a/drivers/media/i2c/rdacm21.c

> +++ b/drivers/media/i2c/rdacm21.c

> @@ -469,7 +469,10 @@ static int rdacm21_initialize(struct rdacm21_device *dev)

>  	if (ret)

>  		return ret;

>  

> -	/* Enable GPIO1 and hold OV490 in reset during max9271 configuration. */

> +	/*

> +	 * Enable GPIO1 and hold OV490 in reset during max9271 configuration.

> +	 * The reset signal has to be asserted for at least 250 useconds.



Is it worth mentioning here that it is asserted to active low? Just to
make it clear that holding it low for 250 uS is the desired effect?

It might not be worth it - but perhaps that was the reason for some
confusion here.

Eitherway RB tag still stands I think.

--
Kieran

> +	 */

>  	ret = max9271_enable_gpios(&dev->serializer, MAX9271_GPIO1OUT);

>  	if (ret)

>  		return ret;

> @@ -477,6 +480,7 @@ static int rdacm21_initialize(struct rdacm21_device *dev)

>  	ret = max9271_clear_gpios(&dev->serializer, MAX9271_GPIO1OUT);

>  	if (ret)

>  		return ret;

> +	usleep_range(250, 500);

>  

>  	ret = max9271_configure_gmsl_link(&dev->serializer);

>  	if (ret)

>
Laurent Pinchart April 14, 2021, 11:10 p.m. UTC | #2
Hi Jacopo,

Thank you for the patch.

On Mon, Apr 12, 2021 at 11:34:42AM +0200, Jacopo Mondi wrote:
> Add a delay after the OV490 chip is put in reset state. The reset

> signal shall be held for at least 250 useconds.

> 

> Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>


With s/dealy/delay/ in the subject line,

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>


> ---

>  drivers/media/i2c/rdacm21.c | 6 +++++-

>  1 file changed, 5 insertions(+), 1 deletion(-)

> 

> diff --git a/drivers/media/i2c/rdacm21.c b/drivers/media/i2c/rdacm21.c

> index 553e3f03752b..6be8ce130e78 100644

> --- a/drivers/media/i2c/rdacm21.c

> +++ b/drivers/media/i2c/rdacm21.c

> @@ -469,7 +469,10 @@ static int rdacm21_initialize(struct rdacm21_device *dev)

>  	if (ret)

>  		return ret;

>  

> -	/* Enable GPIO1 and hold OV490 in reset during max9271 configuration. */

> +	/*

> +	 * Enable GPIO1 and hold OV490 in reset during max9271 configuration.

> +	 * The reset signal has to be asserted for at least 250 useconds.

> +	 */

>  	ret = max9271_enable_gpios(&dev->serializer, MAX9271_GPIO1OUT);

>  	if (ret)

>  		return ret;

> @@ -477,6 +480,7 @@ static int rdacm21_initialize(struct rdacm21_device *dev)

>  	ret = max9271_clear_gpios(&dev->serializer, MAX9271_GPIO1OUT);

>  	if (ret)

>  		return ret;

> +	usleep_range(250, 500);

>  

>  	ret = max9271_configure_gmsl_link(&dev->serializer);

>  	if (ret)


-- 
Regards,

Laurent Pinchart
diff mbox series

Patch

diff --git a/drivers/media/i2c/rdacm21.c b/drivers/media/i2c/rdacm21.c
index 553e3f03752b..6be8ce130e78 100644
--- a/drivers/media/i2c/rdacm21.c
+++ b/drivers/media/i2c/rdacm21.c
@@ -469,7 +469,10 @@  static int rdacm21_initialize(struct rdacm21_device *dev)
 	if (ret)
 		return ret;
 
-	/* Enable GPIO1 and hold OV490 in reset during max9271 configuration. */
+	/*
+	 * Enable GPIO1 and hold OV490 in reset during max9271 configuration.
+	 * The reset signal has to be asserted for at least 250 useconds.
+	 */
 	ret = max9271_enable_gpios(&dev->serializer, MAX9271_GPIO1OUT);
 	if (ret)
 		return ret;
@@ -477,6 +480,7 @@  static int rdacm21_initialize(struct rdacm21_device *dev)
 	ret = max9271_clear_gpios(&dev->serializer, MAX9271_GPIO1OUT);
 	if (ret)
 		return ret;
+	usleep_range(250, 500);
 
 	ret = max9271_configure_gmsl_link(&dev->serializer);
 	if (ret)