* [PATCH v6 0/7] native wl1271 support on ZOOM
@ 2010-09-15 23:22 Ohad Ben-Cohen
2010-09-15 23:22 ` [PATCH v6 1/7] wl12xx: make wl12xx.h common to both spi and sdio Ohad Ben-Cohen
[not found] ` <1284592929-29616-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>
0 siblings, 2 replies; 14+ messages in thread
From: Ohad Ben-Cohen @ 2010-09-15 23:22 UTC (permalink / raw)
To: linux-wireless, linux-omap
Cc: John W. Linville, Mark Brown, linux-arm-kernel,
Chikkature Rajashekar Madhusudhan, Luciano Coelho, San Mehat,
Roger Quadros, Tony Lindgren, Nicolas Pitre, Ido Yariv,
Kalle Valo, Russell King, Vitaly Wool, Ohad Ben-Cohen
This patchset adds wl1271 support on ZOOM2/3 boards.
Only basic support is included; power manipulation is submitted separately as SDIO Runtime PM support.
Changes since v5:
- rebase to wireless-testing
- remove 2nd copy from wl12xx_platform_data (keep the 1st one though)
Thanks,
Ohad Ben-Cohen (7):
wl12xx: make wl12xx.h common to both spi and sdio
wl1271: propagate set_power's return value
wl12xx: add platform data passing support
wl1271: take irq info from private board data
wl1271: make ref_clock configurable by board
omap: zoom: add fixed regulator device for wlan
omap: zoom: add mmc3/wl1271 device support
arch/arm/mach-omap2/board-zoom-peripherals.c | 54 ++++++++++++++++++++
drivers/net/wireless/Makefile | 2 +
drivers/net/wireless/wl12xx/Kconfig | 5 ++-
drivers/net/wireless/wl12xx/wl1251_sdio.c | 2 +-
drivers/net/wireless/wl12xx/wl1251_spi.c | 2 +-
drivers/net/wireless/wl12xx/wl1271.h | 3 +-
drivers/net/wireless/wl12xx/wl1271_boot.c | 11 +++--
drivers/net/wireless/wl12xx/wl1271_boot.h | 1 -
drivers/net/wireless/wl12xx/wl1271_io.h | 9 ++-
drivers/net/wireless/wl12xx/wl1271_main.c | 4 +-
drivers/net/wireless/wl12xx/wl1271_sdio.c | 31 +++++++-----
drivers/net/wireless/wl12xx/wl1271_spi.c | 8 ++-
drivers/net/wireless/wl12xx/wl12xx_platform_data.c | 28 ++++++++++
include/linux/spi/wl12xx.h | 34 ------------
include/linux/wl12xx.h | 38 ++++++++++++++
15 files changed, 170 insertions(+), 62 deletions(-)
create mode 100644 drivers/net/wireless/wl12xx/wl12xx_platform_data.c
delete mode 100644 include/linux/spi/wl12xx.h
create mode 100644 include/linux/wl12xx.h
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH v6 1/7] wl12xx: make wl12xx.h common to both spi and sdio 2010-09-15 23:22 [PATCH v6 0/7] native wl1271 support on ZOOM Ohad Ben-Cohen @ 2010-09-15 23:22 ` Ohad Ben-Cohen 2010-09-16 10:17 ` Teemu Paasikivi [not found] ` <1284592929-29616-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> 1 sibling, 1 reply; 14+ messages in thread From: Ohad Ben-Cohen @ 2010-09-15 23:22 UTC (permalink / raw) To: linux-wireless, linux-omap Cc: John W. Linville, Mark Brown, linux-arm-kernel, Chikkature Rajashekar Madhusudhan, Luciano Coelho, San Mehat, Roger Quadros, Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool, Ohad Ben-Cohen Move wl12xx.h outside of the spi-specific location, so it can be shared with both spi and sdio solutions. Signed-off-by: Ohad Ben-Cohen <ohad@wizery.com> --- drivers/net/wireless/wl12xx/wl1251_sdio.c | 2 +- drivers/net/wireless/wl12xx/wl1251_spi.c | 2 +- drivers/net/wireless/wl12xx/wl1271_spi.c | 2 +- include/linux/spi/wl12xx.h | 34 ----------------------------- include/linux/wl12xx.h | 34 +++++++++++++++++++++++++++++ 5 files changed, 37 insertions(+), 37 deletions(-) delete mode 100644 include/linux/spi/wl12xx.h create mode 100644 include/linux/wl12xx.h diff --git a/drivers/net/wireless/wl12xx/wl1251_sdio.c b/drivers/net/wireless/wl12xx/wl1251_sdio.c index c0b68b0..74ba9ce 100644 --- a/drivers/net/wireless/wl12xx/wl1251_sdio.c +++ b/drivers/net/wireless/wl12xx/wl1251_sdio.c @@ -24,7 +24,7 @@ #include <linux/mmc/sdio_func.h> #include <linux/mmc/sdio_ids.h> #include <linux/platform_device.h> -#include <linux/spi/wl12xx.h> +#include <linux/wl12xx.h> #include <linux/irq.h> #include "wl1251.h" diff --git a/drivers/net/wireless/wl12xx/wl1251_spi.c b/drivers/net/wireless/wl12xx/wl1251_spi.c index 334ded9..320de79 100644 --- a/drivers/net/wireless/wl12xx/wl1251_spi.c +++ b/drivers/net/wireless/wl12xx/wl1251_spi.c @@ -24,7 +24,7 @@ #include <linux/slab.h> #include <linux/crc7.h> #include <linux/spi/spi.h> -#include <linux/spi/wl12xx.h> +#include <linux/wl12xx.h> #include "wl1251.h" #include "wl1251_reg.h" diff --git a/drivers/net/wireless/wl12xx/wl1271_spi.c b/drivers/net/wireless/wl12xx/wl1271_spi.c index 4cb99c5..c3fdab7 100644 --- a/drivers/net/wireless/wl12xx/wl1271_spi.c +++ b/drivers/net/wireless/wl12xx/wl1271_spi.c @@ -25,7 +25,7 @@ #include <linux/module.h> #include <linux/crc7.h> #include <linux/spi/spi.h> -#include <linux/spi/wl12xx.h> +#include <linux/wl12xx.h> #include <linux/slab.h> #include "wl1271.h" diff --git a/include/linux/spi/wl12xx.h b/include/linux/spi/wl12xx.h deleted file mode 100644 index a20bccf..0000000 --- a/include/linux/spi/wl12xx.h +++ /dev/null @@ -1,34 +0,0 @@ -/* - * This file is part of wl12xx - * - * Copyright (C) 2009 Nokia Corporation - * - * Contact: Luciano Coelho <luciano.coelho@nokia.com> - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU General Public License - * version 2 as published by the Free Software Foundation. - * - * This program is distributed in the hope that it will be useful, but - * WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA - * 02110-1301 USA - * - */ - -#ifndef _LINUX_SPI_WL12XX_H -#define _LINUX_SPI_WL12XX_H - -struct wl12xx_platform_data { - void (*set_power)(bool enable); - /* SDIO only: IRQ number if WLAN_IRQ line is used, 0 for SDIO IRQs */ - int irq; - bool use_eeprom; -}; - -#endif diff --git a/include/linux/wl12xx.h b/include/linux/wl12xx.h new file mode 100644 index 0000000..015687a --- /dev/null +++ b/include/linux/wl12xx.h @@ -0,0 +1,34 @@ +/* + * This file is part of wl12xx + * + * Copyright (C) 2009 Nokia Corporation + * + * Contact: Luciano Coelho <luciano.coelho@nokia.com> + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * version 2 as published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA + * 02110-1301 USA + * + */ + +#ifndef _LINUX_WL12XX_H +#define _LINUX_WL12XX_H + +struct wl12xx_platform_data { + void (*set_power)(bool enable); + /* SDIO only: IRQ number if WLAN_IRQ line is used, 0 for SDIO IRQs */ + int irq; + bool use_eeprom; +}; + +#endif -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v6 1/7] wl12xx: make wl12xx.h common to both spi and sdio 2010-09-15 23:22 ` [PATCH v6 1/7] wl12xx: make wl12xx.h common to both spi and sdio Ohad Ben-Cohen @ 2010-09-16 10:17 ` Teemu Paasikivi 2010-09-16 10:45 ` Ohad Ben-Cohen 0 siblings, 1 reply; 14+ messages in thread From: Teemu Paasikivi @ 2010-09-16 10:17 UTC (permalink / raw) To: ext Ohad Ben-Cohen Cc: linux-wireless@vger.kernel.org, linux-omap@vger.kernel.org, John W. Linville, Mark Brown, linux-arm-kernel@lists.infradead.org, Chikkature Rajashekar Madhusudhan, Coelho Luciano (Nokia-MS/Helsinki), San Mehat, Quadros Roger (Nokia-MS/Helsinki), Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool On Thu, 2010-09-16 at 01:22 +0200, ext Ohad Ben-Cohen wrote: > Move wl12xx.h outside of the spi-specific location, > so it can be shared with both spi and sdio solutions. > > Signed-off-by: Ohad Ben-Cohen <ohad@wizery.com> > --- Hi, wl12xx.h seems to be included also in arch/arm/mach-omap2/board-rx51-peripherals.c, so that should be changed too. Best Regards, Teemu Paasikivi ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 1/7] wl12xx: make wl12xx.h common to both spi and sdio 2010-09-16 10:17 ` Teemu Paasikivi @ 2010-09-16 10:45 ` Ohad Ben-Cohen [not found] ` <AANLkTikYd+TojQC-R+Vrdou6KSRwFNczBeTnsX7eTDn4-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 14+ messages in thread From: Ohad Ben-Cohen @ 2010-09-16 10:45 UTC (permalink / raw) To: Teemu Paasikivi Cc: linux-wireless@vger.kernel.org, linux-omap@vger.kernel.org, John W. Linville, Mark Brown, linux-arm-kernel@lists.infradead.org, Chikkature Rajashekar Madhusudhan, Coelho Luciano (Nokia-MS/Helsinki), San Mehat, Quadros Roger (Nokia-MS/Helsinki), Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool On Thu, Sep 16, 2010 at 12:17 PM, Teemu Paasikivi <ext-teemu.3.paasikivi@nokia.com> wrote: > On Thu, 2010-09-16 at 01:22 +0200, ext Ohad Ben-Cohen wrote: >> Move wl12xx.h outside of the spi-specific location, >> so it can be shared with both spi and sdio solutions. >> >> Signed-off-by: Ohad Ben-Cohen <ohad@wizery.com> >> --- > Hi, > > > wl12xx.h seems to be included also in > arch/arm/mach-omap2/board-rx51-peripherals.c, so that should be changed > too. Thanks for catching this, I now see there are several spi/wl12xx.h users (rx51, omap3pandora and MAINTAINERS). An update is coming. > > > Best Regards, > > Teemu Paasikivi > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
[parent not found: <AANLkTikYd+TojQC-R+Vrdou6KSRwFNczBeTnsX7eTDn4-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* [PATCH v6 01/17] wl12xx: make wl12xx.h common to both spi and sdio [not found] ` <AANLkTikYd+TojQC-R+Vrdou6KSRwFNczBeTnsX7eTDn4-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2010-09-16 11:16 ` Ohad Ben-Cohen 2010-09-16 11:50 ` [PATCH v6 1/7] " Ohad Ben-Cohen [not found] ` <1284635762-12335-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> 0 siblings, 2 replies; 14+ messages in thread From: Ohad Ben-Cohen @ 2010-09-16 11:16 UTC (permalink / raw) To: linux-wireless-u79uwXL29TY76Z2rM5mHXA, linux-omap-u79uwXL29TY76Z2rM5mHXA Cc: John W. Linville, Mark Brown, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Chikkature Rajashekar Madhusudhan, Luciano Coelho, San Mehat, Roger Quadros, Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool, Teemu Paasikivi, Ohad Ben-Cohen Move wl12xx.h outside of the spi-specific location, so it can be shared with both spi and sdio solutions. Update all users of spi/wl12xx.h accordingly Signed-off-by: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> --- MAINTAINERS | 2 +- arch/arm/mach-omap2/board-omap3pandora.c | 2 +- arch/arm/mach-omap2/board-rx51-peripherals.c | 2 +- drivers/net/wireless/wl12xx/wl1251_sdio.c | 2 +- drivers/net/wireless/wl12xx/wl1251_spi.c | 2 +- drivers/net/wireless/wl12xx/wl1271_spi.c | 2 +- include/linux/spi/wl12xx.h | 34 -------------------------- include/linux/wl12xx.h | 34 ++++++++++++++++++++++++++ 8 files changed, 40 insertions(+), 40 deletions(-) delete mode 100644 include/linux/spi/wl12xx.h create mode 100644 include/linux/wl12xx.h diff --git a/MAINTAINERS b/MAINTAINERS index 14ab8ca..b656c14 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -6416,7 +6416,7 @@ W: http://wireless.kernel.org T: git git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-testing.git S: Maintained F: drivers/net/wireless/wl12xx/wl1271* -F: include/linux/spi/wl12xx.h +F: include/linux/wl12xx.h WL3501 WIRELESS PCMCIA CARD DRIVER M: Arnaldo Carvalho de Melo <acme-f8uhVLnGfZaxAyOMLChx1axOck334EZe@public.gmane.org> diff --git a/arch/arm/mach-omap2/board-omap3pandora.c b/arch/arm/mach-omap2/board-omap3pandora.c index dd3af2b..7ea1eb4 100644 --- a/arch/arm/mach-omap2/board-omap3pandora.c +++ b/arch/arm/mach-omap2/board-omap3pandora.c @@ -25,7 +25,7 @@ #include <linux/spi/ads7846.h> #include <linux/regulator/machine.h> #include <linux/i2c/twl.h> -#include <linux/spi/wl12xx.h> +#include <linux/wl12xx.h> #include <linux/mtd/partitions.h> #include <linux/mtd/nand.h> #include <linux/leds.h> diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c b/arch/arm/mach-omap2/board-rx51-peripherals.c index 9a5eb87..ce28a85 100644 --- a/arch/arm/mach-omap2/board-rx51-peripherals.c +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c @@ -14,7 +14,7 @@ #include <linux/input.h> #include <linux/input/matrix_keypad.h> #include <linux/spi/spi.h> -#include <linux/spi/wl12xx.h> +#include <linux/wl12xx.h> #include <linux/i2c.h> #include <linux/i2c/twl.h> #include <linux/clk.h> diff --git a/drivers/net/wireless/wl12xx/wl1251_sdio.c b/drivers/net/wireless/wl12xx/wl1251_sdio.c index c0b68b0..74ba9ce 100644 --- a/drivers/net/wireless/wl12xx/wl1251_sdio.c +++ b/drivers/net/wireless/wl12xx/wl1251_sdio.c @@ -24,7 +24,7 @@ #include <linux/mmc/sdio_func.h> #include <linux/mmc/sdio_ids.h> #include <linux/platform_device.h> -#include <linux/spi/wl12xx.h> +#include <linux/wl12xx.h> #include <linux/irq.h> #include "wl1251.h" diff --git a/drivers/net/wireless/wl12xx/wl1251_spi.c b/drivers/net/wireless/wl12xx/wl1251_spi.c index 334ded9..320de79 100644 --- a/drivers/net/wireless/wl12xx/wl1251_spi.c +++ b/drivers/net/wireless/wl12xx/wl1251_spi.c @@ -24,7 +24,7 @@ #include <linux/slab.h> #include <linux/crc7.h> #include <linux/spi/spi.h> -#include <linux/spi/wl12xx.h> +#include <linux/wl12xx.h> #include "wl1251.h" #include "wl1251_reg.h" diff --git a/drivers/net/wireless/wl12xx/wl1271_spi.c b/drivers/net/wireless/wl12xx/wl1271_spi.c index 4cb99c5..c3fdab7 100644 --- a/drivers/net/wireless/wl12xx/wl1271_spi.c +++ b/drivers/net/wireless/wl12xx/wl1271_spi.c @@ -25,7 +25,7 @@ #include <linux/module.h> #include <linux/crc7.h> #include <linux/spi/spi.h> -#include <linux/spi/wl12xx.h> +#include <linux/wl12xx.h> #include <linux/slab.h> #include "wl1271.h" diff --git a/include/linux/spi/wl12xx.h b/include/linux/spi/wl12xx.h deleted file mode 100644 index a20bccf..0000000 --- a/include/linux/spi/wl12xx.h +++ /dev/null @@ -1,34 +0,0 @@ -/* - * This file is part of wl12xx - * - * Copyright (C) 2009 Nokia Corporation - * - * Contact: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU General Public License - * version 2 as published by the Free Software Foundation. - * - * This program is distributed in the hope that it will be useful, but - * WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA - * 02110-1301 USA - * - */ - -#ifndef _LINUX_SPI_WL12XX_H -#define _LINUX_SPI_WL12XX_H - -struct wl12xx_platform_data { - void (*set_power)(bool enable); - /* SDIO only: IRQ number if WLAN_IRQ line is used, 0 for SDIO IRQs */ - int irq; - bool use_eeprom; -}; - -#endif diff --git a/include/linux/wl12xx.h b/include/linux/wl12xx.h new file mode 100644 index 0000000..015687a --- /dev/null +++ b/include/linux/wl12xx.h @@ -0,0 +1,34 @@ +/* + * This file is part of wl12xx + * + * Copyright (C) 2009 Nokia Corporation + * + * Contact: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * version 2 as published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA + * 02110-1301 USA + * + */ + +#ifndef _LINUX_WL12XX_H +#define _LINUX_WL12XX_H + +struct wl12xx_platform_data { + void (*set_power)(bool enable); + /* SDIO only: IRQ number if WLAN_IRQ line is used, 0 for SDIO IRQs */ + int irq; + bool use_eeprom; +}; + +#endif -- 1.7.0.4 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v6 1/7] wl12xx: make wl12xx.h common to both spi and sdio 2010-09-16 11:16 ` [PATCH v6 01/17] " Ohad Ben-Cohen @ 2010-09-16 11:50 ` Ohad Ben-Cohen [not found] ` <1284635762-12335-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> 1 sibling, 0 replies; 14+ messages in thread From: Ohad Ben-Cohen @ 2010-09-16 11:50 UTC (permalink / raw) To: John W. Linville Cc: linux-wireless, linux-omap, Mark Brown, linux-arm-kernel, Chikkature Rajashekar Madhusudhan, Luciano Coelho, San Mehat, Roger Quadros, Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool, Teemu Paasikivi, Ohad Ben-Cohen Hi John, On Thu, Sep 16, 2010 at 1:16 PM, Ohad Ben-Cohen <ohad@wizery.com> wrote: > Move wl12xx.h outside of the spi-specific location, > so it can be shared with both spi and sdio solutions. > > Update all users of spi/wl12xx.h accordingly > > Signed-off-by: Ohad Ben-Cohen <ohad@wizery.com> > --- This updates the first patch in the patchset. Please tell me if you prefer me to resubmit the whole set. Thanks, Ohad. > MAINTAINERS | 2 +- > arch/arm/mach-omap2/board-omap3pandora.c | 2 +- > arch/arm/mach-omap2/board-rx51-peripherals.c | 2 +- > drivers/net/wireless/wl12xx/wl1251_sdio.c | 2 +- > drivers/net/wireless/wl12xx/wl1251_spi.c | 2 +- > drivers/net/wireless/wl12xx/wl1271_spi.c | 2 +- > include/linux/spi/wl12xx.h | 34 -------------------------- > include/linux/wl12xx.h | 34 ++++++++++++++++++++++++++ > 8 files changed, 40 insertions(+), 40 deletions(-) > delete mode 100644 include/linux/spi/wl12xx.h > create mode 100644 include/linux/wl12xx.h > > diff --git a/MAINTAINERS b/MAINTAINERS > index 14ab8ca..b656c14 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -6416,7 +6416,7 @@ W: http://wireless.kernel.org > T: git git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-testing.git > S: Maintained > F: drivers/net/wireless/wl12xx/wl1271* > -F: include/linux/spi/wl12xx.h > +F: include/linux/wl12xx.h > > WL3501 WIRELESS PCMCIA CARD DRIVER > M: Arnaldo Carvalho de Melo <acme@ghostprotocols.net> > diff --git a/arch/arm/mach-omap2/board-omap3pandora.c b/arch/arm/mach-omap2/board-omap3pandora.c > index dd3af2b..7ea1eb4 100644 > --- a/arch/arm/mach-omap2/board-omap3pandora.c > +++ b/arch/arm/mach-omap2/board-omap3pandora.c > @@ -25,7 +25,7 @@ > #include <linux/spi/ads7846.h> > #include <linux/regulator/machine.h> > #include <linux/i2c/twl.h> > -#include <linux/spi/wl12xx.h> > +#include <linux/wl12xx.h> > #include <linux/mtd/partitions.h> > #include <linux/mtd/nand.h> > #include <linux/leds.h> > diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c b/arch/arm/mach-omap2/board-rx51-peripherals.c > index 9a5eb87..ce28a85 100644 > --- a/arch/arm/mach-omap2/board-rx51-peripherals.c > +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c > @@ -14,7 +14,7 @@ > #include <linux/input.h> > #include <linux/input/matrix_keypad.h> > #include <linux/spi/spi.h> > -#include <linux/spi/wl12xx.h> > +#include <linux/wl12xx.h> > #include <linux/i2c.h> > #include <linux/i2c/twl.h> > #include <linux/clk.h> > diff --git a/drivers/net/wireless/wl12xx/wl1251_sdio.c b/drivers/net/wireless/wl12xx/wl1251_sdio.c > index c0b68b0..74ba9ce 100644 > --- a/drivers/net/wireless/wl12xx/wl1251_sdio.c > +++ b/drivers/net/wireless/wl12xx/wl1251_sdio.c > @@ -24,7 +24,7 @@ > #include <linux/mmc/sdio_func.h> > #include <linux/mmc/sdio_ids.h> > #include <linux/platform_device.h> > -#include <linux/spi/wl12xx.h> > +#include <linux/wl12xx.h> > #include <linux/irq.h> > > #include "wl1251.h" > diff --git a/drivers/net/wireless/wl12xx/wl1251_spi.c b/drivers/net/wireless/wl12xx/wl1251_spi.c > index 334ded9..320de79 100644 > --- a/drivers/net/wireless/wl12xx/wl1251_spi.c > +++ b/drivers/net/wireless/wl12xx/wl1251_spi.c > @@ -24,7 +24,7 @@ > #include <linux/slab.h> > #include <linux/crc7.h> > #include <linux/spi/spi.h> > -#include <linux/spi/wl12xx.h> > +#include <linux/wl12xx.h> > > #include "wl1251.h" > #include "wl1251_reg.h" > diff --git a/drivers/net/wireless/wl12xx/wl1271_spi.c b/drivers/net/wireless/wl12xx/wl1271_spi.c > index 4cb99c5..c3fdab7 100644 > --- a/drivers/net/wireless/wl12xx/wl1271_spi.c > +++ b/drivers/net/wireless/wl12xx/wl1271_spi.c > @@ -25,7 +25,7 @@ > #include <linux/module.h> > #include <linux/crc7.h> > #include <linux/spi/spi.h> > -#include <linux/spi/wl12xx.h> > +#include <linux/wl12xx.h> > #include <linux/slab.h> > > #include "wl1271.h" > diff --git a/include/linux/spi/wl12xx.h b/include/linux/spi/wl12xx.h > deleted file mode 100644 > index a20bccf..0000000 > --- a/include/linux/spi/wl12xx.h > +++ /dev/null > @@ -1,34 +0,0 @@ > -/* > - * This file is part of wl12xx > - * > - * Copyright (C) 2009 Nokia Corporation > - * > - * Contact: Luciano Coelho <luciano.coelho@nokia.com> > - * > - * This program is free software; you can redistribute it and/or > - * modify it under the terms of the GNU General Public License > - * version 2 as published by the Free Software Foundation. > - * > - * This program is distributed in the hope that it will be useful, but > - * WITHOUT ANY WARRANTY; without even the implied warranty of > - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > - * General Public License for more details. > - * > - * You should have received a copy of the GNU General Public License > - * along with this program; if not, write to the Free Software > - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > - * 02110-1301 USA > - * > - */ > - > -#ifndef _LINUX_SPI_WL12XX_H > -#define _LINUX_SPI_WL12XX_H > - > -struct wl12xx_platform_data { > - void (*set_power)(bool enable); > - /* SDIO only: IRQ number if WLAN_IRQ line is used, 0 for SDIO IRQs */ > - int irq; > - bool use_eeprom; > -}; > - > -#endif > diff --git a/include/linux/wl12xx.h b/include/linux/wl12xx.h > new file mode 100644 > index 0000000..015687a > --- /dev/null > +++ b/include/linux/wl12xx.h > @@ -0,0 +1,34 @@ > +/* > + * This file is part of wl12xx > + * > + * Copyright (C) 2009 Nokia Corporation > + * > + * Contact: Luciano Coelho <luciano.coelho@nokia.com> > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License > + * version 2 as published by the Free Software Foundation. > + * > + * This program is distributed in the hope that it will be useful, but > + * WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, write to the Free Software > + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA > + * > + */ > + > +#ifndef _LINUX_WL12XX_H > +#define _LINUX_WL12XX_H > + > +struct wl12xx_platform_data { > + void (*set_power)(bool enable); > + /* SDIO only: IRQ number if WLAN_IRQ line is used, 0 for SDIO IRQs */ > + int irq; > + bool use_eeprom; > +}; > + > +#endif > -- > 1.7.0.4 > > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 14+ messages in thread
[parent not found: <1284635762-12335-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH v6 01/17] wl12xx: make wl12xx.h common to both spi and sdio [not found] ` <1284635762-12335-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> @ 2010-09-16 18:35 ` Luciano Coelho 2010-09-16 19:56 ` Ohad Ben-Cohen 2010-09-16 20:46 ` Tony Lindgren 0 siblings, 2 replies; 14+ messages in thread From: Luciano Coelho @ 2010-09-16 18:35 UTC (permalink / raw) To: ext Ohad Ben-Cohen Cc: linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, John W. Linville, Mark Brown, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Chikkature Rajashekar Madhusudhan, San Mehat, Quadros Roger (Nokia-MS/Helsinki), Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool, Paasikivi Teemu.3 (EXT-Ixonos/Tampere) On Thu, 2010-09-16 at 13:16 +0200, ext Ohad Ben-Cohen wrote: > Move wl12xx.h outside of the spi-specific location, > so it can be shared with both spi and sdio solutions. > > Update all users of spi/wl12xx.h accordingly > > Signed-off-by: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> > --- Acked-by: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> I just wonder why the wl12xx.h move wasn't detected as a rename. Did you forget to use the -M option in format-patch? -- Cheers, Luca. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 01/17] wl12xx: make wl12xx.h common to both spi and sdio 2010-09-16 18:35 ` [PATCH v6 01/17] " Luciano Coelho @ 2010-09-16 19:56 ` Ohad Ben-Cohen 2010-09-16 20:00 ` Luciano Coelho 2010-09-16 20:46 ` Tony Lindgren 1 sibling, 1 reply; 14+ messages in thread From: Ohad Ben-Cohen @ 2010-09-16 19:56 UTC (permalink / raw) To: Luciano Coelho Cc: linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, John W. Linville, Mark Brown, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Chikkature Rajashekar Madhusudhan, San Mehat, Quadros Roger (Nokia-MS/Helsinki), Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool, Paasikivi Teemu.3 (EXT-Ixonos/Tampere) On Thu, Sep 16, 2010 at 8:35 PM, Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> wrote: > Acked-by: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> > > I just wonder why the wl12xx.h move wasn't detected as a rename. Did > you forget to use the -M option in format-patch? I thought it's because I stripped the SPI_ from the #ifndef _LINUX_SPI_WL12XX_H, but it should be smarter than that. Probably the -M. Thanks, Ohad. > > -- > Cheers, > Luca. > > -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 01/17] wl12xx: make wl12xx.h common to both spi and sdio 2010-09-16 19:56 ` Ohad Ben-Cohen @ 2010-09-16 20:00 ` Luciano Coelho 0 siblings, 0 replies; 14+ messages in thread From: Luciano Coelho @ 2010-09-16 20:00 UTC (permalink / raw) To: ext Ohad Ben-Cohen Cc: linux-wireless@vger.kernel.org, linux-omap@vger.kernel.org, John W. Linville, Mark Brown, linux-arm-kernel@lists.infradead.org, Chikkature Rajashekar Madhusudhan, San Mehat, Quadros Roger (Nokia-MS/Helsinki), Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool, Paasikivi Teemu.3 (EXT-Ixonos/Tampere) On Thu, 2010-09-16 at 21:56 +0200, ext Ohad Ben-Cohen wrote: > On Thu, Sep 16, 2010 at 8:35 PM, Luciano Coelho > <luciano.coelho@nokia.com> wrote: > > Acked-by: Luciano Coelho <luciano.coelho@nokia.com> > > > > I just wonder why the wl12xx.h move wasn't detected as a rename. Did > > you forget to use the -M option in format-patch? > > I thought it's because I stripped the SPI_ from the > #ifndef _LINUX_SPI_WL12XX_H, but it should be smarter than that. > Probably the -M. Yes, it is smarter than that. With small changes like this, -M will still show it as a rename and show the diff instead of the whole file -ed and +ed back ;) -- Cheers, Luca. ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 01/17] wl12xx: make wl12xx.h common to both spi and sdio 2010-09-16 18:35 ` [PATCH v6 01/17] " Luciano Coelho 2010-09-16 19:56 ` Ohad Ben-Cohen @ 2010-09-16 20:46 ` Tony Lindgren 1 sibling, 0 replies; 14+ messages in thread From: Tony Lindgren @ 2010-09-16 20:46 UTC (permalink / raw) To: Luciano Coelho Cc: ext Ohad Ben-Cohen, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, John W. Linville, Mark Brown, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Chikkature Rajashekar Madhusudhan, San Mehat, Quadros Roger (Nokia-MS/Helsinki), Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool, Paasikivi Teemu.3 (EXT-Ixonos/Tampere) * Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> [100916 11:28]: > On Thu, 2010-09-16 at 13:16 +0200, ext Ohad Ben-Cohen wrote: > > Move wl12xx.h outside of the spi-specific location, > > so it can be shared with both spi and sdio solutions. > > > > Update all users of spi/wl12xx.h accordingly > > > > Signed-off-by: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> > > --- > > Acked-by: Luciano Coelho <luciano.coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org> Looks good to me too: Acked-by: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 14+ messages in thread
[parent not found: <1284592929-29616-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>]
* [PATCH v6 2/7] wl1271: propagate set_power's return value [not found] ` <1284592929-29616-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> @ 2010-09-15 23:22 ` Ohad Ben-Cohen [not found] ` <1284592929-29616-3-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> 0 siblings, 1 reply; 14+ messages in thread From: Ohad Ben-Cohen @ 2010-09-15 23:22 UTC (permalink / raw) To: linux-wireless-u79uwXL29TY76Z2rM5mHXA, linux-omap-u79uwXL29TY76Z2rM5mHXA Cc: John W. Linville, Mark Brown, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Chikkature Rajashekar Madhusudhan, Luciano Coelho, San Mehat, Roger Quadros, Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool, Ohad Ben-Cohen Make it possible for the set power method to indicate a success/failure return value. This is needed to support more complex power on/off operations such as SDIO power manipulations. Signed-off-by: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> --- drivers/net/wireless/wl12xx/wl1271.h | 2 +- drivers/net/wireless/wl12xx/wl1271_io.h | 9 ++++++--- drivers/net/wireless/wl12xx/wl1271_main.c | 4 +++- drivers/net/wireless/wl12xx/wl1271_sdio.c | 15 +++++++++------ drivers/net/wireless/wl12xx/wl1271_spi.c | 4 +++- 5 files changed, 22 insertions(+), 12 deletions(-) diff --git a/drivers/net/wireless/wl12xx/wl1271.h b/drivers/net/wireless/wl12xx/wl1271.h index dd3cee6..faa5925 100644 --- a/drivers/net/wireless/wl12xx/wl1271.h +++ b/drivers/net/wireless/wl12xx/wl1271.h @@ -313,7 +313,7 @@ struct wl1271_if_operations { bool fixed); void (*reset)(struct wl1271 *wl); void (*init)(struct wl1271 *wl); - void (*power)(struct wl1271 *wl, bool enable); + int (*power)(struct wl1271 *wl, bool enable); struct device* (*dev)(struct wl1271 *wl); void (*enable_irq)(struct wl1271 *wl); void (*disable_irq)(struct wl1271 *wl); diff --git a/drivers/net/wireless/wl12xx/wl1271_io.h b/drivers/net/wireless/wl12xx/wl1271_io.h index bc806c7..c1f92e6 100644 --- a/drivers/net/wireless/wl12xx/wl1271_io.h +++ b/drivers/net/wireless/wl12xx/wl1271_io.h @@ -144,10 +144,13 @@ static inline void wl1271_power_off(struct wl1271 *wl) clear_bit(WL1271_FLAG_GPIO_POWER, &wl->flags); } -static inline void wl1271_power_on(struct wl1271 *wl) +static inline int wl1271_power_on(struct wl1271 *wl) { - wl->if_ops->power(wl, true); - set_bit(WL1271_FLAG_GPIO_POWER, &wl->flags); + int ret = wl->if_ops->power(wl, true); + if (ret == 0) + set_bit(WL1271_FLAG_GPIO_POWER, &wl->flags); + + return ret; } diff --git a/drivers/net/wireless/wl12xx/wl1271_main.c b/drivers/net/wireless/wl12xx/wl1271_main.c index 8e55cf8..af26150 100644 --- a/drivers/net/wireless/wl12xx/wl1271_main.c +++ b/drivers/net/wireless/wl12xx/wl1271_main.c @@ -621,7 +621,9 @@ static int wl1271_chip_wakeup(struct wl1271 *wl) int ret = 0; msleep(WL1271_PRE_POWER_ON_SLEEP); - wl1271_power_on(wl); + ret = wl1271_power_on(wl); + if (ret < 0) + goto out; msleep(WL1271_POWER_ON_SLEEP); wl1271_io_reset(wl); wl1271_io_init(wl); diff --git a/drivers/net/wireless/wl12xx/wl1271_sdio.c b/drivers/net/wireless/wl12xx/wl1271_sdio.c index b5d9565..1d5dc72 100644 --- a/drivers/net/wireless/wl12xx/wl1271_sdio.c +++ b/drivers/net/wireless/wl12xx/wl1271_sdio.c @@ -159,35 +159,38 @@ static void wl1271_sdio_raw_write(struct wl1271 *wl, int addr, void *buf, wl1271_error("sdio write failed (%d)", ret); } -static void wl1271_sdio_power_on(struct wl1271 *wl) +static int wl1271_sdio_power_on(struct wl1271 *wl) { struct sdio_func *func = wl_to_func(wl); sdio_claim_host(func); sdio_enable_func(func); sdio_release_host(func); + + return 0; } -static void wl1271_sdio_power_off(struct wl1271 *wl) +static int wl1271_sdio_power_off(struct wl1271 *wl) { struct sdio_func *func = wl_to_func(wl); sdio_claim_host(func); sdio_disable_func(func); sdio_release_host(func); + + return 0; } -static void wl1271_sdio_set_power(struct wl1271 *wl, bool enable) +static int wl1271_sdio_set_power(struct wl1271 *wl, bool enable) { /* Let the SDIO stack handle wlan_enable control, so we * keep host claimed while wlan is in use to keep wl1271 * alive. */ if (enable) - wl1271_sdio_power_on(wl); + return wl1271_sdio_power_on(wl); else - wl1271_sdio_power_off(wl); - + return wl1271_sdio_power_off(wl); } static struct wl1271_if_operations sdio_ops = { diff --git a/drivers/net/wireless/wl12xx/wl1271_spi.c b/drivers/net/wireless/wl12xx/wl1271_spi.c index c3fdab7..de56d8d 100644 --- a/drivers/net/wireless/wl12xx/wl1271_spi.c +++ b/drivers/net/wireless/wl12xx/wl1271_spi.c @@ -312,10 +312,12 @@ static irqreturn_t wl1271_irq(int irq, void *cookie) return IRQ_HANDLED; } -static void wl1271_spi_set_power(struct wl1271 *wl, bool enable) +static int wl1271_spi_set_power(struct wl1271 *wl, bool enable) { if (wl->set_power) wl->set_power(enable); + + return 0; } static struct wl1271_if_operations spi_ops = { -- 1.7.0.4 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 14+ messages in thread
[parent not found: <1284592929-29616-3-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH v6 2/7] wl1271: propagate set_power's return value [not found] ` <1284592929-29616-3-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> @ 2010-09-16 19:40 ` Luciano Coelho 2010-09-16 19:53 ` Ohad Ben-Cohen 0 siblings, 1 reply; 14+ messages in thread From: Luciano Coelho @ 2010-09-16 19:40 UTC (permalink / raw) To: ext Ohad Ben-Cohen Cc: linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, John W. Linville, Mark Brown, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Chikkature Rajashekar Madhusudhan, San Mehat, Quadros Roger (Nokia-MS/Helsinki), Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool On Thu, 2010-09-16 at 01:22 +0200, ext Ohad Ben-Cohen wrote: > Make it possible for the set power method to indicate a > success/failure return value. This is needed to support > more complex power on/off operations such as SDIO > power manipulations. > > Signed-off-by: Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org> > --- Some comments below. > diff --git a/drivers/net/wireless/wl12xx/wl1271_io.h b/drivers/net/wireless/wl12xx/wl1271_io.h > index bc806c7..c1f92e6 100644 > --- a/drivers/net/wireless/wl12xx/wl1271_io.h > +++ b/drivers/net/wireless/wl12xx/wl1271_io.h > @@ -144,10 +144,13 @@ static inline void wl1271_power_off(struct wl1271 *wl) > clear_bit(WL1271_FLAG_GPIO_POWER, &wl->flags); > } > > -static inline void wl1271_power_on(struct wl1271 *wl) > +static inline int wl1271_power_on(struct wl1271 *wl) > { > - wl->if_ops->power(wl, true); > - set_bit(WL1271_FLAG_GPIO_POWER, &wl->flags); > + int ret = wl->if_ops->power(wl, true); I think it look nicer if you keep the "int ret" in one line by itself and then do a ret = wl->if_ops... on another one. > + if (ret == 0) > + set_bit(WL1271_FLAG_GPIO_POWER, &wl->flags); > + > + return ret; > } > diff --git a/drivers/net/wireless/wl12xx/wl1271_sdio.c b/drivers/net/wireless/wl12xx/wl1271_sdio.c > index b5d9565..1d5dc72 100644 > --- a/drivers/net/wireless/wl12xx/wl1271_sdio.c > +++ b/drivers/net/wireless/wl12xx/wl1271_sdio.c > @@ -159,35 +159,38 @@ static void wl1271_sdio_raw_write(struct wl1271 *wl, int addr, void *buf, > wl1271_error("sdio write failed (%d)", ret); > } > > -static void wl1271_sdio_power_on(struct wl1271 *wl) > +static int wl1271_sdio_power_on(struct wl1271 *wl) > { > struct sdio_func *func = wl_to_func(wl); > > sdio_claim_host(func); > sdio_enable_func(func); > sdio_release_host(func); > + > + return 0; > } You seem to always return 0, so the whole chain to pass the value up seems unnecessary. Is this just a preparation for a future patch? -- Cheers, Luca. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 2/7] wl1271: propagate set_power's return value 2010-09-16 19:40 ` Luciano Coelho @ 2010-09-16 19:53 ` Ohad Ben-Cohen 2010-09-16 19:56 ` Luciano Coelho 0 siblings, 1 reply; 14+ messages in thread From: Ohad Ben-Cohen @ 2010-09-16 19:53 UTC (permalink / raw) To: Luciano Coelho Cc: linux-wireless@vger.kernel.org, linux-omap@vger.kernel.org, John W. Linville, Mark Brown, linux-arm-kernel@lists.infradead.org, Chikkature Rajashekar Madhusudhan, San Mehat, Quadros Roger (Nokia-MS/Helsinki), Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool On Thu, Sep 16, 2010 at 9:40 PM, Luciano Coelho <luciano.coelho@nokia.com> wrote: >> + int ret = wl->if_ops->power(wl, true); > > I think it look nicer if you keep the "int ret" in one line by itself > and then do a ret = wl->if_ops... on another one. Fixed. >> +static int wl1271_sdio_power_on(struct wl1271 *wl) >> { >> struct sdio_func *func = wl_to_func(wl); >> >> sdio_claim_host(func); >> sdio_enable_func(func); >> sdio_release_host(func); >> + >> + return 0; >> } > > You seem to always return 0, so the whole chain to pass the value up > seems unnecessary. Is this just a preparation for a future patch? Yes, it's soon going to be: static int wl1271_sdio_power_on(struct wl1271 *wl) { struct sdio_func *func = wl_to_func(wl); int ret; ret = pm_runtime_get_sync(&func->dev); if (ret) goto out; sdio_claim_host(func); sdio_enable_func(func); sdio_release_host(func); out: return ret; } Thanks, Ohad. > > -- > Cheers, > Luca. > > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v6 2/7] wl1271: propagate set_power's return value 2010-09-16 19:53 ` Ohad Ben-Cohen @ 2010-09-16 19:56 ` Luciano Coelho 0 siblings, 0 replies; 14+ messages in thread From: Luciano Coelho @ 2010-09-16 19:56 UTC (permalink / raw) To: ext Ohad Ben-Cohen Cc: linux-wireless@vger.kernel.org, linux-omap@vger.kernel.org, John W. Linville, Mark Brown, linux-arm-kernel@lists.infradead.org, Chikkature Rajashekar Madhusudhan, San Mehat, Quadros Roger (Nokia-MS/Helsinki), Tony Lindgren, Nicolas Pitre, Ido Yariv, Kalle Valo, Russell King, Vitaly Wool On Thu, 2010-09-16 at 21:53 +0200, ext Ohad Ben-Cohen wrote: > On Thu, Sep 16, 2010 at 9:40 PM, Luciano Coelho > <luciano.coelho@nokia.com> wrote: > >> + int ret = wl->if_ops->power(wl, true); > > > > I think it look nicer if you keep the "int ret" in one line by itself > > and then do a ret = wl->if_ops... on another one. > > Fixed. > > >> +static int wl1271_sdio_power_on(struct wl1271 *wl) > >> { > >> struct sdio_func *func = wl_to_func(wl); > >> > >> sdio_claim_host(func); > >> sdio_enable_func(func); > >> sdio_release_host(func); > >> + > >> + return 0; > >> } > > > > You seem to always return 0, so the whole chain to pass the value up > > seems unnecessary. Is this just a preparation for a future patch? > > Yes, it's soon going to be: > > static int wl1271_sdio_power_on(struct wl1271 *wl) > { > struct sdio_func *func = wl_to_func(wl); > int ret; > > ret = pm_runtime_get_sync(&func->dev); > if (ret) > goto out; > > sdio_claim_host(func); > sdio_enable_func(func); > sdio_release_host(func); > > out: > return ret; > } > Ok, that was the only explanation I could think of ;) Acked-by: Luciano Coelho <luciano.coelho@nokia.com> -- Cheers, Luca. ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2010-09-16 20:46 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-15 23:22 [PATCH v6 0/7] native wl1271 support on ZOOM Ohad Ben-Cohen
2010-09-15 23:22 ` [PATCH v6 1/7] wl12xx: make wl12xx.h common to both spi and sdio Ohad Ben-Cohen
2010-09-16 10:17 ` Teemu Paasikivi
2010-09-16 10:45 ` Ohad Ben-Cohen
[not found] ` <AANLkTikYd+TojQC-R+Vrdou6KSRwFNczBeTnsX7eTDn4-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-09-16 11:16 ` [PATCH v6 01/17] " Ohad Ben-Cohen
2010-09-16 11:50 ` [PATCH v6 1/7] " Ohad Ben-Cohen
[not found] ` <1284635762-12335-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>
2010-09-16 18:35 ` [PATCH v6 01/17] " Luciano Coelho
2010-09-16 19:56 ` Ohad Ben-Cohen
2010-09-16 20:00 ` Luciano Coelho
2010-09-16 20:46 ` Tony Lindgren
[not found] ` <1284592929-29616-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>
2010-09-15 23:22 ` [PATCH v6 2/7] wl1271: propagate set_power's return value Ohad Ben-Cohen
[not found] ` <1284592929-29616-3-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>
2010-09-16 19:40 ` Luciano Coelho
2010-09-16 19:53 ` Ohad Ben-Cohen
2010-09-16 19:56 ` Luciano Coelho
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).