[PATCH 1/6] regulator: max1586: Constify regulator

public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed

* [PATCH 1/6] regulator: max1586: Constify regulator_desc
@ 2012-04-06  0:24 Axel Lin
  2012-04-06  0:26 ` [PATCH 2/6] regulator: max8649: " Axel Lin
                   ` (5 more replies)
  0 siblings, 6 replies; 9+ messages in thread
From: Axel Lin @ 2012-04-06  0:24 UTC (permalink / raw)
  To: linux-kernel; +Cc: Robert Jarzmik, Liam Girdwood, Mark Brown

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 drivers/regulator/max1586.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/regulator/max1586.c b/drivers/regulator/max1586.c
index 282d2ee..fad0bee 100644
--- a/drivers/regulator/max1586.c
+++ b/drivers/regulator/max1586.c
@@ -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,
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 2/6] regulator: max8649: Constify regulator_desc
  2012-04-06  0:24 [PATCH 1/6] regulator: max1586: Constify regulator_desc Axel Lin
@ 2012-04-06  0:26 ` Axel Lin
  2012-04-06  0:26 ` [PATCH 3/6] regulator: max8660: " Axel Lin
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 9+ messages in thread
From: Axel Lin @ 2012-04-06  0:26 UTC (permalink / raw)
  To: linux-kernel; +Cc: Jonghwan Choi, Haojian Zhuang, Mark Brown, Liam Girdwood

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 drivers/regulator/max8649.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/regulator/max8649.c b/drivers/regulator/max8649.c
index 2e9ffbe..dca7835 100644
--- a/drivers/regulator/max8649.c
+++ b/drivers/regulator/max8649.c
@@ -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,
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 3/6] regulator: max8660: Constify regulator_desc
  2012-04-06  0:24 [PATCH 1/6] regulator: max1586: Constify regulator_desc Axel Lin
  2012-04-06  0:26 ` [PATCH 2/6] regulator: max8649: " Axel Lin
@ 2012-04-06  0:26 ` Axel Lin
  2012-04-06  0:29 ` [PATCH 4/6] regulator: max8952: " Axel Lin
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 9+ messages in thread
From: Axel Lin @ 2012-04-06  0:26 UTC (permalink / raw)
  To: linux-kernel; +Cc: Wolfram Sang, Liam Girdwood, Mark Brown

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 drivers/regulator/max8660.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/regulator/max8660.c b/drivers/regulator/max8660.c
index e42db53..0e32787 100644
--- a/drivers/regulator/max8660.c
+++ b/drivers/regulator/max8660.c
@@ -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,
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 4/6] regulator: max8952: Constify regulator_desc
  2012-04-06  0:24 [PATCH 1/6] regulator: max1586: Constify regulator_desc Axel Lin
  2012-04-06  0:26 ` [PATCH 2/6] regulator: max8649: " Axel Lin
  2012-04-06  0:26 ` [PATCH 3/6] regulator: max8660: " Axel Lin
@ 2012-04-06  0:29 ` Axel Lin
  2012-04-06  0:30 ` [PATCH 5/6] regulator: tps6105x: " Axel Lin
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 9+ messages in thread
From: Axel Lin @ 2012-04-06  0:29 UTC (permalink / raw)
  To: linux-kernel; +Cc: Liam Girdwood, Mark Brown, MyungJoo Ham, Kyungmin Park

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 drivers/regulator/max8952.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/regulator/max8952.c b/drivers/regulator/max8952.c
index c35236a..b408431 100644
--- a/drivers/regulator/max8952.c
+++ b/drivers/regulator/max8952.c
@@ -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,
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 5/6] regulator: tps6105x: Constify regulator_desc
  2012-04-06  0:24 [PATCH 1/6] regulator: max1586: Constify regulator_desc Axel Lin
                   ` (2 preceding siblings ...)
  2012-04-06  0:29 ` [PATCH 4/6] regulator: max8952: " Axel Lin
@ 2012-04-06  0:30 ` Axel Lin
  2012-04-09 21:12   ` Linus Walleij
  2012-04-06  0:31 ` [PATCH 6/6] regulator: tps65217: " Axel Lin
  2012-04-06  9:07 ` [PATCH 1/6] regulator: max1586: " Mark Brown
  5 siblings, 1 reply; 9+ messages in thread
From: Axel Lin @ 2012-04-06  0:30 UTC (permalink / raw)
  To: linux-kernel; +Cc: Linus Walleij, Liam Girdwood, Mark Brown

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 drivers/regulator/tps6105x-regulator.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/regulator/tps6105x-regulator.c b/drivers/regulator/tps6105x-regulator.c
index d9278da..3b78897 100644
--- a/drivers/regulator/tps6105x-regulator.c
+++ b/drivers/regulator/tps6105x-regulator.c
@@ -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,
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH 5/6] regulator: tps6105x: Constify regulator_desc
  2012-04-06  0:30 ` [PATCH 5/6] regulator: tps6105x: " Axel Lin
@ 2012-04-09 21:12   ` Linus Walleij
  0 siblings, 0 replies; 9+ messages in thread
From: Linus Walleij @ 2012-04-09 21:12 UTC (permalink / raw)
  To: Axel Lin; +Cc: linux-kernel, Liam Girdwood, Mark Brown

On Fri, Apr 6, 2012 at 2:30 AM, Axel Lin <axel.lin@gmail.com> wrote:

> Signed-off-by: Axel Lin <axel.lin@gmail.com>
> ---
>  drivers/regulator/tps6105x-regulator.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Looks like a good idea:
Acked-by: Linus Walleij <linus.walleij@linaro.org>

Thanks,
Linus Walleij

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH 6/6] regulator: tps65217: Constify regulator_desc
  2012-04-06  0:24 [PATCH 1/6] regulator: max1586: Constify regulator_desc Axel Lin
                   ` (3 preceding siblings ...)
  2012-04-06  0:30 ` [PATCH 5/6] regulator: tps6105x: " Axel Lin
@ 2012-04-06  0:31 ` Axel Lin
  2012-04-06  7:03   ` AnilKumar, Chimata
  2012-04-06  9:07 ` [PATCH 1/6] regulator: max1586: " Mark Brown
  5 siblings, 1 reply; 9+ messages in thread
From: Axel Lin @ 2012-04-06  0:31 UTC (permalink / raw)
  To: linux-kernel; +Cc: AnilKumar Ch, Mark Brown, Liam Girdwood

Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
 drivers/regulator/tps65217-regulator.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/regulator/tps65217-regulator.c b/drivers/regulator/tps65217-regulator.c
index e39521b..80fad2d 100644
--- a/drivers/regulator/tps65217-regulator.c
+++ b/drivers/regulator/tps65217-regulator.c
@@ -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),
-- 
1.7.5.4




^ permalink raw reply related	[flat|nested] 9+ messages in thread

* RE: [PATCH 6/6] regulator: tps65217: Constify regulator_desc
  2012-04-06  0:31 ` [PATCH 6/6] regulator: tps65217: " Axel Lin
@ 2012-04-06  7:03   ` AnilKumar, Chimata
  0 siblings, 0 replies; 9+ messages in thread
From: AnilKumar, Chimata @ 2012-04-06  7:03 UTC (permalink / raw)
  To: Axel Lin, linux-kernel@vger.kernel.org; +Cc: Mark Brown, Girdwood, Liam

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1190 bytes --]

Hi Axel Lin,

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

ACK: AnilKumar Ch <anilkumar@ti.com>

On Fri, Apr 06, 2012 at 06:01:51, Axel Lin wrote:
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
> ---
>  drivers/regulator/tps65217-regulator.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/regulator/tps65217-regulator.c b/drivers/regulator/tps65217-regulator.c
> index e39521b..80fad2d 100644
> --- a/drivers/regulator/tps65217-regulator.c
> +++ b/drivers/regulator/tps65217-regulator.c
> @@ -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),
> -- 
> 1.7.5.4
> 
> 
> 
> 

ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éÝ¶\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dÊ‡Ú™ë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 1/6] regulator: max1586: Constify regulator_desc
  2012-04-06  0:24 [PATCH 1/6] regulator: max1586: Constify regulator_desc Axel Lin
                   ` (4 preceding siblings ...)
  2012-04-06  0:31 ` [PATCH 6/6] regulator: tps65217: " Axel Lin
@ 2012-04-06  9:07 ` Mark Brown
  5 siblings, 0 replies; 9+ messages in thread
From: Mark Brown @ 2012-04-06  9:07 UTC (permalink / raw)
  To: Axel Lin; +Cc: linux-kernel, Robert Jarzmik, Liam Girdwood

[-- Attachment #1: Type: text/plain, Size: 127 bytes --]

On Fri, Apr 06, 2012 at 08:24:27AM +0800, Axel Lin wrote:
> Signed-off-by: Axel Lin <axel.lin@gmail.com>

Applied all, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2012-04-09 21:12 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-04-06  0:24 [PATCH 1/6] regulator: max1586: Constify regulator_desc Axel Lin
2012-04-06  0:26 ` [PATCH 2/6] regulator: max8649: " Axel Lin
2012-04-06  0:26 ` [PATCH 3/6] regulator: max8660: " Axel Lin
2012-04-06  0:29 ` [PATCH 4/6] regulator: max8952: " Axel Lin
2012-04-06  0:30 ` [PATCH 5/6] regulator: tps6105x: " Axel Lin
2012-04-09 21:12   ` Linus Walleij
2012-04-06  0:31 ` [PATCH 6/6] regulator: tps65217: " Axel Lin
2012-04-06  7:03   ` AnilKumar, Chimata
2012-04-06  9:07 ` [PATCH 1/6] regulator: max1586: " Mark Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox