Re: [PATCH 0/2] mmc: pwrseq: Add support to multiple gpios in pwrseq simple

2015-01-28 Thread Srinivas Kandagatla


On 28/01/15 13:35, Ulf Hansson wrote:

On 28 January 2015 at 14:16, Srinivas Kandagatla
 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/2] mmc: pwrseq: Add support to multiple gpios in pwrseq simple

2015-01-28 Thread Ulf Hansson
On 28 January 2015 at 14:16, Srinivas Kandagatla
 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.
:-)

I am going to review Javier's patchset, would be nice if you could
have a look at it as well.

Kind regards
Uffe
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 0/2] mmc: pwrseq: Add support to multiple gpios in pwrseq simple

2015-01-28 Thread Srinivas Kandagatla
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

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html