BrushlessPower / SBUS2-Telemetry

Arduino Library for SBUS and SBUS2 with Futaba Telemetry
GNU General Public License v3.0
86 stars 24 forks source link

[ESP32][BUG] Not working with ESP32 Arduino V2.0.0 #40

Open BrushlessPower opened 3 years ago

BrushlessPower commented 3 years ago

Code doesnt compile with ESP32 Arduino Core V2.0.0

emilverwoerd commented 2 years ago

I bought a couple of M5Stack C3U boards I thought something to play with and put this code on a small form factor. However these boards are only usable with v2.x since they are so new.

Is there a change this framework would be updated to v2.x

BrushlessPower commented 2 years ago

Yes,

i have a local version running with the actual ESP32 core V2.x.x

The are just some Variable names that changed from V1.X.X to V2.X.X. These Names have to be changed. I'll try to work on the new version next week. There are a couple of changes and Bug fixes to do.

i just looked at the Code: maybe it just one line:

in SBUS_usart_ESP32.cpp in void SBUS2_uart_setup()

change

ESP_ERROR_CHECK(uart_set_line_inverse(uart_num, UART_INVERSE_RXD|UART_INVERSE_TXD));

to

if(ESP_ARDUINO_VERSION_MAJOR == 2)

ESP_ERROR_CHECK(uart_set_line_inverse(UART_NUM_2, UART_SIGNAL_TXD_INV | UART_SIGNAL_RXD_INV)); // V2.0.0

else

ESP_ERROR_CHECK(uart_set_line_inverse(UART_NUM_2, UART_INVERSE_RXD | UART_INVERSE_TXD)); // V1.6.0

endif

BrushlessPower commented 2 years ago

It need some more lines

i will upload my local changes for testing in the next minutes

ericlangel commented 2 years ago

it should work with the actual master branch

after testing i can make a release and make it available in library manager

emilverwoerd commented 2 years ago

Thx for the quick fixes. I tried to compile it and that works. However with boad ESP32C3 selected it doesn't work its missing stuff

ericlangel commented 2 years ago

Oh dear...

never tried. I opened issue #44 for that