Thread (90 messages) 90 messages, 10 authors, 2022-06-13

Re: [PATCH v10 01/21] dt-bindings: mediatek,dpi: Add DPINTF compatible

From: Rex-BC Chen <hidden>
Date: 2022-06-07 02:36:02
Also in: dri-devel, linux-arm-kernel, linux-devicetree, linux-mediatek, linux-phy, lkml

On Wed, 2022-05-25 at 14:49 +0200, Maxime Ripard wrote:
Hi,

On Mon, May 23, 2022 at 12:47:34PM +0200, Guillaume Ranquet wrote:
quoted
From: Markus Schneider-Pargmann <msp@baylibre.com>

DPINTF is similar to DPI but does not have the exact same feature
set
or register layouts.

DPINTF 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>
---
 .../bindings/display/mediatek/mediatek,dpi.yaml     | 13 ++++++++-
----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git
a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.y
aml
b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.y
aml
index dd2896a40ff0..6d9f6c11806e 100644
---
a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.y
aml
+++
b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.y
aml
@@ -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/DPINTF Controller
 
 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
-  provides 8-bit RGB/YUV444 or 8/10/10-bit YUV422 pixel data on a
parallel
-  output bus.
+  The Mediatek DPI and DPINTF function blocks are a sink of the
display
+  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
Weren't you supposed to change it to have a separator between dp and
intf?

If it's no longer in your plans, the second patch should have
s/dp_intf/dpintf/

Maxime
Hello Maxime,

Thank for your review.
I will do this in next version.

BRs,
Bo-Chen
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help