| Message ID | 20231023-display-support-v4-6-ed82eb168fb1@baylibre.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Add display support for the MT8365-EVK board | expand |
Hi, Alexandre: On Thu, 2024-05-23 at 14:49 +0200, amergnat@baylibre.com wrote: > > External email : Please do not click links or open attachments until you have verified the sender or the content. > From: Fabien Parent <fparent@baylibre.com> > > 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. Reviewed-by: CK Hu <ck.hu@mediatek.com> > > Fixes: 9273cf7d3942 ("dt-bindings: display: mediatek: convert the dpi bindings to yaml") > Signed-off-by: Fabien Parent <fparent@baylibre.com> > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > Acked-by: Rob Herring (Arm) <robh@kernel.org> > Signed-off-by: Alexandre Mergnat <amergnat@baylibre.com> > --- > Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml > index 6607cb1c6e0a..169d054d81ff 100644 > --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml > @@ -80,6 +80,9 @@ properties: > - port@0 > - port@1 > > + power-domains: > + maxItems: 1 > + > required: > - compatible > - reg > @@ -99,11 +102,13 @@ examples: > - | > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/clock/mt8173-clk.h> > + #include <dt-bindings/power/mt8173-power.h> > > dpi: dpi@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>; > > -- > 2.25.1 >
Hi, Alexandre: <amergnat@baylibre.com> 於 2024年5月23日 週四 下午8:49寫道： > > From: Fabien Parent <fparent@baylibre.com> > > 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. I've tired to apply this patch but has conflict. Please rebase this patch onto latest mainline kernel. Other binding patches in this series is applied to mediatek-drm-next [1]. [1] https://git.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git/log/?h=mediatek-drm-next Regards, Chun-Kuang. > > Fixes: 9273cf7d3942 ("dt-bindings: display: mediatek: convert the dpi bindings to yaml") > Signed-off-by: Fabien Parent <fparent@baylibre.com> > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > Acked-by: Rob Herring (Arm) <robh@kernel.org> > Signed-off-by: Alexandre Mergnat <amergnat@baylibre.com> > --- > Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml > index 6607cb1c6e0a..169d054d81ff 100644 > --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml > @@ -80,6 +80,9 @@ properties: > - port@0 > - port@1 > > + power-domains: > + maxItems: 1 > + > required: > - compatible > - reg > @@ -99,11 +102,13 @@ examples: > - | > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/clock/mt8173-clk.h> > + #include <dt-bindings/power/mt8173-power.h> > > dpi: dpi@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>; > > -- > 2.25.1 >
On 21/06/2024 17:10, Chun-Kuang Hu wrote: > Hi, Alexandre: > > <amergnat@baylibre.com> 於 2024年5月23日 週四 下午8:49寫道： >> From: Fabien Parent<fparent@baylibre.com> >> >> 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. > I've tired to apply this patch but has conflict. Please rebase this > patch onto latest mainline kernel. > Other binding patches in this series is applied to mediatek-drm-next [1]. > > [1]https://git.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git/log/?h=mediatek-drm-next Hello Chun-Kuang, First, thank you for the review and merge. This serie has been rebased on top of Angelo's series [1] to use the OF graphs support. I have to rebase on the lastest Angelo's serie [2]. [1] https://lore.kernel.org/all/20240521075717.50330-1-angelogioacchino.delregno@collabora.com/ [2] https://lore.kernel.org/all/20240618101726.110416-1-angelogioacchino.delregno@collabora.com/
diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml index 6607cb1c6e0a..169d054d81ff 100644 --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml @@ -80,6 +80,9 @@ properties: - port@0 - port@1 + power-domains: + maxItems: 1 + required: - compatible - reg @@ -99,11 +102,13 @@ examples: - | #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/clock/mt8173-clk.h> + #include <dt-bindings/power/mt8173-power.h> dpi: dpi@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>;