* [PATCH 1/2] Bluetooth: hci_h5: Add suspend / resume ops
@ 2018-10-27 11:33 Hans de Goede
2018-10-27 11:33 ` [PATCH 2/2] Bluetooth: hci_h5: Turn off RTL8723BS on suspend, reprobe on resume Hans de Goede
0 siblings, 1 reply; 3+ messages in thread
From: Hans de Goede @ 2018-10-27 11:33 UTC (permalink / raw)
To: Marcel Holtmann, Johan Hedberg
Cc: Hans de Goede, Jeremy Cline, linux-bluetooth, linux-serial, linux-acpi
Add support for vendor specific suspend / resume callbacks.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/bluetooth/hci_h5.c | 29 +++++++++++++++++++++++++++++
1 file changed, 29 insertions(+)
diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
index 8eede1197cd2..beddb89a00f2 100644
--- a/drivers/bluetooth/hci_h5.c
+++ b/drivers/bluetooth/hci_h5.c
@@ -115,6 +115,8 @@ struct h5_vnd {
int (*setup)(struct h5 *h5);
void (*open)(struct h5 *h5);
void (*close)(struct h5 *h5);
+ int (*suspend)(struct h5 *h5);
+ int (*resume)(struct h5 *h5);
const struct acpi_gpio_mapping *acpi_gpio_map;
};
@@ -841,6 +843,28 @@ static void h5_serdev_remove(struct serdev_device *serdev)
hci_uart_unregister_device(&h5->serdev_hu);
}
+static int __maybe_unused h5_serdev_suspend(struct device *dev)
+{
+ struct h5 *h5 = dev_get_drvdata(dev);
+ int ret = 0;
+
+ if (h5->vnd && h5->vnd->suspend)
+ ret = h5->vnd->suspend(h5);
+
+ return ret;
+}
+
+static int __maybe_unused h5_serdev_resume(struct device *dev)
+{
+ struct h5 *h5 = dev_get_drvdata(dev);
+ int ret = 0;
+
+ if (h5->vnd && h5->vnd->resume)
+ ret = h5->vnd->resume(h5);
+
+ return ret;
+}
+
#ifdef CONFIG_BT_HCIUART_RTL
static int h5_btrtl_setup(struct h5 *h5)
{
@@ -935,12 +959,17 @@ static const struct acpi_device_id h5_acpi_match[] = {
MODULE_DEVICE_TABLE(acpi, h5_acpi_match);
#endif
+static const struct dev_pm_ops h5_serdev_pm_ops = {
+ SET_SYSTEM_SLEEP_PM_OPS(h5_serdev_suspend, h5_serdev_resume)
+};
+
static struct serdev_device_driver h5_serdev_driver = {
.probe = h5_serdev_probe,
.remove = h5_serdev_remove,
.driver = {
.name = "hci_uart_h5",
.acpi_match_table = ACPI_PTR(h5_acpi_match),
+ .pm = &h5_serdev_pm_ops,
},
};
--
2.19.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] Bluetooth: hci_h5: Turn off RTL8723BS on suspend, reprobe on resume
2018-10-27 11:33 [PATCH 1/2] Bluetooth: hci_h5: Add suspend / resume ops Hans de Goede
@ 2018-10-27 11:33 ` Hans de Goede
2018-10-27 11:57 ` Bastien Nocera
0 siblings, 1 reply; 3+ messages in thread
From: Hans de Goede @ 2018-10-27 11:33 UTC (permalink / raw)
To: Marcel Holtmann, Johan Hedberg
Cc: Hans de Goede, Jeremy Cline, linux-bluetooth, linux-serial, linux-acpi
On many devices the RTL8723BS device gets reset during suspend/resume,
causing it to loose its firmware and all state.
Testing has shown it drops back to communicating at 115200 bps and sends
sync-request packages, indicating it has been fully reset.
This commit fixes this by queueing a reprobe on resume.
This mirrors how USB RTL BT devices, which have the same problem, are
handled in the btusb driver, there we set the USB_QUIRK_RESET_RESUME for
all RTL devices, which also causes a reprobe on resume. The only difference
is that here we need to do the reprobe ourselves.
Since we are doing a full reprobe on resume now, we can also turn of the
device on suspend to save power while suspended.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/bluetooth/hci_h5.c | 52 ++++++++++++++++++++++++++++++++++++++
1 file changed, 52 insertions(+)
diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
index beddb89a00f2..654987b47051 100644
--- a/drivers/bluetooth/hci_h5.c
+++ b/drivers/bluetooth/hci_h5.c
@@ -931,6 +931,56 @@ static void h5_btrtl_close(struct h5 *h5)
gpiod_set_value_cansleep(h5->enable_gpio, 0);
}
+/* Suspend/resume support. On many devices the RTL BT device looses power during
+ * suspend/resume, causing it to loose its firmware and all state. So we simply
+ * turn it off on suspend and reprobe on resume. This mirrors how RTL devices
+ * are handled in the USB driver, where the USB_QUIRK_RESET_RESUME is used which
+ * also causes a reprobe on resume.
+ */
+static int h5_btrtl_suspend(struct h5 *h5)
+{
+ serdev_device_set_flow_control(h5->hu->serdev, false);
+ gpiod_set_value_cansleep(h5->device_wake_gpio, 0);
+ gpiod_set_value_cansleep(h5->enable_gpio, 0);
+ return 0;
+}
+
+struct h5_btrtl_reprobe {
+ struct device *dev;
+ struct work_struct work;
+};
+
+static void h5_btrtl_reprobe_worker(struct work_struct *work)
+{
+ struct h5_btrtl_reprobe *reprobe =
+ container_of(work, struct h5_btrtl_reprobe, work);
+ int ret;
+
+ ret = device_reprobe(reprobe->dev);
+ if (ret && ret != -EPROBE_DEFER)
+ dev_err(reprobe->dev, "Reprobe error %d\n", ret);
+
+ put_device(reprobe->dev);
+ kfree(reprobe);
+ module_put(THIS_MODULE);
+}
+
+static int h5_btrtl_resume(struct h5 *h5)
+{
+ struct h5_btrtl_reprobe *reprobe;
+
+ reprobe = kzalloc(sizeof(*reprobe), GFP_KERNEL);
+ if (!reprobe)
+ return -ENOMEM;
+
+ __module_get(THIS_MODULE);
+
+ INIT_WORK(&reprobe->work, h5_btrtl_reprobe_worker);
+ reprobe->dev = get_device(&h5->hu->serdev->dev);
+ queue_work(system_long_wq, &reprobe->work);
+ return 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 };
@@ -945,6 +995,8 @@ static struct h5_vnd rtl_vnd = {
.setup = h5_btrtl_setup,
.open = h5_btrtl_open,
.close = h5_btrtl_close,
+ .suspend = h5_btrtl_suspend,
+ .resume = h5_btrtl_resume,
.acpi_gpio_map = acpi_btrtl_gpios,
};
#endif
--
2.19.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 2/2] Bluetooth: hci_h5: Turn off RTL8723BS on suspend, reprobe on resume
2018-10-27 11:33 ` [PATCH 2/2] Bluetooth: hci_h5: Turn off RTL8723BS on suspend, reprobe on resume Hans de Goede
@ 2018-10-27 11:57 ` Bastien Nocera
0 siblings, 0 replies; 3+ messages in thread
From: Bastien Nocera @ 2018-10-27 11:57 UTC (permalink / raw)
To: Hans de Goede, Marcel Holtmann, Johan Hedberg
Cc: Jeremy Cline, linux-bluetooth, linux-serial, linux-acpi
On Sat, 2018-10-27 at 13:33 +0200, Hans de Goede wrote:
> On many devices the RTL8723BS device gets reset during
> suspend/resume,
> causing it to loose its firmware and all state.
"lose" (and in other places)
> Testing has shown it drops back to communicating at 115200 bps and
> sends
> sync-request packages, indicating it has been fully reset.
>
> This commit fixes this by queueing a reprobe on resume.
>
> This mirrors how USB RTL BT devices, which have the same problem, are
> handled in the btusb driver, there we set the USB_QUIRK_RESET_RESUME
> for
> all RTL devices, which also causes a reprobe on resume. The only
> difference
> is that here we need to do the reprobe ourselves.
>
> Since we are doing a full reprobe on resume now, we can also turn of
"off".
> the
> device on suspend to save power while suspended.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
> drivers/bluetooth/hci_h5.c | 52
> ++++++++++++++++++++++++++++++++++++++
> 1 file changed, 52 insertions(+)
>
> diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c
> index beddb89a00f2..654987b47051 100644
> --- a/drivers/bluetooth/hci_h5.c
> +++ b/drivers/bluetooth/hci_h5.c
> @@ -931,6 +931,56 @@ static void h5_btrtl_close(struct h5 *h5)
> gpiod_set_value_cansleep(h5->enable_gpio, 0);
> }
>
> +/* Suspend/resume support. On many devices the RTL BT device looses
> power during
> + * suspend/resume, causing it to loose its firmware and all state.
> So we simply
> + * turn it off on suspend and reprobe on resume. This mirrors how
> RTL devices
> + * are handled in the USB driver, where the USB_QUIRK_RESET_RESUME
> is used which
> + * also causes a reprobe on resume.
> + */
> +static int h5_btrtl_suspend(struct h5 *h5)
> +{
> + serdev_device_set_flow_control(h5->hu->serdev, false);
> + gpiod_set_value_cansleep(h5->device_wake_gpio, 0);
> + gpiod_set_value_cansleep(h5->enable_gpio, 0);
> + return 0;
> +}
> +
> +struct h5_btrtl_reprobe {
> + struct device *dev;
> + struct work_struct work;
> +};
> +
> +static void h5_btrtl_reprobe_worker(struct work_struct *work)
> +{
> + struct h5_btrtl_reprobe *reprobe =
> + container_of(work, struct h5_btrtl_reprobe, work);
> + int ret;
> +
> + ret = device_reprobe(reprobe->dev);
> + if (ret && ret != -EPROBE_DEFER)
> + dev_err(reprobe->dev, "Reprobe error %d\n", ret);
> +
> + put_device(reprobe->dev);
> + kfree(reprobe);
> + module_put(THIS_MODULE);
> +}
> +
> +static int h5_btrtl_resume(struct h5 *h5)
> +{
> + struct h5_btrtl_reprobe *reprobe;
> +
> + reprobe = kzalloc(sizeof(*reprobe), GFP_KERNEL);
> + if (!reprobe)
> + return -ENOMEM;
> +
> + __module_get(THIS_MODULE);
> +
> + INIT_WORK(&reprobe->work, h5_btrtl_reprobe_worker);
> + reprobe->dev = get_device(&h5->hu->serdev->dev);
> + queue_work(system_long_wq, &reprobe->work);
> + return 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 };
> @@ -945,6 +995,8 @@ static struct h5_vnd rtl_vnd = {
> .setup = h5_btrtl_setup,
> .open = h5_btrtl_open,
> .close = h5_btrtl_close,
> + .suspend = h5_btrtl_suspend,
> + .resume = h5_btrtl_resume,
> .acpi_gpio_map = acpi_btrtl_gpios,
> };
> #endif
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-10-27 11:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-27 11:33 [PATCH 1/2] Bluetooth: hci_h5: Add suspend / resume ops Hans de Goede
2018-10-27 11:33 ` [PATCH 2/2] Bluetooth: hci_h5: Turn off RTL8723BS on suspend, reprobe on resume Hans de Goede
2018-10-27 11:57 ` Bastien Nocera
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.