ESP32 S2 crash in static void IRAM_ATTR uart_intr_handle()

[chrisDupuis]

Crash with ESP32 S2 mini.

(Code is running on ESP32)

Someone have test on ESP32 S2 ?
Any idea how to solve that ?

[ericlangel]

i dont have a S2 for testing

interesting that it is running on ESP32

which ESP32 Hardware Core Version?
can u use the Exception decoder?

[chrisDupuis]

S2 Mini V1.0.0,

Hailege 3pcs ESP32 S2 Mini WiFi Development Board WIFI IOT Wireless Board Based on ESP32-S2FN4R2 ESP32-S2 4MB Flash Type-C MicroPython Arduino Compatible

I have no exception shown,
only constantly rebooting...

[ericlangel]

I need the version of your ESP32 arduino core version
is it already 3.0.0?
or 2.0.16?
or?

[chrisDupuis]

I use 2.0.16
In 3.0.0 I can't compile, error...

[ericlangel]

did you had a look at #44 ?

there are several reasons for crashing. It's hard to get the right one without exception decoder
i could try to get a S2 Board, but i need some time to rework the code

there are several known issues, but i had no time for fix

[chrisDupuis]

Yes I use this topic and done some test since 3 days.

If I remove
ESP_ERROR_CHECK(uart_isr_register(uart_num, uart_intr_handle, NULL, ESP_INTR_FLAG_IRAM, 0 /*handle_console*/));
No crash.

If I return in ISR uart, then crash.

static void IRAM_ATTR uart_intr_handle(void *arg)
{
uint16_t status = UART1.int_st.val; // read UART interrupt Status
// status = UART1.int_st.val; // read UART interrupt Status
uart_clear_intr_status(uart_num, UART_RXFIFO_FULL_INT_CLR | UART_RXFIFO_TOUT_INT_CLR);
uart_clear_intr_status(uart_num, status);

return;
}

[ericlangel]

ESP_ERROR_CHECK(uart_isr_register(uart_num, uart_intr_handle, NULL, ESP_INTR_FLAG_IRAM, 0 /handle_console/));

this call register the function uart_intr_handle to the Interrupt
without this, the handler is never called so of course there is no Crash

you could try to register the function without ESP_INTR_FLAG_IRAM

[chrisDupuis]

Same crash without ESP_INTR_FLAG_IRAM.

I try to have very simple uart_intr_handle with minimum code,
in ordre to not have crash/reboot.

[chrisDupuis]

What about that for minimum code

static void IRAM_ATTR uart_intr_handle(void *arg)
{
  uart_clear_intr_status(uart_num, UART_RXFIFO_FULL_INT_CLR | UART_RXFIFO_TOUT_INT_CLR);
  return;
}

[chrisDupuis]

One more, info,
If no SBUS2 signal, no crash.

[ericlangel]

Ok lets try to work it out.

• No SBUS Signal no crash. Thats right. without SBUS signal the UART Interrupt is never executed

please uncomment the following lines/functions one by one and report back

• disable_receiving(); line321
• start_transmit_sequencer(((rxbuf[24]&0x30) >> 4)); line 330
• pinMatrixOutDetach(tx_pin,false,true); line265

the first 2 are working with the Timer, and i'am sure it is more or less a Timer Problem, but who knows

[chrisDupuis]

Test comment this line:

disable_receiving(); line321 => reboot
start_transmit_sequencer(((rxbuf[24]&0x30) >> 4)); line 330 => reboot
pinMatrixOutDetach(tx_pin,false,true); line265 => reboot

[chrisDupuis]

On more info,
with the same module and pin assignment, I have test with iBus telemetry, and it s running ok
https://github.com/bmellink/IBusBM

So for me the hardware, and wire is ok.

[chrisDupuis]

I add a rtc_get_reset_reason, print_reset_reason, and the cause is SW_RESET, RESET_REASON = 3

[ericlangel]

Test comment this line:

disable_receiving(); line321 => reboot start_transmit_sequencer(((rxbuf[24]&0x30) >> 4)); line 330 => reboot pinMatrixOutDetach(tx_pin,false,true); line265 => reboot

And all 3 disabled together?