Re: [PATCH 2/4] media: ov5640: fix get_light_freq on auto

2018-10-10 Thread jacopo mondi 
Hi Sam,

On Mon, Oct 08, 2018 at 11:48:00PM -0700, Sam Bobrowicz wrote:
> Light frequency was not properly returned when in auto
> mode and the detected frequency was 60Hz. Fix this.
>
> Signed-off-by: Sam Bobrowicz 

This is indeed a bugfix

Acked-by: Jacopo Mondi 

Thanks
  j
> ---
>  drivers/media/i2c/ov5640.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/media/i2c/ov5640.c b/drivers/media/i2c/ov5640.c
> index 5031aab..f183222 100644
> --- a/drivers/media/i2c/ov5640.c
> +++ b/drivers/media/i2c/ov5640.c
> @@ -1295,6 +1295,7 @@ static int ov5640_get_light_freq(struct ov5640_dev 
> *sensor)
>   light_freq = 50;
>   } else {
>   /* 60Hz */
> + light_freq = 60;
>   }
>   }
>
> --
> 2.7.4
>


signature.asc
Description: PGP signature

[PATCH 2/4] media: ov5640: fix get_light_freq on auto

2018-10-08 Thread Sam Bobrowicz 
Light frequency was not properly returned when in auto
mode and the detected frequency was 60Hz. Fix this.

Signed-off-by: Sam Bobrowicz 
---
 drivers/media/i2c/ov5640.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/media/i2c/ov5640.c b/drivers/media/i2c/ov5640.c
index 5031aab..f183222 100644
--- a/drivers/media/i2c/ov5640.c
+++ b/drivers/media/i2c/ov5640.c
@@ -1295,6 +1295,7 @@ static int ov5640_get_light_freq(struct ov5640_dev 
*sensor)
light_freq = 50;
} else {
/* 60Hz */
+   light_freq = 60;
}
}
 
-- 
2.7.4