regulator: max1586: Constify regulator_desc

PATCH Applied Axel Lin 4 participants 9 messages Apr 6, 2012 – Apr 9, 2012

Subsystems (2)

OMAP2+ support Voltage and current regulator framework

lore

Share & tools

lore

Axel Lin

author

2012-04-06 00:24 UTC #0 lore raw

PATCH 1/6 regulator: max1586: Constify regulator_desc

Signed-off-by Axel Lin <[email protected]>

---
 drivers/regulator/max1586.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

drivers/regulator/max1586.c +1 -1

index 282d2ee..fad0bee 100644
@@ -161,7 +161,7 @@ static struct regulator_ops max1586_v6_ops = {
 	.list_voltage = max1586_v6_list,
 };
 
-static struct regulator_desc max1586_reg[] = {
+static const struct regulator_desc max1586_reg[] = {
 	{
 		.name = "Output_V3",
 		.id = MAX1586_V3,

Signature · 1 line

1.7.5.4

Axel Lin

author

2012-04-06 00:26 UTC #1 lore raw

PATCH 2/6 regulator: max8649: Constify regulator_desc

Signed-off-by Axel Lin <[email protected]>

---
 drivers/regulator/max8649.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

drivers/regulator/max8649.c +1 -1

index 2e9ffbe..dca7835 100644
@@ -207,7 +207,7 @@ static struct regulator_ops max8649_dcdc_ops = {
 
 };
 
-static struct regulator_desc dcdc_desc = {
+static const struct regulator_desc dcdc_desc = {
 	.name		= "max8649",
 	.ops		= &max8649_dcdc_ops,
 	.type		= REGULATOR_VOLTAGE,

Signature · 1 line

1.7.5.4

Axel Lin

author

2012-04-06 00:27 UTC #2 lore raw

PATCH 3/6 regulator: max8660: Constify regulator_desc

In reply to Axel Lin

Signed-off-by Axel Lin <[email protected]>

---
 drivers/regulator/max8660.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

drivers/regulator/max8660.c +1 -1

index e42db53..0e32787 100644
@@ -313,7 +313,7 @@ static struct regulator_ops max8660_ldo67_ops = {
 	.set_voltage = max8660_ldo67_set,
 };
 
-static struct regulator_desc max8660_reg[] = {
+static const struct regulator_desc max8660_reg[] = {
 	{
 		.name = "V3(DCDC)",
 		.id = MAX8660_V3,

Signature · 1 line

1.7.5.4

Axel Lin

author

2012-04-06 00:29 UTC #3 lore raw

PATCH 4/6 regulator: max8952: Constify regulator_desc

In reply to Axel Lin

Signed-off-by Axel Lin <[email protected]>

---
 drivers/regulator/max8952.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

drivers/regulator/max8952.c +1 -1

index c35236a..b408431 100644
@@ -159,7 +159,7 @@ static struct regulator_ops max8952_ops = {
 	.set_suspend_disable	= max8952_disable,
 };
 
-static struct regulator_desc regulator = {
+static const struct regulator_desc regulator = {
 	.name		= "MAX8952_VOUT",
 	.id		= 0,
 	.n_voltages	= MAX8952_NUM_DVS_MODE,

Signature · 1 line

1.7.5.4

Axel Lin

author

2012-04-06 00:30 UTC #4 lore raw

PATCH 5/6 regulator: tps6105x: Constify regulator_desc

In reply to Axel Lin

Signed-off-by Axel Lin <[email protected]>

---
 drivers/regulator/tps6105x-regulator.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

drivers/regulator/tps6105x-regulator.c +1 -1

index d9278da..3b78897 100644
@@ -123,7 +123,7 @@ static struct regulator_ops tps6105x_regulator_ops = {
 	.list_voltage	= tps6105x_regulator_list_voltage,
 };
 
-static struct regulator_desc tps6105x_regulator_desc = {
+static const struct regulator_desc tps6105x_regulator_desc = {
 	.name		= "tps6105x-boost",
 	.ops		= &tps6105x_regulator_ops,
 	.type		= REGULATOR_VOLTAGE,

Signature · 1 line

1.7.5.4

Linus Walleij

Acked-by on v1 5/6

2012-04-09 21:12 UTC #5 lore raw

Re: [PATCH 5/6] regulator: tps6105x: Constify regulator_desc

Quoted 4 lines · Axel Lin

Signed-off-by Axel Lin <[email protected]>
---
drivers/regulator/tps6105x-regulator.c | 2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Looks like a good idea:

Acked-by Linus Walleij <[email protected]>

Thanks,
Linus Walleij

Axel Lin

author

2012-04-06 00:31 UTC #6 lore raw

PATCH 6/6 regulator: tps65217: Constify regulator_desc

In reply to Axel Lin

Signed-off-by Axel Lin <[email protected]>

---
 drivers/regulator/tps65217-regulator.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

drivers/regulator/tps65217-regulator.c +1 -1

index e39521b..80fad2d 100644
@@ -312,7 +312,7 @@ static struct regulator_ops tps65217_pmic_ldo1_ops = {
 	.list_voltage		= tps65217_pmic_list_voltage,
 };
 
-static struct regulator_desc regulators[] = {
+static const struct regulator_desc regulators[] = {
 	TPS65217_REGULATOR("DCDC1", TPS65217_DCDC_1, tps65217_pmic_ops, 64),
 	TPS65217_REGULATOR("DCDC2", TPS65217_DCDC_2, tps65217_pmic_ops, 64),
 	TPS65217_REGULATOR("DCDC3", TPS65217_DCDC_3, tps65217_pmic_ops, 64),

Signature · 1 line

1.7.5.4

AnilKumar, Chimata

2012-04-06 07:03 UTC #7 lore raw

RE: [PATCH 6/6] regulator: tps65217: Constify regulator_desc

Hi Axel Lin,

It's completely missed from my side, thanks for the patch

ACK: AnilKumar Ch <[email protected]>

Quoted 24 lines · Axel Lin

Signed-off-by Axel Lin <[email protected]>

---
 drivers/regulator/tps65217-regulator.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

drivers/regulator/tps65217-regulator.c +1 -1

index e39521b..80fad2d 100644
@@ -312,7 +312,7 @@ static struct regulator_ops tps65217_pmic_ldo1_ops = {
 	.list_voltage		= tps65217_pmic_list_voltage,
 };
 
-static struct regulator_desc regulators[] = {
+static const struct regulator_desc regulators[] = {
 	TPS65217_REGULATOR("DCDC1", TPS65217_DCDC_1, tps65217_pmic_ops, 64),
 	TPS65217_REGULATOR("DCDC2", TPS65217_DCDC_2, tps65217_pmic_ops, 64),
 	TPS65217_REGULATOR("DCDC3", TPS65217_DCDC_3, tps65217_pmic_ops, 64),

Signature · 1 line

1.7.5.4

��{.n�+��+%��ݶ��w��{.n�+��{��G��{ay�ʇڙ�,j��f��h��z_�(�階�ݢj"��m��G��?��&��~��iO��z��v�^�m��?�I�

Mark Brown

Applied

2012-04-06 09:07 UTC #8 lore raw

In reply to Axel Lin

Quoted 1 line · Axel Lin

Signed-off-by Axel Lin <[email protected]>

Applied all, thanks.