From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753193AbbEKV5E (ORCPT ); Mon, 11 May 2015 17:57:04 -0400 Received: from gloria.sntech.de ([95.129.55.99]:51802 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752136AbbEKV5A convert rfc822-to-8bit (ORCPT ); Mon, 11 May 2015 17:57:00 -0400 From: Heiko Stuebner To: linux-rockchip@lists.infradead.org Cc: Michael =?ISO-8859-1?Q?Niew=F6hner?= , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux@arm.linux.org.uk Subject: Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846 Date: Mon, 11 May 2015 23:57:50 +0200 Message-ID: <2270228.43yCmaWq86@phil> User-Agent: KMail/4.14.1 (Linux/3.19.0+; KDE/4.14.2; x86_64; ; ) In-Reply-To: <64DE272A-FD25-406C-A51B-6C5B76DA5171@stud.hs-offenburg.de> References: <64DE272A-FD25-406C-A51B-6C5B76DA5171@stud.hs-offenburg.de> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="iso-8859-1" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 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>; From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Stuebner Subject: Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846 Date: Mon, 11 May 2015 23:57:50 +0200 Message-ID: <2270228.43yCmaWq86@phil> References: <64DE272A-FD25-406C-A51B-6C5B76DA5171@stud.hs-offenburg.de> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <64DE272A-FD25-406C-A51B-6C5B76DA5171-0BV18Q16miHGpFsiFCTsGXqWYbMAw+HU@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org Cc: Michael =?ISO-8859-1?Q?Niew=F6hner?= , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org List-Id: devicetree@vger.kernel.org Am Montag, 11. Mai 2015, 22:57:21 schrieb Michael Niew=F6hner: > Add system-power-controller to act8846 in rk3188-radxarock.dts in add= ition > to the act8846 patch fixing the reboot kernel panic >=20 > Signed-off-by: Michael Niewoehner I have to ask, are you ok with the pending dts relicensing to a gpl2/x1= 1 dual=20 license [0]? Otherwise this looks fine and the act8846 is the power-controller of th= e=20 radxarock in any case, so I'll take this one in any case. Heiko [0] http://lists.infradead.org/pipermail/linux-rockchip/2015-March/0026= 44.html > --- > arch/arm/boot/dts/rk3188-radxarock.dts | 1 + > 1 file changed, 1 insertion(+) >=20 > diff --git a/arch/arm/boot/dts/rk3188-radxarock.dts > b/arch/arm/boot/dts/rk3188-radxarock.dts index bdf8570..225a0eb 10064= 4 > --- a/arch/arm/boot/dts/rk3188-radxarock.dts > +++ b/arch/arm/boot/dts/rk3188-radxarock.dts > @@ -152,6 +152,7 @@ > compatible =3D "active-semi,act8846"; > reg =3D <0x5a>; > status =3D "okay"; > + system-power-controller; >=20 > pinctrl-names =3D "default"; > pinctrl-0 =3D <&act8846_dvs0_ctl>; -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: heiko@sntech.de (Heiko Stuebner) Date: Mon, 11 May 2015 23:57:50 +0200 Subject: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846 In-Reply-To: <64DE272A-FD25-406C-A51B-6C5B76DA5171@stud.hs-offenburg.de> References: <64DE272A-FD25-406C-A51B-6C5B76DA5171@stud.hs-offenburg.de> Message-ID: <2270228.43yCmaWq86@phil> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 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>;