From mboxrd@z Thu Jan 1 00:00:00 1970 From: Caesar Wang Subject: Re: [PATCH v2 06/12] mmc: pwrseq: add support for power-on sequencing through DT Date: Fri, 15 Jan 2016 17:15:21 +0800 Message-ID: <5698B8A9.9040002@gmail.com> References: <1452155155-16232-1-git-send-email-wxt@rock-chips.com> <1452155155-16232-7-git-send-email-wxt@rock-chips.com> <568FAA07.1010509@osg.samsung.com> <6960259.dpUH8KWsNS@phil> <5693D226.2000307@osg.samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <5693D226.2000307-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+glpar-linux-rockchip=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Javier Martinez Canillas Cc: Ulf Hansson , Heiko Stuebner , linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, leecam-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org, keescook-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org, NeilBrown , Srinivas Kandagatla , Alexandre Courbot , leozwang-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org, Caesar Wang List-Id: linux-mmc@vger.kernel.org SGkgSmF2aWVyLAoK5ZyoIDIwMTblubQwMeaciDEy5pelIDAwOjAyLCBKYXZpZXIgTWFydGluZXog Q2FuaWxsYXMg5YaZ6YGTOgo+IEhlbGxvIEhlaWtvLAo+Cj4gT24gMDEvMDgvMjAxNiAxMTo0MiBQ TSwgSGVpa28gU3R1ZWJuZXIgd3JvdGU6Cj4+IEFtIEZyZWl0YWcsIDguIEphbnVhciAyMDE2LCAw OToyMjozMSBzY2hyaWViIEphdmllciBNYXJ0aW5leiBDYW5pbGxhczoKPj4+PiBGb3IgZXhhbXBs ZToKPj4+PiB3ZSBuZWVkIGVuYWJsZSB3aWZpIG1vZHVsZSBwb3dlciB0byB2aWEgdGhlIFdMX1JF R19PTgo+Pj4+IHBpbiwgd2UgbmVlZCBlbmFibGUgaXQgYXMgdGhlIHJlZ3VsYXRvciBpZiB0aGlz IHBpbiBpcyBjb25uZWN0ZWQgdG8KPj4+PiB0aGUgZ3BpbyBvZiBjcHUuCj4+PiBUaGlzIHBhcnQg Y29uZnVzZXMgbWUsIHNvIGRvZXMgeW91ciBjaGlwIGhhdmUgYW4gYWN0dWFsIHJlZ3VsYXRvciB0 aGF0Cj4+PiBuZWVkcyB0byBiZSBlbmFibGVkIG9yIGlzIGp1c3QgYSBmYWtlIHJlZ3VsYXRvciB3 aG9zZSBncGlvIHByb3BlcnR5IGlzCj4+PiB1c2VkIG5vdCB0byBlbmFibGUgdGhlIHJlZ3VsYXRv ciBidXQgdG8gdG9nZ2xlIHRoZSBXTF9SRUdfT04gcGluIG9mCj4+PiB0aGUgV2lGaSBjaGlwPwo+ PiBhbm90aGVyIG9wdGlvbiB3b3VsZCBiZSB0byB1c2UgdGhlIHJlc2V0LWdwaW8taGFuZGxlcy4g cmszMjg4LXZleXJvbiBhbmQgSQo+PiB0aGluayBzb21lIEV4eW5vcyBhcyB3ZWxsIHVzZSBpdCB0 aGF0IHdheS4KPj4KPiBZZXMgSSBrbm93LCBteSBwb2ludCB3YXMgdGhhdCB0aGUgcmVzZXQtZ3Bp b3MgcHJvcGVydHkgc2hvdWxkIGJlIHVzZWQKPiBpbnN0ZWFkIG9mIGEgZmFrZSByZWd1bGF0b3Ig aWYgd2hhdCdzIG5lZWRlZCBpcyB0byB0b2dnbGUgYSBjaGlwIHBpbi4KPgo+Pj4+IE1heWJlLCBz b21lb25lIHdpbGwgc2F5IHRoYXQgY2FuIHB1bGwgdXAvZG93biBmcm9tIGR0cy4KPj4+PiBVbmZv cnR1bmF0ZWx5IHNvbWUgU29DcyBjYW4ndCBzdXBwb3J0IHBpbmN0cmwgcHVsbCB1cC9kb3duIGlu Cj4+Pj4gaW50ZXJuYWwuCj4+PiBDYW4geW91IHBsZWFzZSBlbGFib3JhdGUgb24gdGhpcz8gQUZB SVUgdGhpcyBsaW1pdGF0aW9uIGlzIHRoZSByZWFzb24KPj4+IHdoeSB5b3Ugd2VudCB3aXRoIHRo ZSByZWd1bGF0b3IgYXBwcm9hY2ggc28gSSB0aGluayBpdCBkZXNlcnZlIGEgbW9yZQo+Pj4gZGVl cCBleHBsYW5hdGlvbi4KPj4gT24gdGhlIHJrMzAzNiBlYWNoIHBpbiBoYXMgYW4gaW5kaXZpZHVh bCB1bmNoYW5nYWJsZSBwdWxsIGRpcmVjdGlvbi4gU28gaXQncwo+PiBlaXRoZXIgbm8gYmlhcyBv ciBwdWxsaW5nIGluIHRoZSBwcmVkZWZpbmVkIGRpcmVjdGlvbiAodGhlIHBpbl9kZWZhdWx0IGJp YXMKPj4gb3B0aW9uKS4KPj4KPiBJIHRoaW5rIGVhY2ggY2hhbmdlIGhhcyB0byBiZSBqdXN0aWZp ZWQgb24gaXRzIG93biBzbyBJIHdvdWxkIHNheSB0aGF0Cj4gaGF2aW5nIGEgcmVndWxhdG9yIGVu YWJsZWQgYXMgYSBwYXJ0IG9mIGEgU0RJTyBjaGlwJ3MgcG93ZXIgc2VxdWVuY2luZwo+IGlzIHNv bWV0aGluZyBuZWVkZWQgZm9yIG1hbnkgcGxhdGZvcm1zLCBhbmQgdGhhdCB0aGlzIHByb3ZpZGVy IHNob3VsZAo+IGJlIGV4dGVuZGVkIHRvIHN1cHBvcnQgdGhhdCAoc29tZXRoaW5nIGxpa2UgY29t bWl0IG1zZyBpbiBwYXRjaCAwNS8xMikuCj4KPiBBbmQgdGhlbiBpbiB0aGUga3lsaW4gRFRTIGNo YW5nZSAocGF0Y2ggMDgvMTIpLCBJIHdvdWxkIGV4cGxhaW4gd2h5IGEKPiBjaGFpbmVkIHJlZ3Vs YXRvcnMgYXBwcm9hY2ggaXMgdXNlZC9uZWVkZWQgaW5zdGVhZCBvZiB0aGUgcmVzZXQtZ3Bpb3MK PiBkdWUgYW55IHBsYXRmb3JtIGxpbWl0YXRpb25zLgoKT2theSwKSSAnbSBhZ3JlZWQgd2l0aCB5 b3VyIHBvaW50cyBpbiBoZXJlLgoKVGhlIHJlc2V0LWdwaW9zL3B3cnNxIGNhbiBtZWV0IHRoZSBk ZW1hbmQgb2Ygc29tZSB3bGFuIGNoaXBzIHRyaWdnZXIgCmNvbmRpdGlvbi4KTm8gbWF0dGVyIHdo YXRldmVyIGlzIHRoZSBCVF9FTiBvciBXTF9FTiAgdHJpZ2dlcnMgcGluLgoKPj4gSGVpa28KPj4K PiBCZXN0IHJlZ2FyZHMsCgoKLS0gClRoYW5rcywKQ2Flc2FyCgoKX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX18KTGludXgtcm9ja2NoaXAgbWFpbGluZyBsaXN0 CkxpbnV4LXJvY2tjaGlwQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVh ZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1yb2NrY2hpcAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756283AbcAOJQK (ORCPT ); Fri, 15 Jan 2016 04:16:10 -0500 Received: from mail-pf0-f193.google.com ([209.85.192.193]:35737 "EHLO mail-pf0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751112AbcAOJP2 (ORCPT ); Fri, 15 Jan 2016 04:15:28 -0500 Subject: Re: [PATCH v2 06/12] mmc: pwrseq: add support for power-on sequencing through DT To: Javier Martinez Canillas References: <1452155155-16232-1-git-send-email-wxt@rock-chips.com> <1452155155-16232-7-git-send-email-wxt@rock-chips.com> <568FAA07.1010509@osg.samsung.com> <6960259.dpUH8KWsNS@phil> <5693D226.2000307@osg.samsung.com> Cc: Heiko Stuebner , Ulf Hansson , linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, leecam@google.com, keescook@google.com, NeilBrown , Srinivas Kandagatla , Alexandre Courbot , leozwang@google.com, Caesar Wang From: Caesar Wang Message-ID: <5698B8A9.9040002@gmail.com> Date: Fri, 15 Jan 2016 17:15:21 +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: <5693D226.2000307@osg.samsung.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Javier, 在 2016年01月12日 00:02, Javier Martinez Canillas 写道: > Hello Heiko, > > On 01/08/2016 11:42 PM, Heiko Stuebner wrote: >> Am Freitag, 8. Januar 2016, 09:22:31 schrieb Javier Martinez Canillas: >>>> For example: >>>> we need enable wifi module power to via the WL_REG_ON >>>> pin, we need enable it as the regulator if this pin is connected to >>>> the gpio of cpu. >>> This part confuses me, so does your chip have an actual regulator that >>> needs to be enabled or is just a fake regulator whose gpio property is >>> used not to enable the regulator but to toggle the WL_REG_ON pin of >>> the WiFi chip? >> another option would be to use the reset-gpio-handles. rk3288-veyron and I >> think some Exynos as well use it that way. >> > Yes I know, my point was that the reset-gpios property should be used > instead of a fake regulator if what's needed is to toggle a chip pin. > >>>> Maybe, someone will say that can pull up/down from dts. >>>> Unfortunately some SoCs can't support pinctrl pull up/down in >>>> internal. >>> Can you please elaborate on this? AFAIU this limitation is the reason >>> why you went with the regulator approach so I think it deserve a more >>> deep explanation. >> On the rk3036 each pin has an individual unchangable pull direction. So it's >> either no bias or pulling in the predefined direction (the pin_default bias >> option). >> > I think each change has to be justified on its own so I would say that > having a regulator enabled as a part of a SDIO chip's power sequencing > is something needed for many platforms, and that this provider should > be extended to support that (something like commit msg in patch 05/12). > > And then in the kylin DTS change (patch 08/12), I would explain why a > chained regulators approach is used/needed instead of the reset-gpios > due any platform limitations. Okay, I 'm agreed with your points in here. The reset-gpios/pwrsq can meet the demand of some wlan chips trigger condition. No matter whatever is the BT_EN or WL_EN triggers pin. >> Heiko >> > Best regards, -- Thanks, Caesar