From patchwork Sun Nov 23 12:10:16 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Tunin X-Patchwork-Id: 5360551 Return-Path: X-Original-To: patchwork-linux-wireless@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 3030EC11AC for ; Sun, 23 Nov 2014 12:10:40 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id CD8792024C for ; Sun, 23 Nov 2014 12:10:38 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 33C5B20165 for ; Sun, 23 Nov 2014 12:10:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751132AbaKWMKW (ORCPT ); Sun, 23 Nov 2014 07:10:22 -0500 Received: from mail-lb0-f177.google.com ([209.85.217.177]:61630 "EHLO mail-lb0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750929AbaKWMKU (ORCPT ); Sun, 23 Nov 2014 07:10:20 -0500 Received: by mail-lb0-f177.google.com with SMTP id 10so3218282lbg.22 for ; Sun, 23 Nov 2014 04:10:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=hrYsJNUct46rcNHdUwBymokYOR/BdO8LrPEhCscmfro=; b=FzWE+Ik1lnQ5s5kwpF3WXIRVnNCnQmisDukP52uLbVLt+AEE21/zMxj/K6e18jlWSb bh/EineNSwt5xTvl4T1IBAY7SFebqKR3750Clbif/35wih5WwS248acwXnSCrFTb1GEe PY+iXQh1a65qTPG4KNtTSN8pEByX3nq1AmZolJ/E9SwVj1vd3pNnGfPok3oJWNzyD3Fk 1FNe7hUb93QM1aaXaC2c3F1SSzk/iIfqncWL6QOEhLbXahMrfplvtSpf+4KBqCWbloqB +gImGWqk+wCxN1Wk2HiHtEFIPJ4IK76uwn7dU0Bhmu1kuRFKlnFzVWiLuw43405yf9uy DQ2Q== X-Received: by 10.152.205.11 with SMTP id lc11mr15115252lac.34.1416744618063; Sun, 23 Nov 2014 04:10:18 -0800 (PST) Received: from [192.168.1.2] (89-178-185-14.broadband.corbina.ru. [89.178.185.14]) by mx.google.com with ESMTPSA id j9sm2678511lab.5.2014.11.23.04.10.17 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 23 Nov 2014 04:10:17 -0800 (PST) Message-ID: <5471CEA8.4030101@gmail.com> Date: Sun, 23 Nov 2014 15:10:16 +0300 From: Dmitry Tunin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Michel Memeteau - EKIMIA CC: Marcel Holtmann , linux-bluetooth , linux-kernel , linux-wireless Subject: Re: [PATCH v3] ath3k: Add support of MCI 13d3:3408 bt device References: <546C3CCE.400@gmail.com> <546C6BB9.4040705@gmail.com> In-Reply-To: <546C6BB9.4040705@gmail.com> Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org X-Spam-Status: No, score=-6.8 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, RCVD_IN_DNSWL_HI, T_DKIM_INVALID, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP After some testing on a few laptops it looks like the firmware loads always correctly only using AR3012 variant. So the correct patch follows --- -- 19.11.2014 13:06, Dmitry Tunin wrote: > It seems that I am too stupid. > It is neither AR3012 or AR3011 device. > It is AR9565 with sflash. > Probably it makes sense to do a separate section for these devices. > > But still > { USB_DEVICE(0x13d3, 0x3408), .driver_info = BTUSB_IGNORE }, > works better than > > { USB_DEVICE(0x13d3, 0x3408), .driver_info = BTUSB_ATH3012 }, > > With the latter I have some issues with turning bt on and off. > > 19.11.2014 12:40, Michel Memeteau - EKIMIA wrote: >> Hi, >> >> I tried to understand why this "0x13d3, 0x3408" device support >> wouldn't go in btusb as it's really close to IMC BT chip found in >> Realtek Wifi devices which is on the path to go in btusb [1] >> >> then I guess there is a lot of code that could be shared to support >> all IMC usb devices in the btusb module. >> >> Regards. >> >> [1] https://github.com/lwfinger/rtl8723au_bt/tree/new >> >> <------------------------------------------------------------------------------------------------------------------> >> Michel Memeteau - Directeur. >> >> Notre Boutique Ordinateurs GNU/Linux : http://shop.ekimia.fr >> 49 chemin de l'union 13720 La Bouilladisse - France. >> Fixe : +33 (0) 972308334 Mobile : +33(0) 624808051 >> <------------------------------------------------------------------------------------------------------------------> >> >> 2014-11-19 7:46 GMT+01:00 Dmitry Tunin : >>> Hi Marcel, >>> >>> Here is information from /sys/kernel/debug/usb/devices >>> >>> T: Bus=01 Lev=02 Prnt=03 Port=00 Cnt=01 Dev#= 20 Spd=12 MxCh= 0 >>> D: Ver= 1.10 Cls=e0(wlcon) Sub=01 Prot=01 MxPS=64 #Cfgs= 1 >>> P: Vendor=13d3 ProdID=3408 Rev= 0.02 >>> C:* #Ifs= 2 Cfg#= 1 Atr=e0 MxPwr=100mA >>> A: FirstIf#= 0 IfCount= 2 Cls=e0(wlcon) Sub=01 Prot=01 >>> I:* If#= 0 Alt= 0 #EPs= 3 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb >>> E: Ad=81(I) Atr=03(Int.) MxPS= 16 Ivl=1ms >>> E: Ad=82(I) Atr=02(Bulk) MxPS= 64 Ivl=0ms >>> E: Ad=02(O) Atr=02(Bulk) MxPS= 64 Ivl=0ms >>> I:* If#= 1 Alt= 0 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb >>> E: Ad=83(I) Atr=01(Isoc) MxPS= 0 Ivl=1ms >>> E: Ad=03(O) Atr=01(Isoc) MxPS= 0 Ivl=1ms >>> I: If#= 1 Alt= 1 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb >>> E: Ad=83(I) Atr=01(Isoc) MxPS= 9 Ivl=1ms >>> E: Ad=03(O) Atr=01(Isoc) MxPS= 9 Ivl=1ms >>> I: If#= 1 Alt= 2 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb >>> E: Ad=83(I) Atr=01(Isoc) MxPS= 17 Ivl=1ms >>> E: Ad=03(O) Atr=01(Isoc) MxPS= 17 Ivl=1ms >>> I: If#= 1 Alt= 3 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb >>> E: Ad=83(I) Atr=01(Isoc) MxPS= 25 Ivl=1ms >>> E: Ad=03(O) Atr=01(Isoc) MxPS= 25 Ivl=1ms >>> I: If#= 1 Alt= 4 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb >>> E: Ad=83(I) Atr=01(Isoc) MxPS= 33 Ivl=1ms >>> E: Ad=03(O) Atr=01(Isoc) MxPS= 33 Ivl=1ms >>> I: If#= 1 Alt= 5 #EPs= 2 Cls=e0(wlcon) Sub=01 Prot=01 Driver=btusb >>> E: Ad=83(I) Atr=01(Isoc) MxPS= 49 Ivl=1ms >>> E: Ad=03(O) Atr=01(Isoc) MxPS= 49 Ivl=1ms >>> >>> >>> I hope this mail client is OK now. >>> >>> Add support for bluetooth MCI WB335 (AR9565) Wi-Fi+bt module. >>> It is AR3011 device according to iProduct == 0. >>> So btusb should be blacklisted >>> I submitted a wrong patch before asif it was an AR3012 device, >>> but it works both ways. This is the right one. >>> >>> Signed-off-by: Dmitry Tunin >>> >>> --- >>> >>> diff --git a/drivers/bluetooth/ath3k.c b/drivers/bluetooth/ath3k.c >>> index d85ced2..3b53116 100644 >>> --- a/drivers/bluetooth/ath3k.c >>> +++ b/drivers/bluetooth/ath3k.c >>> @@ -69,6 +69,7 @@ static const struct usb_device_id ath3k_table[] = { >>> { USB_DEVICE(0x0CF3, 0x3002) }, >>> { USB_DEVICE(0x0CF3, 0xE019) }, >>> { USB_DEVICE(0x13d3, 0x3304) }, >>> + { USB_DEVICE(0x13d3, 0x3408) }, >>> >>> /* Atheros AR9285 Malbec with sflash firmware */ >>> { USB_DEVICE(0x03F0, 0x311D) }, >>> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c >>> index edfc17b..bbf5c1c 100644 >>> --- a/drivers/bluetooth/btusb.c >>> +++ b/drivers/bluetooth/btusb.c >>> @@ -146,6 +146,7 @@ static const struct usb_device_id >>> blacklist_table[] = { >>> { USB_DEVICE(0x0cf3, 0x3002), .driver_info = BTUSB_IGNORE }, >>> { USB_DEVICE(0x0cf3, 0xe019), .driver_info = BTUSB_IGNORE }, >>> { USB_DEVICE(0x13d3, 0x3304), .driver_info = BTUSB_IGNORE }, >>> + { USB_DEVICE(0x13d3, 0x3408), .driver_info = BTUSB_IGNORE }, >>> >>> /* Atheros AR9285 Malbec with sflash firmware */ >>> { USB_DEVICE(0x03f0, 0x311d), .driver_info = BTUSB_IGNORE }, >>> >>> >>> 19.11.2014 01:56, Marcel Holtmann wrote: >>>> Hi Dmitry, >>>> >>>>> Add support for bluetooth MCI WB335 (AR9565) Wi-Fi+bt module. >>>>> It is AR3011 device according to iProduct == 0. >>>>> So btusb should be blacklisted >>>>> I submitted a wrong patch before asif it was an AR3012 device, >>>>> but it works both ways. This is the right one. >>>> include the device details from /sys/kernel/debug/usb/devices. >>>> >>>>> Signed-off-by: Dmitry Tunin >>>>> are */ >>>> No idea what this are */ is. >>>> >>>>> --- >>>>> >>>>> diff --git a/drivers/bluetooth/ath3k.c b/drivers/bluetooth/ath3k.c >>>>> index d85ced2..3b53116 100644 >>>>> --- a/drivers/bluetooth/ath3k.c >>>>> +++ b/drivers/bluetooth/ath3k.c >>>>> @@ -69,6 +69,7 @@ static const struct usb_device_id ath3k_table[] = { >>>>> { USB_DEVICE(0x0CF3, 0x3002) }, >>>>> { USB_DEVICE(0x0CF3, 0xE019) }, >>>>> { USB_DEVICE(0x13d3, 0x3304) }, >>>>> + { USB_DEVICE(0x13d3, 0x3408) }, >>>>> >>>>> /* Atheros AR9285 Malbec with sflash firmware */ >>>>> { USB_DEVICE(0x03F0, 0x311D) }, >>>>> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c >>>>> index edfc17b..bbf5c1c 100644 >>>>> --- a/drivers/bluetooth/btusb.c >>>>> +++ b/drivers/bluetooth/btusb.c >>>>> @@ -146,6 +146,7 @@ static const struct usb_device_id >>>>> blacklist_table[] = { >>>> { USB_DEVICE(0x0cf3, 0x3002), .driver_info = BTUSB_IGNORE }, >>>>> { USB_DEVICE(0x0cf3, 0xe019), .driver_info = BTUSB_IGNORE }, >>>>> { USB_DEVICE(0x13d3, 0x3304), .driver_info = BTUSB_IGNORE }, >>>>> + { USB_DEVICE(0x13d3, 0x3408), .driver_info = BTUSB_IGNORE }, >>>> Fix whitespace here or use a proper mail client. >>>> >>>> Regards >>>> >>>> Marcel >>>> >>> -- >>> 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 > -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/bluetooth/ath3k.c b/drivers/bluetooth/ath3k.c index d85ced2..086240c 100644 --- a/drivers/bluetooth/ath3k.c +++ b/drivers/bluetooth/ath3k.c @@ -105,6 +105,7 @@ static const struct usb_device_id ath3k_table[] = { { USB_DEVICE(0x13d3, 0x3375) }, { USB_DEVICE(0x13d3, 0x3393) }, { USB_DEVICE(0x13d3, 0x3402) }, + { USB_DEVICE(0x13d3, 0x3408) }, { USB_DEVICE(0x13d3, 0x3432) }, /* Atheros AR5BBU12 with sflash firmware */ @@ -156,6 +157,7 @@ static const struct usb_device_id ath3k_blist_tbl[] = { { USB_DEVICE(0x13d3, 0x3375), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x13d3, 0x3393), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x13d3, 0x3402), .driver_info = BTUSB_ATH3012 }, + { USB_DEVICE(0x13d3, 0x3408), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x13d3, 0x3432), .driver_info = BTUSB_ATH3012 }, /* Atheros AR5BBU22 with sflash firmware */ diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c index edfc17b..091c813 100644 --- a/drivers/bluetooth/btusb.c +++ b/drivers/bluetooth/btusb.c @@ -182,6 +182,7 @@ static const struct usb_device_id blacklist_table[] = { { USB_DEVICE(0x13d3, 0x3375), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x13d3, 0x3393), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x13d3, 0x3402), .driver_info = BTUSB_ATH3012 }, + { USB_DEVICE(0x13d3, 0x3408), .driver_info = BTUSB_ATH3012 }, { USB_DEVICE(0x13d3, 0x3432), .driver_info = BTUSB_ATH3012 }, /* Atheros AR5BBU12 with sflash firmware */