All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kalle Valo <kalle.valo@nokia.com>
To: "linux-wireless\@vger.kernel.org" <linux-wireless@vger.kernel.org>
Cc: "Coelho Luciano \(Nokia-D\/Helsinki\)" <Luciano.Coelho@nokia.com>
Subject: Re: wl12xx: driver for TI wl1251 chipset
Date: Thu, 19 Feb 2009 11:19:50 +0200	[thread overview]
Message-ID: <87tz6qokax.fsf@nokia.com> (raw)
In-Reply-To: <87iqnishuj.fsf@nokia.com> (Valo Kalle's message of "Tue\, 10 Feb 2009 11\:32\:04 +0100")

"Valo Kalle (Nokia-D/Tampere)" <Kalle.Valo@nokia.com> writes:

> Hello all,
>
> in Nokia we have been working on a mac80211 driver for TI wl1251
> chipset. We named it as wl12xx, and in the future we are planning to
> add wl1271 support to the driver as well.

[...]

> http://www.valot.fi/kalle/tmp/wl12xx/wl12xx-2.6.28-1.patch

[...]

> Currently known issues:
>
> o short slot and preamble support missing
> o rts/cts and cts-to-self missing
> o no proper bitrate control support
> o using deprecated omap_get_config(), need to convert to use platform_data
> o firmware wakeup not working, deepest sleep state in firmware not yet
>   enabled

I received few comments, I'll list them here so that I don't forget
them:

o netlink interface might create problems, I'll remove netlink. cfor
  now but will revisit it later
o wl12xx_80211.h is, well, ugly
o fragmentation in the hardware might not be worth the trouble
o "*fc &= ~IEEE80211_FCTL_PROTECTED;" shouldn't be necessary, just set
  the status properly
o get_tsf() doesn't need to be atomic anymore, ad-hoc tsf hack can be
  removed

If anyone has more comments, please send them soon. I'm planning to
prepare a patch for adding wl12xx to wireless-testing next week. Let's
see if John takes it :)

-- 
Kalle Valo

      parent reply	other threads:[~2009-02-19  9:20 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-10 10:32 wl12xx: driver for TI wl1251 chipset Kalle Valo
2009-02-10 15:21 ` Dan Williams
2009-02-10 16:23   ` Kalle Valo
2009-02-10 16:25     ` Dan Williams
2009-02-10 17:18       ` Kalle Valo
2009-02-19  9:19 ` Kalle Valo [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87tz6qokax.fsf@nokia.com \
    --to=kalle.valo@nokia.com \
    --cc=Luciano.Coelho@nokia.com \
    --cc=linux-wireless@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.