NCS1.9.0 or NCS1.9.1 periodic_adv work error in nRF5340-DK

[yoyou]

Hello,

NCS1.9.0 or NCS1.9.1,  Windows10 X64,VS Code,periodic_adv.default codes

with nRF52840-DK work fine.

but nRF5340-DK got error,as folow logs

"Failed to create advertising set (err -5)"

*** Booting Zephyr OS build v2.7.99-ncs1  ***
Starting Periodic Advertising Demo
Failed to create advertising set (err -5)
[00:00:00.283,355] <inf> bt_hci_core: HW Platform: Nordic Semiconductor (0x0002)
[00:00:00.283,355] <inf> bt_hci_core: HW Variant: nRF53x (0x0003)
[00:00:00.283,386] <inf> bt_hci_core: Firmware: Standard Bluetooth controller (0x00) Version 14.50663 Build 1008232294
[00:00:00.286,712] <inf> bt_hci_core: Identity: C7:71:77:EB:8D:C1 (random)
[00:00:00.286,712] <inf> bt_hci_core: HCI: version 5.2 (0x0b) revision 0x22fe, manufacturer 0x0059
[00:00:00.286,712] <inf> bt_hci_core: LMP: version 5.2 (0x0b) subver 0x22fe
[00:00:00.289,093] <wrn> bt_hci_core: opcode 0x2036 status 0x01

  

thanks

Best Regards

[ovrebekk]

Hi

Could you try this sample instead?

periodic_adv_5340_fix.zip

I believe the issue is that extended advertising is not set up properly for the network core, since the standard sample doesn't include the child_image configuration. 

Best regards
Torbjørn

[yoyou]

Hello

thank you very much,

it work,but can't enter sent callback function,no recv callback.

i want change next data in sent callback(when send finished).

#include <bluetooth/bluetooth.h>
#include "common/log.h"

static uint8_t mfg_data[9] = { 0xff, 0xff, 0x00 };
static struct bt_le_ext_adv_cb cb;//call back

static const struct bt_data ad[] = {
	BT_DATA(BT_DATA_MANUFACTURER_DATA, mfg_data, 9 ),
};
static const struct bt_data sd[] = {
	BT_DATA(BT_DATA_MANUFACTURER_DATA, mfg_data, 9 ),
};

void sent(struct bt_le_ext_adv *adv, struct bt_le_ext_adv_sent_info *info)
{
        printk("Sent Data...");
}
void connected(struct bt_le_ext_adv *adv, struct bt_le_ext_adv_connected_info *info)
{
      printk("connected...");
}
void scanned(struct bt_le_ext_adv *adv, struct bt_le_ext_adv_scanned_info *info)
{
      printk("scanned...");
}

void main(void)
{
	struct bt_le_ext_adv *adv;
        cb.sent=sent;//call back
        cb.connected=connected;//call back
        cb.scanned=scanned;//call back
	int err;

	printk("Starting Periodic Advertising Demo\n");

	// Initialize the Bluetooth Subsystem
	err = bt_enable(NULL);
	if (err) {
		printk("Bluetooth init failed (err %d)\n", err);
		return;
	}
 
        // Non-connectable extended advertising with @ref BT_LE_ADV_OPT_USE_NAME BT_LE_ADV_OPT_CODED
       #define BT_LE_EXT_ADV_NCONN_NAME_CODED  BT_LE_ADV_PARAM(BT_LE_ADV_OPT_EXT_ADV | \
						 BT_LE_ADV_OPT_USE_NAME | BT_LE_ADV_OPT_USE_TX_POWER | \
                                                 BT_LE_ADV_OPT_CODED, \
						 BT_GAP_ADV_FAST_INT_MIN_1, \
						 BT_GAP_ADV_FAST_INT_MAX_1, \
						 NULL)   //Peer address, set to NULL for undirected advertising
	// Create a non-connectable non-scannable unDIRECTED advertising set
	err = bt_le_ext_adv_create(BT_LE_EXT_ADV_NCONN_NAME_CODED, &cb, &adv);
	if (err) {
		printk("Failed to create advertising set (err %d)\n", err);
		return;
	}

        #define BT_LE_PER_ADV_P   BT_LE_PER_ADV_PARAM(BT_GAP_PER_ADV_FAST_INT_MIN_1, \
						  BT_GAP_PER_ADV_FAST_INT_MIN_1, \
						  BT_LE_PER_ADV_OPT_USE_TX_POWER)
	// Set periodic advertising parameters
	err = bt_le_per_adv_set_param(adv,BT_LE_PER_ADV_DEFAULT);
	if (err) {
		printk("Failed to set periodic advertising parameters (err %d)\n", err);
		return;
	}
        err = bt_le_ext_adv_set_data(adv,ad, ARRAY_SIZE(ad),sd, ARRAY_SIZE(sd) );
	if (err) {
		printk("Failed to set  bt_le_ext_adv_set_data (err %d)\n", err);
		return;
	}

	// Enable Periodic Advertising
	err = bt_le_per_adv_start(adv);
	if (err) {
		printk("Failed to enable periodic advertising (err %d)\n", err);
		return;
	}

	// Start extended advertising
	err = bt_le_ext_adv_start(adv, BT_LE_EXT_ADV_START_DEFAULT);
	if (err) {
		printk("Failed to start extended advertising (err %d)\n", err);
		return;
	}

    for(int i=0;i<sizeof(mfg_data);i++){mfg_data[i]=0;}
	while (true) 
	{
		k_sleep(K_SECONDS(1));

		mfg_data[2]++;

		printk("Set Periodic Advertising Data...");
		err = bt_le_per_adv_set_data(adv, ad, ARRAY_SIZE(ad));
		if (err) {
			printk("Failed (err %d)\n", err);
			return;
		}
		printk("done.\n");
	}
}

  

thanks

Best Regards

[ovrebekk]

Hi 

Have you configured a timeout on the advertising?

Looking at your code I can't see that you configure a timeout anywhere. 

As explained here the sent callback won't be triggered until the advertising times out, which won't happen unless you enable it. 

Best regards
Torbjørn

[yoyou]

Hi,

how to configured a timeout？is it.

the starting example code  is not perfect, some difficult for novice,

can help debug the basic function:

1. bt_le_per_adv_set_data() in sent function;

     prepare the next data at finished last sending；

2. set Peer address, for directed advertising;

    for only some one hear;

3. enable LE_CODE for long rang;

    for low rate 125kbit/s;

4. periodic_sync can recv directed advertising data;

     now it can't recv data.

    

thanks

Best Regards

[ovrebekk]

Hi 

The second parameter of the bt_le_ext_adv_start(..) function provides a pointer to a struct of type bt_le_ext_adv_start_param, which allows you to define the timeout either as a time (in 10ms units), or as a number of advertising events:

struct bt_le_ext_adv_start_param {
	/**
	 * @brief Advertiser timeout (N * 10 ms).
	 *
	 * Application will be notified by the advertiser sent callback.
	 * Set to zero for no timeout.
	 *
	 * When using high duty cycle directed connectable advertising then
	 * this parameters must be set to a non-zero value less than or equal
	 * to the maximum of @ref BT_GAP_ADV_HIGH_DUTY_CYCLE_MAX_TIMEOUT.
	 *
	 * If privacy @kconfig{CONFIG_BT_PRIVACY} is enabled then the timeout
	 * must be less than @kconfig{CONFIG_BT_RPA_TIMEOUT}.
	 */
	uint16_t timeout;
	/**
	 * @brief Number of advertising events.
	 *
	 * Application will be notified by the advertiser sent callback.
	 * Set to zero for no limit.
	 */
	uint8_t  num_events;
};

If you create a struct like this, define either the timeout or num_event fields as you want, and provide a pointer to the structure in the bt_le_ext_adv_start(..) call, then the advertising should time out accordingly. 

Best regards
Torbjørn

[yoyou]

Hi,

my project directed advertising low quality audio（2kByte/s), One-way communication,no reply.

Because need to send continuous audio data, so prepare the next data in sent function. 

data must be send one by one,

it means :

send next data at finshed last data sending，

not at timeout.

as:

i'll periodic_adv send "123456789",  three bytes are sent at a time.

1.bt_le_per_adv_set_data("123")

2.when "123" send finshed, in sent() function

   bt_le_per_adv_set_data("456")

3.when "456" send finshed, in sent() function

   bt_le_per_adv_set_data("789")

 

Make sure periodic_sync receives correctly "123456789".

Error received:

"123123456789"

"456123789"

"789123"

"123789"

You know what I mean?

  

thanks

Best Regards

[ovrebekk]

Hi

Most likely you need to include some kind of sequence counter in the payload, so that the receiver knows when it is receiving a repeated payload, and when it is receiving new data. Then the receiver should be able to assemble the data in the right order, and also recognize if it has lost some packets completely (if the sequence number suddenly increases by more than one). 

The sequence number is essentially just a counter that you increment every time you are updating the payload with new data. 

Best regards
Torbjørn

[yoyou]

Hi,

thanks for reply,

I know how to do the project of ble_app_uart in nRF SDK,

but i do not know how to do in periodic_adv/sync NCS.

i just want to get a callback when send one data finished,

and then immediately modify it to the next data for next advertising.

why is it difficult to achieve? NCS1.9 not support it at all.

 

Best Regards

[ovrebekk]

Hi 

I made some modifications to the periodic_adv sample in order to do this:

285510_periodic_adv.zip

Essentially I configure the sent callback, start advertising with a 3 second timeout, and reconfigure the data every time the sent callback occurs. 

Please note that I don't update the advertising directly in the sent callback, instead I use a semaphore to signal to the main loop that the advertising is ready to be restarted. 

Best regards
Torbjørn