| 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
| ||
|
Message-ID: <20251205151158.GA158904-robh@kernel.org> Date: Fri, 5 Dec 2025 09:11:58 -0600 From: Rob Herring <robh@...nel.org> To: Yemike Abhilash Chandra <y-abhilashchandra@...com> Cc: tomi.valkeinen@...asonboard.com, mchehab@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org, hverkuil@...all.nl, sakari.ailus@...ux.intel.com, laurent.pinchart@...asonboard.com, hansg@...nel.org, mehdi.djait@...ux.intel.com, ribalda@...omium.org, git@...tzsch.eu, vladimir.zapolskiy@...aro.org, benjamin.mugnier@...s.st.com, dongcheng.yan@...el.com, u-kumar1@...com, jai.luthra@...ux.dev, linux-media@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH V2 1/4] media: dt-bindings: ti,ds90ub960: Refactor port definitions On Tue, Dec 02, 2025 at 03:52:05PM +0530, Yemike Abhilash Chandra wrote: > The current bindings duplicate the port definitions for each FPD-Link RX > and CSI-2 TX ports. This results in a large amount of repeated schema > blocks and makes it harder to extend the bindings for new devices. > > Refactor the bindings by introducing shared deftinitions for FPD-Link > input ports and CSI-2 output ports. No functional change intended. > > Signed-off-by: Yemike Abhilash Chandra <y-abhilashchandra@...com> > --- > .../bindings/media/i2c/ti,ds90ub960.yaml | 120 +++++++----------- > 1 file changed, 44 insertions(+), 76 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml b/Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml > index 0539d52de422..6a78288aebaa 100644 > --- a/Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml > @@ -125,102 +125,35 @@ properties: > > ports: > $ref: /schemas/graph.yaml#/properties/ports > + description: | Don't need '|' unless you have formatting to preserve. > + Ports represent FPD-Link inputs to the deserializer and CSI TX outputs from the deserializer. > + Their number is model-dependent. 'The number of ports is...' Wrap lines at 80. > > properties: > port@0: > - $ref: /schemas/graph.yaml#/$defs/port-base > - unevaluatedProperties: false > + $ref: '#/$defs/FPDLink-input-port' > description: FPD-Link input 0 > > - properties: > - endpoint: > - $ref: /schemas/media/video-interfaces.yaml# > - unevaluatedProperties: false > - description: > - Endpoint for FPD-Link port. If the RX mode for this port is RAW, > - hsync-active and vsync-active must be defined. > - > port@1: > - $ref: /schemas/graph.yaml#/$defs/port-base > - unevaluatedProperties: false > + $ref: '#/$defs/FPDLink-input-port' > description: FPD-Link input 1 > > - properties: > - endpoint: > - $ref: /schemas/media/video-interfaces.yaml# > - unevaluatedProperties: false > - description: > - Endpoint for FPD-Link port. If the RX mode for this port is RAW, > - hsync-active and vsync-active must be defined. > - > port@2: > - $ref: /schemas/graph.yaml#/$defs/port-base > - unevaluatedProperties: false > + $ref: '#/$defs/FPDLink-input-port' > description: FPD-Link input 2 > > - properties: > - endpoint: > - $ref: /schemas/media/video-interfaces.yaml# > - unevaluatedProperties: false > - description: > - Endpoint for FPD-Link port. If the RX mode for this port is RAW, > - hsync-active and vsync-active must be defined. > - > port@3: > - $ref: /schemas/graph.yaml#/$defs/port-base > - unevaluatedProperties: false > + $ref: '#/$defs/FPDLink-input-port' > description: FPD-Link input 3 > > - properties: > - endpoint: > - $ref: /schemas/media/video-interfaces.yaml# > - unevaluatedProperties: false > - description: > - Endpoint for FPD-Link port. If the RX mode for this port is RAW, > - hsync-active and vsync-active must be defined. > - > port@4: > - $ref: /schemas/graph.yaml#/$defs/port-base > - unevaluatedProperties: false > + $ref: '#/$defs/CSI2-output-port' > description: CSI-2 Output 0 > > - properties: > - endpoint: > - $ref: /schemas/media/video-interfaces.yaml# > - unevaluatedProperties: false > - > - properties: > - data-lanes: > - minItems: 1 > - maxItems: 4 > - link-frequencies: > - maxItems: 1 > - > - required: > - - data-lanes > - - link-frequencies > - > port@5: > - $ref: /schemas/graph.yaml#/$defs/port-base > - unevaluatedProperties: false > + $ref: '#/$defs/CSI2-output-port' > description: CSI-2 Output 1 > > - properties: > - endpoint: > - $ref: /schemas/media/video-interfaces.yaml# > - unevaluatedProperties: false > - > - properties: > - data-lanes: > - minItems: 1 > - maxItems: 4 > - link-frequencies: > - maxItems: 1 > - > - required: > - - data-lanes > - - link-frequencies > - > required: > - port@0 > - port@1 > @@ -236,6 +169,41 @@ required: > - clock-names > - ports > > +$defs: > + FPDLink-input-port: > + $ref: /schemas/graph.yaml#/$defs/port-base > + unevaluatedProperties: false > + description: FPD-Link input > + > + properties: > + endpoint: > + $ref: /schemas/media/video-interfaces.yaml# > + unevaluatedProperties: false > + description: > + Endpoint for FPD-Link port. If the RX mode for this port is RAW, > + hsync-active and vsync-active must be defined. > + > + CSI2-output-port: > + $ref: /schemas/graph.yaml#/$defs/port-base > + unevaluatedProperties: false > + description: CSI-2 Output > + > + properties: > + endpoint: > + $ref: /schemas/media/video-interfaces.yaml# > + unevaluatedProperties: false > + > + properties: > + data-lanes: > + minItems: 1 > + maxItems: 4 > + link-frequencies: > + maxItems: 1 > + > + required: > + - data-lanes > + - link-frequencies > + > unevaluatedProperties: false > > examples: > -- > 2.34.1 >
Powered by blists - more mailing lists