search

turgu1 / bt-keyboard

Other
71 stars 8 forks source link

Made code work with ESP-IDF v5.2.3. #8

Closed jthiem closed 1 month ago

jthiem commented 1 month ago

Code compiles and runs with ESP-IDF 5.2.3 in Linux. Tested with an Apple keyboard.

RBDSeb commented 1 month ago

ahah nice! I was busy on the same thing this morning... (with esp-idf 5.3.1) and obviously had to make the same changes. However, when testing it, I get a reboot of the esp32 as soon as I get a key event... rst:0x8 (TG1WDT_SYS_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT) have you seen something similar (didn't search of it yet)?

tested with keychron k3 keyboard and esp32Wroom...

thanks :)

jthiem commented 1 month ago

Huh, I get reboots of the ESP32 on certain keys, especially the "fn" key. An event buffer or a callback buffer not big enough?

jthiem commented 1 month ago

ahah nice! I was busy on the same thing this morning... (with esp-idf 5.3.1) and obviously had to make the same changes. However, when testing it, I get a reboot of the esp32 as soon as I get a key event... rst:0x8 (TG1WDT_SYS_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT) have you seen something similar (didn't search of it yet)?

tested with keychron k3 keyboard and esp32Wroom...

thanks :)

I may have found the reason, please see issue #9.

Cheers!

jthiem commented 1 month ago

Added bugfix for #9, with some simplification to develop a decoding state machine. Need to adapt BTKeyboard::wait_for_ascii_char for the new code.

RBDSeb commented 1 month ago

Nice catch! works well for me on ESP-IDF v5.3.1.

turgu1 commented 1 month ago

Thanks a lot for your contribution!