From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
To: Ulf Hansson <ulf.hansson@linaro.org>
Cc: linux-mmc <linux-mmc@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
Kumar Gala <galak@codeaurora.org>,
Rob Herring <robh+dt@kernel.org>
Subject: Re: [PATCH 0/2] mmc: pwrseq: Add support to multiple gpios in pwrseq simple
Date: Wed, 28 Jan 2015 13:46:47 +0000 [thread overview]
Message-ID: <54C8E847.10506@linaro.org> (raw)
In-Reply-To: <CAPDyKFquQgut1ZC9Jj=rbNwRKs3Qwr5LsUy_cmY7g2L8MG657Q@mail.gmail.com>
On 28/01/15 13:35, Ulf Hansson wrote:
> On 28 January 2015 at 14:16, Srinivas Kandagatla
> <srinivas.kandagatla@linaro.org> wrote:
>> This patchset adds support to reset/powerup multiple gpio pins on a given
>> sdio bus. The use case is simple, on sdio we could have multiple devices
>> like WLAN, BT which are controlled by there own reset lines. So having
>> multiple reset is something more useful in such cases.
>>
>> Without this patch I could not get BT and WLAN working at same time on IFC6410.
>>
>> Thanks,
>> srini
>>
>> Srinivas Kandagatla (2):
>> mmc: pwrseq: Add support to control multiple gpios in simple pwrseq
>> mmc: pwrseq: Update document with multiple gpios support
>>
>> .../devicetree/bindings/mmc/mmc-pwrseq-simple.txt | 7 ++-
>> drivers/mmc/core/pwrseq_simple.c | 64 +++++++++++++++-------
>> 2 files changed, 49 insertions(+), 22 deletions(-)
>>
>> --
>> 1.9.1
>>
>
> Hi Srinivas,
>
> I noticed that Javier a few hours ago posted a patchset which
> implement the same thing as yours, I guess this is a wanted feature.
> :-)
yes.
>
> I am going to review Javier's patchset, would be nice if you could
> have a look at it as well.
Sure.. they look very similar..
!!! Should have sent my patches yesterday :-)
>
> Kind regards
> Uffe
>
WARNING: multiple messages have this Message-ID (diff)
From: Srinivas Kandagatla <srinivas.kandagatla-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
To: Ulf Hansson <ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: linux-mmc <linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Subject: Re: [PATCH 0/2] mmc: pwrseq: Add support to multiple gpios in pwrseq simple
Date: Wed, 28 Jan 2015 13:46:47 +0000 [thread overview]
Message-ID: <54C8E847.10506@linaro.org> (raw)
In-Reply-To: <CAPDyKFquQgut1ZC9Jj=rbNwRKs3Qwr5LsUy_cmY7g2L8MG657Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
On 28/01/15 13:35, Ulf Hansson wrote:
> On 28 January 2015 at 14:16, Srinivas Kandagatla
> <srinivas.kandagatla-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote:
>> This patchset adds support to reset/powerup multiple gpio pins on a given
>> sdio bus. The use case is simple, on sdio we could have multiple devices
>> like WLAN, BT which are controlled by there own reset lines. So having
>> multiple reset is something more useful in such cases.
>>
>> Without this patch I could not get BT and WLAN working at same time on IFC6410.
>>
>> Thanks,
>> srini
>>
>> Srinivas Kandagatla (2):
>> mmc: pwrseq: Add support to control multiple gpios in simple pwrseq
>> mmc: pwrseq: Update document with multiple gpios support
>>
>> .../devicetree/bindings/mmc/mmc-pwrseq-simple.txt | 7 ++-
>> drivers/mmc/core/pwrseq_simple.c | 64 +++++++++++++++-------
>> 2 files changed, 49 insertions(+), 22 deletions(-)
>>
>> --
>> 1.9.1
>>
>
> Hi Srinivas,
>
> I noticed that Javier a few hours ago posted a patchset which
> implement the same thing as yours, I guess this is a wanted feature.
> :-)
yes.
>
> I am going to review Javier's patchset, would be nice if you could
> have a look at it as well.
Sure.. they look very similar..
!!! Should have sent my patches yesterday :-)
>
> Kind regards
> Uffe
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-01-29 1:08 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-28 13:16 [PATCH 0/2] mmc: pwrseq: Add support to multiple gpios in pwrseq simple Srinivas Kandagatla
2015-01-28 13:16 ` Srinivas Kandagatla
2015-01-28 13:16 ` [PATCH 1/2] mmc: pwrseq: Add support to control multiple gpios in simple pwrseq Srinivas Kandagatla
2015-01-28 13:17 ` [PATCH 2/2] mmc: pwrseq: Update document with multiple gpios support Srinivas Kandagatla
2015-01-28 13:35 ` [PATCH 0/2] mmc: pwrseq: Add support to multiple gpios in pwrseq simple Ulf Hansson
2015-01-28 13:35 ` Ulf Hansson
2015-01-28 13:46 ` Srinivas Kandagatla [this message]
2015-01-28 13:46 ` Srinivas Kandagatla
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54C8E847.10506@linaro.org \
--to=srinivas.kandagatla@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=ulf.hansson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.