Re: [PATCH 1/2] regulator: add QCOM RPMh regulator driver
From: Mark Brown
Date: Tue Mar 27 2018 - 22:28:25 EST
On Tue, Mar 27, 2018 at 01:51:56PM -0700, Doug Anderson wrote:
> Assuming I didn't mess up my analysis, the entire job of of_map_mode()
> is to convert from one integer to another. It should take the number
> that was specified in the device tree and convert it to a
> REGULATOR_MODE_XXX. That means that the regulator framework is
> enforcing a distinct and per-regulator numbering system for the mode
> (I called this "device tree mode").
OK, the confusion comes from your terminology invention rather than the
driver then :(
> So basically it sounds like everyone makes up some arbitrary numbering
> system that is only used in their device tree files and needs to be
> mapped into the standard numbering system...
It's not just that. The very modes themselves are not defined at all
consistently between regulators - the set of options varies wildly as
does the range of applications they are suitable for. We want people to
not only define numbers here but also names that make sense in the
context of the regulator documentation that can then be mapped into the
concept of modes we've inherited in the regulator subsystem.
Attachment:
signature.asc
Description: PGP signature