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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250402001809.GB4845@pendragon.ideasonboard.com>
Date: Wed, 2 Apr 2025 03:18:09 +0300
From: Laurent Pinchart <laurent.pinchart@...asonboard.com>
To: "Nirujogi, Pratap" <pnirujog@....com>
Cc: Pratap Nirujogi <pratap.nirujogi@....com>, mchehab@...nel.org,
	sakari.ailus@...ux.intel.com, hverkuil@...all.nl,
	dave.stevenson@...pberrypi.com, linux-media@...r.kernel.org,
	linux-kernel@...r.kernel.org, benjamin.chan@....com, bin.du@....com,
	gjorgji.rosikopulos@....com, king.li@....com,
	dominic.antony@....com
Subject: Re: [PATCH] media: i2c: Add OV05C camera sensor driver

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@...ps 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@....com>
> >> ---
> >>   drivers/media/i2c/Kconfig  |   10 +
> >>   drivers/media/i2c/Makefile |    1 +
> >>   drivers/media/i2c/ov05c.c  | 1031 ++++++++++++++++++++++++++++++++++++

[snip]

-- 
Regards,

Laurent Pinchart