Re: [PATCH] regulator: Allow set voltage on fixed regulators

From: Mark Brown
Date: Fri Apr 18 2014 - 14:53:36 EST


On Fri, Apr 18, 2014 at 11:30:10AM -0700, Tim Kryger wrote:
> If a regulator consumer requests a voltage range that can be satisfied,
> the return value should indicate success even if that regulator has a
> fixed voltage. Since there is already logic to check if the requested
> voltage range overlaps the allowed range, set REGULATOR_CHANGE_VOLTAGE
> for regulators with constraints that include a positive voltage.

This seems like the wrong place to fix this, it's nothing to do with DT
and we shouldn't require that nonsensical permissions are set. Instead
we should fix this at the point where we're implementing the permission
check, have the failure case check the current voltage before returning
an error.

Attachment: signature.asc
Description: Digital signature