From: Larry Finger <Larry.Finger@lwfinger.net>
To: Hin-Tak Leung <hintak_leung@yahoo.co.uk>
Cc: "John W. Linville" <linville@tuxdriver.com>,
herton@mandriva.com.br, linux-wireless@vger.kernel.org,
flamingice@sourmilk.net, andreamrl@tiscali.it,
linville@redhat.com
Subject: Re: [RFC][PATCH] Realtek 8187B wireless support with product id 0x8197/0x8189
Date: Wed, 25 Jun 2008 09:59:05 -0500 [thread overview]
Message-ID: <48625D39.30700@lwfinger.net> (raw)
In-Reply-To: <20080625140809.GA9111@tuxdriver.com>
John W. Linville wrote:
>
> Thanks for your effort on this issue. I am very proud of the way you
> and others have participated in improving this patch as documented
> in the bugzilla link above.
>
> I apologize for having been slow to respond but I seem to constantly
> be buried in patches... That is even more true today since I have
> recently been on vacation. Still, this patch has been languishing
> in my mailbox too long!
>
> Please review the guidelines here:
>
> http://linux.yyz.us/patch-format.html
>
> If possible, please wrap your changelog comment lines at 80 columns
> or less. Also you should probably include Signed-off-by lines from
> Herton and Pavel. And please include the patch inline if possible
> for easier review -- use a mailer that preserves whitespace correctly.
>
> I may have more specific comments about the patch itself later, but now
> I really need to get caught-up on some other merging. I just wanted to
> ensure that you knew I was not (completely) ignoring your efforts! :-)
I have been using this patch on my Level 1 WNC-0301USB device. There
are still problems with the TX code as it can associate only about 1
try in 10, but RX seems to be fine. I have been too busy to sort out
the problems, but I have fixed the patch to get rid of checkpatch
warnings and errors, which includes the "Signed-off-by" lines.
Would you like me to send the updated version to you, or should I send
it to linux-wireless and John?
Larry
next prev parent reply other threads:[~2008-06-25 14:58 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-15 0:59 [RFC][PATCH] Realtek 8187B wireless support with product id 0x8197/0x8189 Hin-Tak Leung
2008-06-18 22:31 ` Uwe Hermann
2008-06-19 0:17 ` Hin-Tak Leung
2008-06-25 14:08 ` John W. Linville
2008-06-25 14:59 ` Larry Finger [this message]
2008-06-25 15:44 ` Hin-Tak Leung
2008-06-26 0:26 ` Matthew Garrett
2008-06-26 0:44 ` Hin-Tak Leung
2008-06-26 3:41 ` Larry Finger
2008-06-26 17:22 ` Hin-Tak Leung
2008-06-26 18:33 ` Larry Finger
2008-06-26 18:44 ` Pavel Roskin
2008-06-26 19:45 ` Hin-Tak Leung
2008-06-26 20:40 ` Herton Ronaldo Krzesinski
2008-06-26 21:24 ` Herton Ronaldo Krzesinski
2008-06-26 21:53 ` Larry Finger
2008-06-26 20:44 ` Larry Finger
2008-06-26 21:15 ` Herton Ronaldo Krzesinski
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=48625D39.30700@lwfinger.net \
--to=larry.finger@lwfinger.net \
--cc=andreamrl@tiscali.it \
--cc=flamingice@sourmilk.net \
--cc=herton@mandriva.com.br \
--cc=hintak_leung@yahoo.co.uk \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@redhat.com \
--cc=linville@tuxdriver.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).