On Wed, Apr 17, 2019 at 11:25:29AM +0000, Eric Hyeung Dong Jeong wrote: > On Wednesday, April 17, 2019 12:25 AM +0900, Mark Brown wrote:
> > This looks like it should be a get_status() operation as it's reading > > status bits rather than the command we sent to the device - for that > > just use regulator_is_enabled_regmap(). > I thought that it needs to return current status of a regulator when the > function is called. > I am wondering that the *is_enabled()* function is just to check > If a regulator has been turned on or not rather than getting current status > of the regulator. It should say if we told the hardware to enable the regulator so the former - basically, is the hardware in the same state it'd be in after we called _enable() or _disable(). If there's a fault then that should only affect is_enabled() if it resets the enable bit that _enable() set.
signature.asc
Description: PGP signature
