Hi Yang, On Mon, Jun 16, 2025 at 7:05 AM Yang Li via B4 Relay <devnull+yang.li.amlogic.com@xxxxxxxxxx> wrote: > > From: Yang Li <yang.li@xxxxxxxxxxx> > > When PA Create Sync is enabled, advertising resumes unexpectedly. > Therefore, it's necessary to check whether advertising is currently > active before attempting to pause it. > > < HCI Command: LE Add Device To... (0x08|0x0011) plen 7 #1345 [hci0] 48.306205 > Address type: Random (0x01) > Address: 4F:84:84:5F:88:17 (Resolvable) > Identity type: Random (0x01) > Identity: FC:5B:8C:F7:5D:FB (Static) > < HCI Command: LE Set Address Re.. (0x08|0x002d) plen 1 #1347 [hci0] 48.308023 > Address resolution: Enabled (0x01) > ... > < HCI Command: LE Set Extended A.. (0x08|0x0039) plen 6 #1349 [hci0] 48.309650 > Extended advertising: Enabled (0x01) > Number of sets: 1 (0x01) > Entry 0 > Handle: 0x01 > Duration: 0 ms (0x00) > Max ext adv events: 0 > ... > < HCI Command: LE Periodic Adve.. (0x08|0x0044) plen 14 #1355 [hci0] 48.314575 > Options: 0x0000 > Use advertising SID, Advertiser Address Type and address > Reporting initially enabled > SID: 0x02 > Adv address type: Random (0x01) > Adv address: 4F:84:84:5F:88:17 (Resolvable) > Identity type: Random (0x01) > Identity: FC:5B:8C:F7:5D:FB (Static) > Skip: 0x0000 > Sync timeout: 20000 msec (0x07d0) > Sync CTE type: 0x0000 > > Signed-off-by: Yang Li <yang.li@xxxxxxxxxxx> > --- > net/bluetooth/hci_sync.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/net/bluetooth/hci_sync.c b/net/bluetooth/hci_sync.c > index 6687f2a4d1eb..4ea5a87f017d 100644 > --- a/net/bluetooth/hci_sync.c > +++ b/net/bluetooth/hci_sync.c > @@ -2481,6 +2481,10 @@ static int hci_pause_advertising_sync(struct hci_dev *hdev) > int err; > int old_state; > > + /* If controller is not advertising we are done. */ > + if (!hci_dev_test_flag(hdev, HCI_ADVERTISING)) > + return 0; HCI_ADVERTISING is for directed advertising, it shall probably be changed to HCI_LE_ADV like in hci_disable_advertising_sync. > /* If already been paused there is nothing to do. */ > if (hdev->advertising_paused) > return 0; > > --- > base-commit: 6c31dab4ff1e1f4e0f3412efb1cfc88033358d1f > change-id: 20250616-bluetooth_adv-6496e1ca6165 > > Best regards, > -- > Yang Li <yang.li@xxxxxxxxxxx> > > -- Luiz Augusto von Dentz
