From: "John W. Linville" <linville@tuxdriver.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Bing Zhao <bzhao@marvell.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
Dan Williams <dcbw@redhat.com>,
David Woodhouse <dwmw2@infradead.org>,
Amitkumar Karwar <akarwar@marvell.com>,
Kiran Divekar <dkiran@marvell.com>, Greg KH <greg@kroah.com>,
"David S. Miller" <davem@davemloft.net>
Subject: Re: [PATCH 00/31] wireless: Marvell mwifiex driver for SD8787
Date: Fri, 11 Mar 2011 09:12:30 -0500 [thread overview]
Message-ID: <20110311141230.GA14583@tuxdriver.com> (raw)
In-Reply-To: <1299833115.3755.1.camel@jlt3.sipsolutions.net>
On Fri, Mar 11, 2011 at 09:45:15AM +0100, Johannes Berg wrote:
> Hi,
>
> > During the past 3 months, we have made 56 mwifiex patches (listed
> > below) since the initial mwifiex driver submission. Most of these
> > patches address the above issues; and the ioctl multiple abstraction
> > layers and the static forward declarations as you commented in another
> > e-mail. Some other patches are for code cleanups, mainly coding style.
>
> I've only loosely followed these patches, but I'm sure the driver is in
> *much* better shape now. If you think you're done, it'd be worthwhile to
> just resubmit the entire driver in its new version.
>
> johannes
Agreed -- please resubmit the driver as either a single patch
for inclusion or possibly as a small series of patches grouped by
functionality for further review.
Thanks!
John
--
John W. Linville Someday the world will need a hero, and you
linville@tuxdriver.com might be all we have. Be ready.
next prev parent reply other threads:[~2011-03-11 14:16 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-02 1:31 [PATCH 00/31] wireless: Marvell mwifiex driver for SD8787 Bing Zhao
2010-12-02 19:33 ` John W. Linville
2010-12-02 19:58 ` Johannes Berg
2010-12-03 12:53 ` Johannes Berg
2010-12-03 15:06 ` Greg KH
[not found] ` <477F20668A386D41ADCC57781B1F704307D59F9D6D@SC-VEXCH1.marvell.com>
[not found] ` <77956EDC1B917843AC9B7965A3BD78B058FEF48E24@SC-VEXCH2.marvell.com>
2010-12-03 18:39 ` Johannes Berg
2010-12-03 19:11 ` John W. Linville
2010-12-03 21:17 ` [PATCH 0/3] mwifiex: remove some stuff John W. Linville
2010-12-03 22:22 ` Bing Zhao
2010-12-04 0:34 ` Bing Zhao
2010-12-06 19:14 ` Bing Zhao
2010-12-03 21:17 ` [PATCH 1/3] mwifiex: remove uAP functionality John W. Linville
2010-12-03 21:17 ` [PATCH 2/3] mwifiex: remove NETLINK_MARVELL and associate madness John W. Linville
2010-12-03 21:17 ` [PATCH 3/3] mwifiex: remove questionable wireless_send_event stuff John W. Linville
2010-12-04 0:59 ` [PATCH 00/31] wireless: Marvell mwifiex driver for SD8787 Bing Zhao
2010-12-03 22:15 ` Bing Zhao
2011-02-23 3:37 ` Bing Zhao
2011-02-23 8:23 ` Johannes Berg
2011-02-23 19:26 ` Bing Zhao
2011-03-11 4:04 ` Bing Zhao
2011-03-11 8:45 ` Johannes Berg
2011-03-11 14:12 ` John W. Linville [this message]
2011-03-11 18:31 ` Bing Zhao
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=20110311141230.GA14583@tuxdriver.com \
--to=linville@tuxdriver.com \
--cc=akarwar@marvell.com \
--cc=bzhao@marvell.com \
--cc=davem@davemloft.net \
--cc=dcbw@redhat.com \
--cc=dkiran@marvell.com \
--cc=dwmw2@infradead.org \
--cc=greg@kroah.com \
--cc=johannes@sipsolutions.net \
--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.