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
