diff mbox

[RFC] rfkill: Add new ACPI device for Realtek serial BT

Message ID 1468609696-8828-1-git-send-email-Larry.Finger@lwfinger.net (mailing list archive)
State RFC
Delegated to: Johannes Berg
Headers show

Commit Message

Larry Finger July 15, 2016, 7:08 p.m. UTC
The Realtek RTL8723BS chips, which are connected via SDIO, also contain
a serial Bluetooth device. In order for BT to work, the device must be
added to the acpi_device_id table.

Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
---

I have no idea if this device should be added here, or if the device could
be added to some structure supplied at device startup. If anyone has any
suggestions, please let me know.

Thanks,

Larry
---
 net/rfkill/rfkill-gpio.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Marcel Holtmann July 15, 2016, 7:39 p.m. UTC | #1
Hi Larry,

> The Realtek RTL8723BS chips, which are connected via SDIO, also contain
> a serial Bluetooth device. In order for BT to work, the device must be
> added to the acpi_device_id table.
> 
> Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
> ---
> 
> I have no idea if this device should be added here, or if the device could
> be added to some structure supplied at device startup. If anyone has any
> suggestions, please let me know.

it should not be added here. Since most likely more is needed. For example within the hci_bcm.c and hci_intel.c drivers we handle the GPIO when attaching the UART to the Bluetooth subsystem. This means at minimum some sort of hci_rtl.c is needed. Can be for the beginning just an enhanced hci_h4.c.

Regards

Marcel

--
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
Kalle Valo July 18, 2016, 7:18 a.m. UTC | #2
Larry Finger <Larry.Finger@lwfinger.net> writes:

> The Realtek RTL8723BS chips, which are connected via SDIO, also contain
> a serial Bluetooth device. In order for BT to work, the device must be
> added to the acpi_device_id table.
>
> Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
> ---
>
> I have no idea if this device should be added here, or if the device could
> be added to some structure supplied at device startup. If anyone has any
> suggestions, please let me know.
>
> Thanks,
>
> Larry
> ---
>  net/rfkill/rfkill-gpio.c | 1 +
>  1 file changed, 1 insertion(+)

BTW Johannes maintains rfkill, not me.
diff mbox

Patch

diff --git a/net/rfkill/rfkill-gpio.c b/net/rfkill/rfkill-gpio.c
index 76c01cb..50ca65e 100644
--- a/net/rfkill/rfkill-gpio.c
+++ b/net/rfkill/rfkill-gpio.c
@@ -163,6 +163,7 @@  static int rfkill_gpio_remove(struct platform_device *pdev)
 static const struct acpi_device_id rfkill_acpi_match[] = {
 	{ "BCM4752", RFKILL_TYPE_GPS },
 	{ "LNV4752", RFKILL_TYPE_GPS },
+	{ "OBDA8723", RFKILL_TYPE_BLUETOOTH },
 	{ },
 };
 MODULE_DEVICE_TABLE(acpi, rfkill_acpi_match);