Thread (17 messages) 17 messages, 3 authors, 2026-05-29

Re: [PATCH v23 5/8] dt-bindings: display: bridge: Add Cadence MHDP8501

From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2026-05-26 06:10:54
Also in: dri-devel, imx, linux-arm-kernel, linux-phy, lkml

On 26/05/2026 08:08, Krzysztof Kozlowski wrote:
On Tue, May 19, 2026 at 02:42:28PM +0000, Laurentiu Palcu wrote:
quoted
From: Sandor Yu <redacted>

Add bindings for Cadence MHDP8501 DisplayPort/HDMI bridge.

Signed-off-by: Sandor Yu <redacted>
Signed-off-by: Laurentiu Palcu <laurentiu.palcu@oss.nxp.com>
---
 .../bindings/display/bridge/cdns,mhdp8501.yaml     | 136 +++++++++++++++++++++
 1 file changed, 136 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/bridge/cdns,mhdp8501.yaml b/Documentation/devicetree/bindings/display/bridge/cdns,mhdp8501.yaml
new file mode 100644
index 0000000000000..57e7e95199777
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/bridge/cdns,mhdp8501.yaml
Filename must match compatible.
quoted
@@ -0,0 +1,136 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/bridge/cdns,mhdp8501.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Cadence MHDP8501 DP/HDMI bridge
+
+maintainers:
+  - Laurentiu Palcu <laurentiu.palcu@oss.nxp.com>
+
+description:
+  Cadence MHDP8501 DisplayPort/HDMI interface.
+
+properties:
+  compatible:
+    enum:
+      - fsl,imx8mq-mhdp8501-hdmi
+      - fsl,imx8mq-mhdp8501-dp
We are at v23 and you will be getting the same questions till you
finally fix that commit msg.

Why bus/connector is part of the compatible? Device is exactly the same.
Please read writing bindings - it covers exactly this case.
And this was BTW completely different in previous version.

NAK

Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help