linux-kernel - Re: [PATCH v2 1/1] media: dt-bindings: video-interfaces: add video-interfaces.h information

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] [day] [month] [year] [list]

Message-ID: <aP8oibCu-UsOFtNX@kekkonen.localdomain>
Date: Mon, 27 Oct 2025 10:08:41 +0200
From: Sakari Ailus <sakari.ailus@...ux.intel.com>
To: Laurent Pinchart <laurent.pinchart@...asonboard.com>
Cc: Frank Li <Frank.Li@....com>, Mauro Carvalho Chehab <mchehab@...nel.org>,
	Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	"open list:MEDIA INPUT INFRASTRUCTURE (V4L/DVB)" <linux-media@...r.kernel.org>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@...r.kernel.org>,
	open list <linux-kernel@...r.kernel.org>, imx@...ts.linux.dev
Subject: Re: [PATCH v2 1/1] media: dt-bindings: video-interfaces: add
 video-interfaces.h information

Hi Laurent,

On Sun, Oct 26, 2025 at 08:49:33PM +0200, Laurent Pinchart wrote:
> Hi Frank,
> 
> Thank you for the patch.
> 
> On Tue, Oct 21, 2025 at 11:49:22AM -0400, Frank Li wrote:
> > Mention include/dt-bindings/media/video-interfaces.h in descriptions to
> > help avoid use hardcode in dts.
> > 
> > Signed-off-by: Frank Li <Frank.Li@....com>
> > ---
> > changes in v2
> > - update commit message "mention ..."
> > - use full path include/dt-bindings/media/video-interfaces.h
> > ---
> >  Documentation/devicetree/bindings/media/video-interfaces.yaml | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/media/video-interfaces.yaml b/Documentation/devicetree/bindings/media/video-interfaces.yaml
> > index 038e85b45befa..6ed4695cacf7e 100644
> > --- a/Documentation/devicetree/bindings/media/video-interfaces.yaml
> > +++ b/Documentation/devicetree/bindings/media/video-interfaces.yaml
> > @@ -95,7 +95,7 @@ properties:
> >        - 6 # BT.656
> >        - 7 # DPI
> 
> I wondered if we should reference the macros here as well:
> 
>     enum:
>       - 1 # MEDIA_BUS_TYPE_CSI2_CPHY
>       - 2 # MEDIA_BUS_TYPE_CSI1
>       - 3 # MEDIA_BUS_TYPE_CCP2
>       - 4 # MEDIA_BUS_TYPE_CSI2_DPHY
>       - 5 # MEDIA_BUS_TYPE_PARALLEL
>       - 6 # MEDIA_BUS_TYPE_BT656
>       - 7 # DPI
> 
> (Annoyingly we don't have a macro for DPI.)

I'd actually add one: there's certainly no harm from doing so even though
it's for display, not for cameras (or receivers alike).

This patch is already in my tree so the macro name comments should go to a
new one.

> 
> This can be done separately if desired, so
> 
> Reviewed-by: Laurent Pinchart <laurent.pinchart@...asonboard.com>

Thanks!

-- 
Sakari Ailus