projects
/
linux-2.6-microblaze.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
98e48cd
)
regulator: scmi: Fix off-by-one for linear regulators .n_voltages setting
author
Axel Lin
<axel.lin@ingics.com>
Fri, 21 May 2021 07:30:20 +0000
(15:30 +0800)
committer
Mark Brown
<broonie@kernel.org>
Fri, 21 May 2021 19:05:48 +0000
(20:05 +0100)
For linear regulators, the .n_voltages is (max_uv - min_uv) / uv_step + 1.
Fixes:
0fbeae70ee7c
("regulator: add SCMI driver")
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Reviewed-by: Cristian Marussi <cristian.marussi@arm.com>
Link:
https://lore.kernel.org/r/20210521073020.1944981-1-axel.lin@ingics.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/scmi-regulator.c
patch
|
blob
|
history
diff --git
a/drivers/regulator/scmi-regulator.c
b/drivers/regulator/scmi-regulator.c
index
a917c81
..
51eb978
100644
(file)
--- a/
drivers/regulator/scmi-regulator.c
+++ b/
drivers/regulator/scmi-regulator.c
@@
-176,7
+176,7
@@
scmi_config_linear_regulator_mappings(struct scmi_regulator *sreg,
sreg->desc.uV_step =
vinfo->levels_uv[SCMI_VOLTAGE_SEGMENT_STEP];
sreg->desc.linear_min_sel = 0;
- sreg->desc.n_voltages =
delta_uV / sreg->desc.uV_step
;
+ sreg->desc.n_voltages =
(delta_uV / sreg->desc.uV_step) + 1
;
sreg->desc.ops = &scmi_reg_linear_ops;
}