All of lore.kernel.org
 help / color / mirror / Atom feed
From: Luciano Coelho <luciano.coelho@nokia.com>
To: ext Shahar Levi <shahar_levi@ti.com>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH 00/04] wl1271: 11n Support, add support to 80211n spec
Date: Wed, 15 Sep 2010 22:25:40 +0300	[thread overview]
Message-ID: <1284578740.1569.33.camel@powerslave> (raw)
In-Reply-To: <1284577092.1569.8.camel@powerslave>

On Wed, 2010-09-15 at 20:58 +0200, Luciano Coelho wrote:
> Hi Shahar,
> 
> On Wed, 2010-09-15 at 20:31 +0200, ext Shahar Levi wrote:
> > Series that add 80211n spec support to wl1271: scan, connection via MCS rates
> 
> Very nice! Thank you for this implementation!
> 
> It would be nice to describe this a bit more thoroughly, what is already
> implemented, what is still missing etc.  So that people who want to use
> can know the status of the implementation.

I had a few comments to your patches, but I'm really happy you have sent
these patches upstream! Just some minor non-functional comments.  Please
make the changes I proposed and send the series again as v2.

I owe you a drink, if we meet some day ;)

-- 
Cheers,
Luca.


  reply	other threads:[~2010-09-15 19:25 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-15 18:31 [PATCH 00/04] wl1271: 11n Support, add support to 80211n spec Shahar Levi
2010-09-15 18:31 ` [PATCH 01/04] wl1271: 11n Support, Add Definitions Shahar Levi
2010-09-15 19:15   ` Luciano Coelho
2010-09-15 18:31 ` [PATCH 02/04] wl1271: 11n Support, ACX Commands Shahar Levi
2010-09-15 18:31 ` [PATCH 03/04] wl1271: 11n Support, Scan, Connection, MCS rates Shahar Levi
2010-09-15 18:31 ` [PATCH 04/04] wl1271: 11n Support, 11n Kconfig Configurable Shahar Levi
2010-09-15 18:53   ` Johannes Berg
2010-09-15 18:56     ` Luciano Coelho
2010-09-15 19:21   ` Luciano Coelho
2010-09-16  0:39     ` Julian Calaby
2010-09-16 15:54       ` Levi, Shahar
2010-09-16 17:16         ` Julian Calaby
2010-09-18 17:56           ` Levi, Shahar
2010-09-19  1:19             ` Julian Calaby
2010-09-19 12:23               ` Levi, Shahar
2010-09-15 18:58 ` [PATCH 00/04] wl1271: 11n Support, add support to 80211n spec Luciano Coelho
2010-09-15 19:25   ` Luciano Coelho [this message]
2010-09-15 19:34     ` Levi, Shahar
2010-09-15 20:20       ` Luis R. Rodriguez
2010-09-15 20:40         ` Luciano Coelho
2010-09-16 15:24           ` Levi, Shahar

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=1284578740.1569.33.camel@powerslave \
    --to=luciano.coelho@nokia.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=shahar_levi@ti.com \
    /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.