2013-01-03 08:01:05

by Yogesh Ashok Powar

[permalink] [raw]
Subject: [PATCH 5/5] mwl8k: bump firmware API to 3

New firmware is required to support
a) Add support for additional ampdu streams
b) Handle corresponding watchdog events to destroy ampdu streams

Bumping firmware API version by one.

Signed-off-by: Nishant Sarmukadam <[email protected]>
Signed-off-by: Yogesh Ashok Powar <[email protected]>
---
drivers/net/wireless/mwl8k.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/mwl8k.c b/drivers/net/wireless/mwl8k.c
index 51ae9e9..5717e15 100644
--- a/drivers/net/wireless/mwl8k.c
+++ b/drivers/net/wireless/mwl8k.c
@@ -5319,7 +5319,7 @@ enum {
MWL8366,
};

-#define MWL8K_8366_AP_FW_API 2
+#define MWL8K_8366_AP_FW_API 3
#define _MWL8K_8366_AP_FW(api) "mwl8k/fmimage_8366_ap-" #api ".fw"
#define MWL8K_8366_AP_FW(api) _MWL8K_8366_AP_FW(api)

--
1.8.0.3



2013-02-06 15:44:08

by Johannes Berg

[permalink] [raw]
Subject: Re: [PATCH 5/5] mwl8k: bump firmware API to 3

On Wed, 2013-02-06 at 20:43 +0530, Yogesh Ashok Powar wrote:
> On Wed, Feb 06, 2013 at 06:31:12AM -0800, Jonas Gorski wrote:
> > On 3 January 2013 08:54, Yogesh Ashok Powar <[email protected]> wrote:
> > > New firmware is required to support
> > > a) Add support for additional ampdu streams
> > > b) Handle corresponding watchdog events to destroy ampdu streams
> > >
> > > Bumping firmware API version by one.
> >
> > Soo ... where does one get this new firmware version? I don't see it
> > anywhere (especially not in linux-firmware.git). AFAICT mwl8k in
> > ap-mode is currently broken since this patch got in.
>
> Hi Jonas,
>
> We have already mailed the API-3 firmware to Ben Hutching for adding
> it to linux-firmware.git.
>
> Meanwhile please get the new AP firmware from
> https://github.com/yogeshpowar/mwl8k_ap_firmware

I think you really ought to support previous firmware and next firmware
for a while at least so not everybody is forced to upgrade
immediately...

johannes


2013-02-06 15:22:48

by Yogesh Ashok Powar

[permalink] [raw]
Subject: Re: [PATCH 5/5] mwl8k: bump firmware API to 3

On Wed, Feb 06, 2013 at 06:31:12AM -0800, Jonas Gorski wrote:
> On 3 January 2013 08:54, Yogesh Ashok Powar <[email protected]> wrote:
> > New firmware is required to support
> > a) Add support for additional ampdu streams
> > b) Handle corresponding watchdog events to destroy ampdu streams
> >
> > Bumping firmware API version by one.
>
> Soo ... where does one get this new firmware version? I don't see it
> anywhere (especially not in linux-firmware.git). AFAICT mwl8k in
> ap-mode is currently broken since this patch got in.

Hi Jonas,

We have already mailed the API-3 firmware to Ben Hutching for adding
it to linux-firmware.git.

Meanwhile please get the new AP firmware from
https://github.com/yogeshpowar/mwl8k_ap_firmware


Thanks
Yogesh

2013-02-06 14:31:39

by Jonas Gorski

[permalink] [raw]
Subject: Re: [PATCH 5/5] mwl8k: bump firmware API to 3

On 3 January 2013 08:54, Yogesh Ashok Powar <[email protected]> wrote:
> New firmware is required to support
> a) Add support for additional ampdu streams
> b) Handle corresponding watchdog events to destroy ampdu streams
>
> Bumping firmware API version by one.

Soo ... where does one get this new firmware version? I don't see it
anywhere (especially not in linux-firmware.git). AFAICT mwl8k in
ap-mode is currently broken since this patch got in.

> Signed-off-by: Nishant Sarmukadam <[email protected]>
> Signed-off-by: Yogesh Ashok Powar <[email protected]>
> ---
> drivers/net/wireless/mwl8k.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/mwl8k.c b/drivers/net/wireless/mwl8k.c
> index 51ae9e9..5717e15 100644
> --- a/drivers/net/wireless/mwl8k.c
> +++ b/drivers/net/wireless/mwl8k.c
> @@ -5319,7 +5319,7 @@ enum {
> MWL8366,
> };
>
> -#define MWL8K_8366_AP_FW_API 2
> +#define MWL8K_8366_AP_FW_API 3
> #define _MWL8K_8366_AP_FW(api) "mwl8k/fmimage_8366_ap-" #api ".fw"
> #define MWL8K_8366_AP_FW(api) _MWL8K_8366_AP_FW(api)
>
> --
> 1.8.0.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html

2013-02-07 14:35:00

by Jonas Gorski

[permalink] [raw]
Subject: Re: [PATCH 5/5] mwl8k: bump firmware API to 3

On 6 February 2013 16:13, Yogesh Ashok Powar <[email protected]> wrote:
> Hi Jonas,
>
> We have already mailed the API-3 firmware to Ben Hutching for adding
> it to linux-firmware.git.
>
> Meanwhile please get the new AP firmware from
> https://github.com/yogeshpowar/mwl8k_ap_firmware

Thanks, with it I can get AP mode to work on both a 2.4 GHz single
band and a dual band card, but a 5 GHz single band card craps out when
bringing up the interface (see the oops at the end).

I haven't looked to deeply into what's happening, and I haven't made
sure yet the problem isn't caused by any OpenWrt modifications (which
is using compat-wirless 2013-01-07). I'll take a closer look tomorrow.

Regards
Jonas


P.S: The oops:
[ 665.669993] Unable to handle kernel NULL pointer dereference at
virtual address 00000016
[ 665.678194] pgd = c6d58000
[ 665.680941] [00000016] *pgd=06f8a831, *pte=00000000, *ppte=00000000
[ 665.687303] Internal error: Oops: 17 [#1]
[ 665.691326] Modules linked in: nf_nat_irc nf_nat_ftp
nf_conntrack_irc nf_conntrack_ftp ipt_MASQUERADE iptable_nat nf_nat
pppoe xt_conntrack xt_CT xt_NOTRACK iptable_raw xt_state
nf_conntrack_ipv4 nf_defrag_ipv4 nf_conntrack pppox ipt_REJECT
xt_TCPMSS ipt_LOG xt_comment xt_multiport xt_mac xt_limit
iptable_mangle iptable_filter ip_tables xt_tcpudp x_tables ppp_async
ppp_generic slhc mwl8k(O) mac80211(O) crc_ccitt cfg80211(O) compat(O)
arc4 aes_generic crypto_algapi
[ 665.732940] CPU: 0 Tainted: G O (3.3.8 #52)
[ 665.738387] PC is at cfg80211_chandef_usable+0x7c/0x1bc [cfg80211]
[ 665.744611] LR is at cfg80211_chandef_usable+0x20/0x1bc [cfg80211]
(...)
[ 666.116373] Backtrace:
[ 666.118866] [<bf0368dc>] (cfg80211_chandef_usable+0x0/0x1bc
[cfg80211]) from [<bf025e64>] (nl80211_leave_mesh+0x244/0x264
[cfg80211])
[ 666.130919] r7:c6d12100 r6:0000143c r5:c0611c48 r4:c0611b98
[ 666.136668] [<bf025d84>] (nl80211_leave_mesh+0x164/0x264
[cfg80211]) from [<bf02634c>] (nl80211_remain_on_channel+0x2a0/0x358
[cfg80211])
[ 666.149074] r7:c6d12000 r6:c6d12000 r5:c6f4f368 r4:00000003
[ 666.154814] [<bf0262ec>] (nl80211_remain_on_channel+0x240/0x358
[cfg80211]) from [<bf02ddb0>] (nl80211_set_wiphy+0x264/0x560
[cfg80211])
[ 666.167150] [<bf02db4c>] (nl80211_set_wiphy+0x0/0x560 [cfg80211])
from [<c01f94e0>] (genl_rcv_msg+0x1b8/0x1f8)
[ 666.177205] [<c01f9328>] (genl_rcv_msg+0x0/0x1f8) from [<c01f89a0>]
(netlink_rcv_skb+0x58/0xb4)
[ 666.185949] [<c01f8948>] (netlink_rcv_skb+0x0/0xb4) from
[<c01f931c>] (genl_rcv+0x20/0x2c)
[ 666.194251] r6:c6f70780 r5:0000002c r4:c6f70780 r3:00000001
[ 666.199973] [<c01f92fc>] (genl_rcv+0x0/0x2c) from [<c01f8418>]
(netlink_unicast+0x154/0x1f4)
[ 666.208449] r4:c785ea00 r3:c01f92fc
[ 666.212057] [<c01f82c4>] (netlink_unicast+0x0/0x1f4) from
[<c01f8790>] (netlink_sendmsg+0x230/0x2b0)
[ 666.221240] [<c01f8560>] (netlink_sendmsg+0x0/0x2b0) from
[<c01cccf8>] (sock_sendmsg+0x90/0xa4)
[ 666.229986] [<c01ccc68>] (sock_sendmsg+0x0/0xa4) from [<c01cdcb0>]
(__sys_sendmsg+0x290/0x298)
[ 666.238637] r9:00000000 r8:c0611ec8 r6:0000002c r5:c0610000 r4:c0611f64
[ 666.245411] [<c01cda20>] (__sys_sendmsg+0x0/0x298) from
[<c01cf52c>] (sys_sendmsg+0x44/0x6c)
[ 666.253897] [<c01cf4e8>] (sys_sendmsg+0x0/0x6c) from [<c00090a0>]
(ret_fast_syscall+0x0/0x2c)

Full stack at <http://pastebin.com/v0tYysE9>