Merging nRF52 blinky into uart for peripheral as well as central role.

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Thu, 27 May 2021 09:40:15 GMT

Hi,

Thanks a lot for correct suggestion.

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

Sigurd — Wed, 26 May 2021 09:43:40 GMT

Hi,

The LBS does by default only support 1 LED/button, if you are using a lot of LEDs/Buttons, then the LBS service/characteristic implementation needs some modifications. But if you only have a few LEDs/buttons, then this approach shown in this post here might be the easiest to use:

https://devzone.nordicsemi.com/f/nordic-q-a/70023/controlling-multiple-buttons-and-leds-from-central-or-peripheral

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Fri, 21 May 2021 13:16:12 GMT

Hi,

Kindly reply to last query.

Regards,

Megha Chodhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Thu, 20 May 2021 09:03:45 GMT

Hi,

When can I expect reply?

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Fri, 14 May 2021 13:24:12 GMT

Hi,

1. Peripheral & Central side, build & test again. Both works fine. Means while pressing button1 from Peripheral, LED 3 from Central blinks.

2. After that added data send from Peripheral logic in main.c,

/**@brief Function for handling events from the button handler module.

* @param[in] pin_no The pin that the event applies to.

* @param[in] button_action The button action (press/release).

static void button_event_handler(uint8_t pin_no, uint8_t button_action)

{

ret_code_t err_code;

switch (pin_no)

{

case LEDBUTTON_BUTTON:

NRF_LOG_INFO("Send button0 state change.");

err_code = ble_lbs_on_button_change(m_conn_handle, &m_lbs, button_action);

if (err_code != NRF_SUCCESS &&

err_code != BLE_ERROR_INVALID_CONN_HANDLE &&

err_code != NRF_ERROR_INVALID_STATE &&

err_code != BLE_ERROR_GATTS_SYS_ATTR_MISSING)

{

APP_ERROR_CHECK(err_code);

}

if (m_conn_handle != BLE_CONN_HANDLE_INVALID)

{

err_code = ble_nus_data_send(&m_nus, "BT0PRES\r\n",9, m_conn_handle);

}

break;

default:

APP_ERROR_HANDLER(pin_no);

break;

}

3. Build & test again. Able to get data also on Central while pressing Button1 from Peripheral.

4. Now to add second button done below changes in main.c,

Peripheral:

#define LEDBUTTON_BUTTON1 BSP_BUTTON_1

/**@brief Function for initializing the button handler module.

static void buttons_init(void)

{

ret_code_t err_code;

//The array must be static because a pointer to it will be saved in the button handler module.

static app_button_cfg_t buttons[] =

{

{LEDBUTTON_BUTTON, false, BUTTON_PULL, button_event_handler},

{LEDBUTTON_BUTTON1, false, BUTTON_PULL, button_event_handler}

};

err_code = app_button_init(buttons, ARRAY_SIZE(buttons),

BUTTON_DETECTION_DELAY);

APP_ERROR_CHECK(err_code);

}

/**@brief Function for handling events from the button handler module.

* @param[in] pin_no The pin that the event applies to.

* @param[in] button_action The button action (press/release).

static void button_event_handler(uint8_t pin_no, uint8_t button_action)

{

ret_code_t err_code;

switch (pin_no)

{

case LEDBUTTON_BUTTON:

NRF_LOG_INFO("Send button0 state change.");

err_code = ble_lbs_on_button_change(m_conn_handle, &m_lbs, button_action);

if (err_code != NRF_SUCCESS &&

err_code != BLE_ERROR_INVALID_CONN_HANDLE &&

err_code != NRF_ERROR_INVALID_STATE &&

err_code != BLE_ERROR_GATTS_SYS_ATTR_MISSING)

{

APP_ERROR_CHECK(err_code);

}

if (m_conn_handle != BLE_CONN_HANDLE_INVALID)

{

err_code = ble_nus_data_send(&m_nus, "BT0PRES\r\n",9, m_conn_handle);

}

break;

case LEDBUTTON_BUTTON1:

NRF_LOG_INFO("Send button1 state change.");

err_code = ble_lbs_on_button_change(m_conn_handle, &m_lbs, button_action);

if (err_code != NRF_SUCCESS &&

err_code != BLE_ERROR_INVALID_CONN_HANDLE &&

err_code != NRF_ERROR_INVALID_STATE &&

err_code != BLE_ERROR_GATTS_SYS_ATTR_MISSING)

{

APP_ERROR_CHECK(err_code);

}

if (m_conn_handle != BLE_CONN_HANDLE_INVALID)

{

err_code = ble_nus_data_send(&m_nus, "BT1PRES\r\n",9, m_conn_handle);

}

break;

default:

APP_ERROR_HANDLER(pin_no);

break;

}

Central:

#define LEDBUTTON_LED1 BSP_BOARD_LED_3

/**@brief Handles events coming from the LED Button central module.

static void lbs_c_evt_handler(ble_lbs_c_t * p_lbs_c, ble_lbs_c_evt_t * p_lbs_c_evt)

{

switch (p_lbs_c_evt->evt_type)

{

case BLE_LBS_C_EVT_DISCOVERY_COMPLETE:

{

ret_code_t err_code;

err_code = ble_lbs_c_handles_assign(&m_ble_lbs_c,

p_lbs_c_evt->conn_handle,

&p_lbs_c_evt->params.peer_db);

NRF_LOG_INFO("LED Button service discovered on conn_handle 0x%x.", p_lbs_c_evt->conn_handle);

err_code = app_button_enable();

APP_ERROR_CHECK(err_code);

// LED Button service discovered. Enable notification of Button.

err_code = ble_lbs_c_button_notif_enable(p_lbs_c);

APP_ERROR_CHECK(err_code);

} break; // BLE_LBS_C_EVT_DISCOVERY_COMPLETE

case BLE_LBS_C_EVT_BUTTON_NOTIFICATION:

{

NRF_LOG_INFO("Button state changed on peer to 0x%x.", p_lbs_c_evt->params.button.button_state);

if (p_lbs_c_evt->params.button.button_state)

{

bsp_board_led_on(LEDBUTTON_LED);

}

else

{

bsp_board_led_off(LEDBUTTON_LED);

}

} break; // BLE_LBS_C_EVT_BUTTON_NOTIFICATION

case BLE_LBS_C_EVT_BUTTON1_NOTIFICATION:

{

NRF_LOG_INFO("Button state changed on peer to 0x%x.", p_lbs_c_evt->params.button.button_state);

if (p_lbs_c_evt->params.button.button_state)

{

bsp_board_led_on(LEDBUTTON_LED1);

}

else

{

bsp_board_led_off(LEDBUTTON_LED1);

}

} break; // BLE_LBS_C_EVT_BUTTON_NOTIFICATION

default:

// No implementation needed.

break;

}

5. Build & test again, observe that even though pressing second button from Peripheral & LED3 blinks from Central. Beside this LED4 from Central should glow.

For this debug into code, found that

Central:

ble_lbs_c.c

static void on_hvx(ble_lbs_c_t * p_ble_lbs_c, ble_evt_t const * p_ble_evt)

{

// Check if the event is on the link for this instance.

if (p_ble_lbs_c->conn_handle != p_ble_evt->evt.gattc_evt.conn_handle)

{

return;

}

// Check if this is a Button notification.

if (p_ble_evt->evt.gattc_evt.params.hvx.handle == p_ble_lbs_c->peer_lbs_db.button_handle)

{

if (p_ble_evt->evt.gattc_evt.params.hvx.len == 1)

{

ble_lbs_c_evt_t ble_lbs_c_evt;

ble_lbs_c_evt.evt_type = BLE_LBS_C_EVT_BUTTON_NOTIFICATION;

ble_lbs_c_evt.conn_handle = p_ble_lbs_c->conn_handle;

ble_lbs_c_evt.params.button.button_state = p_ble_evt->evt.gattc_evt.params.hvx.data[0];

p_ble_lbs_c->evt_handler(p_ble_lbs_c, &ble_lbs_c_evt);

}

In this function BLE_LBS_C_EVT_BUTTON_NOTIFICATION is assigned. So that it always executes only this case.

How to add second here i.e. BLE_LBS_C_EVT_BUTTON1_NOTIFICATION, so that both Button functionality work on individual buttons.

Used SDK 17.0.2

Peripheral :

nRF5_SDK_17.0.2_d674dde\examples\ble_peripheral

link- https://smetgroup-my.sharepoint.com/:u:/g/personal/megha_choudhari_mysmindia_com/EcAgxjSaa4ZDgGUaAkc4RccBnUajSAo8G8iTqsJl28f4gg?e=LFDyaN

Central :

nRF5_SDK_17.0.2_d674dde\examples\ble_central

link- https://smetgroup-my.sharepoint.com/:u:/g/personal/megha_choudhari_mysmindia_com/EfN0RUjYB-lCuHd5gFP7DI8BwoppdFW30F8N349diTKU2Q?e=hgkbvx

Almost is it done, kindly help here.

Looking for your reply.

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Thu, 13 May 2021 11:11:47 GMT

Waiting for reply.

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Wed, 12 May 2021 12:21:46 GMT

Hi,

I have tested files sent by you. In that from central board when key pressed, LED gets turn ON from peripheral board. But we need viceversa.

Can you help to figureout this.

Waiting for your reply.

Thanks & Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Tue, 11 May 2021 12:57:47 GMT

Hi,

Thanks, will check and get back to you.

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

Sigurd — Tue, 11 May 2021 12:54:17 GMT

Hi,

[quote userid="96290" url="~/f/nordic-q-a/74695/merging-nrf52-blinky-into-uart-for-peripheral-as-well-as-central-role/308860#308860"]

If it's working at your side, Can you resend me that sdk?

[/quote]

Here are the project files.

devzone.nordicsemi.com/.../8637.ble_5F00_app_5F00_uart_5F00_c.rar

devzone.nordicsemi.com/.../0743.ble_5F00_app_5F00_uart.rar

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Tue, 11 May 2021 12:46:47 GMT

Hi,

Have a nice day!!!

Waiting for reply. When I can expect?

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Mon, 10 May 2021 06:02:54 GMT

Hi,

Have a nice day!!!!!

Sorry to revert you late.

I have done changes in db_disc_handler(); as suggested. But still while pressing button from peripheral no LED indication observed on central board. Please find below link for video:

https://smetgroup-my.sharepoint.com/:v:/g/personal/megha_choudhari_mysmindia_com/EevKoIF0g0FHsMoe2dRAIvkBKJLytnGN5S9IfelY2uSQmw?e=IdAL1P

Also, I have checked with in main.c added just to check on button press goes to button_event_handler(); or not. But its goes to NRF_BREAKPOINT_CONDITION;

/**@brief Application main function.
 */
int main(void)
{
    bool erase_bonds;

    // Initialize.
    log_init();
    uart_init();
    leds_init();  //03/05/21     
    timers_init();
    buttons_init(); //03/05/21
//    buttons_leds_init(&erase_bonds);
    power_management_init();
    ble_stack_init();
    gap_params_init();
    gatt_init();
    services_init();
    advertising_init();
    conn_params_init();

    // Start execution.
    printf("\r\nUART started.\r\n");
    NRF_LOG_INFO("Debug logging for UART over RTT started.");
    advertising_start();

    // Enter main loop.
    for (;;)
    {
        idle_state_handle();
       //ble_nus_data_send(&m_nus,"SMET\r\n",5,m_conn_handle);
       if(LEDBUTTON_BUTTON == true)
       {
        bsp_board_led_on(LEDBUTTON_LED);
       }
    }
    
   
}

If it's working at your side, Can you resend me that sdk?

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

Sigurd — Fri, 07 May 2021 15:37:48 GMT

[quote user="MeghaC"]

But when I'm pressing button from peripheral the LED from central should turn ON. This is not happening.

[/quote]

I took a look at the code, in db_disc_handler() you need to add ble_lbs_on_db_disc_evt(&m_ble_lbs_c, p_evt); , so that the function looks like this:

/**@brief Function for handling database discovery events.
 *
 * @details This function is a callback function to handle events from the database discovery module.
 *          Depending on the UUIDs that are discovered, this function forwards the events
 *          to their respective services.
 *
 * @param[in] p_event  Pointer to the database discovery event.
 */
static void db_disc_handler(ble_db_discovery_evt_t * p_evt)
{
    ble_nus_c_on_db_disc_evt(&m_ble_nus_c, p_evt);
    ble_lbs_on_db_disc_evt(&m_ble_lbs_c, p_evt);
}

[quote user="MeghaC"]

Also I have observed while debugging that the when switch pressed from peripheral it again goes to NRF_BREAKPOINT_COND;

[/quote]

This did not happen when I tested here. Maybe this will be resolved when you fix the db_disc_handler() function.

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Fri, 07 May 2021 12:32:17 GMT

Hi,

Also I have observed while debugging that the when switch pressed from peripheral it again goes to NRF_BREAKPOINT_COND;

Why it so?

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Fri, 07 May 2021 07:58:56 GMT

Hi,

Done changes suggested on central side also, now peripheral getting connected with central device. Please find below screenshot for your reference.

But when I'm pressing button from peripheral the LED from central should turn ON. This is not happening.

I have added all necessary logic from ble_app_blinky to ble_app_uart & from ble_app_blinky_c to ble_app_uart_c.

If I missed out anything, kindly suggest me. My sdk is already with you. I have maked peripheral as nRF52832 DK & central as nRF52840 DK.

Waiting for your reply.

Thanks for your valuable guidance.

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

Sigurd — Thu, 06 May 2021 16:34:05 GMT

[quote user="MeghaC"]

What will be the changes required in central side. Can you just check in same sdk & let me know.

[/quote]

For the central, the first steps to get it to connect is:
in sdk_config.h set these:

NRF_BLE_SCAN_NAME_CNT to 1
NRF_SDH_BLE_VS_UUID_COUNT to 2

You then will get this in the log:
<warning> nrf_sdh_ble: Change the RAM start location from 0x20002A38 to 0x20002A48.
<warning> nrf_sdh_ble: Maximum RAM size for application is 0x3D5B8.

So you need to set these values for the linker:

RAM_START=0x20002A48
RAM_SIZE=0x3D5B8

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Thu, 06 May 2021 13:45:18 GMT

Hi,

Checked by doing this changes in peripheral code. Now able discover on nRF tool box. But not connects with Central ble_app_uart_c(nRF52840)

What will be the changes required in central side. Can you just check in same sdk & let me know.

Regards,

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

Sigurd — Thu, 06 May 2021 13:17:27 GMT

Hi,

After setting

NRF_FPRINTF_FLAG_AUTOMATIC_CR_ON_LF_ENABLED to 0

I get this in the log window:

I right click on "ble_app_uart_pca10040_s132" -> Options, and find the RAM size the linker uses:

I changed the RAM values so it matches what the log said, and after that it worked fine.

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Thu, 06 May 2021 12:24:26 GMT

Hi,

I have checked your suggestion.

Added led_write_handler() & init LBS service. Also, change in sdk_config.h as NRF_SDH_BLE_VS_UUID_COUNT in sdk_config.h to 2

Now, I'm unable to see it on nRF toolbox also.

Before adding these changes atleast I'm able to see it in nRF toolbox.

Checked in debug mode, both continuously goes into NRF_BREAKPOINT_COND;

Please find below link of updated sdk:

https://smetgroup-my.sharepoint.com/:u:/g/personal/megha_choudhari_mysmindia_com/Ef_06NnNAFhKjzGPNLRt8s0Bn7x9q9FgYLiibla92wL_IA?e=7VSmin

Kindly suggested what is wrong in this.

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

Sigurd — Thu, 06 May 2021 10:24:18 GMT

Hi,

I took a look at the ble_app_uart. In services_init(), you need to init the LBS service, and add the led_write_handler().

/**@brief Function for handling write events to the LED characteristic.
 *
 * @param[in] p_lbs     Instance of LED Button Service to which the write applies.
 * @param[in] led_state Written/desired state of the LED.
 */
static void led_write_handler(uint16_t conn_handle, ble_lbs_t * p_lbs, uint8_t led_state)
{
    if (led_state)
    {
        bsp_board_led_on(LEDBUTTON_LED);
        NRF_LOG_INFO("Received LED ON!");
    }
    else
    {
        bsp_board_led_off(LEDBUTTON_LED);
        NRF_LOG_INFO("Received LED OFF!");
    }
}


/**@snippet [Handling the data received over BLE] */


/**@brief Function for initializing services that will be used by the application.
 */
static void services_init(void)
{
    uint32_t           err_code;
    ble_nus_init_t     nus_init;
    ble_lbs_init_t     init     = {0};
    nrf_ble_qwr_init_t qwr_init = {0};

    // Initialize Queued Write Module.
    qwr_init.error_handler = nrf_qwr_error_handler;

    err_code = nrf_ble_qwr_init(&m_qwr, &qwr_init);
    APP_ERROR_CHECK(err_code);

    // Initialize NUS.
    memset(&nus_init, 0, sizeof(nus_init));

    nus_init.data_handler = nus_data_handler;

    err_code = ble_nus_init(&m_nus, &nus_init);
    APP_ERROR_CHECK(err_code);

    // Initialize LBS.
    init.led_write_handler = led_write_handler;

    err_code = ble_lbs_init(&m_lbs, &init);
    APP_ERROR_CHECK(err_code);
}

Since you are adding a second custom service, you need to increase the value of NRF_SDH_BLE_VS_UUID_COUNT in sdk_config.h to 2

If you are having trouble viewing the RTT log inside SES, then set NRF_FPRINTF_FLAG_AUTOMATIC_CR_ON_LF_ENABLED in sdk_config.h to 0

After adding the 2nd service, you need to allocate more RAM to the SoftDevice. You typically will get something like this printed in the log when this is needed:

<warning> nrf_sdh_ble: Insufficient RAM allocated for the SoftDevice.
<warning> nrf_sdh_ble: Change the RAM start location from 0x20002AD8 to 0x20002AE8.
<warning> nrf_sdh_ble: Maximum RAM size for application is 0xD518.
<error> nrf_sdh_ble: sd_ble_enable() returned NRF_ERROR_NO_MEM.

You then need to adjust this values, like this:

If you connect with a phone, using e.g. nRF Connect for Mobile app, you should see that the LBS Service has been added.

Let me know if you need more help with this.

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Thu, 06 May 2021 09:46:45 GMT

Hi,

When can I get reply?

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Wed, 05 May 2021 13:43:34 GMT

Hi,

Waiting for reply.

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Tue, 04 May 2021 12:49:33 GMT

Hi,

Please find below link to download the project:

https://smetgroup-my.sharepoint.com/:u:/g/personal/megha_choudhari_mysmindia_com/ESMV-B9MJLdEvC0eyhJ2ghgBXCsVIT4OQg3A2CFfrWDq9w?e=lYWprR

Added comments with //03/05/21 for easy understanding of changes, I have merged -

1. ble_peripheral - ble_app_blinky into ble_app_uart

2. ble_central - ble_app_blinky_c into ble_app_uart_c

Now both project builds successfully but enable to connect ble_app_uart to ble_app_uart_c. Also, the advertising, connection LED status not visible.

Checked in debug mode both goes to NRF_BREAKPOINT_COND; in app_error_weak.c file

Looking forward for your reply.

Regards,

Megha Choudhari

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

Sigurd — Tue, 04 May 2021 12:15:02 GMT

Hi,

I don't think we have any ready examples that merges ble_app_blinky and ble_app_uart.

If you can upload your project here, then I will take a look at what's missing.

RE: Merging nRF52 blinky into uart for peripheral as well as central role.

MeghaC — Tue, 04 May 2021 08:10:31 GMT

Kindly help here. Waiting for reply.

Regards,

Megha Choudhari