* [PATCH] libertas: fix power save issue in libertas_sdio module
@ 2009-02-11 4:57 Bing Zhao
2009-02-11 18:14 ` Dan Williams
0 siblings, 1 reply; 2+ messages in thread
From: Bing Zhao @ 2009-02-11 4:57 UTC (permalink / raw)
To: libertas-dev; +Cc: linux-wireless
Hi,
This is to fix an issue in IEEE power save mode in libertas_sdio module.
The patch (below) was generated by git-format-patch command against
"linux-2.6" git tree.
Please review and your feedbacks are welcome.
Thanks,
Bing
-----------------------------------------------------------------------
>From 4a925690dc76965bf7173bfc0198112093bf131e Mon Sep 17 00:00:00 2001
From: Bing Zhao <bzhao@marvell.com>
Date: Wed, 4 Feb 2009 22:22:39 -0800
Subject: [PATCH] libertas: fix power save issue in libertas_sdio module
The problem: "iwconfig ethX power on" returns error
The cause: "ps_supported" flag was never set for SD8385/8686
The fix: check firmware capabilities returned by GET_HW_SPEC command.
Set "ps_supported" to 1 if FW_CAPINFO_PS bit is on. This fix applies
to SDIO interface only.
Signed-off-by: Bing Zhao <bzhao@marvell.com>
---
drivers/net/wireless/libertas/defs.h | 1 +
drivers/net/wireless/libertas/if_sdio.c | 6 ++++++
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/libertas/defs.h
b/drivers/net/wireless/libertas/defs.h
index c364e4c..10a43c5 100644
--- a/drivers/net/wireless/libertas/defs.h
+++ b/drivers/net/wireless/libertas/defs.h
@@ -263,6 +263,7 @@ static inline void lbs_deb_hex(unsigned int grp,
const char *prompt, u8 *buf, in
#define CMD_F_HOSTCMD (1 << 0)
#define FW_CAPINFO_WPA (1 << 0)
+#define FW_CAPINFO_PS (1 << 1)
#define FW_CAPINFO_FIRMWARE_UPGRADE (1 << 13)
#define FW_CAPINFO_BOOT2_UPGRADE (1<<14)
#define FW_CAPINFO_PERSISTENT_CONFIG (1<<15)
diff --git a/drivers/net/wireless/libertas/if_sdio.c
b/drivers/net/wireless/libertas/if_sdio.c
index 4519d73..9878368 100644
--- a/drivers/net/wireless/libertas/if_sdio.c
+++ b/drivers/net/wireless/libertas/if_sdio.c
@@ -209,6 +209,9 @@ static int if_sdio_handle_event(struct if_sdio_card
*card,
event = sdio_readb(card->func, IF_SDIO_EVENT, &ret);
if (ret)
goto out;
+
+ /* right shift 3 bits to get the event id */
+ event >>= 3;
} else {
if (size < 4) {
lbs_deb_sdio("event packet too small (%d
bytes)\n",
@@ -921,6 +924,9 @@ static int if_sdio_probe(struct sdio_func *func,
if (ret)
goto err_activate_card;
+ if (priv->fwcapinfo & FW_CAPINFO_PS)
+ priv->ps_supported = 1;
+
out:
lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
--
1.5.3.6
-----------------------------------------------------------------------
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] libertas: fix power save issue in libertas_sdio module
2009-02-11 4:57 [PATCH] libertas: fix power save issue in libertas_sdio module Bing Zhao
@ 2009-02-11 18:14 ` Dan Williams
0 siblings, 0 replies; 2+ messages in thread
From: Dan Williams @ 2009-02-11 18:14 UTC (permalink / raw)
To: Bing Zhao; +Cc: libertas-dev, linux-wireless
On Tue, 2009-02-10 at 20:57 -0800, Bing Zhao wrote:
> Hi,
>
> This is to fix an issue in IEEE power save mode in libertas_sdio module.
> The patch (below) was generated by git-format-patch command against
> "linux-2.6" git tree.
>
> Please review and your feedbacks are welcome.
>
> Thanks,
>
> Bing
>
> -----------------------------------------------------------------------
> From 4a925690dc76965bf7173bfc0198112093bf131e Mon Sep 17 00:00:00 2001
> From: Bing Zhao <bzhao@marvell.com>
> Date: Wed, 4 Feb 2009 22:22:39 -0800
> Subject: [PATCH] libertas: fix power save issue in libertas_sdio module
>
> The problem: "iwconfig ethX power on" returns error
>
> The cause: "ps_supported" flag was never set for SD8385/8686
>
> The fix: check firmware capabilities returned by GET_HW_SPEC command.
> Set "ps_supported" to 1 if FW_CAPINFO_PS bit is on. This fix applies
> to SDIO interface only.
>
> Signed-off-by: Bing Zhao <bzhao@marvell.com>
Good catch, thanks!
Acked-by: Dan Williams <dcbw@redhat.com>
> ---
> drivers/net/wireless/libertas/defs.h | 1 +
> drivers/net/wireless/libertas/if_sdio.c | 6 ++++++
> 2 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/libertas/defs.h
> b/drivers/net/wireless/libertas/defs.h
> index c364e4c..10a43c5 100644
> --- a/drivers/net/wireless/libertas/defs.h
> +++ b/drivers/net/wireless/libertas/defs.h
> @@ -263,6 +263,7 @@ static inline void lbs_deb_hex(unsigned int grp,
> const char *prompt, u8 *buf, in
>
> #define CMD_F_HOSTCMD (1 << 0)
> #define FW_CAPINFO_WPA (1 << 0)
> +#define FW_CAPINFO_PS (1 << 1)
> #define FW_CAPINFO_FIRMWARE_UPGRADE (1 << 13)
> #define FW_CAPINFO_BOOT2_UPGRADE (1<<14)
> #define FW_CAPINFO_PERSISTENT_CONFIG (1<<15)
> diff --git a/drivers/net/wireless/libertas/if_sdio.c
> b/drivers/net/wireless/libertas/if_sdio.c
> index 4519d73..9878368 100644
> --- a/drivers/net/wireless/libertas/if_sdio.c
> +++ b/drivers/net/wireless/libertas/if_sdio.c
> @@ -209,6 +209,9 @@ static int if_sdio_handle_event(struct if_sdio_card
> *card,
> event = sdio_readb(card->func, IF_SDIO_EVENT, &ret);
> if (ret)
> goto out;
> +
> + /* right shift 3 bits to get the event id */
> + event >>= 3;
> } else {
> if (size < 4) {
> lbs_deb_sdio("event packet too small (%d
> bytes)\n",
> @@ -921,6 +924,9 @@ static int if_sdio_probe(struct sdio_func *func,
> if (ret)
> goto err_activate_card;
>
> + if (priv->fwcapinfo & FW_CAPINFO_PS)
> + priv->ps_supported = 1;
> +
> out:
> lbs_deb_leave_args(LBS_DEB_SDIO, "ret %d", ret);
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-02-11 18:16 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-02-11 4:57 [PATCH] libertas: fix power save issue in libertas_sdio module Bing Zhao
2009-02-11 18:14 ` Dan Williams
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.