Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757934Ab3G3H76 (ORCPT ); Tue, 30 Jul 2013 03:59:58 -0400 Received: from mail1.bemta8.messagelabs.com ([216.82.243.196]:19722 "EHLO mail1.bemta8.messagelabs.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757847Ab3G3H74 (ORCPT ); Tue, 30 Jul 2013 03:59:56 -0400 X-Env-Sender: Hector.Palacios@digi.com X-Msg-Ref: server-5.tower-46.messagelabs.com!1375171193!29053365!3 X-Originating-IP: [66.77.174.13] X-StarScan-Received: X-StarScan-Version: 6.9.11; banners=-,-,- X-VirusChecked: Checked Message-ID: <51F7722E.9050708@digi.com> Date: Tue, 30 Jul 2013 09:58:38 +0200 From: Hector Palacios Organization: Digi International User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6 MIME-Version: 1.0 To: Marcel Holtmann CC: "linux-bluetooth@vger.kernel.org" , "gustavo@padovan.org" , "johan.hedberg@gmail.com" , "linux-kernel@vger.kernel.org" , , "suraj@atheros.com" Subject: Re: [PATCH] bluetooth: remove wrong dependency for BT_ATH3K References: <1375114325-25498-1-git-send-email-hector.palacios@digi.com> <5438CD3C-E28C-485C-94B8-F743E8B13D4A@holtmann.org> In-Reply-To: <5438CD3C-E28C-485C-94B8-F743E8B13D4A@holtmann.org> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-KSE-Antivirus-Interceptor-Info: scan successful X-KSE-Antivirus-Info: Clean Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1904 Lines: 50 Hi Marcel, On 07/29/2013 06:43 PM, Marcel Holtmann wrote: > Hi Hector, > >> The Atheros firmware download driver is not dependent on HCI >> USB driver. This driver is needed to complement any Atheros >> driver like BT_HCIUART_ATH3K. >> >> Signed-off-by: Hector Palacios >> CC: Suraj Sumangala >> CC: Vikram Kandukuri >> --- >> drivers/bluetooth/Kconfig | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/drivers/bluetooth/Kconfig b/drivers/bluetooth/Kconfig >> index 11a6104..a1d482b 100644 >> --- a/drivers/bluetooth/Kconfig >> +++ b/drivers/bluetooth/Kconfig >> @@ -222,7 +222,6 @@ config BT_MRVL_SDIO >> >> config BT_ATH3K >> tristate "Atheros firmware download driver" >> - depends on BT_HCIBTUSB >> select FW_LOADER >> help >> Bluetooth firmware download driver. >> > > I am confused now. How can an USB driver work with an UART driver? If you do not want to depend on BT_HCIBTUSB then that is fine, but at least you need to depend on USB. The thing is I have an Atheros AR6233 chipset which has wireless and Bluetooth. Bluetooth part in this chip operates as a UART. The driver for this is hci_ath.c but I also need to download the firmware to the chip, and for this I need to compile ath3k.c. However I don't need BT_HCIBTUSB. But you are right, looking at the source code of ath3k.c it is calling usb commands, so it needs to depend on USB at least. This brings an interesting question: shouldn't the firmware download part be isolated from the USB driver? After all, I want to communicate with a UART bluetooth chip. Best regards, -- Hector Palacios -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/