Re: [PATCH v4 3/3] ARM: dts: rockchip: Add the OTP gpio pinctrl

From: Caesar Wang <caesar.upstream@gmail.com>
To: "Heiko Stübner" <heiko@sntech.de>
Cc: Doug Anderson <dianders@chromium.org>,
	Mark Rutland <mark.rutland@arm.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Russell King <linux@arm.linux.org.uk>,
	Pawel Moll <pawel.moll@arm.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Eduardo Valentin <edubezval@gmail.com>,
	"open list:ARM/Rockchip SoC..."
	<linux-rockchip@lists.infradead.org>,
	Rob Herring <robh+dt@kernel.org>,
	Kumar Gala <galak@codeaurora.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	Caesar Wang <wxt@rock-chips.com>
Subject: Re: [PATCH v4 3/3] ARM: dts: rockchip: Add the OTP gpio pinctrl
Date: Thu, 19 Nov 2015 10:44:02 +0800	[thread overview]
Message-ID: <564D3772.9050402@gmail.com> (raw)
In-Reply-To: <2741625.HAsG2fX6yg@diego>

Hi Heiko,

在 2015年10月25日 15:55, Heiko Stübner 写道:
> Am Freitag, 23. Oktober 2015, 08:25:08 schrieb Doug Anderson:
>> On Fri, Oct 23, 2015 at 4:25 AM, Caesar Wang <wxt@rock-chips.com> wrote:
>>> Add the "init" anf "sleep" pinctrl as the OTP gpio state.
>>> We need the OTP pin is gpio state before resetting the TSADC controller,
>>> since the tshut polarity will generate a high signal.
>>>
>>> "init" pinctrl property is defined by Doug's Patch[0].
>>>
>>> Patch[0]:
>>> https://patchwork.kernel.org/patch/7454311/
>>>
>>> Signed-off-by: Caesar Wang <wxt@rock-chips.com>
>>> Reviewed-by: Douglas Anderson <dianders@chromium.org>
>>> ---
>>>
>>> Changes in v4: None
>>>
>>> Changes in v3:
>>>    - Add the "sleep" pinctrl as the gpio state in PATCH[3/3]
>>>
>>> Changes in v2:
>>>    - Add some commits for more obvious in PATCH[2/2]
>>>
>>> Changes in v1:
>>>    - As the Doug comments, drop the thermal driver patchs since
>>>    
>>>      we can with pinctrl changing to work.
>>>    
>>>    - As the Doug's patch to add the 'init' property.
>>>   
>>>   arch/arm/boot/dts/rk3288.dtsi | 10 ++++++++--
>>>   1 file changed, 8 insertions(+), 2 deletions(-)
>> I realized that the subject of this patch should probably contain the
>> word rk3288, but I presume Heiko would rather add that himself than
>> for you to spin this again.  ;)
> yep :-) ... no need to respin for such an easy change

That's seem this patch didn't merge into your v4.5-armsoc/soc branch.:-)

I guess this patch should merge into  kernel-4.4 since the Patch[1/3] / 
[2/3] have been merged into 4.4-rc1.:-P

>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip

-- 
Thanks,
Caesar