Re: [PATCH] Bluetooth: Add Kconfig option to enable Realtek protocol for serial devices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Hans,

>>> Add a new BT_HCIUART_RTL Kconfig option to enable the Realtek protocol for
>>> Bluetooth HCI over serial port interface, mirroring the existing
>>> BT_HCIUART_BCM Kconfig option.
>>> 
>>> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
>>> ---
>>> drivers/bluetooth/Kconfig | 14 ++++++++++++++
>>> 1 file changed, 14 insertions(+)
>>> 
>>> diff --git a/drivers/bluetooth/Kconfig b/drivers/bluetooth/Kconfig
>>> index 5f953ca8ac5b..1124034d1c7e 100644
>>> --- a/drivers/bluetooth/Kconfig
>>> +++ b/drivers/bluetooth/Kconfig
>>> @@ -195,6 +195,20 @@ config BT_HCIUART_BCM
>>> 
>>> 	  Say Y here to compile support for Broadcom protocol.
>>> 
>>> +config BT_HCIUART_RTL
>>> +	bool "Realtek protocol support"
>>> +	depends on BT_HCIUART
>>> +	depends on BT_HCIUART_SERDEV
>>> +	depends on (!ACPI || SERIAL_DEV_CTRL_TTYPORT)
>> is this one really needed?
> 
> I mirrored this from the equivalent BCM Kconfig option, this bit comes from:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=4294625e02902
> 
> Reading the commit message it seems that this indeed is not necessary for the
> RTL_BT case.
> 
>>> +	depends on GPIOLIB
>>> +	select BT_HCIUART_3WIRE
>>> +	select BT_RTL
>>> +	help
>>> +	  The Realtek protocol support enables Bluetooth HCI over serial
>>> +	  port interface for Realtek Bluetooth controllers.
>> I like the text from my patch better since it mentions 3-Wire serial which is what this one is using.
> 
> I've not seen your patch (I'm not subscribed to the list) but if you think
> your version is better feel free to go with it.

you can find it in the archives, but I think my patch is the cleaner one.

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux