From: Romain Gantois <romain.gantois@bootlin.com>
To: "H. Nikolaus Schaller" <hns@goldelico.com>
Cc: "Liam Girdwood" <lgirdwood@gmail.com>,
"Mark Brown" <broonie@kernel.org>,
"Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Jonathan Cameron" <jic23@kernel.org>,
"David Lechner" <dlechner@baylibre.com>,
"Nuno Sá" <nuno.sa@analog.com>,
"Andy Shevchenko" <andy@kernel.org>,
"Guenter Roeck" <linux@roeck-us.net>,
"Thomas Petazzoni" <thomas.petazzoni@bootlin.com>,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
linux-iio@vger.kernel.org,
"Conor Dooley" <conor.dooley@microchip.com>,
"MyungJoo Ham" <myungjoo.ham@samsung.com>,
"Chanwoo Choi" <cw00.choi@samsung.com>,
"Peter Rosin" <peda@axentia.se>,
"Mariel Tinaco" <Mariel.Tinaco@analog.com>,
"Lars-Peter Clausen" <lars@metafoo.de>,
"Michael Hennerich" <Michael.Hennerich@analog.com>,
"Kevin Tsai" <ktsai@capellamicro.com>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
"Eugen Hristev" <eugen.hristev@linaro.org>,
"Vinod Koul" <vkoul@kernel.org>,
"Kishon Vijay Abraham I" <kishon@kernel.org>,
"Sebastian Reichel" <sre@kernel.org>,
"Chen-Yu Tsai" <wens@csie.org>,
"Support Opensource" <support.opensource@diasemi.com>,
"Paul Cercueil" <paul@crapouillou.net>,
"Iskren Chernev" <me@iskren.info>,
"Marek Szyprowski" <m.szyprowski@samsung.com>,
"Matheus Castello" <matheus@castello.eng.br>,
"Saravanan Sekar" <sravanhome@gmail.com>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"AngeloGioacchino Del Regno"
<angelogioacchino.delregno@collabora.com>,
"Casey Connolly" <casey.connolly@linaro.org>,
"Pali Rohár" <pali@kernel.org>,
"Orson Zhai" <orsonzhai@gmail.com>,
"Baolin Wang" <baolin.wang@linux.alibaba.com>,
"Chunyan Zhang" <zhang.lyra@gmail.com>,
"Amit Kucheria" <amitk@kernel.org>,
"Thara Gopinath" <thara.gopinath@gmail.com>,
"Rafael J. Wysocki" <rafael@kernel.org>,
"Daniel Lezcano" <daniel.lezcano@linaro.org>,
"Zhang Rui" <rui.zhang@intel.com>,
"Lukasz Luba" <lukasz.luba@arm.com>,
"Claudiu Beznea" <claudiu.beznea.uj@bp.renesas.com>,
"Jaroslav Kysela" <perex@perex.cz>,
"Takashi Iwai" <tiwai@suse.com>,
"Sylwester Nawrocki" <s.nawrocki@samsung.com>,
"Olivier Moysan" <olivier.moysan@foss.st.com>,
"Arnaud Pouliquen" <arnaud.pouliquen@foss.st.com>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
"Dixit Parmar" <dixitparmar19@gmail.com>,
linux-hwmon@vger.kernel.org, linux-input@vger.kernel.org,
linux-phy@lists.infradead.org, linux-pm@vger.kernel.org,
linux-mips@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org,
linux-arm-msm@vger.kernel.org, linux-sound@vger.kernel.org,
linux-stm32@st-md-mailman.stormreply.com,
"Andy Shevchenko" <andriy.shevchenko@intel.com>
Subject: Re: [PATCH v4 0/6] Add support for the LTM8054 voltage regulator
Date: Thu, 27 Nov 2025 16:06:47 +0100 [thread overview]
Message-ID: <8695751.T7Z3S40VBb@fw-rgant> (raw)
In-Reply-To: <0E900830-E248-4F0F-A048-075EAF1D2440@goldelico.com>
[-- Attachment #1: Type: text/plain, Size: 2608 bytes --]
On Tuesday, 25 November 2025 11:25:24 CET H. Nikolaus Schaller wrote:
> Hi,
>
> > Am 25.11.2025 um 09:41 schrieb Romain Gantois
> > <romain.gantois@bootlin.com>:
> >
> >
> > This is planned support for a voltage regulator chip.
>
> Well, but one which is not by itself programmable. So IMHO, it does not
> support that chip, but the circuit it is used in.
>
The boundary is a bit blurry in this case, sure.
> > > Are you looking for a virtual "glue" driver to logically combine several
> > > low level functions?
> >
> > I'm looking for a clean userspace abstraction for this component, the low-
> > level functions in this case are those of a voltage regulator.
>
> As far as I understood it has
> - constant voltage
> - current can be limited
> - it can be turned on/off
>
> That means it is a fixed-regulator (for constant voltage and turn on/off)
> and a mechanism to program the current limit (iio-dac). Both have clean
> userspace abstraction.
>
> What am I missing?
>
In my case, the regulator has a DAC tapping into the feedback divider with
allows modifying the output voltage level. This isn't specific to the LTM8054
though, you can theoretically do this with any regulator chip that has a
feedback pin.
...
> The question remains if you want to solve something for a single board which
> happens to have an LTM8054 or if you are solving a more general design
> pattern.
>
> In summary my view is that the LTM8054 is just a "fixed-regulator" which
> gets an additional current-limiter feature by adding a DAC chip (which needs
> a driver of course). So software control is required not by the LTM8054 but
> by adding a DAC chip.
>
> Another suggestion: what extending the "regulator-fixed", "regulator-gpio",
> "regulator-fixed-clock" pattern by some
> "regulator-gpio-iio-dac-current-limiter" driver to make it independent of
> your specific chip?
>
> By the way, are you aware of this feature of the regulator-gpio driver?
>
> https://elixir.bootlin.com/linux/v6.18-rc7/source/drivers/regulator/gpio-reg
> ulator.c#L97
>
That could be a preferable solution given that similar current limit and
output voltage limit control methods could apply to other regulator chips...
I'll have to think about it some more.
> Just to note: I am neither maintainer nor doing any decisions on this, just
> asking questions for curiosity and from experience and giving hints for
> alternative approaches, where I hope they help to find the really best
> solution.
Sure, I appreciate that.
Thanks,
--
Romain Gantois, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2025-11-27 15:07 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-24 14:48 [PATCH v4 0/6] Add support for the LTM8054 voltage regulator Romain Gantois
2025-11-24 14:48 ` [PATCH v4 2/6] iio: inkern: Use namespaced exports Romain Gantois
2025-11-24 14:57 ` [PATCH v4 0/6] Add support for the LTM8054 voltage regulator Guenter Roeck
2025-11-24 15:07 ` Andy Shevchenko
2025-11-24 15:13 ` Romain Gantois
2025-11-24 15:35 ` H. Nikolaus Schaller
2025-11-24 15:57 ` Romain Gantois
2025-11-24 16:19 ` H. Nikolaus Schaller
2025-11-25 8:41 ` Romain Gantois
2025-11-25 10:25 ` H. Nikolaus Schaller
2025-11-25 16:37 ` Guenter Roeck
2025-12-07 18:48 ` Jonathan Cameron
2025-12-08 8:57 ` Romain Gantois
2025-11-27 15:06 ` Romain Gantois [this message]
2025-11-25 8:49 ` Romain Gantois
2025-11-24 15:40 ` Guenter Roeck
2025-11-24 16:02 ` Romain Gantois
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=8695751.T7Z3S40VBb@fw-rgant \
--to=romain.gantois@bootlin.com \
--cc=Mariel.Tinaco@analog.com \
--cc=Michael.Hennerich@analog.com \
--cc=alexandre.torgue@foss.st.com \
--cc=amitk@kernel.org \
--cc=andriy.shevchenko@intel.com \
--cc=andy@kernel.org \
--cc=angelogioacchino.delregno@collabora.com \
--cc=arnaud.pouliquen@foss.st.com \
--cc=baolin.wang@linux.alibaba.com \
--cc=broonie@kernel.org \
--cc=casey.connolly@linaro.org \
--cc=claudiu.beznea.uj@bp.renesas.com \
--cc=conor+dt@kernel.org \
--cc=conor.dooley@microchip.com \
--cc=cw00.choi@samsung.com \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=dixitparmar19@gmail.com \
--cc=dlechner@baylibre.com \
--cc=dmitry.torokhov@gmail.com \
--cc=eugen.hristev@linaro.org \
--cc=hns@goldelico.com \
--cc=jic23@kernel.org \
--cc=kishon@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=ktsai@capellamicro.com \
--cc=lars@metafoo.de \
--cc=lgirdwood@gmail.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-mips@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-sound@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=linux@roeck-us.net \
--cc=lukasz.luba@arm.com \
--cc=m.szyprowski@samsung.com \
--cc=matheus@castello.eng.br \
--cc=matthias.bgg@gmail.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=me@iskren.info \
--cc=myungjoo.ham@samsung.com \
--cc=nuno.sa@analog.com \
--cc=olivier.moysan@foss.st.com \
--cc=orsonzhai@gmail.com \
--cc=pali@kernel.org \
--cc=paul@crapouillou.net \
--cc=peda@axentia.se \
--cc=perex@perex.cz \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=rui.zhang@intel.com \
--cc=s.nawrocki@samsung.com \
--cc=sravanhome@gmail.com \
--cc=sre@kernel.org \
--cc=support.opensource@diasemi.com \
--cc=thara.gopinath@gmail.com \
--cc=thomas.petazzoni@bootlin.com \
--cc=tiwai@suse.com \
--cc=vkoul@kernel.org \
--cc=wens@csie.org \
--cc=zhang.lyra@gmail.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).