Bluetooth: hci_h5: Add support for enable and device-wake GPIOs

Add support for the enable and device-wake GPIOs used on ACPI enumerated
RTL8723BS devices.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>

diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
index 3513dad43f3c59c7a9ed7e3b24f3033b26b25327..63c0dcbc49141c53e74b9158abbdff3e785e60a2 100644 (file)
--- a/drivers/bluetooth/hci_h5.c
+++ b/drivers/bluetooth/hci_h5.c
@@ -23,6 +23,7 @@
 
 #include <linux/acpi.h>
 #include <linux/errno.h>
+#include <linux/gpio/consumer.h>
 #include <linux/kernel.h>
 #include <linux/mod_devicetable.h>
 #include <linux/serdev.h>
@@ -105,12 +106,16 @@ struct h5 {
 
        const struct h5_vnd *vnd;
        const char *id;
+
+       struct gpio_desc *enable_gpio;
+       struct gpio_desc *device_wake_gpio;
 };
 
 struct h5_vnd {
        int (*setup)(struct h5 *h5);
        void (*open)(struct h5 *h5);
        void (*close)(struct h5 *h5);
+       const struct acpi_gpio_mapping *acpi_gpio_map;
 };
 
 static void h5_reset_rx(struct h5 *h5);
@@ -811,8 +816,21 @@ static int h5_serdev_probe(struct serdev_device *serdev)
 
                h5->vnd = (const struct h5_vnd *)match->driver_data;
                h5->id  = (char *)match->id;
+
+               if (h5->vnd->acpi_gpio_map)
+                       devm_acpi_dev_add_driver_gpios(dev,
+                                                      h5->vnd->acpi_gpio_map);
        }
 
+       h5->enable_gpio = devm_gpiod_get_optional(dev, "enable", GPIOD_OUT_LOW);
+       if (IS_ERR(h5->enable_gpio))
+               return PTR_ERR(h5->enable_gpio);
+
+       h5->device_wake_gpio = devm_gpiod_get_optional(dev, "device-wake",
+                                                      GPIOD_OUT_LOW);
+       if (IS_ERR(h5->device_wake_gpio))
+               return PTR_ERR(h5->device_wake_gpio);
+
        return hci_uart_register_device(&h5->serdev_hu, &h5p);
 }
 
@@ -875,11 +893,34 @@ static void h5_btrtl_open(struct h5 *h5)
        serdev_device_set_flow_control(h5->hu->serdev, false);
        serdev_device_set_parity(h5->hu->serdev, SERDEV_PARITY_EVEN);
        serdev_device_set_baudrate(h5->hu->serdev, 115200);
+
+       /* The controller needs up to 500ms to wakeup */
+       gpiod_set_value_cansleep(h5->enable_gpio, 1);
+       gpiod_set_value_cansleep(h5->device_wake_gpio, 1);
+       msleep(500);
 }
 
+static void h5_btrtl_close(struct h5 *h5)
+{
+       gpiod_set_value_cansleep(h5->device_wake_gpio, 0);
+       gpiod_set_value_cansleep(h5->enable_gpio, 0);
+}
+
+static const struct acpi_gpio_params btrtl_device_wake_gpios = { 0, 0, false };
+static const struct acpi_gpio_params btrtl_enable_gpios = { 1, 0, false };
+static const struct acpi_gpio_params btrtl_host_wake_gpios = { 2, 0, false };
+static const struct acpi_gpio_mapping acpi_btrtl_gpios[] = {
+       { "device-wake-gpios", &btrtl_device_wake_gpios, 1 },
+       { "enable-gpios", &btrtl_enable_gpios, 1 },
+       { "host-wake-gpios", &btrtl_host_wake_gpios, 1 },
+       {},
+};
+
 static struct h5_vnd rtl_vnd = {
        .setup          = h5_btrtl_setup,
        .open           = h5_btrtl_open,
+       .close          = h5_btrtl_close,
+       .acpi_gpio_map  = acpi_btrtl_gpios,
 };
 
 #ifdef CONFIG_ACPI