[PATCH 1/7] dt-bindings: display: mediatek: dpi: add power-domains property
CK Hu
ck.hu at mediatek.com
Thu Jun 16 22:31:29 PDT 2022
Hi, Fabien:
On Mon, 2022-05-30 at 22:14 +0200, Fabien Parent wrote:
> DPI is part of the display / multimedia block in MediaTek SoCs, and
> always have a power-domain (at least in the upstream device-trees).
> Add the power-domains property to the binding documentation.
>
> Signed-off-by: Fabien Parent <fparent at baylibre.com>
> ---
> .../devicetree/bindings/display/mediatek/mediatek,dpi.yaml | 6
> ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git
> a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yam
> l
> b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yam
> l
> index 77ee1b923991..caf4c88708f4 100644
> ---
> a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yam
> l
> +++
> b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yam
> l
> @@ -57,6 +57,9 @@ properties:
> Output port node. This port should be connected to the input
> port of an
> attached HDMI or LVDS encoder chip.
>
> + power-domains:
> + maxItems: 1
> +
> required:
> - compatible
> - reg
> @@ -64,6 +67,7 @@ required:
> - clocks
> - clock-names
> - port
> + - power-domains
>
> additionalProperties: false
>
> @@ -71,11 +75,13 @@ examples:
> - |
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> #include <dt-bindings/clock/mt8173-clk.h>
> + #include <dt-bindings/power/mt8183-power.h>
Why do you include mt8183 power header file for mt8173 dpi node?
Regards,
CK
>
> dpi0: dpi at 1401d000 {
> compatible = "mediatek,mt8173-dpi";
> reg = <0x1401d000 0x1000>;
> interrupts = <GIC_SPI 194 IRQ_TYPE_LEVEL_LOW>;
> + power-domains = <&spm MT8173_POWER_DOMAIN_MM>;
> clocks = <&mmsys CLK_MM_DPI_PIXEL>,
> <&mmsys CLK_MM_DPI_ENGINE>,
> <&apmixedsys CLK_APMIXED_TVDPLL>;
More information about the Linux-mediatek
mailing list