From: allen-kh.cheng <allen-kh.cheng@mediatek.com> To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>, Wim Van Sebroeck <wim@linux-watchdog.org>, Guenter Roeck <linux@roeck-us.net>, "Krzysztof Kozlowski" <krzk+dt@kernel.org>, Matthias Brugger <matthias.bgg@gmail.com>, Rob Herring <robh+dt@kernel.org>, <nfraprado@collabora.com> Cc: <Project_Global_Chrome_Upstream_Group@mediatek.com>, <devicetree@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org>, <linux-mediatek@lists.infradead.org>, <srv_heupstream@mediatek.com>, Chen-Yu Tsai <wenst@chromium.org>, Ryder Lee <ryder.lee@kernel.org> Subject: Re: [PATCH 2/2] dt-bindings: watchdog: mediatek: Convert binding to YAML Date: Mon, 25 Apr 2022 15:37:29 +0800 [thread overview] Message-ID: <b850f88f5044eb6cd57bec4fb864e5670618c9e7.camel@mediatek.com> (raw) In-Reply-To: <e2e4ead0-d8eb-1302-93e9-f330807d811e@linaro.org> Hi Krzysztof, Thanks for your suggestion On Fri, 2022-04-22 at 17:44 +0200, Krzysztof Kozlowski wrote: > On 22/04/2022 14:10, Allen-KH Cheng wrote: > > Convert Mediatek watchdog devicetree binding to YAML. > > > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > > --- > > .../devicetree/bindings/watchdog/mtk-wdt.txt | 42 ---------- > > .../devicetree/bindings/watchdog/mtk-wdt.yaml | 76 > > +++++++++++++++++++ > > 2 files changed, 76 insertions(+), 42 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/watchdog/mtk- > > wdt.txt > > create mode 100644 Documentation/devicetree/bindings/watchdog/mtk- > > wdt.yaml > > > > diff --git a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > deleted file mode 100644 > > index 762c62e428ef..000000000000 > > --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > +++ /dev/null > > @@ -1,42 +0,0 @@ > > -Mediatek SoCs Watchdog timer > > - > > -The watchdog supports a pre-timeout interrupt that fires timeout- > > sec/2 > > -before the expiry. > > - > > -Required properties: > > - > > -- compatible should contain: > > - "mediatek,mt2701-wdt", "mediatek,mt6589-wdt": for MT2701 > > - "mediatek,mt2712-wdt": for MT2712 > > - "mediatek,mt6582-wdt", "mediatek,mt6589-wdt": for MT6582 > > - "mediatek,mt6589-wdt": for MT6589 > > - "mediatek,mt6797-wdt", "mediatek,mt6589-wdt": for MT6797 > > - "mediatek,mt7622-wdt", "mediatek,mt6589-wdt": for MT7622 > > - "mediatek,mt7623-wdt", "mediatek,mt6589-wdt": for MT7623 > > - "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629 > > - "mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for MT7986 > > - "mediatek,mt8183-wdt": for MT8183 > > - "mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for MT8186 > > - "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516 > > - "mediatek,mt8192-wdt": for MT8192 > > - "mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for MT8195 > > - > > -- reg : Specifies base physical address and size of the registers. > > - > > -Optional properties: > > -- mediatek,disable-extrst: disable send output reset signal > > -- interrupts: Watchdog pre-timeout (bark) interrupt. > > -- timeout-sec: contains the watchdog timeout in seconds. > > -- #reset-cells: Should be 1. > > - > > -Example: > > - > > -watchdog: watchdog@10007000 { > > - compatible = "mediatek,mt8183-wdt", > > - "mediatek,mt6589-wdt"; > > - mediatek,disable-extrst; > > - reg = <0 0x10007000 0 0x100>; > > - interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; > > - timeout-sec = <10>; > > - #reset-cells = <1>; > > -}; > > diff --git a/Documentation/devicetree/bindings/watchdog/mtk- > > wdt.yaml b/Documentation/devicetree/bindings/watchdog/mtk-wdt.yaml > > new file mode 100644 > > index 000000000000..5788617027c8 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.yaml > > @@ -0,0 +1,76 @@ > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: > > https://urldefense.com/v3/__http://devicetree.org/schemas/watchdog/mtk-wdt.yaml*__;Iw!!CTRNKA9wMg0ARbw!xERWr2MUvqHyyGH9pqm2mwLVX2kDQAF0wr8QJrl5tQnRanjP0cFLXaLZ4-lCdeJ2erdbFvS3b8m5CyqauSNxK-PDlQ$ > > > > File name: mtk,wdt.yaml > > I assume all future MediaTek watchdogs (even these unpublished and > developed in some years) will fit into this schema. :) If not, then > maybe better to name the file by the compatible... but it's not a > must. > Will correct file name. > > +$schema: > > https://urldefense.com/v3/__http://devicetree.org/meta-schemas/core.yaml*__;Iw!!CTRNKA9wMg0ARbw!xERWr2MUvqHyyGH9pqm2mwLVX2kDQAF0wr8QJrl5tQnRanjP0cFLXaLZ4-lCdeJ2erdbFvS3b8m5CyqauSOMMN0Duw$ > > > > + > > +title: Mediatek SoCs Watchdog timer > > + > > +maintainers: > > + - Runyang Chen <runyang.chen@mediatek.com> > > + - Allen-KH Cheng <allen-kh.cheng@mediatek.com> > > + > > +description: |+ > > + The watchdog supports a pre-timeout interrupt that fires > > timeout-sec/2 > > + before the expiry. > > + > > +allOf: > > + - $ref: "watchdog.yaml#" > > + > > +properties: > > + compatible: > > + oneOf: > > + - const: mediatek,mt2712-wdt > > + - const: mediatek,mt6589-wdt > > + - const: mediatek,mt8183-wdt > > + - const: mediatek,mt8192-wdt > > All these four entries should be an enum. > ok. > > + > > + - items: > > + - enum: > > + - mediatek,mt2701-wdt > > + - mediatek,mt6582-wdt > > + - mediatek,mt6797-wdt > > + - mediatek,mt7622-wdt > > + - mediatek,mt7623-wdt > > + - mediatek,mt7629-wdt > > + - mediatek,mt7986-wdt > > + - mediatek,mt8173-wdt > > + - mediatek,mt8183-wdt > > + - mediatek,mt8186-wdt > > + - mediatek,mt8192-wdt > > + - mediatek,mt8195-wdt > > + - mediatek,mt8516-wdt > > + - const: mediatek,mt6589-wdt > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + maxItems: 1 > > + > > + '#reset-cells': > > + const: 1 > > + > > + timeout-sec: true > > You do not need this, it's coming from watchdog schema. > ok. > > + > > +required: > > + - compatible > > + - reg > > + > > +unevaluatedProperties: false > > + > > +examples: > > + - | > > + #include <dt-bindings/interrupt-controller/arm-gic.h> > > + > > + watchdog@10007000 { > > + compatible = "mediatek,mt8183-wdt", > > + "mediatek,mt6589-wdt"; > > + mediatek,disable-extrst; > > Did you test your bindings? Does not look like. Please run > dt_binding_check (see writing-schema for instructions). > Yes, I have run dt_binding_check. But example is from original binding txt file. Is it ok I can write a new expample? > > + reg = <0x10007000 0x100>; > > + interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; > > Wrong interrupt flag. I wil check this. Thanks, Allen > > > + timeout-sec = <10>; > > + #reset-cells = <1>; > > + }; > > + > > +... > > > Best regards, > Krzysztof _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek
WARNING: multiple messages have this Message-ID (diff)
From: allen-kh.cheng <allen-kh.cheng@mediatek.com> To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>, Wim Van Sebroeck <wim@linux-watchdog.org>, Guenter Roeck <linux@roeck-us.net>, "Krzysztof Kozlowski" <krzk+dt@kernel.org>, Matthias Brugger <matthias.bgg@gmail.com>, Rob Herring <robh+dt@kernel.org>, <nfraprado@collabora.com> Cc: <Project_Global_Chrome_Upstream_Group@mediatek.com>, <devicetree@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org>, <linux-mediatek@lists.infradead.org>, <srv_heupstream@mediatek.com>, Chen-Yu Tsai <wenst@chromium.org>, Ryder Lee <ryder.lee@kernel.org> Subject: Re: [PATCH 2/2] dt-bindings: watchdog: mediatek: Convert binding to YAML Date: Mon, 25 Apr 2022 15:37:29 +0800 [thread overview] Message-ID: <b850f88f5044eb6cd57bec4fb864e5670618c9e7.camel@mediatek.com> (raw) In-Reply-To: <e2e4ead0-d8eb-1302-93e9-f330807d811e@linaro.org> Hi Krzysztof, Thanks for your suggestion On Fri, 2022-04-22 at 17:44 +0200, Krzysztof Kozlowski wrote: > On 22/04/2022 14:10, Allen-KH Cheng wrote: > > Convert Mediatek watchdog devicetree binding to YAML. > > > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > > --- > > .../devicetree/bindings/watchdog/mtk-wdt.txt | 42 ---------- > > .../devicetree/bindings/watchdog/mtk-wdt.yaml | 76 > > +++++++++++++++++++ > > 2 files changed, 76 insertions(+), 42 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/watchdog/mtk- > > wdt.txt > > create mode 100644 Documentation/devicetree/bindings/watchdog/mtk- > > wdt.yaml > > > > diff --git a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > deleted file mode 100644 > > index 762c62e428ef..000000000000 > > --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > +++ /dev/null > > @@ -1,42 +0,0 @@ > > -Mediatek SoCs Watchdog timer > > - > > -The watchdog supports a pre-timeout interrupt that fires timeout- > > sec/2 > > -before the expiry. > > - > > -Required properties: > > - > > -- compatible should contain: > > - "mediatek,mt2701-wdt", "mediatek,mt6589-wdt": for MT2701 > > - "mediatek,mt2712-wdt": for MT2712 > > - "mediatek,mt6582-wdt", "mediatek,mt6589-wdt": for MT6582 > > - "mediatek,mt6589-wdt": for MT6589 > > - "mediatek,mt6797-wdt", "mediatek,mt6589-wdt": for MT6797 > > - "mediatek,mt7622-wdt", "mediatek,mt6589-wdt": for MT7622 > > - "mediatek,mt7623-wdt", "mediatek,mt6589-wdt": for MT7623 > > - "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629 > > - "mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for MT7986 > > - "mediatek,mt8183-wdt": for MT8183 > > - "mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for MT8186 > > - "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516 > > - "mediatek,mt8192-wdt": for MT8192 > > - "mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for MT8195 > > - > > -- reg : Specifies base physical address and size of the registers. > > - > > -Optional properties: > > -- mediatek,disable-extrst: disable send output reset signal > > -- interrupts: Watchdog pre-timeout (bark) interrupt. > > -- timeout-sec: contains the watchdog timeout in seconds. > > -- #reset-cells: Should be 1. > > - > > -Example: > > - > > -watchdog: watchdog@10007000 { > > - compatible = "mediatek,mt8183-wdt", > > - "mediatek,mt6589-wdt"; > > - mediatek,disable-extrst; > > - reg = <0 0x10007000 0 0x100>; > > - interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; > > - timeout-sec = <10>; > > - #reset-cells = <1>; > > -}; > > diff --git a/Documentation/devicetree/bindings/watchdog/mtk- > > wdt.yaml b/Documentation/devicetree/bindings/watchdog/mtk-wdt.yaml > > new file mode 100644 > > index 000000000000..5788617027c8 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.yaml > > @@ -0,0 +1,76 @@ > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: > > https://urldefense.com/v3/__http://devicetree.org/schemas/watchdog/mtk-wdt.yaml*__;Iw!!CTRNKA9wMg0ARbw!xERWr2MUvqHyyGH9pqm2mwLVX2kDQAF0wr8QJrl5tQnRanjP0cFLXaLZ4-lCdeJ2erdbFvS3b8m5CyqauSNxK-PDlQ$ > > > > File name: mtk,wdt.yaml > > I assume all future MediaTek watchdogs (even these unpublished and > developed in some years) will fit into this schema. :) If not, then > maybe better to name the file by the compatible... but it's not a > must. > Will correct file name. > > +$schema: > > https://urldefense.com/v3/__http://devicetree.org/meta-schemas/core.yaml*__;Iw!!CTRNKA9wMg0ARbw!xERWr2MUvqHyyGH9pqm2mwLVX2kDQAF0wr8QJrl5tQnRanjP0cFLXaLZ4-lCdeJ2erdbFvS3b8m5CyqauSOMMN0Duw$ > > > > + > > +title: Mediatek SoCs Watchdog timer > > + > > +maintainers: > > + - Runyang Chen <runyang.chen@mediatek.com> > > + - Allen-KH Cheng <allen-kh.cheng@mediatek.com> > > + > > +description: |+ > > + The watchdog supports a pre-timeout interrupt that fires > > timeout-sec/2 > > + before the expiry. > > + > > +allOf: > > + - $ref: "watchdog.yaml#" > > + > > +properties: > > + compatible: > > + oneOf: > > + - const: mediatek,mt2712-wdt > > + - const: mediatek,mt6589-wdt > > + - const: mediatek,mt8183-wdt > > + - const: mediatek,mt8192-wdt > > All these four entries should be an enum. > ok. > > + > > + - items: > > + - enum: > > + - mediatek,mt2701-wdt > > + - mediatek,mt6582-wdt > > + - mediatek,mt6797-wdt > > + - mediatek,mt7622-wdt > > + - mediatek,mt7623-wdt > > + - mediatek,mt7629-wdt > > + - mediatek,mt7986-wdt > > + - mediatek,mt8173-wdt > > + - mediatek,mt8183-wdt > > + - mediatek,mt8186-wdt > > + - mediatek,mt8192-wdt > > + - mediatek,mt8195-wdt > > + - mediatek,mt8516-wdt > > + - const: mediatek,mt6589-wdt > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + maxItems: 1 > > + > > + '#reset-cells': > > + const: 1 > > + > > + timeout-sec: true > > You do not need this, it's coming from watchdog schema. > ok. > > + > > +required: > > + - compatible > > + - reg > > + > > +unevaluatedProperties: false > > + > > +examples: > > + - | > > + #include <dt-bindings/interrupt-controller/arm-gic.h> > > + > > + watchdog@10007000 { > > + compatible = "mediatek,mt8183-wdt", > > + "mediatek,mt6589-wdt"; > > + mediatek,disable-extrst; > > Did you test your bindings? Does not look like. Please run > dt_binding_check (see writing-schema for instructions). > Yes, I have run dt_binding_check. But example is from original binding txt file. Is it ok I can write a new expample? > > + reg = <0x10007000 0x100>; > > + interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; > > Wrong interrupt flag. I wil check this. Thanks, Allen > > > + timeout-sec = <10>; > > + #reset-cells = <1>; > > + }; > > + > > +... > > > Best regards, > Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-04-25 7:42 UTC|newest] Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-04-22 12:10 [PATCH 0/2] dt-bindings: watchdog: mediatek: Convert binding to YAML Allen-KH Cheng 2022-04-22 12:10 ` Allen-KH Cheng 2022-04-22 12:10 ` Allen-KH Cheng 2022-04-22 12:10 ` [PATCH 1/2] arm64: dts: mt8516: align watchdog node name with dtschema Allen-KH Cheng 2022-04-22 12:10 ` Allen-KH Cheng 2022-04-22 12:10 ` Allen-KH Cheng 2022-04-22 15:45 ` Krzysztof Kozlowski 2022-04-22 15:45 ` Krzysztof Kozlowski 2022-04-22 15:45 ` Krzysztof Kozlowski 2022-04-22 12:10 ` [PATCH 2/2] dt-bindings: watchdog: mediatek: Convert binding to YAML Allen-KH Cheng 2022-04-22 12:10 ` Allen-KH Cheng 2022-04-22 12:10 ` Allen-KH Cheng 2022-04-22 15:44 ` Krzysztof Kozlowski 2022-04-22 15:44 ` Krzysztof Kozlowski 2022-04-22 15:44 ` Krzysztof Kozlowski 2022-04-25 7:37 ` allen-kh.cheng [this message] 2022-04-25 7:37 ` allen-kh.cheng 2022-04-25 7:53 ` Krzysztof Kozlowski 2022-04-25 7:53 ` Krzysztof Kozlowski 2022-04-25 7:53 ` Krzysztof Kozlowski 2022-04-25 16:02 ` Rob Herring 2022-04-25 16:02 ` Rob Herring 2022-04-25 16:02 ` Rob Herring
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=b850f88f5044eb6cd57bec4fb864e5670618c9e7.camel@mediatek.com \ --to=allen-kh.cheng@mediatek.com \ --cc=Project_Global_Chrome_Upstream_Group@mediatek.com \ --cc=devicetree@vger.kernel.org \ --cc=krzk+dt@kernel.org \ --cc=krzysztof.kozlowski@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=linux@roeck-us.net \ --cc=matthias.bgg@gmail.com \ --cc=nfraprado@collabora.com \ --cc=robh+dt@kernel.org \ --cc=ryder.lee@kernel.org \ --cc=srv_heupstream@mediatek.com \ --cc=wenst@chromium.org \ --cc=wim@linux-watchdog.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.