From: "Heiko Stübner" <heiko@sntech.de> To: Rob Herring <robh+dt@kernel.org>, Paul Walmsley <paul.walmsley@sifive.com>, Palmer Dabbelt <palmer@dabbelt.com>, Albert Ou <aou@eecs.berkeley.edu>, Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>, Qiu Wenbo <qiuwenbo@kylinos.com.cn>, Yash Shah <yash.shah@sifive.com>, devicetree@vger.kernel.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, David Abdurachmanov <david.abdurachmanov@sifive.com> Cc: Vincent Pelletier <plr.vincent@gmail.com> Subject: Re: [PATCH] riscv: dts: sifive unmatched: Expose the FU740 core supply regulator. Date: Tue, 16 Nov 2021 10:53:26 +0100 [thread overview] Message-ID: <2266648.AD6qrfpaa2@diego> (raw) In-Reply-To: <0879c5b0c72b9bf6bf71f880def166f8804f41c7.1637023980.git.plr.vincent@gmail.com> Hi Vincent, Am Dienstag, 16. November 2021, 01:52:59 CET schrieb Vincent Pelletier: > Provides monitoring of core voltage and current: > tps544b20-i2c-0-1e > Adapter: i2c-ocores > vout1: 906.00 mV > temp1: -40.0°C (high = +125.0°C, crit = +150.0°C) > iout1: 5.06 A (max = +20.00 A, crit max = +26.00 A) > > Signed-off-by: Vincent Pelletier <plr.vincent@gmail.com> > > -- > Note for review: this patch has one warning from checkpatch.pl: > WARNING: DT compatible string "tps544b20" appears un-documented -- check ./Documentation/devicetree/bindings/ > #32: FILE: arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts:55: > + compatible = "tps544b20"; > This chip is handled by the existing pmbus module, and there is indeed no > matching entry in Documentation/devicetree/bindings/hwmon/pmbus. I am not > especially knowledgeable about this chip, I only know it is used by this > board, so I am not sure I can do the best job in putting such a file > together. > If needed I can git it a try. Devicetree bindings are supposed to be stable into the future, so an actually reviewed binding is quite necessary ;-) . In the case of your tps544b20 it should also be pretty easy to do, as Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml is probably a pretty good match to what you need in terms of Yaml notation. Just need to replace the naming in your copy and drop in the correct description from https://www.ti.com/lit/ds/symlink/tps544b20.pdf?ts=1637055780278 and you have a working binding. Then just add another patch to your series that mimics https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=8a36e38d8b0fbb92609e837a67f919202ec7ec51 and include the relevant maintainers that scripts/get_maintainer.pl will give you, and you're all set :-) > --- > arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts b/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts > index 270360b258b7..e327831d0d48 100644 > --- a/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts > +++ b/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts > @@ -51,6 +51,11 @@ &uart1 { > &i2c0 { > status = "okay"; > > + tps544b20@1e { > + compatible = "tps544b20"; This should definitly be compatible = "ti,tps544b20"; i.e. include the ti vendor-prefix. The i2c-core will automatically remove this when matching against the i2c devices. Heiko > + reg = <0x1e>; > + }; > + > temperature-sensor@4c { > compatible = "ti,tmp451"; > reg = <0x4c>; >
WARNING: multiple messages have this Message-ID (diff)
From: "Heiko Stübner" <heiko@sntech.de> To: Rob Herring <robh+dt@kernel.org>, Paul Walmsley <paul.walmsley@sifive.com>, Palmer Dabbelt <palmer@dabbelt.com>, Albert Ou <aou@eecs.berkeley.edu>, Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>, Qiu Wenbo <qiuwenbo@kylinos.com.cn>, Yash Shah <yash.shah@sifive.com>, devicetree@vger.kernel.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, David Abdurachmanov <david.abdurachmanov@sifive.com> Cc: Vincent Pelletier <plr.vincent@gmail.com> Subject: Re: [PATCH] riscv: dts: sifive unmatched: Expose the FU740 core supply regulator. Date: Tue, 16 Nov 2021 10:53:26 +0100 [thread overview] Message-ID: <2266648.AD6qrfpaa2@diego> (raw) In-Reply-To: <0879c5b0c72b9bf6bf71f880def166f8804f41c7.1637023980.git.plr.vincent@gmail.com> Hi Vincent, Am Dienstag, 16. November 2021, 01:52:59 CET schrieb Vincent Pelletier: > Provides monitoring of core voltage and current: > tps544b20-i2c-0-1e > Adapter: i2c-ocores > vout1: 906.00 mV > temp1: -40.0°C (high = +125.0°C, crit = +150.0°C) > iout1: 5.06 A (max = +20.00 A, crit max = +26.00 A) > > Signed-off-by: Vincent Pelletier <plr.vincent@gmail.com> > > -- > Note for review: this patch has one warning from checkpatch.pl: > WARNING: DT compatible string "tps544b20" appears un-documented -- check ./Documentation/devicetree/bindings/ > #32: FILE: arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts:55: > + compatible = "tps544b20"; > This chip is handled by the existing pmbus module, and there is indeed no > matching entry in Documentation/devicetree/bindings/hwmon/pmbus. I am not > especially knowledgeable about this chip, I only know it is used by this > board, so I am not sure I can do the best job in putting such a file > together. > If needed I can git it a try. Devicetree bindings are supposed to be stable into the future, so an actually reviewed binding is quite necessary ;-) . In the case of your tps544b20 it should also be pretty easy to do, as Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml is probably a pretty good match to what you need in terms of Yaml notation. Just need to replace the naming in your copy and drop in the correct description from https://www.ti.com/lit/ds/symlink/tps544b20.pdf?ts=1637055780278 and you have a working binding. Then just add another patch to your series that mimics https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=8a36e38d8b0fbb92609e837a67f919202ec7ec51 and include the relevant maintainers that scripts/get_maintainer.pl will give you, and you're all set :-) > --- > arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts b/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts > index 270360b258b7..e327831d0d48 100644 > --- a/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts > +++ b/arch/riscv/boot/dts/sifive/hifive-unmatched-a00.dts > @@ -51,6 +51,11 @@ &uart1 { > &i2c0 { > status = "okay"; > > + tps544b20@1e { > + compatible = "tps544b20"; This should definitly be compatible = "ti,tps544b20"; i.e. include the ti vendor-prefix. The i2c-core will automatically remove this when matching against the i2c devices. Heiko > + reg = <0x1e>; > + }; > + > temperature-sensor@4c { > compatible = "ti,tmp451"; > reg = <0x4c>; > _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2021-11-16 9:53 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-11-16 0:52 [PATCH] riscv: dts: sifive unmatched: Name gpio lines Vincent Pelletier 2021-11-16 0:52 ` Vincent Pelletier 2021-11-16 0:52 ` [PATCH] riscv: dts: sifive unmatched: Expose the board ID eeprom Vincent Pelletier 2021-11-16 0:52 ` Vincent Pelletier 2021-11-16 0:52 ` [PATCH] riscv: dts: sifive unmatched: Expose the PMIC sub-functions Vincent Pelletier 2021-11-16 0:52 ` Vincent Pelletier 2021-11-16 0:52 ` [PATCH] riscv: dts: sifive unmatched: Expose the FU740 core supply regulator Vincent Pelletier 2021-11-16 0:52 ` Vincent Pelletier 2021-11-16 9:53 ` Heiko Stübner [this message] 2021-11-16 9:53 ` Heiko Stübner 2021-11-16 11:08 ` Krzysztof Kozlowski 2021-11-16 11:08 ` Krzysztof Kozlowski 2021-11-16 11:13 ` Heiko Stübner 2021-11-16 11:13 ` Heiko Stübner 2021-11-16 10:44 ` Krzysztof Kozlowski 2021-11-16 10:44 ` Krzysztof Kozlowski 2021-11-16 0:53 ` [PATCH] riscv: dts: sifive unmatched: Fix regulator for board rev3 Vincent Pelletier 2021-11-16 0:53 ` Vincent Pelletier 2021-11-16 0:53 ` [PATCH] riscv: dts: sifive unmatched: Link the tmp451 with its power supply Vincent Pelletier 2021-11-16 0:53 ` Vincent Pelletier 2021-11-16 11:04 ` Krzysztof Kozlowski 2021-11-16 11:04 ` Krzysztof Kozlowski 2021-11-16 9:39 ` [PATCH] riscv: dts: sifive unmatched: Name gpio lines Heiko Stübner 2021-11-16 9:39 ` Heiko Stübner 2021-11-16 10:17 ` Vincent Pelletier 2021-11-16 10:17 ` Vincent Pelletier 2021-11-16 11:05 ` Krzysztof Kozlowski 2021-11-16 11:05 ` Krzysztof Kozlowski 2021-11-16 10:36 ` Krzysztof Kozlowski 2021-11-16 10:36 ` Krzysztof Kozlowski
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=2266648.AD6qrfpaa2@diego \ --to=heiko@sntech.de \ --cc=aou@eecs.berkeley.edu \ --cc=david.abdurachmanov@sifive.com \ --cc=devicetree@vger.kernel.org \ --cc=krzysztof.kozlowski@canonical.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=palmer@dabbelt.com \ --cc=paul.walmsley@sifive.com \ --cc=plr.vincent@gmail.com \ --cc=qiuwenbo@kylinos.com.cn \ --cc=robh+dt@kernel.org \ --cc=yash.shah@sifive.com \ /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.