Skip to content

Commit b01d182

Browse files
Jeffrey Hugobroonie
authored andcommitted
regulator: qcom_spmi: Do NULL check for lvs
Low-voltage switches (lvs) don't have set_points since the voltage ranges of the output are really controlled by the inputs. This is a problem for the newly added linear range support in the probe(), as that will cause a null pointer dereference error on older platforms like msm8974 which happen to need to control some of the implemented lvs. Fix this by adding the appropriate null check. Fixes: 86f4ff7 ("regulator: qcom_spmi: enable linear range info") Reported-by: Bjorn Andersson <[email protected]> Signed-off-by: Jeffrey Hugo <[email protected]> Tested-by: Bjorn Andersson <[email protected]> Signed-off-by: Mark Brown <[email protected]>
1 parent 61d7fdc commit b01d182

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/regulator/qcom_spmi-regulator.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2045,7 +2045,7 @@ static int qcom_spmi_regulator_probe(struct platform_device *pdev)
20452045
}
20462046
}
20472047

2048-
if (vreg->set_points->count == 1) {
2048+
if (vreg->set_points && vreg->set_points->count == 1) {
20492049
/* since there is only one range */
20502050
range = vreg->set_points->range;
20512051
vreg->desc.uV_step = range->step_uV;

0 commit comments

Comments
 (0)