Re: [RESEND v7, PATCH 1/7] dt-bindings: mediatek: add rdma_fifo_size description for mt8183 display
On Sat, Aug 08, 2020 at 10:53:45AM +0800, Yongqiang Niu wrote: > rdma fifo size may be different even in same SOC, add this > property to the corresponding rdma > > Change-Id: I67635ec7f3f59cf4cbc7737285e5e28ff0ab71c9 > Signed-off-by: Yongqiang Niu > --- > .../devicetree/bindings/display/mediatek/mediatek,disp.txt | 14 > ++ > 1 file changed, 14 insertions(+) > > diff --git > a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > index b91e709..e6bbe32 100644 > --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > @@ -66,6 +66,11 @@ Required properties (DMA function blocks): >argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.txt >for details. > > +Optional properties (RDMA function blocks): > +- mediatek,rdma_fifo_size: rdma fifo size may be different even in same SOC, > add this mediatek,rdma-fifo-size What's the range/set of valid values? > + property to the corresponding rdma > + the value is the Max value which defined in hardware data sheet. > + > Examples: > > mmsys: clock-controller@1400 { > @@ -207,3 +212,12 @@ od@14023000 { > power-domains = < MT8173_POWER_DOMAIN_MM>; > clocks = < CLK_MM_DISP_OD>; > }; > + > +rdma1: rdma@1400c000 { Does a new property really need a whole new example? > + compatible = "mediatek,mt8183-disp-rdma"; > + reg = <0 0x1400c000 0 0x1000>; > + interrupts = ; > + power-domains = < MT8183_POWER_DOMAIN_DISP>; > + clocks = < CLK_MM_DISP_RDMA1>; > + mediatek,rdma_fifo_size = <2048>; > +}; > -- > 1.8.1.1.dirty ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[RESEND v7, PATCH 1/7] dt-bindings: mediatek: add rdma_fifo_size description for mt8183 display
rdma fifo size may be different even in same SOC, add this property to the corresponding rdma Change-Id: I67635ec7f3f59cf4cbc7737285e5e28ff0ab71c9 Signed-off-by: Yongqiang Niu --- .../devicetree/bindings/display/mediatek/mediatek,disp.txt | 14 ++ 1 file changed, 14 insertions(+) diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt index b91e709..e6bbe32 100644 --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt @@ -66,6 +66,11 @@ Required properties (DMA function blocks): argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.txt for details. +Optional properties (RDMA function blocks): +- mediatek,rdma_fifo_size: rdma fifo size may be different even in same SOC, add this + property to the corresponding rdma + the value is the Max value which defined in hardware data sheet. + Examples: mmsys: clock-controller@1400 { @@ -207,3 +212,12 @@ od@14023000 { power-domains = < MT8173_POWER_DOMAIN_MM>; clocks = < CLK_MM_DISP_OD>; }; + +rdma1: rdma@1400c000 { + compatible = "mediatek,mt8183-disp-rdma"; + reg = <0 0x1400c000 0 0x1000>; + interrupts = ; + power-domains = < MT8183_POWER_DOMAIN_DISP>; + clocks = < CLK_MM_DISP_RDMA1>; + mediatek,rdma_fifo_size = <2048>; +}; -- 1.8.1.1.dirty ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [RESEND v7, PATCH 1/7] dt-bindings: mediatek: add rdma_fifo_size description for mt8183 display
Hi, Yongqiang: Chun-Kuang Hu 於 2020年8月9日 週日 上午8:56寫道: > > Hi, Yongqiang: > > Yongqiang Niu 於 2020年8月8日 週六 上午11:04寫道: > > > > rdma fifo size may be different even in same SOC, add this > > property to the corresponding rdma > > > > Change-Id: I67635ec7f3f59cf4cbc7737285e5e28ff0ab71c9 > > Remove change-id. > > > Signed-off-by: Yongqiang Niu > > --- > > .../devicetree/bindings/display/mediatek/mediatek,disp.txt | 14 > > ++ > > 1 file changed, 14 insertions(+) > > > > diff --git > > a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > > b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > > index b91e709..e6bbe32 100644 > > --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > > @@ -66,6 +66,11 @@ Required properties (DMA function blocks): > >argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.txt > >for details. > > > > +Optional properties (RDMA function blocks): > > +- mediatek,rdma_fifo_size: rdma fifo size may be different even in same > > SOC, add this > > + property to the corresponding rdma > > + the value is the Max value which defined in hardware data sheet. > > + > > Examples: > > > > mmsys: clock-controller@1400 { > > @@ -207,3 +212,12 @@ od@14023000 { > > power-domains = < MT8173_POWER_DOMAIN_MM>; > > clocks = < CLK_MM_DISP_OD>; > > }; > > + > > +rdma1: rdma@1400c000 { > > + compatible = "mediatek,mt8183-disp-rdma"; > > + reg = <0 0x1400c000 0 0x1000>; > > + interrupts = ; > > + power-domains = < MT8183_POWER_DOMAIN_DISP>; > > + clocks = < CLK_MM_DISP_RDMA1>; > > + mediatek,rdma_fifo_size = <2048>; > > +}; > > I would like you to show rdma0 as well so that could prove two rdma > have different fifo size in the same SoC. Sorry, rdma0 is already define in this file. Just ignore this comment. Regards, Chun-Kuang. > > Regards, > Chun-Kuang. > > > -- > > 1.8.1.1.dirty > > ___ > > Linux-mediatek mailing list > > linux-media...@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-mediatek ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [RESEND v7, PATCH 1/7] dt-bindings: mediatek: add rdma_fifo_size description for mt8183 display
Hi, Yongqiang: Yongqiang Niu 於 2020年8月8日 週六 上午11:04寫道: > > rdma fifo size may be different even in same SOC, add this > property to the corresponding rdma > > Change-Id: I67635ec7f3f59cf4cbc7737285e5e28ff0ab71c9 Remove change-id. > Signed-off-by: Yongqiang Niu > --- > .../devicetree/bindings/display/mediatek/mediatek,disp.txt | 14 > ++ > 1 file changed, 14 insertions(+) > > diff --git > a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > index b91e709..e6bbe32 100644 > --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > @@ -66,6 +66,11 @@ Required properties (DMA function blocks): >argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.txt >for details. > > +Optional properties (RDMA function blocks): > +- mediatek,rdma_fifo_size: rdma fifo size may be different even in same SOC, > add this > + property to the corresponding rdma > + the value is the Max value which defined in hardware data sheet. > + > Examples: > > mmsys: clock-controller@1400 { > @@ -207,3 +212,12 @@ od@14023000 { > power-domains = < MT8173_POWER_DOMAIN_MM>; > clocks = < CLK_MM_DISP_OD>; > }; > + > +rdma1: rdma@1400c000 { > + compatible = "mediatek,mt8183-disp-rdma"; > + reg = <0 0x1400c000 0 0x1000>; > + interrupts = ; > + power-domains = < MT8183_POWER_DOMAIN_DISP>; > + clocks = < CLK_MM_DISP_RDMA1>; > + mediatek,rdma_fifo_size = <2048>; > +}; I would like you to show rdma0 as well so that could prove two rdma have different fifo size in the same SoC. Regards, Chun-Kuang. > -- > 1.8.1.1.dirty > ___ > Linux-mediatek mailing list > linux-media...@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-mediatek ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel