From: Mark Brown <broonie@kernel.org> To: Marco Felsch <m.felsch@pengutronix.de> Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Support Opensource <Support.Opensource@diasemi.com>, "linux-aspeed@lists.ozlabs.org" <linux-aspeed@lists.ozlabs.org>, "bgolaszewski@baylibre.com" <bgolaszewski@baylibre.com>, "andrew@aj.id.au" <andrew@aj.id.au>, "linus.walleij@linaro.org" <linus.walleij@linaro.org>, "lgirdwood@gmail.com" <lgirdwood@gmail.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>, "robh+dt@kernel.org" <robh+dt@kernel.org>, "joel@jms.id.au" <joel@jms.id.au>, "kernel@pengutronix.de" <kernel@pengutronix.de>, Adam Thomson <Adam.Thomson.Opensource@diasemi.com>, "lee.jones@linaro.org" <lee.jones@linaro.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: Re: [PATCH v3 3/6] dt-bindings: mfd: da9062: add regulator voltage selection documentation Date: Tue, 7 Jan 2020 13:09:11 +0000 [thread overview] Message-ID: <20200107130911.GD4877@sirena.org.uk> (raw) In-Reply-To: <20200107083654.atgbjhrnhyax2gqq@pengutronix.de> [-- Attachment #1: Type: text/plain, Size: 1223 bytes --] On Tue, Jan 07, 2020 at 09:36:54AM +0100, Marco Felsch wrote: > On 19-12-17 12:58, Mark Brown wrote: > > This doesn't say anything about how the GPIO input is expected to be > > controlled, for voltage setting any runtime control would need to be > > done by the driver and it sounds like that's all that can be controlled. > > The way this reads I'd expect one use of this to be for fast voltage > > setting for example (you could even combine that with suspend sequencing > > using the internal sequencer if you mux back to the sequencer during > > suspend). > The input signal is routed trough the da9062 gpio block to the > regualtors. You can't set any voltage value using a gpio instead you > decide which voltage setting is applied. The voltage values for > runtime/suspend comes from the dt-data. No it's not just a fast > switching option imagine the system suspend case where the cpu and soc > voltage can be reduced to a very low value. Older soc's like the imx6 > signaling this state by a hard wired gpio line because the soc and > cpu cores don't work properly on such low voltage values. This is > my use case and I can't use the sequencer. My point is that I can't tell any of this from the description. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Mark Brown <broonie@kernel.org> To: Marco Felsch <m.felsch@pengutronix.de> Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Support Opensource <Support.Opensource@diasemi.com>, "linux-aspeed@lists.ozlabs.org" <linux-aspeed@lists.ozlabs.org>, "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>, "andrew@aj.id.au" <andrew@aj.id.au>, "linus.walleij@linaro.org" <linus.walleij@linaro.org>, "lgirdwood@gmail.com" <lgirdwood@gmail.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "bgolaszewski@baylibre.com" <bgolaszewski@baylibre.com>, "robh+dt@kernel.org" <robh+dt@kernel.org>, "joel@jms.id.au" <joel@jms.id.au>, "kernel@pengutronix.de" <kernel@pengutronix.de>, Adam Thomson <Adam.Thomson.Opensource@diasemi.com>, "lee.jones@linaro.org" <lee.jones@linaro.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: Re: [PATCH v3 3/6] dt-bindings: mfd: da9062: add regulator voltage selection documentation Date: Tue, 7 Jan 2020 13:09:11 +0000 [thread overview] Message-ID: <20200107130911.GD4877@sirena.org.uk> (raw) In-Reply-To: <20200107083654.atgbjhrnhyax2gqq@pengutronix.de> [-- Attachment #1.1: Type: text/plain, Size: 1223 bytes --] On Tue, Jan 07, 2020 at 09:36:54AM +0100, Marco Felsch wrote: > On 19-12-17 12:58, Mark Brown wrote: > > This doesn't say anything about how the GPIO input is expected to be > > controlled, for voltage setting any runtime control would need to be > > done by the driver and it sounds like that's all that can be controlled. > > The way this reads I'd expect one use of this to be for fast voltage > > setting for example (you could even combine that with suspend sequencing > > using the internal sequencer if you mux back to the sequencer during > > suspend). > The input signal is routed trough the da9062 gpio block to the > regualtors. You can't set any voltage value using a gpio instead you > decide which voltage setting is applied. The voltage values for > runtime/suspend comes from the dt-data. No it's not just a fast > switching option imagine the system suspend case where the cpu and soc > voltage can be reduced to a very low value. Older soc's like the imx6 > signaling this state by a hard wired gpio line because the soc and > cpu cores don't work properly on such low voltage values. This is > my use case and I can't use the sequencer. My point is that I can't tell any of this from the description. [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ 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-01-07 13:09 UTC|newest] Thread overview: 76+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-11-29 17:25 [PATCH v3 0/6] DA9062 PMIC features Marco Felsch 2019-11-29 17:25 ` Marco Felsch 2019-11-29 17:25 ` [PATCH v3 1/6] gpio: treewide rename gpio_chip_hwgpio to gpiod_to_offset Marco Felsch 2019-11-29 17:25 ` Marco Felsch 2019-12-02 2:59 ` Andrew Jeffery 2019-12-02 2:59 ` Andrew Jeffery 2019-11-29 17:25 ` [PATCH v3 2/6] gpio: make gpiod_to_offset() available for other users Marco Felsch 2019-11-29 17:25 ` Marco Felsch 2019-12-02 3:00 ` Andrew Jeffery 2019-12-02 3:00 ` Andrew Jeffery 2019-11-29 17:25 ` [PATCH v3 3/6] dt-bindings: mfd: da9062: add regulator voltage selection documentation Marco Felsch 2019-11-29 17:25 ` Marco Felsch 2019-12-04 13:46 ` Mark Brown 2019-12-04 13:46 ` Mark Brown 2019-12-10 9:41 ` Marco Felsch 2019-12-10 9:41 ` Marco Felsch 2019-12-11 16:14 ` Adam Thomson 2019-12-11 16:14 ` Adam Thomson 2019-12-11 17:09 ` Marco Felsch 2019-12-11 17:09 ` Marco Felsch 2019-12-12 15:08 ` Linus Walleij 2019-12-12 15:08 ` Linus Walleij 2019-12-12 15:55 ` Marco Felsch 2019-12-12 15:55 ` Marco Felsch 2019-12-12 16:10 ` Mark Brown 2019-12-12 16:10 ` Mark Brown 2019-12-12 16:21 ` Marco Felsch 2019-12-12 16:21 ` Marco Felsch 2019-12-12 16:51 ` Mark Brown 2019-12-12 16:51 ` Mark Brown 2019-12-16 8:55 ` Marco Felsch 2019-12-16 8:55 ` Marco Felsch 2019-12-16 11:44 ` Mark Brown 2019-12-16 11:44 ` Mark Brown 2019-12-17 7:35 ` Marco Felsch 2019-12-17 7:35 ` Marco Felsch 2019-12-17 12:58 ` Mark Brown 2019-12-17 12:58 ` Mark Brown 2020-01-07 8:36 ` Marco Felsch 2020-01-07 8:36 ` Marco Felsch 2020-01-07 13:09 ` Mark Brown [this message] 2020-01-07 13:09 ` Mark Brown 2020-01-07 13:38 ` Marco Felsch 2020-01-07 13:38 ` Marco Felsch 2020-01-14 15:43 ` Mark Brown 2020-01-14 15:43 ` Mark Brown 2019-12-16 16:32 ` Adam Thomson 2019-12-16 16:32 ` Adam Thomson 2019-12-17 9:00 ` Marco Felsch 2019-12-17 9:00 ` Marco Felsch 2019-12-17 9:12 ` Marco Felsch 2019-12-17 9:12 ` Marco Felsch 2019-12-17 9:53 ` Adam Thomson 2019-12-17 9:53 ` Adam Thomson 2019-12-17 12:31 ` Marco Felsch 2019-12-17 12:31 ` Marco Felsch 2019-12-17 13:13 ` Adam Thomson 2019-12-17 13:13 ` Adam Thomson 2019-12-16 16:32 ` Adam Thomson 2019-12-16 16:32 ` Adam Thomson 2019-12-16 12:28 ` Linus Walleij 2019-12-16 12:28 ` Linus Walleij 2019-11-29 17:25 ` [PATCH v3 4/6] regulator: da9062: add voltage selection gpio support Marco Felsch 2019-11-29 17:25 ` Marco Felsch 2019-11-29 17:25 ` [PATCH v3 5/6] dt-bindings: mfd: da9062: add regulator gpio enable/disable documentation Marco Felsch 2019-11-29 17:25 ` Marco Felsch 2019-12-13 22:23 ` Rob Herring 2019-12-13 22:23 ` Rob Herring 2019-12-16 16:31 ` Lee Jones 2019-12-16 16:31 ` Lee Jones 2019-11-29 17:25 ` [PATCH v3 6/6] regulator: da9062: add gpio based regulator dis-/enable support Marco Felsch 2019-11-29 17:25 ` Marco Felsch 2019-12-02 11:44 ` [PATCH v3 0/6] DA9062 PMIC features Linus Walleij 2019-12-02 11:44 ` Linus Walleij 2019-12-02 12:04 ` Lee Jones 2019-12-02 12:04 ` Lee Jones
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=20200107130911.GD4877@sirena.org.uk \ --to=broonie@kernel.org \ --cc=Adam.Thomson.Opensource@diasemi.com \ --cc=Support.Opensource@diasemi.com \ --cc=andrew@aj.id.au \ --cc=bgolaszewski@baylibre.com \ --cc=devicetree@vger.kernel.org \ --cc=joel@jms.id.au \ --cc=kernel@pengutronix.de \ --cc=lee.jones@linaro.org \ --cc=lgirdwood@gmail.com \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-aspeed@lists.ozlabs.org \ --cc=linux-gpio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=m.felsch@pengutronix.de \ --cc=robh+dt@kernel.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.