stable.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [3.0.y, 3.2.y, 3.4.y] Re: [PATCH 1/1] bluetooth: Add support for Foxconn/Hon Hai AR5BBU22 0489:E03C
       [not found] <1769868.QZY8sEHkp9@dagobah>
@ 2012-05-29 12:27 ` Jonathan Nieder
  2012-06-01  9:18   ` Greg KH
  2012-06-06 14:07   ` Ben Hutchings
  0 siblings, 2 replies; 3+ messages in thread
From: Jonathan Nieder @ 2012-05-29 12:27 UTC (permalink / raw)
  To: Michael Grützner
  Cc: stable, marcel, gustavo, johan.hedberg, linux-bluetooth,
	linux-kernel, AceLan Kao

Hi Ben and Greg,

Please consider

  85d59726c5c6 Bluetooth: Add support for Foxconn/Hon Hai AR5BBU22 0489:E03C

for inclusion if appropriate in the 3.0.y, 3.2.y, and 3.4.y trees.

Michael Grützner wrote:

> Add Foxconn/Hon Hai AR5BBU22 Bluetooth Module( 0x489:0xE03C) to 
> the blacklist of btusb module and add it to the ath3k module to properly load 
> the firmware in Kernel 3.3.4
> The device is integrated in  e.g. some  Acer Aspire 7750G.

I am guessing this is tested by Michael against 3.3.y.

I am wondering about one detail about the patch (please forgive my
ignorance):

[...]
> --- linux-3.3.4-vanilla/drivers/bluetooth/ath3k.c	2012-04-27 19:17:35.000000000 +0200
> +++ linux-3.3.4/drivers/bluetooth/ath3k.c	2012-04-30 01:29:47.081838667 +0200
> @@ -79,6 +79,9 @@ static struct usb_device_id ath3k_table[
>  	/* Atheros AR5BBU12 with sflash firmware */
>  	{ USB_DEVICE(0x0489, 0xE02C) },
>  
> +	/* Atheros AR5BBU22 with sflash firmware */
> +	{ USB_DEVICE(0x0489, 0xE03C) },
> +
>  	{ }	/* Terminating entry */

This part makes sense and seems similar to what is done for other
Atheros bluetooth adaptors that need firmware loaded at run time, like
the AR3011.

>  };
>  
> @@ -95,6 +98,9 @@ static struct usb_device_id ath3k_blist_
>  	{ USB_DEVICE(0x13d3, 0x3375), .driver_info = BTUSB_ATH3012 },
>  	{ USB_DEVICE(0x04ca, 0x3005), .driver_info = BTUSB_ATH3012 },
>  
> +	/* Atheros AR5BBU22 with sflash firmware */
> +	{ USB_DEVICE(0x0489, 0xE03C), .driver_info = BTUSB_ATH3012 },
> +
>  	{ }	/* Terminating entry */

No other chip other than the AR3012 seems to add entries here.  Could
you add a comment explaining why the AR5BBU22 is listed but not e.g.
the AR5BBU12?

> --- linux-3.3.4-vanilla/drivers/bluetooth/btusb.c	2012-04-27 19:17:35.000000000 +0200
> +++ linux-3.3.4/drivers/bluetooth/btusb.c	2012-04-30 01:31:49.302182281 +0200
> @@ -135,6 +135,9 @@ static struct usb_device_id blacklist_ta
>  	/* Atheros AR5BBU12 with sflash firmware */
>  	{ USB_DEVICE(0x0489, 0xe02c), .driver_info = BTUSB_IGNORE },
>  
> +	/* Atheros AR5BBU22 with sflash firmware */
> +	{ USB_DEVICE(0x0489, 0xe03c), .driver_info = BTUSB_ATH3012 },
> +

Is the AR5BBU22 an AR3012?

Curious,
Jonathan

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [3.0.y, 3.2.y, 3.4.y] Re: [PATCH 1/1] bluetooth: Add support for Foxconn/Hon Hai AR5BBU22 0489:E03C
  2012-05-29 12:27 ` [3.0.y, 3.2.y, 3.4.y] Re: [PATCH 1/1] bluetooth: Add support for Foxconn/Hon Hai AR5BBU22 0489:E03C Jonathan Nieder
@ 2012-06-01  9:18   ` Greg KH
  2012-06-06 14:07   ` Ben Hutchings
  1 sibling, 0 replies; 3+ messages in thread
From: Greg KH @ 2012-06-01  9:18 UTC (permalink / raw)
  To: Jonathan Nieder
  Cc: Michael Grützner, stable, marcel, gustavo, johan.hedberg,
	linux-bluetooth, linux-kernel, AceLan Kao

On Tue, May 29, 2012 at 07:27:49AM -0500, Jonathan Nieder wrote:
> Hi Ben and Greg,
> 
> Please consider
> 
>   85d59726c5c6 Bluetooth: Add support for Foxconn/Hon Hai AR5BBU22 0489:E03C
> 
> for inclusion if appropriate in the 3.0.y, 3.2.y, and 3.4.y trees.

Applied to 3.0-stable, thanks.

greg k-h

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [3.0.y, 3.2.y, 3.4.y] Re: [PATCH 1/1] bluetooth: Add support for Foxconn/Hon Hai AR5BBU22 0489:E03C
  2012-05-29 12:27 ` [3.0.y, 3.2.y, 3.4.y] Re: [PATCH 1/1] bluetooth: Add support for Foxconn/Hon Hai AR5BBU22 0489:E03C Jonathan Nieder
  2012-06-01  9:18   ` Greg KH
@ 2012-06-06 14:07   ` Ben Hutchings
  1 sibling, 0 replies; 3+ messages in thread
From: Ben Hutchings @ 2012-06-06 14:07 UTC (permalink / raw)
  To: Jonathan Nieder
  Cc: Michael Grützner, stable, marcel, gustavo, johan.hedberg,
	linux-bluetooth, linux-kernel, AceLan Kao

[-- Attachment #1: Type: text/plain, Size: 457 bytes --]

On Tue, 2012-05-29 at 07:27 -0500, Jonathan Nieder wrote:
> Hi Ben and Greg,
> 
> Please consider
> 
>   85d59726c5c6 Bluetooth: Add support for Foxconn/Hon Hai AR5BBU22 0489:E03C
> 
> for inclusion if appropriate in the 3.0.y, 3.2.y, and 3.4.y trees.
[...]

I've queued up this and the various other bluetooth device ID updates
you requested for 3.2.y.

Ben.

-- 
Ben Hutchings
I haven't lost my mind; it's backed up on tape somewhere.


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-06-06 14:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1769868.QZY8sEHkp9@dagobah>
2012-05-29 12:27 ` [3.0.y, 3.2.y, 3.4.y] Re: [PATCH 1/1] bluetooth: Add support for Foxconn/Hon Hai AR5BBU22 0489:E03C Jonathan Nieder
2012-06-01  9:18   ` Greg KH
2012-06-06 14:07   ` Ben Hutchings

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).