Messages in this thread | | | Date | Fri, 13 Mar 2020 13:29:27 +0000 | From | Mark Brown <> | Subject | Re: [PATCH 1/1] regulator: fixed: add system pm routines for pinctrl |
| |
On Fri, Mar 13, 2020 at 01:16:31PM +0000, Peter Chen wrote:
> Most of pins for controlling fixed regulator on or off is GPIO, but how > GPIO driver handles this? We usually configure pin as GPIO function at > its user's node (Eg, reset pin for most drivers), but not GPIO node,
The GPIO driver knows what signals it's supposed to be generating, it should be able to configure pins for this. Setting the pinctrl stuff on the consumer seems like for example setting the pinctrl for a SPI controller on the SPI device rather than the controller device which would be a bit weird.
> GPIO node is usually per SoC, not per board level.
Sure, and it's totally normal for boards to add configuration to nodes that are part of the SoC. Again I'm a bit confused about why GPIOs are expected to be different here.
> So, I am wondering why fixed regulator can't have a pin in pinctrl. > If you grep the dts, there are already several fixed regulator has > pinctrl.
Obviously those pinctrl settings currently don't actually do anything... [unhandled content-type:application/pgp-signature] | |