* [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight
@ 2017-09-18 15:56 Ulrich Hecht
2017-09-19 8:36 ` Laurent Pinchart
2017-09-20 11:55 ` Geert Uytterhoeven
0 siblings, 2 replies; 6+ messages in thread
From: Ulrich Hecht @ 2017-09-18 15:56 UTC (permalink / raw)
To: linux-renesas-soc, geert
Cc: linux-pwm, magnus.damm, laurent.pinchart, horms, Ulrich Hecht
This fixes the warning "pwm-backlight backlight: backlight supply power
not found, using dummy regulator".
Fixes: b33be33670217533 ("arm64: dts: salvator-x: Add panel backlight support")
Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
---
arch/arm64/boot/dts/renesas/salvator-common.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
index e3d8310..fa2a1f7 100644
--- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
+++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
@@ -63,6 +63,7 @@
brightness-levels = <256 128 64 16 8 4 0>;
default-brightness-level = <6>;
+ power-supply = <®_12v>;
enable-gpios = <&gpio6 7 GPIO_ACTIVE_HIGH>;
};
@@ -107,6 +108,15 @@
regulator-always-on;
};
+ reg_12v: regulator2 {
+ compatible = "regulator-fixed";
+ regulator-name = "fixed-12V";
+ regulator-min-microvolt = <12000000>;
+ regulator-max-microvolt = <12000000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
rsnd_ak4613: sound {
compatible = "simple-audio-card";
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight
2017-09-18 15:56 [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight Ulrich Hecht
@ 2017-09-19 8:36 ` Laurent Pinchart
2017-09-20 11:55 ` Geert Uytterhoeven
1 sibling, 0 replies; 6+ messages in thread
From: Laurent Pinchart @ 2017-09-19 8:36 UTC (permalink / raw)
To: Ulrich Hecht; +Cc: linux-renesas-soc, geert, linux-pwm, magnus.damm, horms
Hi Ulrich,
Thank you for the patch.
On Monday, 18 September 2017 18:56:14 EEST Ulrich Hecht wrote:
> This fixes the warning "pwm-backlight backlight: backlight supply power
> not found, using dummy regulator".
>
> Fixes: b33be33670217533 ("arm64: dts: salvator-x: Add panel backlight
> support")
> Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
> ---
> arch/arm64/boot/dts/renesas/salvator-common.dtsi | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> b/arch/arm64/boot/dts/renesas/salvator-common.dtsi index e3d8310..fa2a1f7
> 100644
> --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> @@ -63,6 +63,7 @@
> brightness-levels = <256 128 64 16 8 4 0>;
> default-brightness-level = <6>;
>
> + power-supply = <®_12v>;
> enable-gpios = <&gpio6 7 GPIO_ACTIVE_HIGH>;
> };
>
> @@ -107,6 +108,15 @@
> regulator-always-on;
> };
>
> + reg_12v: regulator2 {
Nitpicking, should be call this reg_12p0v to match the naming scheme of the
other fixed regulators ?
Apart from that, and regardless of whether you want to change the name or not,
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> + compatible = "regulator-fixed";
> + regulator-name = "fixed-12V";
> + regulator-min-microvolt = <12000000>;
> + regulator-max-microvolt = <12000000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> rsnd_ak4613: sound {
> compatible = "simple-audio-card";
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight
2017-09-18 15:56 [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight Ulrich Hecht
2017-09-19 8:36 ` Laurent Pinchart
@ 2017-09-20 11:55 ` Geert Uytterhoeven
2017-09-21 8:04 ` Simon Horman
1 sibling, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2017-09-20 11:55 UTC (permalink / raw)
To: Ulrich Hecht
Cc: Linux-Renesas, Linux PWM List, Magnus Damm, Laurent Pinchart,
Simon Horman
On Mon, Sep 18, 2017 at 5:56 PM, Ulrich Hecht
<ulrich.hecht+renesas@gmail.com> wrote:
> This fixes the warning "pwm-backlight backlight: backlight supply power
> not found, using dummy regulator".
>
> Fixes: b33be33670217533 ("arm64: dts: salvator-x: Add panel backlight support")
> Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight
2017-09-20 11:55 ` Geert Uytterhoeven
@ 2017-09-21 8:04 ` Simon Horman
0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2017-09-21 8:04 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Ulrich Hecht, Linux-Renesas, Linux PWM List, Magnus Damm,
Laurent Pinchart
On Wed, Sep 20, 2017 at 01:55:09PM +0200, Geert Uytterhoeven wrote:
> On Mon, Sep 18, 2017 at 5:56 PM, Ulrich Hecht
> <ulrich.hecht+renesas@gmail.com> wrote:
> > This fixes the warning "pwm-backlight backlight: backlight supply power
> > not found, using dummy regulator".
> >
> > Fixes: b33be33670217533 ("arm64: dts: salvator-x: Add panel backlight support")
> > Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks, applied as a fix for v4.14.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL] Renesas ARM Based SoC Fixes for v4.14
@ 2017-09-25 7:54 Simon Horman
2017-09-25 7:54 ` Simon Horman
0 siblings, 1 reply; 6+ messages in thread
From: Simon Horman @ 2017-09-25 7:54 UTC (permalink / raw)
To: arm
Cc: linux-renesas-soc, Olof Johansson, Kevin Hilman, Arnd Bergmann,
linux-arm-kernel, Magnus Damm, Simon Horman
Hi Olof, Hi Kevin, Hi Arnd,
Please consider these Renesas ARM based SoC fixes for v4.14.
The following changes since commit 2bd6bf03f4c1c59381d62c61d03f6cc3fe71f66e:
Linux 4.14-rc1 (2017-09-16 15:47:51 -0700)
are available in the git repository at:
https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-fixes-for-v4.14
for you to fetch changes up to 829e4887b8676255778e162efc57e4a0f18f8bb3:
arm64: dts: salvator-common: add 12V regulator to backlight (2017-09-21 10:03:51 +0200)
----------------------------------------------------------------
Renesas ARM Based SoC Fixes for v4.14
Add 12V regulator to backlight allowing the power supply
for the backlight to be found.
----------------------------------------------------------------
Ulrich Hecht (1):
arm64: dts: salvator-common: add 12V regulator to backlight
arch/arm64/boot/dts/renesas/salvator-common.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight
2017-09-25 7:54 [GIT PULL] Renesas ARM Based SoC Fixes for v4.14 Simon Horman
@ 2017-09-25 7:54 ` Simon Horman
0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2017-09-25 7:54 UTC (permalink / raw)
To: linux-renesas-soc
Cc: linux-arm-kernel, Magnus Damm, Ulrich Hecht, Simon Horman
From: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
This fixes the warning "pwm-backlight backlight: backlight supply power
not found, using dummy regulator".
Fixes: b33be33670217533 ("arm64: dts: salvator-x: Add panel backlight support")
Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
arch/arm64/boot/dts/renesas/salvator-common.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
index 4786c67b5e65..d9d885006a8e 100644
--- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
+++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
@@ -62,6 +62,7 @@
brightness-levels = <256 128 64 16 8 4 0>;
default-brightness-level = <6>;
+ power-supply = <®_12v>;
enable-gpios = <&gpio6 7 GPIO_ACTIVE_HIGH>;
};
@@ -83,6 +84,15 @@
regulator-always-on;
};
+ reg_12v: regulator2 {
+ compatible = "regulator-fixed";
+ regulator-name = "fixed-12V";
+ regulator-min-microvolt = <12000000>;
+ regulator-max-microvolt = <12000000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
rsnd_ak4613: sound {
compatible = "simple-audio-card";
--
2.1.4
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight
@ 2017-09-25 7:54 ` Simon Horman
0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2017-09-25 7:54 UTC (permalink / raw)
To: linux-arm-kernel
From: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
This fixes the warning "pwm-backlight backlight: backlight supply power
not found, using dummy regulator".
Fixes: b33be33670217533 ("arm64: dts: salvator-x: Add panel backlight support")
Reported-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
arch/arm64/boot/dts/renesas/salvator-common.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
index 4786c67b5e65..d9d885006a8e 100644
--- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
+++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
@@ -62,6 +62,7 @@
brightness-levels = <256 128 64 16 8 4 0>;
default-brightness-level = <6>;
+ power-supply = <®_12v>;
enable-gpios = <&gpio6 7 GPIO_ACTIVE_HIGH>;
};
@@ -83,6 +84,15 @@
regulator-always-on;
};
+ reg_12v: regulator2 {
+ compatible = "regulator-fixed";
+ regulator-name = "fixed-12V";
+ regulator-min-microvolt = <12000000>;
+ regulator-max-microvolt = <12000000>;
+ regulator-boot-on;
+ regulator-always-on;
+ };
+
rsnd_ak4613: sound {
compatible = "simple-audio-card";
--
2.1.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-09-25 7:54 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-18 15:56 [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight Ulrich Hecht
2017-09-19 8:36 ` Laurent Pinchart
2017-09-20 11:55 ` Geert Uytterhoeven
2017-09-21 8:04 ` Simon Horman
-- strict thread matches above, loose matches on Subject: below --
2017-09-25 7:54 [GIT PULL] Renesas ARM Based SoC Fixes for v4.14 Simon Horman
2017-09-25 7:54 ` [PATCH] arm64: dts: salvator-common: add 12V regulator to backlight Simon Horman
2017-09-25 7:54 ` Simon Horman
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.