* [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846
@ 2015-05-11 20:57 ` Michael Niewöhner
0 siblings, 0 replies; 12+ messages in thread
From: Michael Niewöhner @ 2015-05-11 20:57 UTC (permalink / raw)
To: linux-kernel, devicetree, linux-rockchip, linux-arm-kernel, linux, heiko
Add system-power-controller to act8846 in rk3188-radxarock.dts in addition to
the act8846 patch fixing the reboot kernel panic
Signed-off-by: Michael Niewoehner <mniewoeh@stud.hs-offenburg.de>
---
arch/arm/boot/dts/rk3188-radxarock.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts b/arch/arm/boot/dts/rk3188-radxarock.dts
index bdf8570..225a0eb 100644
--- a/arch/arm/boot/dts/rk3188-radxarock.dts
+++ b/arch/arm/boot/dts/rk3188-radxarock.dts
@@ -152,6 +152,7 @@
compatible = "active-semi,act8846";
reg = <0x5a>;
status = "okay";
+ system-power-controller;
pinctrl-names = "default";
pinctrl-0 = <&act8846_dvs0_ctl>;
--
2.3.5
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846
@ 2015-05-11 20:57 ` Michael Niewöhner
0 siblings, 0 replies; 12+ messages in thread
From: Michael Niewöhner @ 2015-05-11 20:57 UTC (permalink / raw)
To: linux-arm-kernel
Add system-power-controller to act8846 in rk3188-radxarock.dts in addition to
the act8846 patch fixing the reboot kernel panic
Signed-off-by: Michael Niewoehner <mniewoeh@stud.hs-offenburg.de>
---
arch/arm/boot/dts/rk3188-radxarock.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts b/arch/arm/boot/dts/rk3188-radxarock.dts
index bdf8570..225a0eb 100644
--- a/arch/arm/boot/dts/rk3188-radxarock.dts
+++ b/arch/arm/boot/dts/rk3188-radxarock.dts
@@ -152,6 +152,7 @@
compatible = "active-semi,act8846";
reg = <0x5a>;
status = "okay";
+ system-power-controller;
pinctrl-names = "default";
pinctrl-0 = <&act8846_dvs0_ctl>;
--
2.3.5
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846
@ 2015-05-11 20:57 ` Michael Niewöhner
0 siblings, 0 replies; 12+ messages in thread
From: Michael Niewöhner @ 2015-05-11 20:57 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-lFZ/pmaqli7XmaaqVzeoHQ, heiko-4mtYJXux2i+zQB+pC5nmwQ
Add system-power-controller to act8846 in rk3188-radxarock.dts in addition to
the act8846 patch fixing the reboot kernel panic
Signed-off-by: Michael Niewoehner <mniewoeh-0BV18Q16miHGpFsiFCTsGXqWYbMAw+HU@public.gmane.org>
---
arch/arm/boot/dts/rk3188-radxarock.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts b/arch/arm/boot/dts/rk3188-radxarock.dts
index bdf8570..225a0eb 100644
--- a/arch/arm/boot/dts/rk3188-radxarock.dts
+++ b/arch/arm/boot/dts/rk3188-radxarock.dts
@@ -152,6 +152,7 @@
compatible = "active-semi,act8846";
reg = <0x5a>;
status = "okay";
+ system-power-controller;
pinctrl-names = "default";
pinctrl-0 = <&act8846_dvs0_ctl>;
--
2.3.5
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846
@ 2015-05-11 21:57 ` Heiko Stuebner
0 siblings, 0 replies; 12+ messages in thread
From: Heiko Stuebner @ 2015-05-11 21:57 UTC (permalink / raw)
To: linux-rockchip
Cc: Michael Niewöhner, linux-kernel, devicetree,
linux-arm-kernel, linux
Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niewöhner:
> Add system-power-controller to act8846 in rk3188-radxarock.dts in addition
> to the act8846 patch fixing the reboot kernel panic
>
> Signed-off-by: Michael Niewoehner <mniewoeh@stud.hs-offenburg.de>
I have to ask, are you ok with the pending dts relicensing to a gpl2/x11 dual
license [0]?
Otherwise this looks fine and the act8846 is the power-controller of the
radxarock in any case, so I'll take this one in any case.
Heiko
[0] http://lists.infradead.org/pipermail/linux-rockchip/2015-March/002644.html
> ---
> arch/arm/boot/dts/rk3188-radxarock.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts
> b/arch/arm/boot/dts/rk3188-radxarock.dts index bdf8570..225a0eb 100644
> --- a/arch/arm/boot/dts/rk3188-radxarock.dts
> +++ b/arch/arm/boot/dts/rk3188-radxarock.dts
> @@ -152,6 +152,7 @@
> compatible = "active-semi,act8846";
> reg = <0x5a>;
> status = "okay";
> + system-power-controller;
>
> pinctrl-names = "default";
> pinctrl-0 = <&act8846_dvs0_ctl>;
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846
@ 2015-05-11 21:57 ` Heiko Stuebner
0 siblings, 0 replies; 12+ messages in thread
From: Heiko Stuebner @ 2015-05-11 21:57 UTC (permalink / raw)
To: linux-arm-kernel
Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niew?hner:
> Add system-power-controller to act8846 in rk3188-radxarock.dts in addition
> to the act8846 patch fixing the reboot kernel panic
>
> Signed-off-by: Michael Niewoehner <mniewoeh@stud.hs-offenburg.de>
I have to ask, are you ok with the pending dts relicensing to a gpl2/x11 dual
license [0]?
Otherwise this looks fine and the act8846 is the power-controller of the
radxarock in any case, so I'll take this one in any case.
Heiko
[0] http://lists.infradead.org/pipermail/linux-rockchip/2015-March/002644.html
> ---
> arch/arm/boot/dts/rk3188-radxarock.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts
> b/arch/arm/boot/dts/rk3188-radxarock.dts index bdf8570..225a0eb 100644
> --- a/arch/arm/boot/dts/rk3188-radxarock.dts
> +++ b/arch/arm/boot/dts/rk3188-radxarock.dts
> @@ -152,6 +152,7 @@
> compatible = "active-semi,act8846";
> reg = <0x5a>;
> status = "okay";
> + system-power-controller;
>
> pinctrl-names = "default";
> pinctrl-0 = <&act8846_dvs0_ctl>;
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846
@ 2015-05-11 21:57 ` Heiko Stuebner
0 siblings, 0 replies; 12+ messages in thread
From: Heiko Stuebner @ 2015-05-11 21:57 UTC (permalink / raw)
To: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Cc: Michael Niewöhner, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-lFZ/pmaqli7XmaaqVzeoHQ
Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niewöhner:
> Add system-power-controller to act8846 in rk3188-radxarock.dts in addition
> to the act8846 patch fixing the reboot kernel panic
>
> Signed-off-by: Michael Niewoehner <mniewoeh-0BV18Q16miHGpFsiFCTsGXqWYbMAw+HU@public.gmane.org>
I have to ask, are you ok with the pending dts relicensing to a gpl2/x11 dual
license [0]?
Otherwise this looks fine and the act8846 is the power-controller of the
radxarock in any case, so I'll take this one in any case.
Heiko
[0] http://lists.infradead.org/pipermail/linux-rockchip/2015-March/002644.html
> ---
> arch/arm/boot/dts/rk3188-radxarock.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts
> b/arch/arm/boot/dts/rk3188-radxarock.dts index bdf8570..225a0eb 100644
> --- a/arch/arm/boot/dts/rk3188-radxarock.dts
> +++ b/arch/arm/boot/dts/rk3188-radxarock.dts
> @@ -152,6 +152,7 @@
> compatible = "active-semi,act8846";
> reg = <0x5a>;
> status = "okay";
> + system-power-controller;
>
> pinctrl-names = "default";
> pinctrl-0 = <&act8846_dvs0_ctl>;
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller toact8846
2015-05-11 21:57 ` Heiko Stuebner
(?)
@ 2015-05-12 7:22 ` Michael Niewöhner
-1 siblings, 0 replies; 12+ messages in thread
From: Michael Niewöhner @ 2015-05-12 7:22 UTC (permalink / raw)
To: heiko; +Cc: linux, linux-arm-kernel, linux-rockchip, devicetree, linux-kernel
I agree on relicensing to gpl2/x11.
Michael
> Am 11.05.2015 um 23:58 schrieb Heiko Stuebner <heiko@sntech.de>:
>
> Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niewöhner:
>> Add system-power-controller to act8846 in rk3188-radxarock.dts in
addition
>> to the act8846 patch fixing the reboot kernel panic
>>
>> Signed-off-by: Michael Niewoehner <mniewoeh@stud.hs-offenburg.de>
>
> I have to ask, are you ok with the pending dts relicensing to a
gpl2/x11 dual
> license [0]?
>
> Otherwise this looks fine and the act8846 is the power-controller of
the
> radxarock in any case, so I'll take this one in any case.
>
>
> Heiko
>
> [0]
http://lists.infradead.org/pipermail/linux-rockchip/2015-March/002644.html
>
>
>> ---
>> arch/arm/boot/dts/rk3188-radxarock.dts | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts
>> b/arch/arm/boot/dts/rk3188-radxarock.dts index bdf8570..225a0eb
100644
>> --- a/arch/arm/boot/dts/rk3188-radxarock.dts
>> +++ b/arch/arm/boot/dts/rk3188-radxarock.dts
>> @@ -152,6 +152,7 @@
>> compatible = "active-semi,act8846";
>> reg = <0x5a>;
>> status = "okay";
>> + system-power-controller;
>>
>> pinctrl-names = "default";
>> pinctrl-0 = <&act8846_dvs0_ctl>;
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 2/2] ARM: dts: rockchip: add system-power-controller toact8846
@ 2015-05-12 7:22 ` Michael Niewöhner
0 siblings, 0 replies; 12+ messages in thread
From: Michael Niewöhner @ 2015-05-12 7:22 UTC (permalink / raw)
To: linux-arm-kernel
I agree on relicensing to gpl2/x11.
Michael
> Am 11.05.2015 um 23:58 schrieb Heiko Stuebner <heiko@sntech.de>:
>
> Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niew?hner:
>> Add system-power-controller to act8846 in rk3188-radxarock.dts in
addition
>> to the act8846 patch fixing the reboot kernel panic
>>
>> Signed-off-by: Michael Niewoehner <mniewoeh@stud.hs-offenburg.de>
>
> I have to ask, are you ok with the pending dts relicensing to a
gpl2/x11 dual
> license [0]?
>
> Otherwise this looks fine and the act8846 is the power-controller of
the
> radxarock in any case, so I'll take this one in any case.
>
>
> Heiko
>
> [0]
http://lists.infradead.org/pipermail/linux-rockchip/2015-March/002644.html
>
>
>> ---
>> arch/arm/boot/dts/rk3188-radxarock.dts | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts
>> b/arch/arm/boot/dts/rk3188-radxarock.dts index bdf8570..225a0eb
100644
>> --- a/arch/arm/boot/dts/rk3188-radxarock.dts
>> +++ b/arch/arm/boot/dts/rk3188-radxarock.dts
>> @@ -152,6 +152,7 @@
>> compatible = "active-semi,act8846";
>> reg = <0x5a>;
>> status = "okay";
>> + system-power-controller;
>>
>> pinctrl-names = "default";
>> pinctrl-0 = <&act8846_dvs0_ctl>;
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller toact8846
@ 2015-05-12 7:22 ` Michael Niewöhner
0 siblings, 0 replies; 12+ messages in thread
From: Michael Niewöhner @ 2015-05-12 7:22 UTC (permalink / raw)
To: heiko-4mtYJXux2i+zQB+pC5nmwQ
Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-lFZ/pmaqli7XmaaqVzeoHQ,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA
I agree on relicensing to gpl2/x11.
Michael
> Am 11.05.2015 um 23:58 schrieb Heiko Stuebner <heiko@sntech.de>:
>
> Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niewöhner:
>> Add system-power-controller to act8846 in rk3188-radxarock.dts in
addition
>> to the act8846 patch fixing the reboot kernel panic
>>
>> Signed-off-by: Michael Niewoehner <mniewoeh@stud.hs-offenburg.de>
>
> I have to ask, are you ok with the pending dts relicensing to a
gpl2/x11 dual
> license [0]?
>
> Otherwise this looks fine and the act8846 is the power-controller of
the
> radxarock in any case, so I'll take this one in any case.
>
>
> Heiko
>
> [0]
http://lists.infradead.org/pipermail/linux-rockchip/2015-March/002644.html
>
>
>> ---
>> arch/arm/boot/dts/rk3188-radxarock.dts | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts
>> b/arch/arm/boot/dts/rk3188-radxarock.dts index bdf8570..225a0eb
100644
>> --- a/arch/arm/boot/dts/rk3188-radxarock.dts
>> +++ b/arch/arm/boot/dts/rk3188-radxarock.dts
>> @@ -152,6 +152,7 @@
>> compatible = "active-semi,act8846";
>> reg = <0x5a>;
>> status = "okay";
>> + system-power-controller;
>>
>> pinctrl-names = "default";
>> pinctrl-0 = <&act8846_dvs0_ctl>;
>
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846
@ 2015-05-12 20:44 ` Heiko Stuebner
0 siblings, 0 replies; 12+ messages in thread
From: Heiko Stuebner @ 2015-05-12 20:44 UTC (permalink / raw)
To: Michael Niewöhner
Cc: linux-kernel, devicetree, linux-rockchip, linux-arm-kernel, linux
Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niewöhner:
> Add system-power-controller to act8846 in rk3188-radxarock.dts in addition
> to the act8846 patch fixing the reboot kernel panic
>
> Signed-off-by: Michael Niewoehner <mniewoeh@stud.hs-offenburg.de>
As the act8846 is the system-power-controller in any case (for example to
powerdown the board) I've applied this patch to my dts branch for 4.2 [0],
after adapting the commit message slightly to make it a bit more generic:
----
ARM: dts: rockchip: add system-power-controller to act8846 on radxarock
The act8846 is the main pmic and system-power-controller on radxarock boards,
so add the necessary property.
----
Hope that is ok with you :-)
Heiko
[0] https://git.kernel.org/cgit/linux/kernel/git/mmind/linux-rockchip.git/commit/?h=v4.2-armsoc/dts&id=9ab79be0da98e371f1d2f9c01ac25745639718d1
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846
@ 2015-05-12 20:44 ` Heiko Stuebner
0 siblings, 0 replies; 12+ messages in thread
From: Heiko Stuebner @ 2015-05-12 20:44 UTC (permalink / raw)
To: linux-arm-kernel
Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niew?hner:
> Add system-power-controller to act8846 in rk3188-radxarock.dts in addition
> to the act8846 patch fixing the reboot kernel panic
>
> Signed-off-by: Michael Niewoehner <mniewoeh@stud.hs-offenburg.de>
As the act8846 is the system-power-controller in any case (for example to
powerdown the board) I've applied this patch to my dts branch for 4.2 [0],
after adapting the commit message slightly to make it a bit more generic:
----
ARM: dts: rockchip: add system-power-controller to act8846 on radxarock
The act8846 is the main pmic and system-power-controller on radxarock boards,
so add the necessary property.
----
Hope that is ok with you :-)
Heiko
[0] https://git.kernel.org/cgit/linux/kernel/git/mmind/linux-rockchip.git/commit/?h=v4.2-armsoc/dts&id=9ab79be0da98e371f1d2f9c01ac25745639718d1
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846
@ 2015-05-12 20:44 ` Heiko Stuebner
0 siblings, 0 replies; 12+ messages in thread
From: Heiko Stuebner @ 2015-05-12 20:44 UTC (permalink / raw)
To: Michael Niewöhner
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-lFZ/pmaqli7XmaaqVzeoHQ
Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niewöhner:
> Add system-power-controller to act8846 in rk3188-radxarock.dts in addition
> to the act8846 patch fixing the reboot kernel panic
>
> Signed-off-by: Michael Niewoehner <mniewoeh-0BV18Q16miHGpFsiFCTsGXqWYbMAw+HU@public.gmane.org>
As the act8846 is the system-power-controller in any case (for example to
powerdown the board) I've applied this patch to my dts branch for 4.2 [0],
after adapting the commit message slightly to make it a bit more generic:
----
ARM: dts: rockchip: add system-power-controller to act8846 on radxarock
The act8846 is the main pmic and system-power-controller on radxarock boards,
so add the necessary property.
----
Hope that is ok with you :-)
Heiko
[0] https://git.kernel.org/cgit/linux/kernel/git/mmind/linux-rockchip.git/commit/?h=v4.2-armsoc/dts&id=9ab79be0da98e371f1d2f9c01ac25745639718d1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2015-05-12 20:44 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-11 20:57 [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846 Michael Niewöhner
2015-05-11 20:57 ` Michael Niewöhner
2015-05-11 20:57 ` Michael Niewöhner
2015-05-11 21:57 ` Heiko Stuebner
2015-05-11 21:57 ` Heiko Stuebner
2015-05-11 21:57 ` Heiko Stuebner
2015-05-12 7:22 ` [PATCH 2/2] ARM: dts: rockchip: add system-power-controller toact8846 Michael Niewöhner
2015-05-12 7:22 ` Michael Niewöhner
2015-05-12 7:22 ` Michael Niewöhner
2015-05-12 20:44 ` [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846 Heiko Stuebner
2015-05-12 20:44 ` Heiko Stuebner
2015-05-12 20:44 ` Heiko Stuebner
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.