Re: [PATCH v3 1/7] media: platform: amd: Introduce amd isp4 capture driver

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

On Wed, Sep 03, 2025 at 11:22:27AM +0800, Du, Bin wrote:
> Thanks Laurent Pinchart
> 
> On 9/2/2025 8:49 PM, Laurent Pinchart wrote:
> > Hi Bin,
> > 
> > Could you please provide the v4l2-compliance report (please use the
> > very latest version of v4l2-compliance), as well as the output of
> > `media-ctl -p -d /dev/media0` (replacing media0 with the appropriate
> > device) ?
> > 
> > Generally speaking, you should include a cover letter in patch series,
> > and that information can be included in the cover letter.
> > 
> 
> Yes, there is a cover letter for the patch series 
> https://lore.kernel.org/linux-media/20250828084507.94552-1-Bin.Du@amd.com/, 
> because of some server issue, the cover letter and left patches are sent 
> separately, really sorry for the confusing. It does contain 
> v4l2-compliance report, but the media-ctl output is missing, many thanks 
> for the guide, will include it either in the next patch

My bad, I even replied to that cover letter, but then forgot about it.

> > On Thu, Aug 28, 2025 at 06:08:05PM +0800, Bin Du wrote:
> >> AMD isp4 capture is a v4l2 media device which implements media controller
> >> interface. It has one sub-device (AMD ISP4 sub-device) endpoint which can
> >> be connected to a remote CSI2 TX endpoint. It supports only one physical
> >> interface for now. Also add ISP4 driver related entry info into the
> >> MAINTAINERS file
> >>
> >> Co-developed-by: Svetoslav Stoilov <Svetoslav.Stoilov@amd.com>
> >> Signed-off-by: Svetoslav Stoilov <Svetoslav.Stoilov@amd.com>
> >> Signed-off-by: Bin Du <Bin.Du@amd.com>
> >> ---
> >>   MAINTAINERS                              |  13 +++
> >>   drivers/media/platform/Kconfig           |   1 +
> >>   drivers/media/platform/Makefile          |   1 +
> >>   drivers/media/platform/amd/Kconfig       |   3 +
> >>   drivers/media/platform/amd/Makefile      |   3 +
> >>   drivers/media/platform/amd/isp4/Kconfig  |  13 +++
> >>   drivers/media/platform/amd/isp4/Makefile |   6 ++
> >>   drivers/media/platform/amd/isp4/isp4.c   | 121 +++++++++++++++++++++++
> >>   drivers/media/platform/amd/isp4/isp4.h   |  24 +++++
> >>   9 files changed, 185 insertions(+)
> >>   create mode 100644 drivers/media/platform/amd/Kconfig
> >>   create mode 100644 drivers/media/platform/amd/Makefile
> >>   create mode 100644 drivers/media/platform/amd/isp4/Kconfig
> >>   create mode 100644 drivers/media/platform/amd/isp4/Makefile
> >>   create mode 100644 drivers/media/platform/amd/isp4/isp4.c
> >>   create mode 100644 drivers/media/platform/amd/isp4/isp4.h

-- 
Regards,

Laurent Pinchart