On 12/02/14 12:35, Lars-Peter Clausen wrote:
On 02/12/2014 01:19 PM, Lars-Peter Clausen wrote:
On 02/12/2014 01:01 PM, Paul Cercueil wrote:
Previously the driver would revert to internal supply if the
external supply couldn't be found. This had multiple problems:
- it caused silently ignored errors when a regulator was intended
to be supplied, but was not specified correctly.
- if CONFIG_REGULATOR is disabled, regulator_get() will always
return a dummy regulator, which caused a device to always use
the external vref mode, even though there is none.
This patch addresses the issue by adding a platform data structure,
containing a boolean field use_external_ref. If the platform data
structure is present and if that boolean is set, the external vref
is used; otherwise the internal vref is used.
In the case where an external vref is wanted but regulator_get()
fails, the driver no longer reverts to using the internal vref,
but returns an error instead.
Signed-off-by: Paul Cercueil <paul.cercueil@xxxxxxxxxx>
All three patches:
Acked-by: Lars-Peter Clausen <lars@xxxxxxxxxx>
Uhm, actually no. I think for devices without an internal reference we should default to using the external reference if no platform data is specified. Everything else doesn't make too much sense.
Agreed. If present, we default to the internal reference ifan external
one is not provided. The logic in the driver can check if one is specified,
whether or not config_regulator is enabled.
Jonathan
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
