[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241012180935.GC5212@pendragon.ideasonboard.com>
Date: Sat, 12 Oct 2024 21:09:35 +0300
From: Laurent Pinchart <laurent.pinchart@...asonboard.com>
To: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
Cc: Mauro Carvalho Chehab <mchehab@...nel.org>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Dave Stevenson <dave.stevenson@...pberrypi.com>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>,
Martin Kepplinger <martink@...teo.de>,
Paul Kocialkowski <paul.kocialkowski@...tlin.com>,
"Paul J. Murphy" <paul.j.murphy@...el.com>,
Daniele Alessandrelli <daniele.alessandrelli@...el.com>,
Tommaso Merciai <tomm.merciai@...il.com>,
Martin Hecht <martin.hecht@...et.eu>,
Zhi Mao <zhi.mao@...iatek.com>,
Alain Volmat <alain.volmat@...s.st.com>,
Mikhail Rudenko <mike.rudenko@...il.com>,
Ricardo Ribalda <ribalda@...nel.org>,
Kieran Bingham <kieran.bingham@...asonboard.com>,
Umang Jain <umang.jain@...asonboard.com>,
Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
Vladimir Zapolskiy <vladimir.zapolskiy@...aro.org>,
Dongchun Zhu <dongchun.zhu@...iatek.com>,
Quentin Schulz <quentin.schulz@...obroma-systems.com>,
Todor Tomov <todor.too@...il.com>,
Krzysztof Kozlowski <krzk@...nel.org>, linux-media@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 2/2] media: dt-bindings: Use additionalProperties: false
for endpoint: properties:
Hi Bryan,
Thank you for the patch.
On Sat, Oct 12, 2024 at 04:02:51PM +0100, Bryan O'Donoghue wrote:
> Some of our sensor schemas use unevaluatedProperities: false for endpoint:
s/unevaluatedProperities/unevaluatedProperties/
Same below.
> properties: while other schemas use additionalProperties: false.
>
> The effect of using unevaluatedProperities: false in this instance is that
> any property in media/video-interfaces.yaml can be considered in a dts for
> an endpoint.
>
> Converting to additionalProperties: false and running DT checkers show that
> such a liberal policy is unnecessary.
>
> We should have a consistent way of defining these properties if for no
> other reason than aid other developers in the preferred way of writing
> these schemas for media/i2c in the future.
>
> Convert to additionalProperties: as a result remote-endpoint needs to be
> added to the property list for most sensors. In a few cases some
> additional properties clock data-lanes or clock-lanes need to be added too
> but, for-the-most-part remote-endpoint is the only missing property.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
> ---
> .../devicetree/bindings/media/i2c/alliedvision,alvium-csi2.yaml | 5 ++++-
> Documentation/devicetree/bindings/media/i2c/galaxycore,gc05a2.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/galaxycore,gc08a3.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/galaxycore,gc2145.yaml | 6 +++++-
> Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/imx219.yaml | 6 +++++-
> Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/ovti,og01a1b.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/ovti,ov4689.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/ovti,ov5648.yaml | 5 ++++-
> Documentation/devicetree/bindings/media/i2c/ovti,ov5675.yaml | 3 ++-
> Documentation/devicetree/bindings/media/i2c/ovti,ov7251.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/ovti,ov8865.yaml | 5 ++++-
> Documentation/devicetree/bindings/media/i2c/ovti,ov9282.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/sony,imx258.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/sony,imx283.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/sony,imx412.yaml | 4 +++-
> Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml | 4 +++-
> 23 files changed, 75 insertions(+), 23 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/alliedvision,alvium-csi2.yaml b/Documentation/devicetree/bindings/media/i2c/alliedvision,alvium-csi2.yaml
> index d3329e991d1652936fcf671012b8018e4317ea40..ba166ecf4fcbb77efab69ebcbdb46f5666af8e77 100644
> --- a/Documentation/devicetree/bindings/media/i2c/alliedvision,alvium-csi2.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/alliedvision,alvium-csi2.yaml
> @@ -32,7 +32,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> link-frequencies: true
> @@ -45,9 +45,12 @@ properties:
> - const: 3
> - const: 4
>
> + remote-endpoint: true
> +
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/galaxycore,gc05a2.yaml b/Documentation/devicetree/bindings/media/i2c/galaxycore,gc05a2.yaml
> index 0e7a7b5ac89f618e6cba0d86f6f7ea853e59ae1e..8b42440586aa8c853d8bf6046ccab0c3b23cb907 100644
> --- a/Documentation/devicetree/bindings/media/i2c/galaxycore,gc05a2.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/galaxycore,gc05a2.yaml
> @@ -44,7 +44,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -59,10 +59,12 @@ properties:
> - const: 2
>
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - endpoint
> diff --git a/Documentation/devicetree/bindings/media/i2c/galaxycore,gc08a3.yaml b/Documentation/devicetree/bindings/media/i2c/galaxycore,gc08a3.yaml
> index 51b8ece09c722e057fdb01b38d3e360e7604f39a..c15169ef901139411273e110523a311d87b4322e 100644
> --- a/Documentation/devicetree/bindings/media/i2c/galaxycore,gc08a3.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/galaxycore,gc08a3.yaml
> @@ -44,7 +44,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -59,10 +59,12 @@ properties:
> - const: 2
>
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - endpoint
> diff --git a/Documentation/devicetree/bindings/media/i2c/galaxycore,gc2145.yaml b/Documentation/devicetree/bindings/media/i2c/galaxycore,gc2145.yaml
> index 9eac588de0bc28d85f44663afe5472e35f1e652c..702625962d90ea7fafb4f4f4f865659097b51406 100644
> --- a/Documentation/devicetree/bindings/media/i2c/galaxycore,gc2145.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/galaxycore,gc2145.yaml
> @@ -56,13 +56,17 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> + data-lanes: true
We should have a more precise constraint here. The sensor supports
single or dual data lanes operation, so you can write
data-lanes:
minItems: 1
items:
- const: 1
- const: 2
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> + - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - endpoint
> diff --git a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml
> index d18ead8f7fc43bfacc291aed85b5ca9166c46edb..52bb089bd67fd0f9b5464e068b8db0b8e4406b3d 100644
> --- a/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/hynix,hi846.yaml
> @@ -52,7 +52,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -67,10 +67,12 @@ properties:
> - const: 2
>
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/imx219.yaml b/Documentation/devicetree/bindings/media/i2c/imx219.yaml
> index 07d088cf66e0bde362b12d3494e5c91a1dd96bf3..5f395cf04b95ca47d5e685b8c43b8265db6910ae 100644
> --- a/Documentation/devicetree/bindings/media/i2c/imx219.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/imx219.yaml
> @@ -52,7 +52,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -65,10 +65,14 @@ properties:
> - const: 2
>
> clock-noncontinuous: true
> + clock-lanes: true
This shouldn't be needed, as the sensor doesn't support clock lane
remapping. Could we drop the clock-lanes property from upstream device
tree sources instead ?
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> + - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml b/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
> index f8ace8cbccdbac482ffba733d5b28a3a38aaf822..ce45bd8409597fa6989f632d68cd4aa1a468d152 100644
> --- a/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
> @@ -77,7 +77,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> link-frequencies: true
> @@ -87,11 +87,13 @@ properties:
> - 1 # CSI-2 C-PHY
> - 3 # CCP2
> - 4 # CSI-2 D-PHY
> + remote-endpoint: true
>
> required:
> - link-frequencies
> - data-lanes
> - bus-type
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,og01a1b.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,og01a1b.yaml
> index ca57c01739d2b93100a37db56255ab717c1197ff..9b3738956c482d8826bf64f557c2e91630ea9799 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,og01a1b.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,og01a1b.yaml
> @@ -55,7 +55,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -65,10 +65,12 @@ properties:
> enum: [1, 2]
>
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
> index 67c1c291327b7febb6a039bf6f28c8dc1f32ed7f..b8db4be137085fe31ec2f076c7dc66b30bf0b66c 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov02a10.yaml
> @@ -77,7 +77,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> link-frequencies: true
> @@ -88,9 +88,11 @@ properties:
> the link speed defined by the 'link-frequencies' property.
> If present, the value shall be in the range of 0-4.
> default: 4
> + remote-endpoint: true
>
> required:
> - link-frequencies
> + - remote-endpoint
>
> required:
> - endpoint
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov4689.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov4689.yaml
> index d96199031b66c5c162a034824f195e277f2a1795..7499523a6e0fbd64b9b980333adaa14a0c40a33b 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov4689.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov4689.yaml
> @@ -61,7 +61,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -77,10 +77,12 @@ properties:
> - items:
> - const: 1
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov5648.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov5648.yaml
> index 622243cae03caa5d14aa312df40ef5907e190d2c..358c0422451f7faa8e0ebfc9226a5cfb087e3598 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov5648.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov5648.yaml
> @@ -45,7 +45,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> link-frequencies: true
> @@ -54,9 +54,12 @@ properties:
> minItems: 1
> maxItems: 2
>
> + remote-endpoint: true
> +
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov5675.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov5675.yaml
> index ad07204057f979ade534d29c99c3aff7372bd332..eff212524bf3c7b1ec6aa7e826d4318d58ba53a5 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov5675.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov5675.yaml
> @@ -60,7 +60,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -69,6 +69,7 @@ properties:
>
> # Supports max data transfer of 900 Mbps per lane
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov7251.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov7251.yaml
> index 2e5187acbbb89728cbb8a402559d24766198a3da..cbbe3c9ce151eb33d2b0cc1a44e6ebf66d9b59fa 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov7251.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov7251.yaml
> @@ -53,7 +53,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> clock-lanes:
> @@ -63,10 +63,12 @@ properties:
> maxItems: 1
>
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov8865.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov8865.yaml
> index 382d7de7a89bcea11be384a2a3800512994f9346..dd5c5715fdcfc00e6d851f375f41e4d077b92bc0 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov8865.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov8865.yaml
> @@ -45,7 +45,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> link-frequencies: true
> @@ -54,9 +54,12 @@ properties:
> minItems: 1
> maxItems: 4
>
> + remote-endpoint: true
> +
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov9282.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov9282.yaml
> index 38325cf318f7bd2cd60a4c7bbb6a65b54b855e26..dde4e7426bf00920f1bd9ed1bf4d8594932dedaf 100644
> --- a/Documentation/devicetree/bindings/media/i2c/ovti,ov9282.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov9282.yaml
> @@ -51,15 +51,17 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes: true
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - endpoint
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
> index 0162eec8ca993a7614d29908f89fa9fe6d4b545d..9b78ff6bd5f114c7f63ac90e71fa677445ddf702 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
> @@ -58,7 +58,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -73,10 +73,12 @@ properties:
> - const: 4
>
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx258.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx258.yaml
> index f0f9726a2add89492b8c56e17ed607841baa3a0d..4cf49472c24f1b800f6d2e41b8716e2ac32f959a 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx258.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx258.yaml
> @@ -56,7 +56,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -71,10 +71,12 @@ properties:
> - const: 2
>
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx283.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx283.yaml
> index e4f49f1435a5c2e6e1507d250662ea6ecbf3c7dc..75b78a3e925ed2fd09f56c8349d234a32739f141 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx283.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx283.yaml
> @@ -48,7 +48,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -60,10 +60,12 @@ properties:
> - const: 4
>
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - endpoint
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml
> index bf05ca48601abda53d60a3d03aa556e7b8fd825b..e6aec7a1ba2b22a11111d19a61384f1200041df5 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx290.yaml
> @@ -71,7 +71,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -86,10 +86,12 @@ properties:
> - const: 4
>
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml
> index 872b8288948b2bba743f2365a55165181df156ae..d30ef330e5af225728d1a6c40b26050cd33ba4be 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx334.yaml
> @@ -38,15 +38,17 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes: true
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - endpoint
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml
> index 38bd1c7304a59bb5fea90954c1e4e626a7c86f2f..36c3a0ba7822475770cd903cec3343d31bb66520 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml
> @@ -48,15 +48,17 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes: true
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - endpoint
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx412.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx412.yaml
> index ece1e17fe34553671e61c965eb1833c5eb08131b..0bbdf657a8c0643ffe512ae04c14dfc8e6b4fc94 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx412.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx412.yaml
> @@ -50,15 +50,17 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes: true
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - endpoint
> diff --git a/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml b/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml
> index 1c476b635b690865cff0882578d72b1db2dc7c50..367d669ad864ed6b2a8762f953f58e206c8c8194 100644
> --- a/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml
> @@ -96,7 +96,7 @@ properties:
> properties:
> endpoint:
> $ref: /schemas/media/video-interfaces.yaml#
> - unevaluatedProperties: false
> + additionalProperties: false
>
> properties:
> data-lanes:
> @@ -105,10 +105,12 @@ properties:
>
> clock-noncontinuous: true
> link-frequencies: true
> + remote-endpoint: true
>
> required:
> - data-lanes
> - link-frequencies
> + - remote-endpoint
>
> required:
> - port@0
>
--
Regards,
Laurent Pinchart
Powered by blists - more mailing lists