| Message ID | 20210519161847.3747352-1-fparent@baylibre.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/3] dt-bindings: arm: mediatek: mmsys: convert to YAML format | expand |
On Wed, 19 May 2021 18:18:44 +0200, Fabien Parent wrote: > Convert the mmsys bindings to the YAML format. > > Signed-off-by: Fabien Parent <fparent@baylibre.com> > --- > .../bindings/arm/mediatek/mediatek,mmsys.txt | 31 ---------- > .../bindings/arm/mediatek/mediatek,mmsys.yaml | 57 +++++++++++++++++++ > 2 files changed, 57 insertions(+), 31 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > create mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml > Applied, thanks!
On Thu, May 20, 2021 at 8:48 PM Rob Herring <robh@kernel.org> wrote: > > On Wed, 19 May 2021 18:18:44 +0200, Fabien Parent wrote: > > Convert the mmsys bindings to the YAML format. > > > > Signed-off-by: Fabien Parent <fparent@baylibre.com> > > --- > > .../bindings/arm/mediatek/mediatek,mmsys.txt | 31 ---------- > > .../bindings/arm/mediatek/mediatek,mmsys.yaml | 57 +++++++++++++++++++ > > 2 files changed, 57 insertions(+), 31 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > > create mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml > > > > Applied, thanks! Actually, should go with patch 3. So dropped and: Reviewed-by: Rob Herring <robh@kernel.org>
Hi Rob, Missatge de Rob Herring <robh+dt@kernel.org> del dia dv., 21 de maig 2021 a les 3:51: > > On Thu, May 20, 2021 at 8:48 PM Rob Herring <robh@kernel.org> wrote: > > > > On Wed, 19 May 2021 18:18:44 +0200, Fabien Parent wrote: > > > Convert the mmsys bindings to the YAML format. > > > > > > Signed-off-by: Fabien Parent <fparent@baylibre.com> > > > --- > > > .../bindings/arm/mediatek/mediatek,mmsys.txt | 31 ---------- > > > .../bindings/arm/mediatek/mediatek,mmsys.yaml | 57 +++++++++++++++++++ > > > 2 files changed, 57 insertions(+), 31 deletions(-) > > > delete mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > > > create mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml > > > > > > > Applied, thanks! > > Actually, should go with patch 3. So dropped and: > I think that this patch can be applied alone through your tree, patch 2 and 3 can be applied later for the maintainers when they are fine. There are other patches in the ML that depend on having this meged, so it would be really nice to have it. Although it has already your ack/review if it help I also took a look and I know the hardware so: Reviewed-by: Enric Balletbo i Serra <enric.balletbo@collabora.com> Thanks, Enric > Reviewed-by: Rob Herring <robh@kernel.org> > > _______________________________________________ > Linux-mediatek mailing list > Linux-mediatek@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-mediatek
On Fri, Jul 23, 2021 at 3:45 AM Enric Balletbo Serra <eballetbo@gmail.com> wrote: > > Hi Rob, > > Missatge de Rob Herring <robh+dt@kernel.org> del dia dv., 21 de maig > 2021 a les 3:51: > > > > On Thu, May 20, 2021 at 8:48 PM Rob Herring <robh@kernel.org> wrote: > > > > > > On Wed, 19 May 2021 18:18:44 +0200, Fabien Parent wrote: > > > > Convert the mmsys bindings to the YAML format. > > > > > > > > Signed-off-by: Fabien Parent <fparent@baylibre.com> > > > > --- > > > > .../bindings/arm/mediatek/mediatek,mmsys.txt | 31 ---------- > > > > .../bindings/arm/mediatek/mediatek,mmsys.yaml | 57 +++++++++++++++++++ > > > > 2 files changed, 57 insertions(+), 31 deletions(-) > > > > delete mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > > > > create mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml > > > > > > > > > > Applied, thanks! > > > > Actually, should go with patch 3. So dropped and: > > > > I think that this patch can be applied alone through your tree, patch > 2 and 3 can be applied later for the maintainers when they are fine. > There are other patches in the ML that depend on having this meged, so > it would be really nice to have it. Although it has already your > ack/review if it help I also took a look and I know the hardware so: > > Reviewed-by: Enric Balletbo i Serra <enric.balletbo@collabora.com> Okay, patch 1 and 2 applied. Rob
diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt deleted file mode 100644 index 78c50733985c..000000000000 --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt +++ /dev/null @@ -1,31 +0,0 @@ -Mediatek mmsys controller -============================ - -The Mediatek mmsys system controller provides clock control, routing control, -and miscellaneous control in mmsys partition. - -Required Properties: - -- compatible: Should be one of: - - "mediatek,mt2701-mmsys", "syscon" - - "mediatek,mt2712-mmsys", "syscon" - - "mediatek,mt6765-mmsys", "syscon" - - "mediatek,mt6779-mmsys", "syscon" - - "mediatek,mt6797-mmsys", "syscon" - - "mediatek,mt7623-mmsys", "mediatek,mt2701-mmsys", "syscon" - - "mediatek,mt8167-mmsys", "syscon" - - "mediatek,mt8173-mmsys", "syscon" - - "mediatek,mt8183-mmsys", "syscon" -- #clock-cells: Must be 1 - -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: syscon@14000000 { - compatible = "mediatek,mt8173-mmsys", "syscon"; - reg = <0 0x14000000 0 0x1000>; - #clock-cells = <1>; -}; diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml new file mode 100644 index 000000000000..a419da33e10b --- /dev/null +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml @@ -0,0 +1,57 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: "http://devicetree.org/schemas/arm/mediatek/mediatek,mmsys.yaml#" +$schema: "http://devicetree.org/meta-schemas/core.yaml#" + +title: MediaTek mmsys controller + +maintainers: + - Matthias Brugger <matthias.bgg@gmail.com> + +description: + The MediaTek mmsys system controller provides clock control, routing control, + and miscellaneous control in mmsys partition. + +properties: + $nodename: + pattern: "^syscon@[0-9a-f]+$" + + compatible: + oneOf: + - items: + - enum: + - mediatek,mt2701-mmsys + - mediatek,mt2712-mmsys + - mediatek,mt6765-mmsys + - mediatek,mt6779-mmsys + - mediatek,mt6797-mmsys + - mediatek,mt8167-mmsys + - mediatek,mt8173-mmsys + - mediatek,mt8183-mmsys + - const: syscon + - items: + - const: mediatek,mt7623-mmsys + - const: mediatek,mt2701-mmsys + - const: syscon + + reg: + maxItems: 1 + + "#clock-cells": + const: 1 + +required: + - compatible + - reg + - "#clock-cells" + +additionalProperties: false + +examples: + - | + mmsys: syscon@14000000 { + compatible = "mediatek,mt8173-mmsys", "syscon"; + reg = <0x14000000 0x1000>; + #clock-cells = <1>; + };
Convert the mmsys bindings to the YAML format. Signed-off-by: Fabien Parent <fparent@baylibre.com> --- .../bindings/arm/mediatek/mediatek,mmsys.txt | 31 ---------- .../bindings/arm/mediatek/mediatek,mmsys.yaml | 57 +++++++++++++++++++ 2 files changed, 57 insertions(+), 31 deletions(-) delete mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt create mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml