Customising a bluetooth Zephyr sample

RE: Customising a bluetooth Zephyr sample

adam_atmo — Tue, 23 Aug 2022 01:07:39 GMT

Great, thank you Simon.

RE: Customising a bluetooth Zephyr sample

Simonr — Mon, 22 Aug 2022 13:56:48 GMT

A) When a config has "dependencies" it is dependent on those listed configurations to be valid in a project. I'm not sure how much of a definition you're looking for to be honest.

C) Okay, I double checked this on my end, and there seems to be a known issue here, that was just recently added to the main branch. So if you're using NCS version 2.0.2 or older these BT_CTLR_TX_PWR won't work properly, and you have to use the dynamic TX power control. Sorry about that. So either update to the main branch or use the dynamic power control.

2. As long as you also enable the necessary configs, CONFIG_BT_CTLR_PHY_CODED=y & CONFIG_BT_EXT_ADV=y, I believe so. But make sure that you don't have any other "bt_le_adv_param" structs in the sample that can overwrite or conflict with this.

Best regards,

Simon

RE: Customising a bluetooth Zephyr sample

adam_atmo — Mon, 22 Aug 2022 02:30:07 GMT

Hi Simon,

1. Thank you - I have a few questions/issues regarding this:

A. What is the definition of a dependency with regards to a .conf file?

B. Must the dependency be added into the prj.conf file? If so, this does not solve the problem. There is an error when setting CONFIG_HAS_HW_NRF_RADIO_TX_PWR_HIGH=y. This seems to be an issue that was also identified in this post but no clear answer was suggested.

C. Are you sure it is possible to configure the TX power using the prj.conf alone (this would be most convenient) or do additional functions need to be used in the source code?

2. I am referring to the snippet that you posted - can I add this create_advertising_coded function into the source code in addition to the existing advertising functions in the Bluetooth: Peripheral HT sample?

Thanks again for the assistance,

Adam

RE: Customising a bluetooth Zephyr sample

Simonr — Thu, 18 Aug 2022 08:30:21 GMT

Hi Adam

1. Indeed, you also need to include the dependencies for a config you set, so you also have to set CONFIG_HAS_HW_NRF_RADIO_TX_PWR_HIGH=Y in your config file as well.

2. What is "this function"?

Best regards,

Simon

RE: Customising a bluetooth Zephyr sample

adam_atmo — Wed, 17 Aug 2022 11:29:27 GMT

Thanks Simon,

1. I did try that and it doesn't seem to change the TX power value when I tested it. I added the following line into the prj.conf file: CONFIG_BT_CTLR_TX_PWR_PLUS_8=y. I get the following message next to this line: In build_833: n.

Furthermore, I get the following problem message in VS Code: CONFIG_BT_CTLR_TX_PWR_PLUS_8 was assigned the value y, but got the value n. Missing dependencies: HAS_HW_NRF_RADIO_TX_PWR_HIGH.

2. Can I add this function into the source code in addition to the existing advertising functions in the Bluetooth: Peripheral HT sample?

Thanks very much,

Adam

RE: Customising a bluetooth Zephyr sample

Simonr — Wed, 17 Aug 2022 10:48:07 GMT

1. Okay, to just set the TX power statically to +8dBm you can add the CONFIG_BT_CTLR_TX_PWR_PLUS_8 to the Kconfig file of your project.

2. You can set up a Coded PHY advertising with something like this snippet from the peripheral_hr_coded sample:

static int create_advertising_coded(void)
{
	int err;
	struct bt_le_adv_param param =
		BT_LE_ADV_PARAM_INIT(BT_LE_ADV_OPT_CONNECTABLE |
				     BT_LE_ADV_OPT_EXT_ADV |
				     BT_LE_ADV_OPT_CODED,
				     BT_GAP_ADV_FAST_INT_MIN_2,
				     BT_GAP_ADV_FAST_INT_MAX_2,
				     NULL);

	err = bt_le_ext_adv_create(&param, NULL, &adv);
	if (err) {
		printk("Failed to create advertiser set (%d)\n", err);
		return err;
	}

	printk("Created adv: %p\n", adv);

	err = bt_le_ext_adv_set_data(adv, ad, ARRAY_SIZE(ad), NULL, 0);
	if (err) {
		printk("Failed to set advertising data (%d)\n", err);
		return err;
	}

	return 0;
}

Yes, the Coded PHY sample should be possible to build for the nRF52833 as well, we've just not set it up and tested it on our end, so we can't add it to the sample page.

Best regards,

Simon

RE: Customising a bluetooth Zephyr sample

adam_atmo — Wed, 17 Aug 2022 02:39:14 GMT

Thanks for the help Simon,

1. That is for the HCI power control application where the TX power is modulated according to the RSSI. How can I simply set the default power to 8dBm in the Bluetooth: Peripheral HT sample from Zephyr?

2. Can you please be more specific as to what advertising initialization needs to change in the Bluetooth: Peripheral HT sample? Do I need to change the bt_ready function (code attached below)?

Furthermore, under requirements for the Peripheral Heart Rate Monitor with Coded PHY sample, it does not support the nRF52833 DK which I am using. Surely this sample can be built for the nRF52833 as it supports Coded PHY?

static void bt_ready(void)
	{
	int err;

	printk("Bluetooth initialized\n");

	hts_init();

	err = bt_le_adv_start(BT_LE_ADV_CONN_NAME, ad, ARRAY_SIZE(ad), NULL, 0);
	if (err) {
	printk("Advertising failed to start (err %d)\n", err);
	return;
	}

	printk("Advertising successfully started\n");
	}

RE: Customising a bluetooth Zephyr sample

Simonr — Tue, 16 Aug 2022 09:26:35 GMT

Hi Adam

1. In this case, my colleague Hung has explained how to set the TX power in the nRF Connect SDK (NCS).

2. In order to enable Coded PHY, enable the following Kconfigs in your project:

CONFIG_BT_CTLR_PHY_CODED=y
CONFIG_BT_EXT_ADV=y

You can also check out the Peripheral Heart Rate Monitor with Coded PHY project that already implements Coded PHY. The Coded PHY and extended advertising also needs to be set in the advertising initialization I believe.

Best regards,

Simon