Re: [PATCH] media: i2c: Add OV05C camera sensor driver

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hello Pratap,

On Fri, Mar 28, 2025 at 06:10:41PM -0400, Nirujogi, Pratap wrote:
> Hi Laurent,
> 
> Thanks for reviewing and extremely sorry for the delayed response. We 
> have submitted V2 patch based on your review feedback.
> 
> Can you please help to review latest V2 patch and let us know your feedback.

Sure. It will take a bit of time though, as my review backlog is big.
Please see below for additional comments.

> On 2/28/2025 12:35 PM, Laurent Pinchart wrote:
> > 
> > Hi Pratap,
> > 
> > Thank you for the patch.
> > 
> > A few assorted comments to start with, I'll try to do a more in-depth
> > review later.
> > 
> > On Fri, Feb 28, 2025 at 11:53:12AM -0500, Pratap Nirujogi wrote:
> >> Add driver for OmniVision 5.2M OV05C10 sensor. This driver
> >> supports only the full size normal 2888x1808@30fps 2-lane
> >> sensor profile.
> > 
> > What have you tested this driver with ? I see no OF device ID or ACPI
> > device ID.
> 
> We have tested this driver on the "AMD RYZEN AI MAX PRO 385 w/ Radeon 
> 8050S" target supporting ISP HW v4.1. OMNI5C10 is ACPI HID for this 
> sensor on the target.

That's interesting. Do you have plans to post a driver for the ISP ?

> > Please provide a v4l2-compliance report.
> 
> Please refer the attached screenshot for the v4l2-compliance test 
> report. This test is ran with V2 patch. Fixed some of the compliance 
> test failures on V1 and ensured it is 100% compliant for the supported 
> IOCTLS.

Could you please post the v4l2-compliance report in a reply to v2, in
text format instead of a screenshot ?

> >> Signed-off-by: Pratap Nirujogi <pratap.nirujogi@amd.com>
> >> ---
> >>   drivers/media/i2c/Kconfig  |   10 +
> >>   drivers/media/i2c/Makefile |    1 +
> >>   drivers/media/i2c/ov05c.c  | 1031 ++++++++++++++++++++++++++++++++++++

[snip]

-- 
Regards,

Laurent Pinchart