From: Mark Brown broonie@linaro.org
Since the sht15 driver supports operation without an external vref regulator the driver should use the new devm_regulator_get_optional() to indicate that a stub regulator should not be provided.
Signed-off-by: Mark Brown broonie@linaro.org ---
Since devm_regulator_get_optional() is not yet merged either the branch adding it would need be be merged into the hwmon tree or I'd need to apply this to the regualtor tree.
drivers/hwmon/sht15.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hwmon/sht15.c b/drivers/hwmon/sht15.c index 883d291..97cd45a 100644 --- a/drivers/hwmon/sht15.c +++ b/drivers/hwmon/sht15.c @@ -957,7 +957,7 @@ static int sht15_probe(struct platform_device *pdev) * If a regulator is available, * query what the supply voltage actually is! */ - data->reg = devm_regulator_get(data->dev, "vcc"); + data->reg = devm_regulator_get_optional(data->dev, "vcc"); if (!IS_ERR(data->reg)) { int voltage;
On 08/18/2013 09:43 AM, Mark Brown wrote:
From: Mark Brown broonie@linaro.org
Since the sht15 driver supports operation without an external vref regulator the driver should use the new devm_regulator_get_optional() to indicate that a stub regulator should not be provided.
Signed-off-by: Mark Brown broonie@linaro.org
Acked-by: Guenter Roeck linux@roeck-us.net
Since devm_regulator_get_optional() is not yet merged either the branch adding it would need be be merged into the hwmon tree or I'd need to apply this to the regualtor tree.
Please take it through the regulator tree; that seems to be easier.
Thanks, Guenter
linaro-kernel@lists.linaro.org