Messages in this thread | | | Date | Wed, 22 Mar 2023 14:40:09 +0100 | Subject | Re: [PATCH INTERNAL v1 3/3] regulator: tps6594-regulator: Add driver for TI TPS6594 regulators | From | Julien Panis <> |
| |
On 3/22/23 14:13, Mark Brown wrote: > On Wed, Mar 22, 2023 at 10:10:23AM +0100, Julien Panis wrote: > >> Question @ Mark/Liam: >> Shouldn't we use the generic 'regulator-coupled-with' property >> instead of 'ti,multi-phase-id' ? > My understanding was that this was a hardware configuration where > two regulators operate as one with only one set of registers used > for configuration.
Your understanding was correct.
> >> I am in charge of upstreaming dt-bindings and maintainers >> pointed out the similarity between 'multi-phase' and 'coupled' >> regulator concepts. Does 'regulator-coupled-with' mean that >> outputs of buck converters are combined ? If so, this generic >> property should replace our specific 'ti,multi-phase-id' prop, >> I guess. > No, coupled regulators are regulators where the voltages can vary > but there's a constraint that their configurations need to be > related somehow, for example they must be within 200mV of each > other or something like that.
OK, thank you for this explanation. So, we keep 'ti,multi-phase-id' property.
| |