Re: [PATCH v2 02/23] media: i2c: ov01a10: Fix reported pixel-rate value

[Bingbu Cao <bingbu.cao@linux.intel.com>]

Reviewed-by: Bingbu Cao <bingbu.cao@intel.com>

On 1/12/26 5:59 PM, Sakari Ailus wrote:
> From: Hans de Goede <hansg@kernel.org>
> 
> CSI lanes are double-clocked so with a single lane at 400MHZ the resulting
> pixel-rate for 10-bits pixels is 400 MHz * 2 / 10 = 80 MHz, not 40 MHz.
> 
> This also matches with the observed frame-rate of 60 fps with the default
> vblank setting: 80000000 / (1488 * 896) = 60.
> 
> Fixes: 0827b58dabff ("media: i2c: add ov01a10 image sensor driver")
> Cc: stable@vger.kernel.org
> Signed-off-by: Hans de Goede <hansg@kernel.org>
> Tested-by: Mehdi Djait <mehdi.djait@linux.intel.com> # Dell XPS 9315
> Reviewed-by: Mehdi Djait <mehdi.djait@linux.intel.com>
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> ---
>  drivers/media/i2c/ov01a10.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/media/i2c/ov01a10.c b/drivers/media/i2c/ov01a10.c
> index e5df01f97978..0b1a1ecfffd0 100644
> --- a/drivers/media/i2c/ov01a10.c
> +++ b/drivers/media/i2c/ov01a10.c
> @@ -16,7 +16,7 @@
>  #include <media/v4l2-fwnode.h>
>  
>  #define OV01A10_LINK_FREQ_400MHZ	400000000ULL
> -#define OV01A10_SCLK			40000000LL
> +#define OV01A10_SCLK			80000000LL
>  #define OV01A10_DATA_LANES		1
>  
>  #define OV01A10_REG_CHIP_ID		0x300a
> 

-- 
Best regards,
Bingbu Cao