From: eran <eran@over-here.org>
To: Vinicius Costa Gomes <vinicius.gomes@openbossa.org>
Cc: linux-bluetooth <linux-bluetooth@vger.kernel.org>
Subject: [PATCH] Bluetooth: Adding USB device 13d3:3375 as an Atheros AR3012.
Date: Thu, 24 Nov 2011 11:19:08 +0000 [thread overview]
Message-ID: <20111124111908.GA922@over-here.org> (raw)
In-Reply-To: <20111123190500.GA7179@samus>
From: Eran <eran@over-here.org>
The bluetooth module in the Asus UX31/UX21 is based on Atheros AR3012
and requires a firmware to be uploaded before it's usable.
Signed-off-by: Eran <eran@over-here.org>
---
drivers/bluetooth/ath3k.c | 2 ++
drivers/bluetooth/btusb.c | 1 +
2 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/drivers/bluetooth/ath3k.c b/drivers/bluetooth/ath3k.c
index 106beb1..b94aae6 100644
--- a/drivers/bluetooth/ath3k.c
+++ b/drivers/bluetooth/ath3k.c
@@ -70,6 +70,7 @@ static struct usb_device_id ath3k_table[] = {
/* Atheros AR3012 with sflash firmware*/
{ USB_DEVICE(0x0CF3, 0x3004) },
+ { USB_DEVICE(0x13d3, 0x3375) },
/* Atheros AR5BBU12 with sflash firmware */
{ USB_DEVICE(0x0489, 0xE02C) },
@@ -86,6 +87,7 @@ static struct usb_device_id ath3k_blist_tbl[] = {
/* Atheros AR3012 with sflash firmware*/
{ USB_DEVICE(0x0cf3, 0x3004), .driver_info = BTUSB_ATH3012 },
+ { USB_DEVICE(0x13d3, 0x3375), .driver_info = BTUSB_ATH3012 },
{ } /* Terminating entry */
};
diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index f9b7260..a64f15a 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -122,6 +122,7 @@ static struct usb_device_id blacklist_table[] = {
/* Atheros 3012 with sflash firmware */
{ USB_DEVICE(0x0cf3, 0x3004), .driver_info = BTUSB_ATH3012 },
+ { USB_DEVICE(0x13d3, 0x3375), .driver_info = BTUSB_ATH3012 },
/* Atheros AR5BBU12 with sflash firmware */
{ USB_DEVICE(0x0489, 0xe02c), .driver_info = BTUSB_IGNORE },
--
1.7.2.5
On Wed, Nov 23, 2011 at 04:05:00PM -0300, Vinicius Costa Gomes wrote:
> Hi Eran,
>
> Just a few cosmetic comments, and a disclaimer: I don't have any
> idea why it seems that every new Atheros device needs another
> entry in those tables.
>
> Apart from those minor points it looks good.
>
> On 10:03 Wed 23 Nov, eran wrote:
> > From: Eran <eran@over-here.org>
>
> Keep the subject line shorter, under 60 or so characters, i.e. remove
> the part between parentheses. And prefix your subject with "Bluetooth:".
>
> >
> > The bluetooth module in the Asus UX31/UX21 is based on Atheros AR3012 and requires a firmware to be uploaded before it's usable.
>
> Please break the comment at 70 or so characters.
>
> >
> > Signed-off-by: Eran <eran@over-here.org>
> > ---
> > drivers/bluetooth/ath3k.c | 2 ++
> > drivers/bluetooth/btusb.c | 1 +
> > 2 files changed, 3 insertions(+), 0 deletions(-)
> >
> > diff --git a/drivers/bluetooth/ath3k.c b/drivers/bluetooth/ath3k.c
> > index 106beb1..b94aae6 100644
> > --- a/drivers/bluetooth/ath3k.c
> > +++ b/drivers/bluetooth/ath3k.c
> > @@ -70,6 +70,7 @@ static struct usb_device_id ath3k_table[] = {
> >
> > /* Atheros AR3012 with sflash firmware*/
> > { USB_DEVICE(0x0CF3, 0x3004) },
> > + { USB_DEVICE(0x13d3, 0x3375) },
> >
> > /* Atheros AR5BBU12 with sflash firmware */
> > { USB_DEVICE(0x0489, 0xE02C) },
> > @@ -86,6 +87,7 @@ static struct usb_device_id ath3k_blist_tbl[] = {
> >
> > /* Atheros AR3012 with sflash firmware*/
> > { USB_DEVICE(0x0cf3, 0x3004), .driver_info = BTUSB_ATH3012 },
> > + { USB_DEVICE(0x13d3, 0x3375), .driver_info = BTUSB_ATH3012 },
> >
> > { } /* Terminating entry */
> > };
> > diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
> > index f9b7260..a64f15a 100644
> > --- a/drivers/bluetooth/btusb.c
> > +++ b/drivers/bluetooth/btusb.c
> > @@ -122,6 +122,7 @@ static struct usb_device_id blacklist_table[] = {
> >
> > /* Atheros 3012 with sflash firmware */
> > { USB_DEVICE(0x0cf3, 0x3004), .driver_info = BTUSB_ATH3012 },
> > + { USB_DEVICE(0x13d3, 0x3375), .driver_info = BTUSB_ATH3012 },
> >
> > /* Atheros AR5BBU12 with sflash firmware */
> > { USB_DEVICE(0x0489, 0xe02c), .driver_info = BTUSB_IGNORE },
> > --
> > 1.7.2.5
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
>
> Cheers,
> --
> Vinicius
next prev parent reply other threads:[~2011-11-24 11:19 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-23 10:03 [PATCH] Adding USB device 13d3:3375 (a bluetooth module embedded in the Asus UX31) as an Atheros AR3012 device eran
2011-11-23 19:05 ` Vinicius Costa Gomes
2011-11-24 11:19 ` eran [this message]
2011-12-02 11:15 ` [PATCH] Bluetooth: Adding USB device 13d3:3375 as an Atheros AR3012 Gustavo Padovan
2011-12-05 22:15 ` eran
2012-01-17 11:04 ` Michal Labedzki
2012-03-08 5:17 ` Gustavo Padovan
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=20111124111908.GA922@over-here.org \
--to=eran@over-here.org \
--cc=linux-bluetooth@vger.kernel.org \
--cc=vinicius.gomes@openbossa.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.