* [PATCH] ARM: SMDK6410: Convert to using REGULATOR_SUPPLY()
@ 2011-05-24 0:27 Mark Brown
2011-05-24 0:33 ` Kyungmin Park
2011-05-30 17:23 ` Kukjin Kim
0 siblings, 2 replies; 3+ messages in thread
From: Mark Brown @ 2011-05-24 0:27 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
arch/arm/mach-s3c64xx/mach-smdk6410.c | 26 ++++++--------------------
1 files changed, 6 insertions(+), 20 deletions(-)
diff --git a/arch/arm/mach-s3c64xx/mach-smdk6410.c b/arch/arm/mach-s3c64xx/mach-smdk6410.c
index b416294..8c318a3 100644
--- a/arch/arm/mach-s3c64xx/mach-smdk6410.c
+++ b/arch/arm/mach-s3c64xx/mach-smdk6410.c
@@ -210,17 +210,9 @@ static struct platform_device smdk6410_smsc911x = {
};
#ifdef CONFIG_REGULATOR
-static struct regulator_consumer_supply smdk6410_b_pwr_5v_consumers[] = {
- {
- /* WM8580 */
- .supply = "PVDD",
- .dev_name = "0-001b",
- },
- {
- /* WM8580 */
- .supply = "AVDD",
- .dev_name = "0-001b",
- },
+static struct regulator_consumer_supply smdk6410_b_pwr_5v_consumers[] __initdata = {
+ REGULATOR_SUPPLY("PVDD", "0-001b"),
+ REGULATOR_SUPPLY("AVDD", "0-001b"),
};
static struct regulator_init_data smdk6410_b_pwr_5v_data = {
@@ -345,9 +337,7 @@ static struct platform_device *smdk6410_devices[] __initdata = {
#ifdef CONFIG_REGULATOR
/* ARM core */
static struct regulator_consumer_supply smdk6410_vddarm_consumers[] = {
- {
- .supply = "vddarm",
- }
+ REGULATOR_SUPPLY("vddarm", NULL),
};
/* VDDARM, BUCK1 on J5 */
@@ -485,11 +475,7 @@ static struct regulator_init_data wm8350_dcdc3_data = {
/* USB, EXT, PCM, ADC/DAC, USB, MMC */
static struct regulator_consumer_supply wm8350_dcdc4_consumers[] = {
- {
- /* WM8580 */
- .supply = "DVDD",
- .dev_name = "0-001b",
- },
+ REGULATOR_SUPPLY("DVDD", "0-001b"),
};
static struct regulator_init_data wm8350_dcdc4_data = {
@@ -603,7 +589,7 @@ static struct regulator_init_data wm1192_dcdc3 = {
};
static struct regulator_consumer_supply wm1192_ldo1_consumers[] = {
- { .supply = "DVDD", .dev_name = "0-001b", }, /* WM8580 */
+ REGULATOR_SUPPLY("DVDD", "0-001b"), /* WM8580 */
};
static struct regulator_init_data wm1192_ldo1 = {
--
1.7.5.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* [PATCH] ARM: SMDK6410: Convert to using REGULATOR_SUPPLY()
2011-05-24 0:27 [PATCH] ARM: SMDK6410: Convert to using REGULATOR_SUPPLY() Mark Brown
@ 2011-05-24 0:33 ` Kyungmin Park
2011-05-30 17:23 ` Kukjin Kim
1 sibling, 0 replies; 3+ messages in thread
From: Kyungmin Park @ 2011-05-24 0:33 UTC (permalink / raw)
To: linux-arm-kernel
Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
On Tue, May 24, 2011 at 9:27 AM, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
> ---
> ?arch/arm/mach-s3c64xx/mach-smdk6410.c | ? 26 ++++++--------------------
> ?1 files changed, 6 insertions(+), 20 deletions(-)
>
> diff --git a/arch/arm/mach-s3c64xx/mach-smdk6410.c b/arch/arm/mach-s3c64xx/mach-smdk6410.c
> index b416294..8c318a3 100644
> --- a/arch/arm/mach-s3c64xx/mach-smdk6410.c
> +++ b/arch/arm/mach-s3c64xx/mach-smdk6410.c
> @@ -210,17 +210,9 @@ static struct platform_device smdk6410_smsc911x = {
> ?};
>
> ?#ifdef CONFIG_REGULATOR
> -static struct regulator_consumer_supply smdk6410_b_pwr_5v_consumers[] = {
> - ? ? ? {
> - ? ? ? ? ? ? ? /* WM8580 */
> - ? ? ? ? ? ? ? .supply = "PVDD",
> - ? ? ? ? ? ? ? .dev_name = "0-001b",
> - ? ? ? },
> - ? ? ? {
> - ? ? ? ? ? ? ? /* WM8580 */
> - ? ? ? ? ? ? ? .supply = "AVDD",
> - ? ? ? ? ? ? ? .dev_name = "0-001b",
> - ? ? ? },
> +static struct regulator_consumer_supply smdk6410_b_pwr_5v_consumers[] __initdata = {
> + ? ? ? REGULATOR_SUPPLY("PVDD", "0-001b"),
> + ? ? ? REGULATOR_SUPPLY("AVDD", "0-001b"),
> ?};
>
> ?static struct regulator_init_data smdk6410_b_pwr_5v_data = {
> @@ -345,9 +337,7 @@ static struct platform_device *smdk6410_devices[] __initdata = {
> ?#ifdef CONFIG_REGULATOR
> ?/* ARM core */
> ?static struct regulator_consumer_supply smdk6410_vddarm_consumers[] = {
> - ? ? ? {
> - ? ? ? ? ? ? ? .supply = "vddarm",
> - ? ? ? }
> + ? ? ? REGULATOR_SUPPLY("vddarm", NULL),
> ?};
>
> ?/* VDDARM, BUCK1 on J5 */
> @@ -485,11 +475,7 @@ static struct regulator_init_data wm8350_dcdc3_data = {
>
> ?/* USB, EXT, PCM, ADC/DAC, USB, MMC */
> ?static struct regulator_consumer_supply wm8350_dcdc4_consumers[] = {
> - ? ? ? {
> - ? ? ? ? ? ? ? /* WM8580 */
> - ? ? ? ? ? ? ? .supply = "DVDD",
> - ? ? ? ? ? ? ? .dev_name = "0-001b",
> - ? ? ? },
> + ? ? ? REGULATOR_SUPPLY("DVDD", "0-001b"),
> ?};
>
> ?static struct regulator_init_data wm8350_dcdc4_data = {
> @@ -603,7 +589,7 @@ static struct regulator_init_data wm1192_dcdc3 = {
> ?};
>
> ?static struct regulator_consumer_supply wm1192_ldo1_consumers[] = {
> - ? ? ? { .supply = "DVDD", .dev_name = "0-001b", }, ? /* WM8580 */
> + ? ? ? REGULATOR_SUPPLY("DVDD", "0-001b"), ? /* WM8580 */
> ?};
>
> ?static struct regulator_init_data wm1192_ldo1 = {
> --
> 1.7.5.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
^ permalink raw reply [flat|nested] 3+ messages in thread* [PATCH] ARM: SMDK6410: Convert to using REGULATOR_SUPPLY()
2011-05-24 0:27 [PATCH] ARM: SMDK6410: Convert to using REGULATOR_SUPPLY() Mark Brown
2011-05-24 0:33 ` Kyungmin Park
@ 2011-05-30 17:23 ` Kukjin Kim
1 sibling, 0 replies; 3+ messages in thread
From: Kukjin Kim @ 2011-05-30 17:23 UTC (permalink / raw)
To: linux-arm-kernel
On 05/23/11 17:27, Mark Brown wrote:
> Signed-off-by: Mark Brown<broonie@opensource.wolfsonmicro.com>
> ---
> arch/arm/mach-s3c64xx/mach-smdk6410.c | 26 ++++++--------------------
> 1 files changed, 6 insertions(+), 20 deletions(-)
>
> diff --git a/arch/arm/mach-s3c64xx/mach-smdk6410.c b/arch/arm/mach-s3c64xx/mach-smdk6410.c
> index b416294..8c318a3 100644
> --- a/arch/arm/mach-s3c64xx/mach-smdk6410.c
> +++ b/arch/arm/mach-s3c64xx/mach-smdk6410.c
> @@ -210,17 +210,9 @@ static struct platform_device smdk6410_smsc911x = {
> };
>
> #ifdef CONFIG_REGULATOR
> -static struct regulator_consumer_supply smdk6410_b_pwr_5v_consumers[] = {
> - {
> - /* WM8580 */
> - .supply = "PVDD",
> - .dev_name = "0-001b",
> - },
> - {
> - /* WM8580 */
> - .supply = "AVDD",
> - .dev_name = "0-001b",
> - },
> +static struct regulator_consumer_supply smdk6410_b_pwr_5v_consumers[] __initdata = {
> + REGULATOR_SUPPLY("PVDD", "0-001b"),
> + REGULATOR_SUPPLY("AVDD", "0-001b"),
> };
>
> static struct regulator_init_data smdk6410_b_pwr_5v_data = {
> @@ -345,9 +337,7 @@ static struct platform_device *smdk6410_devices[] __initdata = {
> #ifdef CONFIG_REGULATOR
> /* ARM core */
> static struct regulator_consumer_supply smdk6410_vddarm_consumers[] = {
> - {
> - .supply = "vddarm",
> - }
> + REGULATOR_SUPPLY("vddarm", NULL),
> };
>
> /* VDDARM, BUCK1 on J5 */
> @@ -485,11 +475,7 @@ static struct regulator_init_data wm8350_dcdc3_data = {
>
> /* USB, EXT, PCM, ADC/DAC, USB, MMC */
> static struct regulator_consumer_supply wm8350_dcdc4_consumers[] = {
> - {
> - /* WM8580 */
> - .supply = "DVDD",
> - .dev_name = "0-001b",
> - },
> + REGULATOR_SUPPLY("DVDD", "0-001b"),
> };
>
> static struct regulator_init_data wm8350_dcdc4_data = {
> @@ -603,7 +589,7 @@ static struct regulator_init_data wm1192_dcdc3 = {
> };
>
> static struct regulator_consumer_supply wm1192_ldo1_consumers[] = {
> - { .supply = "DVDD", .dev_name = "0-001b", }, /* WM8580 */
> + REGULATOR_SUPPLY("DVDD", "0-001b"), /* WM8580 */
> };
>
> static struct regulator_init_data wm1192_ldo1 = {
Sorry for late response :(
Oops, I missed this...
Looks ok, will apply :)
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-05-30 17:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-24 0:27 [PATCH] ARM: SMDK6410: Convert to using REGULATOR_SUPPLY() Mark Brown
2011-05-24 0:33 ` Kyungmin Park
2011-05-30 17:23 ` Kukjin Kim
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.