Messages in this thread | | | Date | Wed, 25 Apr 2018 11:31:36 +0100 | From | Mark Brown <> | Subject | Re: [PATCH v2 2/2] regulator: add QCOM RPMh regulator driver |
| |
On Tue, Apr 24, 2018 at 02:09:47PM -0700, David Collins wrote: > On 04/24/2018 10:45 AM, Mark Brown wrote:
> > I think that's probably only OK if we have a specific error code for the > > regulator being limited in this way otherwise our error handling for I/O > > problems involves us trying to reconfigure supplies which seems like it > > would be risky.
> Would you be ok with -EAGAIN being used for this purpose?
Using -EAGAIN for "I can't ever read the configuration from this regulator" doesn't seem right - it's not like any number of retries will ever manage to read the value back. [unhandled content-type:application/pgp-signature] | |