ASSERTION FAIL Error Issue

RE: ASSERTION FAIL Error Issue

Naeem Maroof — Fri, 26 Jul 2024 08:44:00 GMT

Hi Basam,

Apologies for the delayed response due to high work load these days.

Nonetheless, I suggest you not to enable and disable the advertising directly in the button handler, but rather set a flag and submit a work-item, and based on the state of flag the work-handler should enable or disable the advertising. This is so that the button handler should return immediately and any time consuming work is off-loaded to the work-queue. You can do something like this

void button_pressed(const struct device *dev, struct gpio_callback *cb, uint32_t pins)
{
	buttonflag = !buttonflag;
	printk("Button pressed at %" PRIu32 "\n", k_cycle_get_32());
	k_work_submit(&mywork);
}
//===========================================================
/* Work handler function */
void work_handler(struct k_work *work) {
    int err;
	if (buttonflag == 1 )
	{
		printk("Button Pressed to start advertising\n");
		err = bt_le_adv_start(BT_LE_ADV_NCONN_IDENTITY, ad, ARRAY_SIZE(ad),sd, ARRAY_SIZE(sd));
		if (err) {printk("Advertising failed to start (err %d)\n", err);return;}
	}
	else
	{
		printk("Button Pressed to stop advertising\n");
		err = bt_le_adv_stop();
		if (err) {printk("Advertising failed to stop (err %d)\n", err);return;}
	}
}

where work_handler function will be invoked when a work is submitted to mywork struct.

you may find WorkQueues also useful while doing this.

Regards,
Naeem

RE: ASSERTION FAIL Error Issue

Madhu Mohan Reddy — Mon, 15 Jul 2024 05:42:26 GMT

struct k_timer mytimer;
/* timer call back function */
static void my_timer(struct k_timer *dummy)
{
    if(gpio_pin_get_dt(&button))
    {
            button_pressed_var = !button_pressed_var;
            if(button_pressed_var)
            {
                printk("device Off: \n");
                /* Here I want to Stop the advertising*/
            }
            else
            {
               /* Here I want to Start the advertising*/
                printk("device On: \n");
            }
            k_timer_stop(&mytimer);
    }

}

/*Button call back function */
void button_pressed(const struct device *dev, struct gpio_callback *cb, uint32_t pins)
{
    if(BIT(button.pin) & pins)
    {
        if(gpio_pin_get_dt(&button))
        {
            k_timer_start(&mytimer, K_SECONDS(2), K_NO_WAIT);

        }
    }
   
}
/*Define a variable of type static struct gpio_callback */
static struct gpio_callback button_cb_data;

void main()
{
    /* Configure the interrupt on the button's pin */
    ret = gpio_pin_interrupt_configure_dt(&button, GPIO_INT_EDGE_TO_ACTIVE );

    /* Initialize the static struct gpio_callback variable   */
    gpio_init_callback(&button_cb_data, button_pressed, BIT(button.pin));  
   
    /* Add the callback function by calling gpio_add_callback()   */
    gpio_add_callback(button.port, &button_cb_data);

}

Minimal code

Above code is Power Button Implementation when i pressed one time the device is off and when i pressed again the device is switched on.

In Off state I want to stop the Advertisement using bt_le_adv_stop() .
In on state I want to start the advertisement again

If any suggestion to optimize the power its very helpful to me

Thank you in advance.

RE: ASSERTION FAIL Error Issue

Naeem Maroof — Thu, 11 Jul 2024 20:31:32 GMT

I see in line 28 that you are calling bt-enable within that callback.

Do you mind setting a flag in the callback and processing in the main.

I can have a look if you let me know how to reproduce the issue you are facing (minimal code please).

RE: ASSERTION FAIL Error Issue

Madhu Mohan Reddy — Tue, 09 Jul 2024 07:03:14 GMT

[quote userid="119245" url="~/f/nordic-q-a/112757/assertion-fail-error-issue/492701"]Are you disabling the BLE again and again? I mean in the else condition?[/quote]

No, only in if statement.

button_pressed() function is a callback function for button interrupt.

RE: ASSERTION FAIL Error Issue

Naeem Maroof — Mon, 08 Jul 2024 12:35:34 GMT

Hello,

Are you disabling the BLE again and again? I mean in the else condition?

How about setting a flag in the button_pressed() and doing processing in the main based on that flag.