Update extended advert error

RE: Update extended advert error

Sigurd — Wed, 05 Oct 2022 10:38:44 GMT

Hi,

I have attached my sample I used to try to recreate the issue.

For me update_advert_data() seem to be working fine.

devzone.nordicsemi.com/.../multiple_5F00_adv_5F00_sets_5F00_test.zip

RE: Update extended advert error

billywalton — Wed, 05 Oct 2022 08:36:40 GMT

I have even tried to delete the sets using the following code, with the view to recreate them:

but bt_le_ext_adv_delete errors with a -5 (EIO?)

so maybe there is some fundamental problem with the ext_ad construction

RE: Update extended advert error

billywalton — Wed, 05 Oct 2022 08:23:22 GMT

Hi,

Many thanks, i have run out of time now and am being pressed for a deliverable so would certainly appreciate a way forward.

RE: Update extended advert error

Sigurd — Wed, 05 Oct 2022 08:18:35 GMT

Hi,

I have tried to combine your code here with the multiple_adv_sets sample, but here it fails already with the first call at advertising_set_create(). I think it have something to do with how ext_ad is construed, so I'm looking into that now.

RE: Update extended advert error

billywalton — Wed, 05 Oct 2022 07:45:45 GMT

Everybody gone again?

RE: Update extended advert error

billywalton — Thu, 29 Sep 2022 07:31:55 GMT

Hi Sigurd,

Did you have any luck with this issue yet? is there anything else you need?

Thanks

Billy

RE: Update extended advert error

billywalton — Mon, 26 Sep 2022 09:03:13 GMT

typedef struct __attribute__((packed)) ble_network_manufacturer_flags
{
	uint8_t                       ProhibitConnect:1;
    uint8_t                       Schedule:2;
    uint8_t                       unused:1;
    uint8_t                       TXPower:4;
	

} ble_network_manufacturer_flags_t;

RE: Update extended advert error

billywalton — Mon, 26 Sep 2022 09:01:27 GMT

typedef struct __attribute__((packed)) ble_network_manufacturer_data
{
	uint8_t									LismoreID_0;
	uint8_t             					LismoreID_1;
	uint8_t 					   			DeviceType;
	uint8_t 					   			ChannelID;
	uint8_t 					   			AddressID;
	uint8_t 					   			EventID;
	uint16_t 					   			UserID;
    uint16_t                                PreviousUserID;
	ble_network_manufacturer_flags_t 		Flags;
	uint8_t 					   			CurrentPriority;
	int8_t 					   				CalibratedRSSI;
	int8_t 					   				MinimumRSSI;

    uint32_t                                NetworkCRC;



} ble_network_manufacturer_data_t;

RE: Update extended advert error

Sigurd — Mon, 26 Sep 2022 08:55:40 GMT

Hi,

[quote user="billywalton"]Using the code at the top of this ticket[/quote]

I got error: unknown type name 'ble_network_manufacturer_data_t' when building.

Could you post the code that shows how you have defined ble_network_manufacturer_data_t as well?

RE: Update extended advert error

billywalton — Mon, 26 Sep 2022 07:36:07 GMT

Hi,

Many thanks for the reply, CONFIG_BT_EXT_ADV_MAX_ADV_SET is set to two, as all i want is two advertisements, i do not want to add another advertisement, just amend the values (not length) od the current ones:

Using the code at the top of this ticket, setup and start the adverts - everything works perfectly, and i see the two adverts with correct data

now change the advert data, for example

m_network_manufacturer_data.NetworkCRC = 0x12345678

now call update_advert_data() - this is where the error appears; rather than updating the current adverts, it seems to be trying to make them longer or adding a new one

RE: Update extended advert error

Sigurd — Fri, 23 Sep 2022 12:23:06 GMT

Hi,

Sorry for the delay. Joakim is out of office. This case have been assigned to me.

What did you set CONFIG_BT_EXT_ADV_MAX_ADV_SET to? Could to try to increase that value?

If increasing the value does not work, would it be possible for you to create a small sample that reproduces this issue, that I could have a look at? I think that might be the fastest way to solve this now.

RE: Update extended advert error

billywalton — Tue, 20 Sep 2022 12:55:18 GMT

Gentle bump.... Its been 11 days

Is it possible to assign this ticket to someone who has a little more time on their hands please?

RE: Update extended advert error

billywalton — Fri, 09 Sep 2022 20:36:56 GMT

Even when i start and stop the adverts with the following code, i get the same error

RE: Update extended advert error

billywalton — Thu, 08 Sep 2022 14:47:07 GMT

The advertisement works fine when initially started, everything is great

I want to update the adverts manufacturing data value, the size is never changed, its the second call to the bt_le_ext_adv_set_data function that causes the error, not the first call. If it was a size problem, then the first call would also fail, the same array is being used in both calls

I suspect the bt_le_ext_adv_set_data call is trying to pend the array onto the end of the array from the first call, rather than remove the array and replace it - thus increasing the size - but this is not what i want.

All i want to do is update the values being advertised by the first call to bt_le_ext_adv_set_data - how can i do this?

RE: Update extended advert error

Joakim Jakobsen — Thu, 08 Sep 2022 14:06:39 GMT

Hi.

If the advertising set has been configured for extended advertising, then the maximum data length is defined by the controller with the maximum possible of BT_GAP_ADV_MAX_EXT_ADV_DATA_LEN bytes.

When updating the advertising data while advertising the advertising data and scan response data length must be smaller or equal to what can be fit in a single advertising packet. Otherwise the advertiser must be stopped.

Br,
Joakim

RE: Update extended advert error

Joakim Jakobsen — Thu, 08 Sep 2022 09:45:56 GMT

Hi.

Sorry about the delay.

I'll look at your ticket and get back to you by the end of day today!

Br,
Joakim

RE: Update extended advert error

billywalton — Thu, 08 Sep 2022 07:52:16 GMT

anybody there?

RE: Update extended advert error

billywalton — Sun, 04 Sep 2022 20:12:41 GMT

Just like to add to this, when i edit the array data i am not increasing the size of the array, i am only changing the value of an array member - is the call to bt_le_ext_adv_set_data trying to append the entire array onto the end of the current array?

What i want to do is replace it, not append it