* [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
@ 2013-12-20 9:46 ` Thierry Reding
0 siblings, 0 replies; 14+ messages in thread
From: Thierry Reding @ 2013-12-20 9:46 UTC (permalink / raw)
To: Haojian Zhuang, Robert Jarzmik; +Cc: linux-arm-kernel, linux-pwm
Recent changes to the pwm-backlight driver have made the power supply
mandatory. There is code in the regulator core to deal with situations
where no regulator is specified and provide a dummy, but that works on
DT-based boards only.
The situation can be remedied by adding a dummy regulator during board
initialization.
Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---
arch/arm/mach-pxa/mioa701.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
index f70583fee59f..29997bde277d 100644
--- a/arch/arm/mach-pxa/mioa701.c
+++ b/arch/arm/mach-pxa/mioa701.c
@@ -38,6 +38,7 @@
#include <linux/mtd/physmap.h>
#include <linux/usb/gpio_vbus.h>
#include <linux/reboot.h>
+#include <linux/regulator/fixed.h>
#include <linux/regulator/max1586.h>
#include <linux/slab.h>
#include <linux/i2c/pxa-i2c.h>
@@ -714,6 +715,10 @@ static struct gpio global_gpios[] = {
{ GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
};
+static struct regulator_consumer_supply fixed_5v0_consumers[] = {
+ REGULATOR_SUPPLY("power", "pwm-backlight"),
+};
+
static void __init mioa701_machine_init(void)
{
int rc;
@@ -753,6 +758,10 @@ static void __init mioa701_machine_init(void)
pxa_set_i2c_info(&i2c_pdata);
pxa27x_set_i2c_power_info(NULL);
pxa_set_camera_info(&mioa701_pxacamera_platform_data);
+
+ regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
+ ARRAY_SIZE(fixed_5v0_consumers),
+ 5000000);
}
static void mioa701_machine_exit(void)
--
1.8.4.2
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
@ 2013-12-20 9:46 ` Thierry Reding
0 siblings, 0 replies; 14+ messages in thread
From: Thierry Reding @ 2013-12-20 9:46 UTC (permalink / raw)
To: linux-arm-kernel
Recent changes to the pwm-backlight driver have made the power supply
mandatory. There is code in the regulator core to deal with situations
where no regulator is specified and provide a dummy, but that works on
DT-based boards only.
The situation can be remedied by adding a dummy regulator during board
initialization.
Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---
arch/arm/mach-pxa/mioa701.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
index f70583fee59f..29997bde277d 100644
--- a/arch/arm/mach-pxa/mioa701.c
+++ b/arch/arm/mach-pxa/mioa701.c
@@ -38,6 +38,7 @@
#include <linux/mtd/physmap.h>
#include <linux/usb/gpio_vbus.h>
#include <linux/reboot.h>
+#include <linux/regulator/fixed.h>
#include <linux/regulator/max1586.h>
#include <linux/slab.h>
#include <linux/i2c/pxa-i2c.h>
@@ -714,6 +715,10 @@ static struct gpio global_gpios[] = {
{ GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
};
+static struct regulator_consumer_supply fixed_5v0_consumers[] = {
+ REGULATOR_SUPPLY("power", "pwm-backlight"),
+};
+
static void __init mioa701_machine_init(void)
{
int rc;
@@ -753,6 +758,10 @@ static void __init mioa701_machine_init(void)
pxa_set_i2c_info(&i2c_pdata);
pxa27x_set_i2c_power_info(NULL);
pxa_set_camera_info(&mioa701_pxacamera_platform_data);
+
+ regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
+ ARRAY_SIZE(fixed_5v0_consumers),
+ 5000000);
}
static void mioa701_machine_exit(void)
--
1.8.4.2
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
2013-12-20 9:46 ` Thierry Reding
@ 2013-12-22 18:14 ` Robert Jarzmik
-1 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2013-12-22 18:14 UTC (permalink / raw)
To: linux, Marek Vasut, Philipp Zabel, Paul Parsons, Daniel Ribeiro,
Stefan Schmidt, Harald Welte
Cc: Thierry Reding, Haojian Zhuang, linux-pwm, linux-arm-kernel
Hi guys,
I think this kind of patch [1] should end up in your machine files too, as it
impacts all machines not relying on devicetree.
Moreover, your defconfig should be amended, adding
CONFIG_REGULATOR_FIXED_VOLTAGE=y.
So far these are the files I think should be checked :
rj@beldin:~/mio_linux/kernel$ for f in $(grep -l pwm-backlight arch/arm/mach-*/*.c); do echo $f; done
arch/arm/mach-pxa/cm-x300.c
arch/arm/mach-pxa/colibri-pxa270-income.c
arch/arm/mach-pxa/ezx.c
arch/arm/mach-pxa/hx4700.c
arch/arm/mach-pxa/lpd270.c
arch/arm/mach-pxa/magician.c
arch/arm/mach-pxa/mainstone.c
arch/arm/mach-pxa/mioa701.c
arch/arm/mach-pxa/palm27x.c
arch/arm/mach-pxa/palmtc.c
arch/arm/mach-pxa/palmte2.c
arch/arm/mach-pxa/pcm990-baseboard.c
arch/arm/mach-pxa/raumfeld.c
arch/arm/mach-pxa/tavorevb.c
arch/arm/mach-pxa/viper.c
arch/arm/mach-pxa/z2.c
arch/arm/mach-pxa/zylonite.c
arch/arm/mach-s3c24xx/mach-h1940.c
arch/arm/mach-s3c24xx/mach-rx1950.c
arch/arm/mach-s3c64xx/mach-crag6410.c
arch/arm/mach-s3c64xx/mach-hmt.c
arch/arm/mach-s3c64xx/mach-smartq.c
arch/arm/mach-shmobile/board-armadillo800eva.c
Cheers.
--
Robert
[1] Thierry's patch
Thierry Reding <thierry.reding@gmail.com> writes:
> Recent changes to the pwm-backlight driver have made the power supply
> mandatory. There is code in the regulator core to deal with situations
> where no regulator is specified and provide a dummy, but that works on
> DT-based boards only.
>
> The situation can be remedied by adding a dummy regulator during board
> initialization.
>
> Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
> Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> ---
> arch/arm/mach-pxa/mioa701.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
> index f70583fee59f..29997bde277d 100644
> --- a/arch/arm/mach-pxa/mioa701.c
> +++ b/arch/arm/mach-pxa/mioa701.c
> @@ -38,6 +38,7 @@
> #include <linux/mtd/physmap.h>
> #include <linux/usb/gpio_vbus.h>
> #include <linux/reboot.h>
> +#include <linux/regulator/fixed.h>
> #include <linux/regulator/max1586.h>
> #include <linux/slab.h>
> #include <linux/i2c/pxa-i2c.h>
> @@ -714,6 +715,10 @@ static struct gpio global_gpios[] = {
> { GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
> };
>
> +static struct regulator_consumer_supply fixed_5v0_consumers[] = {
> + REGULATOR_SUPPLY("power", "pwm-backlight"),
> +};
> +
> static void __init mioa701_machine_init(void)
> {
> int rc;
> @@ -753,6 +758,10 @@ static void __init mioa701_machine_init(void)
> pxa_set_i2c_info(&i2c_pdata);
> pxa27x_set_i2c_power_info(NULL);
> pxa_set_camera_info(&mioa701_pxacamera_platform_data);
> +
> + regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
> + ARRAY_SIZE(fixed_5v0_consumers),
> + 5000000);
> }
>
> static void mioa701_machine_exit(void)
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
@ 2013-12-22 18:14 ` Robert Jarzmik
0 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2013-12-22 18:14 UTC (permalink / raw)
To: linux-arm-kernel
Hi guys,
I think this kind of patch [1] should end up in your machine files too, as it
impacts all machines not relying on devicetree.
Moreover, your defconfig should be amended, adding
CONFIG_REGULATOR_FIXED_VOLTAGE=y.
So far these are the files I think should be checked :
rj at beldin:~/mio_linux/kernel$ for f in $(grep -l pwm-backlight arch/arm/mach-*/*.c); do echo $f; done
arch/arm/mach-pxa/cm-x300.c
arch/arm/mach-pxa/colibri-pxa270-income.c
arch/arm/mach-pxa/ezx.c
arch/arm/mach-pxa/hx4700.c
arch/arm/mach-pxa/lpd270.c
arch/arm/mach-pxa/magician.c
arch/arm/mach-pxa/mainstone.c
arch/arm/mach-pxa/mioa701.c
arch/arm/mach-pxa/palm27x.c
arch/arm/mach-pxa/palmtc.c
arch/arm/mach-pxa/palmte2.c
arch/arm/mach-pxa/pcm990-baseboard.c
arch/arm/mach-pxa/raumfeld.c
arch/arm/mach-pxa/tavorevb.c
arch/arm/mach-pxa/viper.c
arch/arm/mach-pxa/z2.c
arch/arm/mach-pxa/zylonite.c
arch/arm/mach-s3c24xx/mach-h1940.c
arch/arm/mach-s3c24xx/mach-rx1950.c
arch/arm/mach-s3c64xx/mach-crag6410.c
arch/arm/mach-s3c64xx/mach-hmt.c
arch/arm/mach-s3c64xx/mach-smartq.c
arch/arm/mach-shmobile/board-armadillo800eva.c
Cheers.
--
Robert
[1] Thierry's patch
Thierry Reding <thierry.reding@gmail.com> writes:
> Recent changes to the pwm-backlight driver have made the power supply
> mandatory. There is code in the regulator core to deal with situations
> where no regulator is specified and provide a dummy, but that works on
> DT-based boards only.
>
> The situation can be remedied by adding a dummy regulator during board
> initialization.
>
> Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
> Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> ---
> arch/arm/mach-pxa/mioa701.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
> index f70583fee59f..29997bde277d 100644
> --- a/arch/arm/mach-pxa/mioa701.c
> +++ b/arch/arm/mach-pxa/mioa701.c
> @@ -38,6 +38,7 @@
> #include <linux/mtd/physmap.h>
> #include <linux/usb/gpio_vbus.h>
> #include <linux/reboot.h>
> +#include <linux/regulator/fixed.h>
> #include <linux/regulator/max1586.h>
> #include <linux/slab.h>
> #include <linux/i2c/pxa-i2c.h>
> @@ -714,6 +715,10 @@ static struct gpio global_gpios[] = {
> { GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
> };
>
> +static struct regulator_consumer_supply fixed_5v0_consumers[] = {
> + REGULATOR_SUPPLY("power", "pwm-backlight"),
> +};
> +
> static void __init mioa701_machine_init(void)
> {
> int rc;
> @@ -753,6 +758,10 @@ static void __init mioa701_machine_init(void)
> pxa_set_i2c_info(&i2c_pdata);
> pxa27x_set_i2c_power_info(NULL);
> pxa_set_camera_info(&mioa701_pxacamera_platform_data);
> +
> + regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
> + ARRAY_SIZE(fixed_5v0_consumers),
> + 5000000);
> }
>
> static void mioa701_machine_exit(void)
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
2013-12-20 9:46 ` Thierry Reding
@ 2014-01-06 8:19 ` Robert Jarzmik
-1 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2014-01-06 8:19 UTC (permalink / raw)
To: Haojian Zhuang; +Cc: Thierry Reding, linux-arm-kernel, linux-pwm
Thierry Reding <thierry.reding@gmail.com> writes:
> Recent changes to the pwm-backlight driver have made the power supply
> mandatory. There is code in the regulator core to deal with situations
> where no regulator is specified and provide a dummy, but that works on
> DT-based boards only.
>
> The situation can be remedied by adding a dummy regulator during board
> initialization.
Haojian, ping ?
We're at -rc7 and I've seen no pull of this fix. As the display is broken
without it, I'd really like to have it fixed for v3.13.
Cheers.
--
Robert
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
@ 2014-01-06 8:19 ` Robert Jarzmik
0 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2014-01-06 8:19 UTC (permalink / raw)
To: linux-arm-kernel
Thierry Reding <thierry.reding@gmail.com> writes:
> Recent changes to the pwm-backlight driver have made the power supply
> mandatory. There is code in the regulator core to deal with situations
> where no regulator is specified and provide a dummy, but that works on
> DT-based boards only.
>
> The situation can be remedied by adding a dummy regulator during board
> initialization.
Haojian, ping ?
We're at -rc7 and I've seen no pull of this fix. As the display is broken
without it, I'd really like to have it fixed for v3.13.
Cheers.
--
Robert
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
2014-01-06 8:19 ` Robert Jarzmik
@ 2014-01-12 23:43 ` Robert Jarzmik
-1 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2014-01-12 23:43 UTC (permalink / raw)
To: Arnd Bergmann, Haojian Zhuang; +Cc: linux-pwm, Thierry Reding, linux-arm-kernel
Hi Arnd,
I cannot reach Haojian, and my platform is currently screen-broken without
Thierry's patch (see [1] and [2]). I'm trying to have it since -rc6, with no luck.
Could you handle it in arm-soc fixes for -rc9 (if it happens) please ?
Cheers.
--
Robert
[1] http://www.spinics.net/lists/arm-kernel/msg298187.html
[2] Patch :
From 7ca9a136dc67ac4c302b15a333e8cfa1c3fc051b Mon Sep 17 00:00:00 2001
From: Thierry Reding <thierry.reding@gmail.com>
Date: Fri, 20 Dec 2013 10:46:52 +0100
Subject: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
Recent changes to the pwm-backlight driver have made the power supply
mandatory. There is code in the regulator core to deal with situations
where no regulator is specified and provide a dummy, but that works on
DT-based boards only.
The situation can be remedied by adding a dummy regulator during board
initialization.
Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---
arch/arm/mach-pxa/mioa701.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
index c2d8625..a8fdcb7 100644
--- a/arch/arm/mach-pxa/mioa701.c
+++ b/arch/arm/mach-pxa/mioa701.c
@@ -38,6 +38,7 @@
#include <linux/mtd/physmap.h>
#include <linux/usb/gpio_vbus.h>
#include <linux/reboot.h>
+#include <linux/regulator/fixed.h>
#include <linux/regulator/max1586.h>
#include <linux/slab.h>
#include <linux/i2c/pxa-i2c.h>
@@ -723,6 +724,10 @@ static struct gpio global_gpios[] = {
{ GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
};
+static struct regulator_consumer_supply fixed_5v0_consumers[] = {
+ REGULATOR_SUPPLY("power", "pwm-backlight"),
+};
+
static void __init mioa701_machine_init(void)
{
int rc;
@@ -762,6 +767,10 @@ static void __init mioa701_machine_init(void)
pxa_set_i2c_info(&i2c_pdata);
pxa27x_set_i2c_power_info(NULL);
pxa_set_camera_info(&mioa701_pxacamera_platform_data);
+
+ regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
+ ARRAY_SIZE(fixed_5v0_consumers),
+ 5000000);
}
static void mioa701_machine_exit(void)
--
1.7.10.4
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
@ 2014-01-12 23:43 ` Robert Jarzmik
0 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2014-01-12 23:43 UTC (permalink / raw)
To: linux-arm-kernel
Hi Arnd,
I cannot reach Haojian, and my platform is currently screen-broken without
Thierry's patch (see [1] and [2]). I'm trying to have it since -rc6, with no luck.
Could you handle it in arm-soc fixes for -rc9 (if it happens) please ?
Cheers.
--
Robert
[1] http://www.spinics.net/lists/arm-kernel/msg298187.html
[2] Patch :
>From 7ca9a136dc67ac4c302b15a333e8cfa1c3fc051b Mon Sep 17 00:00:00 2001
From: Thierry Reding <thierry.reding@gmail.com>
Date: Fri, 20 Dec 2013 10:46:52 +0100
Subject: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
Recent changes to the pwm-backlight driver have made the power supply
mandatory. There is code in the regulator core to deal with situations
where no regulator is specified and provide a dummy, but that works on
DT-based boards only.
The situation can be remedied by adding a dummy regulator during board
initialization.
Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---
arch/arm/mach-pxa/mioa701.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
index c2d8625..a8fdcb7 100644
--- a/arch/arm/mach-pxa/mioa701.c
+++ b/arch/arm/mach-pxa/mioa701.c
@@ -38,6 +38,7 @@
#include <linux/mtd/physmap.h>
#include <linux/usb/gpio_vbus.h>
#include <linux/reboot.h>
+#include <linux/regulator/fixed.h>
#include <linux/regulator/max1586.h>
#include <linux/slab.h>
#include <linux/i2c/pxa-i2c.h>
@@ -723,6 +724,10 @@ static struct gpio global_gpios[] = {
{ GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
};
+static struct regulator_consumer_supply fixed_5v0_consumers[] = {
+ REGULATOR_SUPPLY("power", "pwm-backlight"),
+};
+
static void __init mioa701_machine_init(void)
{
int rc;
@@ -762,6 +767,10 @@ static void __init mioa701_machine_init(void)
pxa_set_i2c_info(&i2c_pdata);
pxa27x_set_i2c_power_info(NULL);
pxa_set_camera_info(&mioa701_pxacamera_platform_data);
+
+ regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
+ ARRAY_SIZE(fixed_5v0_consumers),
+ 5000000);
}
static void mioa701_machine_exit(void)
--
1.7.10.4
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
2014-01-12 23:43 ` Robert Jarzmik
@ 2014-01-20 20:11 ` Robert Jarzmik
-1 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2014-01-20 20:11 UTC (permalink / raw)
To: Arnd Bergmann, Haojian Zhuang; +Cc: linux-pwm, Thierry Reding, linux-arm-kernel
Robert Jarzmik <robert.jarzmik@free.fr> writes:
> Hi Arnd,
>
> I cannot reach Haojian, and my platform is currently screen-broken without
> Thierry's patch (see [1] and [2]). I'm trying to have it since -rc6, with no luck.
>
> Could you handle it in arm-soc fixes for -rc9 (if it happens) please ?
>
> Cheers.
>
> --
> Robert
OK, so my platform is broken in 3.13, so be it.
What now ? Do you have a backup for such cases ?
--
Robert
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
@ 2014-01-20 20:11 ` Robert Jarzmik
0 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2014-01-20 20:11 UTC (permalink / raw)
To: linux-arm-kernel
Robert Jarzmik <robert.jarzmik@free.fr> writes:
> Hi Arnd,
>
> I cannot reach Haojian, and my platform is currently screen-broken without
> Thierry's patch (see [1] and [2]). I'm trying to have it since -rc6, with no luck.
>
> Could you handle it in arm-soc fixes for -rc9 (if it happens) please ?
>
> Cheers.
>
> --
> Robert
OK, so my platform is broken in 3.13, so be it.
What now ? Do you have a backup for such cases ?
--
Robert
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
2014-01-20 20:11 ` Robert Jarzmik
@ 2014-02-01 17:55 ` Robert Jarzmik
-1 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2014-02-01 17:55 UTC (permalink / raw)
To: Eric Miao, Russell King
Cc: Haojian Zhuang, linux-pwm, Thierry Reding, linux-arm-kernel,
Arnd Bergmann
Robert Jarzmik <robert.jarzmik@free.fr> writes:
>> Hi Arnd,
>>
>> I cannot reach Haojian, and my platform is currently screen-broken without
>> Thierry's patch (see [1] and [2]). I'm trying to have it since -rc6, with no luck.
... zip ...
Hi Eric and Russell,
Since more than a month I've been trying to have a fix applied in the PXA tree,
since v3.13-rc6. I cannot reach Haojian, and Arnd seems unresponsive too.
The history mails are in [1], the patch in [2].
Your are the 2 other maintainers of the PXA tree, Could you please help me ?
Cheers.
--
Robert
[1]
http://lists.infradead.org/pipermail/linux-arm-kernel/2014-January/223116.html
[2]
http://lists.infradead.org/pipermail/linux-arm-kernel/2013-December/221013.html
---8>---
From: Thierry Reding <thierry.reding@gmail.com>
Subject: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
To: Haojian Zhuang <haojian.zhuang@gmail.com>, Robert Jarzmik <robert.jarzmik@free.fr>
Cc: linux-pwm@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Date: Fri, 20 Dec 2013 10:46:52 +0100 (6 weeks, 1 day, 8 hours ago)
Message-Id: <1387532812-11742-1-git-send-email-thierry.reding@gmail.com>
X-Mailer: git-send-email 1.8.4.2
Recent changes to the pwm-backlight driver have made the power supply
mandatory. There is code in the regulator core to deal with situations
where no regulator is specified and provide a dummy, but that works on
DT-based boards only.
The situation can be remedied by adding a dummy regulator during board
initialization.
Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---
arch/arm/mach-pxa/mioa701.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
index f70583fee59f..29997bde277d 100644
--- a/arch/arm/mach-pxa/mioa701.c
+++ b/arch/arm/mach-pxa/mioa701.c
@@ -38,6 +38,7 @@
#include <linux/mtd/physmap.h>
#include <linux/usb/gpio_vbus.h>
#include <linux/reboot.h>
+#include <linux/regulator/fixed.h>
#include <linux/regulator/max1586.h>
#include <linux/slab.h>
#include <linux/i2c/pxa-i2c.h>
@@ -714,6 +715,10 @@ static struct gpio global_gpios[] = {
{ GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
};
+static struct regulator_consumer_supply fixed_5v0_consumers[] = {
+ REGULATOR_SUPPLY("power", "pwm-backlight"),
+};
+
static void __init mioa701_machine_init(void)
{
int rc;
@@ -753,6 +758,10 @@ static void __init mioa701_machine_init(void)
pxa_set_i2c_info(&i2c_pdata);
pxa27x_set_i2c_power_info(NULL);
pxa_set_camera_info(&mioa701_pxacamera_platform_data);
+
+ regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
+ ARRAY_SIZE(fixed_5v0_consumers),
+ 5000000);
}
static void mioa701_machine_exit(void)
--
1.8.4.2
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
@ 2014-02-01 17:55 ` Robert Jarzmik
0 siblings, 0 replies; 14+ messages in thread
From: Robert Jarzmik @ 2014-02-01 17:55 UTC (permalink / raw)
To: linux-arm-kernel
Robert Jarzmik <robert.jarzmik@free.fr> writes:
>> Hi Arnd,
>>
>> I cannot reach Haojian, and my platform is currently screen-broken without
>> Thierry's patch (see [1] and [2]). I'm trying to have it since -rc6, with no luck.
... zip ...
Hi Eric and Russell,
Since more than a month I've been trying to have a fix applied in the PXA tree,
since v3.13-rc6. I cannot reach Haojian, and Arnd seems unresponsive too.
The history mails are in [1], the patch in [2].
Your are the 2 other maintainers of the PXA tree, Could you please help me ?
Cheers.
--
Robert
[1]
http://lists.infradead.org/pipermail/linux-arm-kernel/2014-January/223116.html
[2]
http://lists.infradead.org/pipermail/linux-arm-kernel/2013-December/221013.html
---8>---
From: Thierry Reding <thierry.reding@gmail.com>
Subject: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
To: Haojian Zhuang <haojian.zhuang@gmail.com>, Robert Jarzmik <robert.jarzmik@free.fr>
Cc: linux-pwm at vger.kernel.org, linux-arm-kernel at lists.infradead.org
Date: Fri, 20 Dec 2013 10:46:52 +0100 (6 weeks, 1 day, 8 hours ago)
Message-Id: <1387532812-11742-1-git-send-email-thierry.reding@gmail.com>
X-Mailer: git-send-email 1.8.4.2
Recent changes to the pwm-backlight driver have made the power supply
mandatory. There is code in the regulator core to deal with situations
where no regulator is specified and provide a dummy, but that works on
DT-based boards only.
The situation can be remedied by adding a dummy regulator during board
initialization.
Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---
arch/arm/mach-pxa/mioa701.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
index f70583fee59f..29997bde277d 100644
--- a/arch/arm/mach-pxa/mioa701.c
+++ b/arch/arm/mach-pxa/mioa701.c
@@ -38,6 +38,7 @@
#include <linux/mtd/physmap.h>
#include <linux/usb/gpio_vbus.h>
#include <linux/reboot.h>
+#include <linux/regulator/fixed.h>
#include <linux/regulator/max1586.h>
#include <linux/slab.h>
#include <linux/i2c/pxa-i2c.h>
@@ -714,6 +715,10 @@ static struct gpio global_gpios[] = {
{ GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
};
+static struct regulator_consumer_supply fixed_5v0_consumers[] = {
+ REGULATOR_SUPPLY("power", "pwm-backlight"),
+};
+
static void __init mioa701_machine_init(void)
{
int rc;
@@ -753,6 +758,10 @@ static void __init mioa701_machine_init(void)
pxa_set_i2c_info(&i2c_pdata);
pxa27x_set_i2c_power_info(NULL);
pxa_set_camera_info(&mioa701_pxacamera_platform_data);
+
+ regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
+ ARRAY_SIZE(fixed_5v0_consumers),
+ 5000000);
}
static void mioa701_machine_exit(void)
--
1.8.4.2
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
2014-02-01 17:55 ` Robert Jarzmik
@ 2014-02-11 3:45 ` Haojian Zhuang
-1 siblings, 0 replies; 14+ messages in thread
From: Haojian Zhuang @ 2014-02-11 3:45 UTC (permalink / raw)
To: Robert Jarzmik, Eric Miao, Russell King
Cc: linux-pwm, Thierry Reding, linux-arm-kernel, Arnd Bergmann
On 02/02/2014 01:55 AM, Robert Jarzmik wrote:
> Robert Jarzmik <robert.jarzmik@free.fr> writes:
>>> Hi Arnd,
>>>
>>> I cannot reach Haojian, and my platform is currently screen-broken without
>>> Thierry's patch (see [1] and [2]). I'm trying to have it since -rc6, with no luck.
> ... zip ...
>
> Hi Eric and Russell,
>
> Since more than a month I've been trying to have a fix applied in the PXA tree,
> since v3.13-rc6. I cannot reach Haojian, and Arnd seems unresponsive too.
>
> The history mails are in [1], the patch in [2].
>
> Your are the 2 other maintainers of the PXA tree, Could you please help me ?
>
> Cheers.
>
> --
> Robert
>
> [1]
> http://lists.infradead.org/pipermail/linux-arm-kernel/2014-January/223116.html
>
>
> [2]
> http://lists.infradead.org/pipermail/linux-arm-kernel/2013-December/221013.html
>
> ---8>---
>
> From: Thierry Reding <thierry.reding@gmail.com>
> Subject: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
> To: Haojian Zhuang <haojian.zhuang@gmail.com>, Robert Jarzmik <robert.jarzmik@free.fr>
> Cc: linux-pwm@vger.kernel.org, linux-arm-kernel@lists.infradead.org
> Date: Fri, 20 Dec 2013 10:46:52 +0100 (6 weeks, 1 day, 8 hours ago)
> Message-Id: <1387532812-11742-1-git-send-email-thierry.reding@gmail.com>
> X-Mailer: git-send-email 1.8.4.2
>
> Recent changes to the pwm-backlight driver have made the power supply
> mandatory. There is code in the regulator core to deal with situations
> where no regulator is specified and provide a dummy, but that works on
> DT-based boards only.
>
> The situation can be remedied by adding a dummy regulator during board
> initialization.
>
> Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
> Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> ---
> arch/arm/mach-pxa/mioa701.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
> index f70583fee59f..29997bde277d 100644
> --- a/arch/arm/mach-pxa/mioa701.c
> +++ b/arch/arm/mach-pxa/mioa701.c
> @@ -38,6 +38,7 @@
> #include <linux/mtd/physmap.h>
> #include <linux/usb/gpio_vbus.h>
> #include <linux/reboot.h>
> +#include <linux/regulator/fixed.h>
> #include <linux/regulator/max1586.h>
> #include <linux/slab.h>
> #include <linux/i2c/pxa-i2c.h>
> @@ -714,6 +715,10 @@ static struct gpio global_gpios[] = {
> { GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
> };
>
> +static struct regulator_consumer_supply fixed_5v0_consumers[] = {
> + REGULATOR_SUPPLY("power", "pwm-backlight"),
> +};
> +
> static void __init mioa701_machine_init(void)
> {
> int rc;
> @@ -753,6 +758,10 @@ static void __init mioa701_machine_init(void)
> pxa_set_i2c_info(&i2c_pdata);
> pxa27x_set_i2c_power_info(NULL);
> pxa_set_camera_info(&mioa701_pxacamera_platform_data);
> +
> + regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
> + ARRAY_SIZE(fixed_5v0_consumers),
> + 5000000);
> }
>
> static void mioa701_machine_exit(void)
>
Hi Robert,
Excuse me to response late. Now it's applied.
Regards
Haojian
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
@ 2014-02-11 3:45 ` Haojian Zhuang
0 siblings, 0 replies; 14+ messages in thread
From: Haojian Zhuang @ 2014-02-11 3:45 UTC (permalink / raw)
To: linux-arm-kernel
On 02/02/2014 01:55 AM, Robert Jarzmik wrote:
> Robert Jarzmik <robert.jarzmik@free.fr> writes:
>>> Hi Arnd,
>>>
>>> I cannot reach Haojian, and my platform is currently screen-broken without
>>> Thierry's patch (see [1] and [2]). I'm trying to have it since -rc6, with no luck.
> ... zip ...
>
> Hi Eric and Russell,
>
> Since more than a month I've been trying to have a fix applied in the PXA tree,
> since v3.13-rc6. I cannot reach Haojian, and Arnd seems unresponsive too.
>
> The history mails are in [1], the patch in [2].
>
> Your are the 2 other maintainers of the PXA tree, Could you please help me ?
>
> Cheers.
>
> --
> Robert
>
> [1]
> http://lists.infradead.org/pipermail/linux-arm-kernel/2014-January/223116.html
>
>
> [2]
> http://lists.infradead.org/pipermail/linux-arm-kernel/2013-December/221013.html
>
> ---8>---
>
> From: Thierry Reding <thierry.reding@gmail.com>
> Subject: [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701
> To: Haojian Zhuang <haojian.zhuang@gmail.com>, Robert Jarzmik <robert.jarzmik@free.fr>
> Cc: linux-pwm at vger.kernel.org, linux-arm-kernel at lists.infradead.org
> Date: Fri, 20 Dec 2013 10:46:52 +0100 (6 weeks, 1 day, 8 hours ago)
> Message-Id: <1387532812-11742-1-git-send-email-thierry.reding@gmail.com>
> X-Mailer: git-send-email 1.8.4.2
>
> Recent changes to the pwm-backlight driver have made the power supply
> mandatory. There is code in the regulator core to deal with situations
> where no regulator is specified and provide a dummy, but that works on
> DT-based boards only.
>
> The situation can be remedied by adding a dummy regulator during board
> initialization.
>
> Tested-by: Robert Jarzmik <robert.jarzmik@free.fr>
> Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> ---
> arch/arm/mach-pxa/mioa701.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/arch/arm/mach-pxa/mioa701.c b/arch/arm/mach-pxa/mioa701.c
> index f70583fee59f..29997bde277d 100644
> --- a/arch/arm/mach-pxa/mioa701.c
> +++ b/arch/arm/mach-pxa/mioa701.c
> @@ -38,6 +38,7 @@
> #include <linux/mtd/physmap.h>
> #include <linux/usb/gpio_vbus.h>
> #include <linux/reboot.h>
> +#include <linux/regulator/fixed.h>
> #include <linux/regulator/max1586.h>
> #include <linux/slab.h>
> #include <linux/i2c/pxa-i2c.h>
> @@ -714,6 +715,10 @@ static struct gpio global_gpios[] = {
> { GPIO56_MT9M111_nOE, GPIOF_OUT_INIT_LOW, "Camera nOE" },
> };
>
> +static struct regulator_consumer_supply fixed_5v0_consumers[] = {
> + REGULATOR_SUPPLY("power", "pwm-backlight"),
> +};
> +
> static void __init mioa701_machine_init(void)
> {
> int rc;
> @@ -753,6 +758,10 @@ static void __init mioa701_machine_init(void)
> pxa_set_i2c_info(&i2c_pdata);
> pxa27x_set_i2c_power_info(NULL);
> pxa_set_camera_info(&mioa701_pxacamera_platform_data);
> +
> + regulator_register_always_on(0, "fixed-5.0V", fixed_5v0_consumers,
> + ARRAY_SIZE(fixed_5v0_consumers),
> + 5000000);
> }
>
> static void mioa701_machine_exit(void)
>
Hi Robert,
Excuse me to response late. Now it's applied.
Regards
Haojian
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2014-02-11 3:45 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-12-20 9:46 [PATCH] ARM: pxa: Add dummy backlight power supply on Mitac Mio A701 Thierry Reding
2013-12-20 9:46 ` Thierry Reding
2013-12-22 18:14 ` Robert Jarzmik
2013-12-22 18:14 ` Robert Jarzmik
2014-01-06 8:19 ` Robert Jarzmik
2014-01-06 8:19 ` Robert Jarzmik
2014-01-12 23:43 ` Robert Jarzmik
2014-01-12 23:43 ` Robert Jarzmik
2014-01-20 20:11 ` Robert Jarzmik
2014-01-20 20:11 ` Robert Jarzmik
2014-02-01 17:55 ` Robert Jarzmik
2014-02-01 17:55 ` Robert Jarzmik
2014-02-11 3:45 ` Haojian Zhuang
2014-02-11 3:45 ` Haojian Zhuang
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.