From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1753271AbcARBMH (ORCPT <rfc822;w@1wt.eu>);
	Sun, 17 Jan 2016 20:12:07 -0500
Received: from regular1.263xmail.com ([211.150.99.140]:37753 "EHLO
	regular1.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1753065AbcARBMF (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Sun, 17 Jan 2016 20:12:05 -0500
X-263anti-spam: KSV:0;
X-MAIL-GRAY: 0
X-MAIL-DELIVERY: 1
X-KSVirus-check: 0
X-ABS-CHECKED: 4
X-ADDR-CHECKED: 0
X-MAE-MAIL_UID: 00569c3bdb05ab69@maes@b71a071c77542579
X-MAE-ID: 223.6.254.61
X-RL-SENDER: andy.yan@rock-chips.com
X-LOGIN-NAME: andy.yan@rock-chips.com
X-SENDER-IP: 121.15.173.1
Subject: Re: [PATCH v3 8/9] ARM: dts: rockchip: add reboot-mode node for
 rk3036 SoCs
To: Heiko Stuebner <heiko@sntech.de>, Caesar Wang <wxt@rock-chips.com>
References: <1452865796-23527-1-git-send-email-wxt@rock-chips.com>
 <1452865796-23527-9-git-send-email-wxt@rock-chips.com>
 <3866840.JFJvXxmQit@phil>
Cc: linux-rockchip@lists.infradead.org, leozwang@google.com,
        Mark Brown <broonie@kernel.org>, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, keescook@google.com
From: Andy Yan <andy.yan@rock-chips.com>
Message-ID: <569C3BDE.1040108@rock-chips.com>
Date: Mon, 18 Jan 2016 09:11:58 +0800
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.4.0
MIME-Version: 1.0
In-Reply-To: <3866840.JFJvXxmQit@phil>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Heiko, Caesar:

On 2016年01月18日 03:18, Heiko Stuebner wrote:
> Hi Caesar, Andy,
>
> Am Freitag, 15. Januar 2016, 21:49:55 schrieb Caesar Wang:
>> Add reboot mode driver DT node for rk3036 platform.
>> This patch adds the needed reboot mode node for rk3036.
>>
>> Signed-off-by: Caesar Wang <wxt@rock-chips.com>
> this obviously depends on Andy's reboot series.
>
> Andy, could you include this patch into further submissions of your series
> please? That way it won't keep floating unconnected on the lists :-)
>
>
> Thanks
> Heiko
>


     Okay, I will add it in my next series.
>> ---
>>
>> Changes in v3: None
>>
>>   arch/arm/boot/dts/rk3036.dtsi | 28 +++++++++++++++++++++++++++-
>>   1 file changed, 27 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/rk3036.dtsi b/arch/arm/boot/dts/rk3036.dtsi
>> index cf0968c..f5ecfff 100644
>> --- a/arch/arm/boot/dts/rk3036.dtsi
>> +++ b/arch/arm/boot/dts/rk3036.dtsi
>> @@ -43,6 +43,7 @@
>>   #include <dt-bindings/interrupt-controller/arm-gic.h>
>>   #include <dt-bindings/pinctrl/rockchip.h>
>>   #include <dt-bindings/clock/rk3036-cru.h>
>> +#include <dt-bindings/soc/rockchip_boot-mode.h>
>>   #include "skeleton.dtsi"
>>
>>   / {
>> @@ -297,8 +298,33 @@
>>   	};
>>
>>   	grf: syscon@20008000 {
>> -		compatible = "rockchip,rk3036-grf", "syscon";
>> +		compatible = "rockchip,rk3036-grf", "syscon", "simple-mfd";
>>   		reg = <0x20008000 0x1000>;
>> +
>> +		reboot-mode {
>> +			compatible = "syscon-reboot-mode";
>> +			offset = <0x1d8>;
>> +
>> +			normal {
>> +				linux,mode = "normal";
>> +				loader,magic = <BOOT_NORMAL>;
>> +			};
>> +
>> +			loader {
>> +				linux,mode = "loader";
>> +				loader,magic = <BOOT_LOADER>;
>> +			};
>> +
>> +			recovery {
>> +				linux,mode = "recovery";
>> +				loader,magic = <BOOT_RECOVERY>;
>> +			};
>> +
>> +			fastboot {
>> +				linux,mode = "fastboot";
>> +				loader,magic = <BOOT_FASTBOOT>;
>> +			};
>> +		};
>>   	};
>>
>>   	acodec: acodec-ana@20030000 {
>
>
>
>