From: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp> To: Punit Agrawal <punit1.agrawal@toshiba.co.jp> Cc: Rob Herring <robh+dt@kernel.org>, Wim Van Sebroeck <wim@linux-watchdog.org>, Guenter Roeck <linux@roeck-us.net>, devicetree@vger.kernel.org, linux-watchdog@vger.kernel.org, yuji2.ishikawa@toshiba.co.jp, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 1/2] watchdog: bindings: Add binding documentation for Toshiba Visconti watchdog device Date: Mon, 21 Sep 2020 17:43:30 +0900 [thread overview] Message-ID: <20200921084330.6cqci4oit6fdzgis@toshiba.co.jp> (raw) In-Reply-To: <878sd3lhcy.fsf@kokedama.swc.toshiba.co.jp> Hi, Thanks for you review. On Mon, Sep 21, 2020 at 04:27:25PM +0900, Punit Agrawal wrote: > Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp> writes: > > > Add documentation for the binding of Toshiba Visconti SoC's watchdog. > > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp> > > --- > > .../watchdog/toshiba,visconti-wdt.yaml | 49 +++++++++++++++++++ > > 1 file changed, 49 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml > > > > diff --git a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml > > new file mode 100644 > > index 000000000000..721e38fa5a0f > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml > > @@ -0,0 +1,49 @@ > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > > +# Copyright 2020 Toshiba Electronic Devices & Storage Corporation > > +%YAML 1.2 > > +--- > > +$id: "http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#" > > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > > + > > +title: Toshiba Visconti SoCs PIUWDT Watchdog timer > > + > > +maintainers: > > + - Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp> > > + > > +allOf: > > + - $ref: watchdog.yaml# > > + > > +properties: > > + compatible: > > + enum: > > + - toshiba,visconti-wdt > > + > > + reg: > > + maxItems: 1 > > + > > + clocks: > > + maxItems: 1 > > + > > +required: > > + - compatible > > + - reg > > + - clocks > > + > > +examples: > > + - | > > + soc { > > + #address-cells = <2>; > > + #size-cells = <2>; > > + > > + wdt_clk: wdt-clk { > > + compatible = "fixed-clock"; > > + clock-frequency = <150000000>; > > + #clock-cells = <0>; > > + }; > > + > > + watchdog@28330000 { > > + compatible = "toshiba,visconti-wdt"; > > + reg = <0 0x28330000 0 0x1000>; > > + clocks = <&wdt_clk>; > > + }; > > I was going to suggest adding the "timeout-sec" property as described in > watchdog.yaml but both code and usage seems to suggest it is optional. > I see. I will add about timeout-sec property. > > + }; > > So, > > Reviewed-by: Punit Agrawal <punit1.agrawal@toshiba.co.jp> > Thanks! > Thanks. > Best regards, Nobuhiro
WARNING: multiple messages have this Message-ID (diff)
From: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp> To: Punit Agrawal <punit1.agrawal@toshiba.co.jp> Cc: devicetree@vger.kernel.org, linux-watchdog@vger.kernel.org, Rob Herring <robh+dt@kernel.org>, linux-arm-kernel@lists.infradead.org, Wim Van Sebroeck <wim@linux-watchdog.org>, yuji2.ishikawa@toshiba.co.jp, Guenter Roeck <linux@roeck-us.net> Subject: Re: [PATCH v3 1/2] watchdog: bindings: Add binding documentation for Toshiba Visconti watchdog device Date: Mon, 21 Sep 2020 17:43:30 +0900 [thread overview] Message-ID: <20200921084330.6cqci4oit6fdzgis@toshiba.co.jp> (raw) In-Reply-To: <878sd3lhcy.fsf@kokedama.swc.toshiba.co.jp> Hi, Thanks for you review. On Mon, Sep 21, 2020 at 04:27:25PM +0900, Punit Agrawal wrote: > Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp> writes: > > > Add documentation for the binding of Toshiba Visconti SoC's watchdog. > > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp> > > --- > > .../watchdog/toshiba,visconti-wdt.yaml | 49 +++++++++++++++++++ > > 1 file changed, 49 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml > > > > diff --git a/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml > > new file mode 100644 > > index 000000000000..721e38fa5a0f > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml > > @@ -0,0 +1,49 @@ > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > > +# Copyright 2020 Toshiba Electronic Devices & Storage Corporation > > +%YAML 1.2 > > +--- > > +$id: "http://devicetree.org/schemas/watchdog/toshiba,visconti-wdt.yaml#" > > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > > + > > +title: Toshiba Visconti SoCs PIUWDT Watchdog timer > > + > > +maintainers: > > + - Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp> > > + > > +allOf: > > + - $ref: watchdog.yaml# > > + > > +properties: > > + compatible: > > + enum: > > + - toshiba,visconti-wdt > > + > > + reg: > > + maxItems: 1 > > + > > + clocks: > > + maxItems: 1 > > + > > +required: > > + - compatible > > + - reg > > + - clocks > > + > > +examples: > > + - | > > + soc { > > + #address-cells = <2>; > > + #size-cells = <2>; > > + > > + wdt_clk: wdt-clk { > > + compatible = "fixed-clock"; > > + clock-frequency = <150000000>; > > + #clock-cells = <0>; > > + }; > > + > > + watchdog@28330000 { > > + compatible = "toshiba,visconti-wdt"; > > + reg = <0 0x28330000 0 0x1000>; > > + clocks = <&wdt_clk>; > > + }; > > I was going to suggest adding the "timeout-sec" property as described in > watchdog.yaml but both code and usage seems to suggest it is optional. > I see. I will add about timeout-sec property. > > + }; > > So, > > Reviewed-by: Punit Agrawal <punit1.agrawal@toshiba.co.jp> > Thanks! > Thanks. > Best regards, Nobuhiro _______________________________________________ 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:[~2020-09-21 8:44 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-09-20 5:18 [PATCH v3 0/2] Add WDT driver for Toshiba Visconti ARM SoC Nobuhiro Iwamatsu 2020-09-20 5:18 ` Nobuhiro Iwamatsu 2020-09-20 5:18 ` [PATCH v3 1/2] watchdog: bindings: Add binding documentation for Toshiba Visconti watchdog device Nobuhiro Iwamatsu 2020-09-20 5:18 ` Nobuhiro Iwamatsu 2020-09-21 7:27 ` Punit Agrawal 2020-09-21 7:27 ` Punit Agrawal 2020-09-21 8:43 ` Nobuhiro Iwamatsu [this message] 2020-09-21 8:43 ` Nobuhiro Iwamatsu 2020-09-20 5:18 ` [PATCH v3 2/2] watchdog: Add Toshiba Visconti watchdog driver Nobuhiro Iwamatsu 2020-09-20 5:18 ` Nobuhiro Iwamatsu 2020-09-21 7:25 ` Punit Agrawal 2020-09-21 7:25 ` Punit Agrawal 2020-09-21 8:46 ` Nobuhiro Iwamatsu 2020-09-21 8:46 ` Nobuhiro Iwamatsu
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=20200921084330.6cqci4oit6fdzgis@toshiba.co.jp \ --to=nobuhiro1.iwamatsu@toshiba.co.jp \ --cc=devicetree@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-watchdog@vger.kernel.org \ --cc=linux@roeck-us.net \ --cc=punit1.agrawal@toshiba.co.jp \ --cc=robh+dt@kernel.org \ --cc=wim@linux-watchdog.org \ --cc=yuji2.ishikawa@toshiba.co.jp \ /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.