[RFC PATCH] Bluetooth: hci_vhci: Fix to set force_wakeup value

[RFC PATCH] Bluetooth: hci_vhci: Fix to set force_wakeup value

[Tedd Ho-Jeong An]

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

This patch sets the wakeup value in the vhci driver data after reading
the value.

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: [RFC PATCH] Bluetooth: hci_vhci: Fix to set force_wakeup value

[Marcel Holtmann]

Hi Tedd,

> This patch sets the wakeup value in the vhci driver data after reading
> the value.
> 
> 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;

patch looks good, but the commit message is confusing.

Regards

Marcel

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

[Luiz Augusto von Dentz]

Hi Marcel, Tedd,

On Sat, Oct 2, 2021 at 2:04 PM Marcel Holtmann <marcel@holtmann.org> wrote:
>
> Hi Tedd,
>
> > This patch sets the wakeup value in the vhci driver data after reading
> > the value.
> >
> > 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;
>
> patch looks good, but the commit message is confusing.

Ive ended up fixing the original patch since I had to force push due
to the pull request problem with committer/sign-off problem.

> Regards
>
> Marcel
>


-- 
Luiz Augusto von Dentz