linux-kernel - Re: [PATCH v2 1/2] media: dt-bindings: Add ST VD56G3 camera sensor binding

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]

Date: Tue, 28 May 2024 09:22:09 +0000
From: Sakari Ailus <sakari.ailus@....fi>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc: Sylvain Petinot <sylvain.petinot@...s.st.com>,
	benjamin.mugnier@...s.st.com, mchehab@...nel.org, robh@...nel.org,
	krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org,
	linux-media@...r.kernel.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/2] media: dt-bindings: Add ST VD56G3 camera sensor
 binding

Hi Krzysztof,

On Mon, May 27, 2024 at 09:04:38PM +0200, Krzysztof Kozlowski wrote:
> On 21/05/2024 18:29, Sylvain Petinot wrote:
> > Add devicetree bindings Documentation for ST VD56G3 & ST VD66GY camera
> > sensors. Update MAINTAINERS file.
> > 
> > Signed-off-by: Sylvain Petinot <sylvain.petinot@...s.st.com>
> 
> 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index ef6be9d95143..554e6861425b 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -20885,6 +20885,15 @@ S:	Maintained
> >  F:	Documentation/hwmon/stpddc60.rst
> >  F:	drivers/hwmon/pmbus/stpddc60.c
> >  
> > +ST VD56G3 DRIVER

I might add this is a sensor, i.e. "ST VD653G IMAGE SENSOR DRIVER".

> > +M:	Benjamin Mugnier <benjamin.mugnier@...s.st.com>
> > +M:	Sylvain Petinot <sylvain.petinot@...s.st.com>
> > +L:	linux-media@...r.kernel.org
> > +S:	Maintained
> > +T:	git git://linuxtv.org/media_tree.git
> 
> This is a friendly reminder during the review process.
> 
> It seems my or other reviewer's previous comments were not fully
> addressed. Maybe the feedback got lost between the quotes, maybe you
> just forgot to apply it. Please go back to the previous discussion and
> either implement all requested changes or keep discussing them.

The above MAINTAINERS entry is roughly in line with what else we have for
the Media tree. I'm in favour of listing the people who would look after
the driver, not just those who merge the patches (or even send PRs to
Linus).

In other words, I think the above entry is fine as-is.

Sylvain: could you also add the file under "V4L2 CAMERA SENSOR DRIVERS"
that lists myself as the maintainer?

Thanks.

-- 
Kind regards,

Sakari Ailus