Hi, Enric: On Thu, 2020-02-27 at 19:08 +0100, Enric Balletbo i Serra wrote: > The mmsys system controller is not only a pure clock controller, so > update the binding documentation to reflect that apart from providing > clocks, it also provides routing and miscellaneous control registers. > Reviewed-by: CK Hu <ck.hu@xxxxxxxxxxxx> > Signed-off-by: Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx> > --- > > Changes in v10: > - Update the binding documentation for the mmsys system controller. > > Changes in v9: None > Changes in v8: None > Changes in v7: None > > .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > index 301eefbe1618..8d6a9d98e7a6 100644 > --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > @@ -1,7 +1,8 @@ > Mediatek mmsys controller > ============================ > > -The Mediatek mmsys controller provides various clocks to the system. > +The Mediatek mmsys system controller provides clock control, routing control, > +and miscellaneous control in mmsys partition. > > Required Properties: > > @@ -15,13 +16,13 @@ Required Properties: > - "mediatek,mt8183-mmsys", "syscon" > - #clock-cells: Must be 1 > > -The mmsys controller uses the common clk binding from > +For the clock control, the mmsys controller uses the common clk binding from > Documentation/devicetree/bindings/clock/clock-bindings.txt > The available clocks are defined in dt-bindings/clock/mt*-clk.h. > > Example: > > -mmsys: clock-controller@14000000 { > +mmsys: syscon@14000000 { > compatible = "mediatek,mt8173-mmsys", "syscon"; > reg = <0 0x14000000 0 0x1000>; > #clock-cells = <1>;