Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-08-02 Thread Ohad Ben-Cohen
Hi Vitaly, On Mon, Aug 2, 2010 at 7:19 PM, Vitaly Wool wrote: > On Mon, Aug 2, 2010 at 5:59 PM, Ohad Ben-Cohen wrote: >> >> Check out patch no. 9: > > sorry for the confusion. However, I guess if the platform doesn't > supply this value, we'll just consider it 0? Without any warning, > right? >

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-08-02 Thread Vitaly Wool
Hi Ohad, On Mon, Aug 2, 2010 at 5:59 PM, Ohad Ben-Cohen wrote: > > Check out patch no. 9: sorry for the confusion. However, I guess if the platform doesn't supply this value, we'll just consider it 0? Without any warning, right? The problem is, we can't really distinguish between this field not

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-08-02 Thread Ohad Ben-Cohen
On Mon, Aug 2, 2010 at 6:12 PM, Vitaly Wool wrote: > Also, specifically to 12xx, I'd like to see REF_CLOCK moved on and > have this set by platform too (I haven't found anything like that in > the patches but maybe I've overlooked that). Check out patch no. 9: http://www.spinics.net/lists/arm-ke

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-08-02 Thread Vitaly Wool
On Mon, Aug 2, 2010 at 1:42 PM, Tony Lindgren wrote: > Nicolas Pitre made a comment saying he wants to look at these patches > more, are there other pending comments? Yep, I'm feeling quite uncomfortable with the way platform data is being passed over to sdio device now. Also, specifically to 1

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-08-02 Thread Ohad Ben-Cohen
On Mon, Aug 2, 2010 at 2:42 PM, Tony Lindgren wrote: > Nicolas Pitre made a comment saying he wants to look at these patches > more, are there other pending comments? I plan to post a v3 series this week with some of the comments that were already discussed after v2. Thanks everyone for the atte

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-08-02 Thread Tony Lindgren
* Luciano Coelho [100802 11:10]: > On Mon, 2010-07-26 at 21:30 +0200, ext John W. Linville wrote: > > On Wed, Jul 21, 2010 at 08:33:34PM +0300, Ohad Ben-Cohen wrote: > > > This patch series adds native support for wl1271 on ZOOM. > > > > Just for the record, I'm fine with the wl1271 bits here goi

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-08-02 Thread Luciano Coelho
On Mon, 2010-07-26 at 21:30 +0200, ext John W. Linville wrote: > On Wed, Jul 21, 2010 at 08:33:34PM +0300, Ohad Ben-Cohen wrote: > > This patch series adds native support for wl1271 on ZOOM. > > Just for the record, I'm fine with the wl1271 bits here going through > the omap tree with the rest of

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-07-27 Thread Ohad Ben-Cohen
On Mon, Jul 26, 2010 at 10:30 PM, John W. Linville wrote: > On Wed, Jul 21, 2010 at 08:33:34PM +0300, Ohad Ben-Cohen wrote: >> This patch series adds native support for wl1271 on ZOOM. > > Just for the record, I'm fine with the wl1271 bits here going through > the omap tree with the rest of the se

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-07-26 Thread John W. Linville
On Wed, Jul 21, 2010 at 08:33:34PM +0300, Ohad Ben-Cohen wrote: > This patch series adds native support for wl1271 on ZOOM. Just for the record, I'm fine with the wl1271 bits here going through the omap tree with the rest of the series. John -- John W. LinvilleSomeday the world w

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-07-22 Thread Ohad Ben-Cohen
On Fri, Jul 23, 2010 at 1:56 AM, Nicolas Pitre wrote: > On Wed, 21 Jul 2010, Ohad Ben-Cohen wrote: > >> This patch series adds native support for wl1271 on ZOOM. >> >> Changes since v1: >> >> - introduce a fixed regulator device to control the power of wl1271 >> - allow to propagate private board-

Re: [PATCH v2 00/20] native support for wl1271 on ZOOM

2010-07-22 Thread Nicolas Pitre
On Wed, 21 Jul 2010, Ohad Ben-Cohen wrote: > This patch series adds native support for wl1271 on ZOOM. > > Changes since v1: > > - introduce a fixed regulator device to control the power of wl1271 > - allow to propagate private board-specific data to SDIO function drivers > - allow SDIO function

[PATCH v2 00/20] native support for wl1271 on ZOOM

2010-07-21 Thread Ohad Ben-Cohen
This patch series adds native support for wl1271 on ZOOM. Changes since v1: - introduce a fixed regulator device to control the power of wl1271 - allow to propagate private board-specific data to SDIO function drivers - allow SDIO function driver to control the card's power via new API - make it