From: "John W. Linville" <linville@tuxdriver.com>
To: Rajkumar Manoharan <Rajkumar.Manoharan@Atheros.com>
Cc: Christian Lamparter <chunkeey@googlemail.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH] ath9k_htc: Add new devices into AR7010
Date: Thu, 11 Nov 2010 10:28:18 -0500 [thread overview]
Message-ID: <20101111152818.GA2559@tuxdriver.com> (raw)
In-Reply-To: <44EE5C37ADC36343B0625A05DD408C4850DAB4D6CE@CHEXMB-01.global.atheros.com>
On Thu, Nov 11, 2010 at 08:51:37PM +0530, Rajkumar Manoharan wrote:
> >On Thursday 11 November 2010 14:23:41 Rajkumar Manoharan wrote:
> >> On Wed, Nov 10, 2010 at 06:43:57PM +0530, Rajkumar Manoharan wrote:
> > >> On Wed, Nov 10, 2010 at 06:05:14PM +0530, Christian Lamparter wrote:
> > > >> On Wednesday 10 November 2010 12:37:21 Rajkumar Manoharan wrote:
> > > > >> Treat PIDs (0xA704, 0x1200) as AR7010 devices.
> > > >>
> > > > >just a though, but instead of relying only on the pid and
> >> > > hope for no collisions. you could put a @driver_info flag
> >> > > for all AR7010 devices into the usb_device_id table and
> > >> > remove the per-device "cases".
> > > >>
> >> > Make sense. Thanks for your review. I'll resend the patch.
> >> >
> >> After adding @driver_info to replace per-device cases,
> >> the patch looks like cleanup one which is not compatible for stable kernel.
> >> So let the previous patches to be integrated, due to minimal changes.
> >> And I'll submit the cleanup patch separately. Is that fine?
> >sure?
> >
> > Or is this a question directed to John?
>
> Both you and John. Since these patches are stable and enable support to new devices,
> I prefer to go with minimal changes. I will submit the cleanup patches to wireless-testing.
>
> Any comments John?
Sounds fine.
--
John W. Linville Someday the world will need a hero, and you
linville@tuxdriver.com might be all we have. Be ready.
prev parent reply other threads:[~2010-11-11 15:29 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-10 11:37 [PATCH] ath9k_htc: Add new devices into AR7010 Rajkumar Manoharan
2010-11-10 12:35 ` Christian Lamparter
2010-11-10 13:13 ` Rajkumar Manoharan
2010-11-11 13:23 ` Rajkumar Manoharan
2010-11-11 14:12 ` Christian Lamparter
2010-11-11 15:21 ` Rajkumar Manoharan
2010-11-11 15:28 ` John W. Linville [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=20101111152818.GA2559@tuxdriver.com \
--to=linville@tuxdriver.com \
--cc=Rajkumar.Manoharan@Atheros.com \
--cc=chunkeey@googlemail.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.