Thread (67 messages) 67 messages, 7 authors, 2022-05-25

Re: [PATCH v9 01/22] dt-bindings: mediatek,dpi: Add DP_INTF compatible

From: Miles Chen <hidden>
Date: 2022-03-30 05:12:30
Also in: dri-devel, linux-arm-kernel, linux-devicetree, linux-mediatek, linux-phy, lkml

Hi Guillaume,
quoted hunk ↗ jump to hunk
DP_INTF is similar to DPI but does not have the exact same feature set
or register layouts.

DP_INTF is the sink of the display pipeline that is connected to the
DisplayPort controller and encoder unit. It takes the same clocks as
DPI.

Signed-off-by: Markus Schneider-Pargmann <msp@baylibre.com>
Signed-off-by: Guillaume Ranquet <redacted>
Reviewed-by: Rob Herring <robh@kernel.org>
---
.../bindings/display/mediatek/mediatek,dpi.yaml       | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
index dd2896a40ff0..2dba80ad3b18 100644
--- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
+++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
@@ -4,16 +4,16 @@
$id: http://devicetree.org/schemas/display/mediatek/mediatek,dpi.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: mediatek DPI Controller Device Tree Bindings
+title: mediatek DPI/DP_INTF Controller
         ^^^^^^^^
Would you mind changing this 'mediatek' to 'MediaTek'?
maintainers:
  - CK Hu [off-list ref]
  - Jitao shi [off-list ref]

description: |
-  The Mediatek DPI function block is a sink of the display subsystem and
s/Mediatek/MediaTek/

Could you replace all 'Mediatek' with 'MediaTek' of this series, please?
We have set up a chcker inside our organization to capture this case,
hopefully we will see less 'Mediatek' in the future.
-  provides 8-bit RGB/YUV444 or 8/10/10-bit YUV422 pixel data on a parallel
-  output bus.
+  The Mediatek DPI and DP_INTF function blocks are a sink of the display
s/Mediatek/MediaTek/

thanks,
Miles
quoted hunk ↗ jump to hunk
+  subsystem and provides 8-bit RGB/YUV444 or 8/10/10-bit YUV422 pixel data on a
+  parallel output bus.

properties:
  compatible:
@@ -23,6 +23,7 @@ properties:
      - mediatek,mt8173-dpi
      - mediatek,mt8183-dpi
      - mediatek,mt8192-dpi
+      - mediatek,mt8195-dpintf

  reg:
    maxItems: 1
@@ -54,7 +55,7 @@ properties:
    $ref: /schemas/graph.yaml#/properties/port
    description:
      Output port node. This port should be connected to the input port of an
-      attached HDMI or LVDS encoder chip.
+      attached HDMI, LVDS or DisplayPort encoder chip.
required:
  - compatible
-- 
2.34.1
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help