[PATCH V2] Bluetooth: hci_vhci: Fix to set the force_wakeup value

[PATCH V2] Bluetooth: hci_vhci: Fix to set the force_wakeup value

[Tedd Ho-Jeong An]

From: Tedd Ho-Jeong An <tedd.an@intel.com>

This patch sets the wakeup state of the vhci driver when the
force_wakeup is updated.

Signed-off-by: Tedd Ho-Jeong An <tedd.an@intel.com>
---
 drivers/bluetooth/hci_vhci.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/bluetooth/hci_vhci.c b/drivers/bluetooth/hci_vhci.c
index b45db0db347c..5fd91106e853 100644
--- a/drivers/bluetooth/hci_vhci.c
+++ b/drivers/bluetooth/hci_vhci.c
@@ -176,6 +176,8 @@ static ssize_t force_wakeup_write(struct file *file,
 	if (data->wakeup == enable)
 		return -EALREADY;
 
+	data->wakeup = enable;
+
 	return count;
 }
 
-- 
2.25.1

Re: [PATCH V2] Bluetooth: hci_vhci: Fix to set the force_wakeup value

[Luiz Augusto von Dentz]

Hi Tedd,

On Mon, Oct 4, 2021 at 3:54 PM Tedd Ho-Jeong An <hj.tedd.an@gmail.com> wrote:
>
> From: Tedd Ho-Jeong An <tedd.an@intel.com>
>
> This patch sets the wakeup state of the vhci driver when the
> force_wakeup is updated.
>
> Signed-off-by: Tedd Ho-Jeong An <tedd.an@intel.com>
> ---
>  drivers/bluetooth/hci_vhci.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/bluetooth/hci_vhci.c b/drivers/bluetooth/hci_vhci.c
> index b45db0db347c..5fd91106e853 100644
> --- a/drivers/bluetooth/hci_vhci.c
> +++ b/drivers/bluetooth/hci_vhci.c
> @@ -176,6 +176,8 @@ static ssize_t force_wakeup_write(struct file *file,
>         if (data->wakeup == enable)
>                 return -EALREADY;
>
> +       data->wakeup = enable;
> +
>         return count;
>  }
>
> --
> 2.25.1

https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git/commit/?id=43f51bd946dab69bb38b05863898dd711b7e4506

It already contains this change as well.


-- 
Luiz Augusto von Dentz

Re: [PATCH V2] Bluetooth: hci_vhci: Fix to set the force_wakeup value

[Marcel Holtmann]

Hi Tedd,

> This patch sets the wakeup state of the vhci driver when the
> force_wakeup is updated.
> 
> Signed-off-by: Tedd Ho-Jeong An <tedd.an@intel.com>
> ---
> drivers/bluetooth/hci_vhci.c | 2 ++
> 1 file changed, 2 insertions(+)

patch has been applied to bluetooth-next tree.

Regards

Marcel