* [PATCH 1/2] regulator: tps65217: Fix off-by-one for latest seletor of tps65217_uv1_ranges
@ 2019-03-24 6:00 Axel Lin
2019-03-24 6:00 ` [PATCH 2/2] regulator: tps65217: Constify regulator_ops Axel Lin
0 siblings, 1 reply; 2+ messages in thread
From: Axel Lin @ 2019-03-24 6:00 UTC (permalink / raw)
To: Mark Brown; +Cc: Keerthy, Russ Dill, Liam Girdwood, linux-kernel, Axel Lin
According to the datasheet the latest seletor 11 1111b = 3.3 V.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
drivers/regulator/tps65217-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/tps65217-regulator.c b/drivers/regulator/tps65217-regulator.c
index ada62f792258..ead31741dc13 100644
--- a/drivers/regulator/tps65217-regulator.c
+++ b/drivers/regulator/tps65217-regulator.c
@@ -60,7 +60,7 @@ static const struct regulator_linear_range tps65217_uv1_ranges[] = {
REGULATOR_LINEAR_RANGE(900000, 0, 24, 25000),
REGULATOR_LINEAR_RANGE(1550000, 25, 52, 50000),
REGULATOR_LINEAR_RANGE(3000000, 53, 55, 100000),
- REGULATOR_LINEAR_RANGE(3300000, 56, 62, 0),
+ REGULATOR_LINEAR_RANGE(3300000, 56, 63, 0),
};
static const struct regulator_linear_range tps65217_uv2_ranges[] = {
--
2.17.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH 2/2] regulator: tps65217: Constify regulator_ops
2019-03-24 6:00 [PATCH 1/2] regulator: tps65217: Fix off-by-one for latest seletor of tps65217_uv1_ranges Axel Lin
@ 2019-03-24 6:00 ` Axel Lin
0 siblings, 0 replies; 2+ messages in thread
From: Axel Lin @ 2019-03-24 6:00 UTC (permalink / raw)
To: Mark Brown; +Cc: Keerthy, Russ Dill, Liam Girdwood, linux-kernel, Axel Lin
These regulator_ops variables never need to be modified, make them const so
compiler can put them to .rodata.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
drivers/regulator/tps65217-regulator.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/regulator/tps65217-regulator.c b/drivers/regulator/tps65217-regulator.c
index ead31741dc13..67ba78da77ec 100644
--- a/drivers/regulator/tps65217-regulator.c
+++ b/drivers/regulator/tps65217-regulator.c
@@ -149,7 +149,7 @@ static int tps65217_pmic_set_suspend_disable(struct regulator_dev *dev)
}
/* Operations permitted on DCDCx, LDO2, LDO3 and LDO4 */
-static struct regulator_ops tps65217_pmic_ops = {
+static const struct regulator_ops tps65217_pmic_ops = {
.is_enabled = regulator_is_enabled_regmap,
.enable = tps65217_pmic_enable,
.disable = tps65217_pmic_disable,
@@ -162,7 +162,7 @@ static struct regulator_ops tps65217_pmic_ops = {
};
/* Operations permitted on LDO1 */
-static struct regulator_ops tps65217_pmic_ldo1_ops = {
+static const struct regulator_ops tps65217_pmic_ldo1_ops = {
.is_enabled = regulator_is_enabled_regmap,
.enable = tps65217_pmic_enable,
.disable = tps65217_pmic_disable,
--
2.17.1
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-03-24 6:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-24 6:00 [PATCH 1/2] regulator: tps65217: Fix off-by-one for latest seletor of tps65217_uv1_ranges Axel Lin
2019-03-24 6:00 ` [PATCH 2/2] regulator: tps65217: Constify regulator_ops Axel Lin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®