[PATCH] Bluetooth: hci_sync: Fix not setting adv set duration

[PATCH] Bluetooth: hci_sync: Fix not setting adv set duration

[Luiz Augusto von Dentz]

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

10bbffa3e88e attempted to fix the use of rotation duration as
advertising duration but it didn't change the if condition which still
uses the duration instead of the timeout.

Fixes: 10bbffa3e88e ("Bluetooth: Fix using advertising instance duration as timeout")
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
---
 net/bluetooth/hci_sync.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/bluetooth/hci_sync.c b/net/bluetooth/hci_sync.c
index fb7ecf24b89a..b794605dc882 100644
--- a/net/bluetooth/hci_sync.c
+++ b/net/bluetooth/hci_sync.c
@@ -895,7 +895,7 @@ int hci_enable_ext_advertising_sync(struct hci_dev *hdev, u8 instance)
 	/* Set duration per instance since controller is responsible for
 	 * scheduling it.
 	 */
-	if (adv && adv->duration) {
+	if (adv && adv->timeout) {
 		u16 duration = adv->timeout * MSEC_PER_SEC;
 
 		/* Time = N * 10 ms */
-- 
2.31.1

Re: [PATCH] Bluetooth: hci_sync: Fix not setting adv set duration

[Marcel Holtmann]

Hi Luiz,

> 10bbffa3e88e attempted to fix the use of rotation duration as
> advertising duration but it didn't change the if condition which still
> uses the duration instead of the timeout.
> 
> Fixes: 10bbffa3e88e ("Bluetooth: Fix using advertising instance duration as timeout")
> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
> ---
> net/bluetooth/hci_sync.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

patch has been applied to bluetooth-next tree.

Regards

Marcel