[PATCH v9 05/22] dt-bindings: ARM: Mediatek: Document bindings of MT8192 clock controllers

Chun-Jie Chen chun-jie.chen at mediatek.com
Thu Jun 3 05:26:29 PDT 2021


On Wed, 2021-06-02 at 12:20 -0500, Rob Herring wrote:
> On Mon, May 24, 2021 at 08:20:36PM +0800, Chun-Jie Chen wrote:
> > This patch adds the binding documentation of topckgen, apmixedsys,
> > infracfg, pericfg and subsystem clocks for Mediatek MT8192.
> 
> I suspect all these could just be merged into 1 schema like the
> others 
> if the only differences are compatible strings.
> 

Did you mean to create 2 schema for mt8192?
1. medaitek,mt8192-clock.yaml: it contains all compatible string that
only provide clocks control.
2. mediatek,mt8192-sys.yaml: it contains all compatible string that
provide clocks and other control.

Best Regards,
Chun-Jie 

> > 
> > Signed-off-by: Weiyi Lu <weiyi.lu at mediatek.com>
> > Signed-off-by: chun-jie.chen <chun-jie.chen at mediatek.com>
> > ---
> >  .../arm/mediatek/mediatek,apmixedsys.txt      |  1 +
> >  .../bindings/arm/mediatek/mediatek,audsys.txt |  1 +
> >  .../bindings/arm/mediatek/mediatek,camsys.txt | 22
> > +++++++++++++++++++
> >  .../bindings/arm/mediatek/mediatek,imgsys.txt |  2 ++
> >  .../arm/mediatek/mediatek,infracfg.txt        |  1 +
> >  .../bindings/arm/mediatek/mediatek,ipesys.txt |  1 +
> >  .../bindings/arm/mediatek/mediatek,mfgcfg.txt |  1 +
> >  .../bindings/arm/mediatek/mediatek,mmsys.txt  |  1 +
> >  .../arm/mediatek/mediatek,pericfg.yaml        |  1 +
> >  .../arm/mediatek/mediatek,topckgen.txt        |  1 +
> >  .../arm/mediatek/mediatek,vdecsys.txt         |  8 +++++++
> >  .../arm/mediatek/mediatek,vencsys.txt         |  1 +
> >  12 files changed, 41 insertions(+)
> > 
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,apmixedsy
> > s.txt
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,apmixedsy
> > s.txt
> > index ea827e8763de..551c30735cd7 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,apmixedsy
> > s.txt
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,apmixedsy
> > s.txt
> > @@ -18,6 +18,7 @@ Required Properties:
> >  	- "mediatek,mt8167-apmixedsys", "syscon"
> >  	- "mediatek,mt8173-apmixedsys"
> >  	- "mediatek,mt8183-apmixedsys", "syscon"
> > +	- "mediatek,mt8192-apmixedsys", "syscon"
> >  	- "mediatek,mt8516-apmixedsys"
> >  - #clock-cells: Must be 1
> >  
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,audsys.tx
> > t
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,audsys.tx
> > t
> > index b32d374193c7..699776be1dd3 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,audsys.tx
> > t
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,audsys.tx
> > t
> > @@ -13,6 +13,7 @@ Required Properties:
> >  	- "mediatek,mt7623-audsys", "mediatek,mt2701-audsys", "syscon"
> >  	- "mediatek,mt8167-audiosys", "syscon"
> >  	- "mediatek,mt8183-audiosys", "syscon"
> > +	- "mediatek,mt8192-audsys", "syscon"
> >  	- "mediatek,mt8516-audsys", "syscon"
> >  - #clock-cells: Must be 1
> >  
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,camsys.tx
> > t
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,camsys.tx
> > t
> > index a0ce82085ad0..7d0b14e5c8ba 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,camsys.tx
> > t
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,camsys.tx
> > t
> > @@ -9,6 +9,10 @@ Required Properties:
> >  	- "mediatek,mt6765-camsys", "syscon"
> >  	- "mediatek,mt6779-camsys", "syscon"
> >  	- "mediatek,mt8183-camsys", "syscon"
> > +	- "mediatek,mt8192-camsys", "syscon"
> > +	- "mediatek,mt8192-camsys_rawa", "syscon"
> > +	- "mediatek,mt8192-camsys_rawb", "syscon"
> > +	- "mediatek,mt8192-camsys_rawc", "syscon"
> >  - #clock-cells: Must be 1
> >  
> >  The camsys controller uses the common clk binding from
> > @@ -22,3 +26,21 @@ camsys: camsys at 1a000000  {
> >  	reg = <0 0x1a000000  0 0x1000>;
> >  	#clock-cells = <1>;
> >  };
> > +
> > +camsys_rawa: syscon at 1a04f000 {
> > +	compatible = "mediatek,mt8192-camsys_rawa", "syscon";
> > +	reg = <0 0x1a04f000 0 0x1000>;
> > +	#clock-cells = <1>;
> > +};
> > +
> > +camsys_rawb: syscon at 1a06f000 {
> > +	compatible = "mediatek,mt8192-camsys_rawb", "syscon";
> > +	reg = <0 0x1a06f000 0 0x1000>;
> > +	#clock-cells = <1>;
> > +};
> > +
> > +camsys_rawc: syscon at 1a08f000 {
> > +	compatible = "mediatek,mt8192-camsys_rawc", "syscon";
> > +	reg = <0 0x1a08f000 0 0x1000>;
> > +	#clock-cells = <1>;
> > +};
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,imgsys.tx
> > t
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,imgsys.tx
> > t
> > index dce4c9241932..b9e599e116dc 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,imgsys.tx
> > t
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,imgsys.tx
> > t
> > @@ -15,6 +15,8 @@ Required Properties:
> >  	- "mediatek,mt8167-imgsys", "syscon"
> >  	- "mediatek,mt8173-imgsys", "syscon"
> >  	- "mediatek,mt8183-imgsys", "syscon"
> > +	- "mediatek,mt8192-imgsys", "syscon"
> > +	- "mediatek,mt8192-imgsys2", "syscon"
> >  - #clock-cells: Must be 1
> >  
> >  The imgsys controller uses the common clk binding from
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,infracfg.
> > txt
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,infracfg.
> > txt
> > index eb3523c7a7be..6e05a0014cf7 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,infracfg.
> > txt
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,infracfg.
> > txt
> > @@ -19,6 +19,7 @@ Required Properties:
> >  	- "mediatek,mt8167-infracfg", "syscon"
> >  	- "mediatek,mt8173-infracfg", "syscon"
> >  	- "mediatek,mt8183-infracfg", "syscon"
> > +	- "mediatek,mt8192-infracfg", "syscon"
> >  	- "mediatek,mt8516-infracfg", "syscon"
> >  - #clock-cells: Must be 1
> >  - #reset-cells: Must be 1
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ipesys.tx
> > t
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ipesys.tx
> > t
> > index 2ce889b023d9..9cd10350ab9b 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,ipesys.tx
> > t
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ipesys.tx
> > t
> > @@ -7,6 +7,7 @@ Required Properties:
> >  
> >  - compatible: Should be one of:
> >  	- "mediatek,mt6779-ipesys", "syscon"
> > +	- "mediatek,mt8192-ipesys", "syscon"
> >  - #clock-cells: Must be 1
> >  
> >  The ipesys controller uses the common clk binding from
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mfgcfg.tx
> > t
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mfgcfg.tx
> > t
> > index 054424fb64b4..6bfb49a43ef9 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mfgcfg.tx
> > t
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mfgcfg.tx
> > t
> > @@ -10,6 +10,7 @@ Required Properties:
> >  	- "mediatek,mt6779-mfgcfg", "syscon"
> >  	- "mediatek,mt8167-mfgcfg", "syscon"
> >  	- "mediatek,mt8183-mfgcfg", "syscon"
> > +	- "mediatek,mt8192-mfgcfg", "syscon"
> >  - #clock-cells: Must be 1
> >  
> >  The mfgcfg controller uses the common clk binding from
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> > index 78c50733985c..9712a6831fab 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> > @@ -16,6 +16,7 @@ Required Properties:
> >  	- "mediatek,mt8167-mmsys", "syscon"
> >  	- "mediatek,mt8173-mmsys", "syscon"
> >  	- "mediatek,mt8183-mmsys", "syscon"
> > +	- "mediatek,mt8192-mmsys", "syscon"
> >  - #clock-cells: Must be 1
> >  
> >  For the clock control, the mmsys controller uses the common clk
> > binding from
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,pericfg.y
> > aml
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,pericfg.y
> > aml
> > index 8723dfe34bab..b405cbcafb90 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,pericfg.y
> > aml
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,pericfg.y
> > aml
> > @@ -26,6 +26,7 @@ properties:
> >                - mediatek,mt8135-pericfg
> >                - mediatek,mt8173-pericfg
> >                - mediatek,mt8183-pericfg
> > +              - mediatek,mt8192-pericfg
> >                - mediatek,mt8516-pericfg
> >            - const: syscon
> >        - items:
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,topckgen.
> > txt
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,topckgen.
> > txt
> > index 5ce7578cf274..1627e384b2ba 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,topckgen.
> > txt
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,topckgen.
> > txt
> > @@ -18,6 +18,7 @@ Required Properties:
> >  	- "mediatek,mt8167-topckgen", "syscon"
> >  	- "mediatek,mt8173-topckgen"
> >  	- "mediatek,mt8183-topckgen", "syscon"
> > +	- "mediatek,mt8192-topckgen", "syscon"
> >  	- "mediatek,mt8516-topckgen"
> >  - #clock-cells: Must be 1
> >  
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,vdecsys.t
> > xt
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,vdecsys.t
> > xt
> > index 98195169176a..376c82ed09ab 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,vdecsys.t
> > xt
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,vdecsys.t
> > xt
> > @@ -14,6 +14,8 @@ Required Properties:
> >  	- "mediatek,mt8167-vdecsys", "syscon"
> >  	- "mediatek,mt8173-vdecsys", "syscon"
> >  	- "mediatek,mt8183-vdecsys", "syscon"
> > +	- "mediatek,mt8192-vdecsys", "syscon"
> > +	- "mediatek,mt8192-vdecsys_soc", "syscon"
> >  - #clock-cells: Must be 1
> >  
> >  The vdecsys controller uses the common clk binding from
> > @@ -27,3 +29,9 @@ vdecsys: clock-controller at 16000000 {
> >  	reg = <0 0x16000000 0 0x1000>;
> >  	#clock-cells = <1>;
> >  };
> > +
> > +vdecsys_soc: syscon at 1600f000 {
> > +	compatible = "mediatek,mt8192-vdecsys_soc", "syscon";
> > +	reg = <0 0x1600f000 0 0x1000>;
> > +	#clock-cells = <1>;
> > +};
> > diff --git
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,vencsys.t
> > xt
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,vencsys.t
> > xt
> > index 6a6a14e15cd7..d22de01c24ec 100644
> > ---
> > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,vencsys.t
> > xt
> > +++
> > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,vencsys.t
> > xt
> > @@ -11,6 +11,7 @@ Required Properties:
> >  	- "mediatek,mt6797-vencsys", "syscon"
> >  	- "mediatek,mt8173-vencsys", "syscon"
> >  	- "mediatek,mt8183-vencsys", "syscon"
> > +	- "mediatek,mt8192-vencsys", "syscon"
> >  - #clock-cells: Must be 1
> >  
> >  The vencsys controller uses the common clk binding from
> > -- 
> > 2.18.0


More information about the linux-arm-kernel mailing list