On Tue 18 Aug 20:17 PDT 2015, Axel Lin wrote: > Just setting fixed_uV is not enough, the regulator core will also check > n_voltages setting. The fixed_uV only works when n_voltages is 1. >
Thanks, I missed that. Reviewed-by: Bjorn Andersson <[email protected]> > Signed-off-by: Axel Lin <[email protected]> > --- > drivers/regulator/qcom_smd-regulator.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/regulator/qcom_smd-regulator.c > b/drivers/regulator/qcom_smd-regulator.c > index 9d093ae..9c6167d 100644 > --- a/drivers/regulator/qcom_smd-regulator.c > +++ b/drivers/regulator/qcom_smd-regulator.c > @@ -203,6 +203,7 @@ static const struct regulator_desc pm8941_nldo = { > > static const struct regulator_desc pm8941_lnldo = { > .fixed_uV = 1740000, > + .n_voltages = 1, > .ops = &rpm_smps_ldo_ops, > }; > > -- > 2.1.0 > > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
