Re: [PATCH v10 07/17] dt-bindings: display: mediatek: merge: add additional prop for mt8195

Hi Philipp,

Thanks for the reviews.

On Wed, 2021-09-08 at 08:39 +0200, Philipp Zabel wrote:

Hi Jason,

On Wed, 2021-09-08 at 14:03 +0800, jason-jh.lin wrote:

quoted

add MERGE additional properties description for mt8195:
1. async clock
2. fifo setting enable
3. reset controller

Signed-off-by: jason-jh.lin <redacted>
---
 .../display/mediatek/mediatek,merge.yaml      | 30
+++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git
a/Documentation/devicetree/bindings/display/mediatek/mediatek,merge
.yaml
b/Documentation/devicetree/bindings/display/mediatek/mediatek,merge
.yaml
index 75beeb207ceb..0fe204d9ad2c 100644
---
a/Documentation/devicetree/bindings/display/mediatek/mediatek,merge
.yaml
+++
b/Documentation/devicetree/bindings/display/mediatek/mediatek,merge
.yaml

@@ -38,6 +38,19 @@ properties:
   clocks:
     items:
       - description: MERGE Clock
+      - description: MERGE Async Clock
+          Controlling the synchronous process between MERGE and

other display
+          function blocks cross clock domain.
+
+  mediatek,merge-fifo-en:
+    description:
+      The setting of merge fifo is mainly provided for the display
latency
+      buffer to ensure that the back-end panel display data will
not be
+      underrun, a little more data is needed in the fifo.
+      According to the merge fifo settings, when the water level
is detected
+      to be insufficient, it will trigger RDMA sending ultra and
preulra
+      command to SMI to speed up the data rate.
+    type: boolean
 
   mediatek,gce-client-reg:
     description:

@@ -50,6 +63,10 @@ properties:
     $ref: /schemas/types.yaml#/definitions/phandle-array
     maxItems: 1
 
+  resets:
+    description: reset controller
+      See Documentation/devicetree/bindings/reset/reset.txt for

details.

From the example this looks like it could have a maxItems: 1.

OK, I think it could have a maxItems: 1 in mt8195 because
merge1~megre5 only have one async clock.

quoted

+
 required:
   - compatible
   - reg

Should the resets property be required for "mediatek,mt8195-disp-
merge"?

I think the resets property is not the required propoerty.
The reset controller is for async clock of MERGE module on vdosys1.
MERGE module on vdosys0 doesn't have async clock, so it doesn't need to
add the resets property.

Regards,
Jason-JH.Lin

quoted

@@ -67,3 +84,16 @@ examples:
         power-domains = <&spm MT8173_POWER_DOMAIN_MM>;
         clocks = <&mmsys CLK_MM_DISP_MERGE>;
     };
+
+    merge5: disp_vpp_merge5@1c110000 {
+        compatible = "mediatek,mt8195-disp-merge";
+        reg = <0 0x1c110000 0 0x1000>;
+        interrupts = <GIC_SPI 507 IRQ_TYPE_LEVEL_HIGH 0>;
+        clocks = <&vdosys1 CLK_VDO1_VPP_MERGE4>,
+                 <&vdosys1 CLK_VDO1_MERGE4_DL_ASYNC>;
+        clock-names = "merge","merge_async";
+        power-domains = <&spm MT8195_POWER_DOMAIN_VDOSYS1>;
+        mediatek,gce-client-reg = <&gce1 SUBSYS_1c11XXXX 0x0000

0x1000>;
+        mediatek,merge-fifo-en = <1>;
+        resets = <&vdosys1
MT8195_VDOSYS1_SW0_RST_B_MERGE4_DL_ASYNC>;
+    };

regards
Philipp

-- 
Jason-JH Lin [off-list ref]


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel