* [PATCH] dt-bindings: display: bridge: ldb: Add check for reg and reg-names
@ 2026-01-04 21:34 Marek Vasut
2026-01-05 22:19 ` Rob Herring
0 siblings, 1 reply; 3+ messages in thread
From: Marek Vasut @ 2026-01-04 21:34 UTC (permalink / raw)
To: dri-devel
Cc: Marek Vasut, Andrzej Hajda, Conor Dooley, David Airlie,
Jernej Skrabec, Jonas Karlman, Krzysztof Kozlowski,
Laurent Pinchart, Maarten Lankhorst, Maxime Ripard,
Neil Armstrong, Rob Herring, Robert Foss, Simona Vetter,
Thomas Zimmermann, devicetree, linux-kernel
Make "reg" property mandatory for all LDB devices and "reg-names"
mandatory for i.MX8MP and i.MX93 which have two "reg" values. The
i.MX6SX has only one "reg" value so the "reg-names" property there
is optional and not needed.
Signed-off-by: Marek Vasut <marek.vasut@mailbox.org>
---
Cc: Andrzej Hajda <andrzej.hajda@intel.com>
Cc: Conor Dooley <conor+dt@kernel.org>
Cc: David Airlie <airlied@gmail.com>
Cc: Jernej Skrabec <jernej.skrabec@gmail.com>
Cc: Jonas Karlman <jonas@kwiboo.se>
Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Maxime Ripard <mripard@kernel.org>
Cc: Neil Armstrong <neil.armstrong@linaro.org>
Cc: Rob Herring <robh@kernel.org>
Cc: Robert Foss <rfoss@kernel.org>
Cc: Simona Vetter <simona@ffwll.ch>
Cc: Thomas Zimmermann <tzimmermann@suse.de>
Cc: devicetree@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linux-kernel@vger.kernel.org
---
.../bindings/display/bridge/fsl,ldb.yaml | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
index 07388bf2b90df..197ae2f2dd36c 100644
--- a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
@@ -59,6 +59,7 @@ required:
- compatible
- clocks
- ports
+ - reg
allOf:
- if:
@@ -73,6 +74,19 @@ allOf:
ports:
properties:
port@2: false
+ - if:
+ not:
+ properties:
+ compatible:
+ contains:
+ const: fsl,imx6sx-ldb
+ then:
+ required:
+ - compatible
+ - clocks
+ - ports
+ - reg
+ - reg-names
additionalProperties: false
--
2.51.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: display: bridge: ldb: Add check for reg and reg-names
2026-01-04 21:34 [PATCH] dt-bindings: display: bridge: ldb: Add check for reg and reg-names Marek Vasut
@ 2026-01-05 22:19 ` Rob Herring
2026-01-06 1:23 ` Marek Vasut
0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring @ 2026-01-05 22:19 UTC (permalink / raw)
To: Marek Vasut
Cc: dri-devel, Andrzej Hajda, Conor Dooley, David Airlie,
Jernej Skrabec, Jonas Karlman, Krzysztof Kozlowski,
Laurent Pinchart, Maarten Lankhorst, Maxime Ripard,
Neil Armstrong, Robert Foss, Simona Vetter, Thomas Zimmermann,
devicetree, linux-kernel
On Sun, Jan 04, 2026 at 10:34:21PM +0100, Marek Vasut wrote:
> Make "reg" property mandatory for all LDB devices and "reg-names"
> mandatory for i.MX8MP and i.MX93 which have two "reg" values. The
> i.MX6SX has only one "reg" value so the "reg-names" property there
> is optional and not needed.
>
> Signed-off-by: Marek Vasut <marek.vasut@mailbox.org>
> ---
> Cc: Andrzej Hajda <andrzej.hajda@intel.com>
> Cc: Conor Dooley <conor+dt@kernel.org>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Jernej Skrabec <jernej.skrabec@gmail.com>
> Cc: Jonas Karlman <jonas@kwiboo.se>
> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: Maxime Ripard <mripard@kernel.org>
> Cc: Neil Armstrong <neil.armstrong@linaro.org>
> Cc: Rob Herring <robh@kernel.org>
> Cc: Robert Foss <rfoss@kernel.org>
> Cc: Simona Vetter <simona@ffwll.ch>
> Cc: Thomas Zimmermann <tzimmermann@suse.de>
> Cc: devicetree@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-kernel@vger.kernel.org
> ---
> .../bindings/display/bridge/fsl,ldb.yaml | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
> index 07388bf2b90df..197ae2f2dd36c 100644
> --- a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
> @@ -59,6 +59,7 @@ required:
> - compatible
> - clocks
> - ports
> + - reg
>
> allOf:
> - if:
> @@ -73,6 +74,19 @@ allOf:
> ports:
> properties:
> port@2: false
> + - if:
> + not:
> + properties:
> + compatible:
> + contains:
> + const: fsl,imx6sx-ldb
> + then:
> + required:
> + - compatible
> + - clocks
> + - ports
> + - reg
All of these are already required up above. You only need reg-names
here.
> + - reg-names
>
> additionalProperties: false
>
> --
> 2.51.0
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: display: bridge: ldb: Add check for reg and reg-names
2026-01-05 22:19 ` Rob Herring
@ 2026-01-06 1:23 ` Marek Vasut
0 siblings, 0 replies; 3+ messages in thread
From: Marek Vasut @ 2026-01-06 1:23 UTC (permalink / raw)
To: Rob Herring
Cc: dri-devel, Andrzej Hajda, Conor Dooley, David Airlie,
Jernej Skrabec, Jonas Karlman, Krzysztof Kozlowski,
Laurent Pinchart, Maarten Lankhorst, Maxime Ripard,
Neil Armstrong, Robert Foss, Simona Vetter, Thomas Zimmermann,
devicetree, linux-kernel
On 1/5/26 11:19 PM, Rob Herring wrote:
[...]
>> + - if:
>> + not:
>> + properties:
>> + compatible:
>> + contains:
>> + const: fsl,imx6sx-ldb
>> + then:
>> + required:
>> + - compatible
>> + - clocks
>> + - ports
>> + - reg
>
> All of these are already required up above. You only need reg-names
> here.
Fixed in V2, thank you.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-01-06 1:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-04 21:34 [PATCH] dt-bindings: display: bridge: ldb: Add check for reg and reg-names Marek Vasut
2026-01-05 22:19 ` Rob Herring
2026-01-06 1:23 ` Marek Vasut
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox