linux-kernel - Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <2270228.43yCmaWq86@phil>
Date:	Mon, 11 May 2015 23:57:50 +0200
From:	Heiko Stuebner <heiko@...ech.de>
To:	linux-rockchip@...ts.infradead.org
Cc:	Michael Niewöhner 
	<mniewoeh@...d.hs-offenburg.de>, linux-kernel@...r.kernel.org,
	devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	linux@....linux.org.uk
Subject: Re: [PATCH 2/2] ARM: dts: rockchip: add system-power-controller to act8846

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@...d.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>;

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/