Removed mainLoopTask to avoid kernal panic due to re-entrancy on TWI …

OpenAstroTech / OpenAstroTracker

3D printed DSLR tracking mount

Other

963 stars 122 forks source link

Removed mainLoopTask to avoid kernal panic due to re-entrancy on TWI … #126

Closed philharrisathome closed 3 years ago

philharrisathome commented 3 years ago

…from multiple loop(). loop() now runs on Core1 (Arduino default) and steppers run on Core 0. This fix allows I2C displays to work on ESP32. It will also allow Gyro to work on ESP32 but need to fix some minor compilation issues first. Removed separate finishSetup() - setup() just continues on. Some minor updates to logging as well.

ClutchplateDude commented 3 years ago

And let's get this one merged before you add any more features not related to kernel panics :-)

andre-stefanov commented 3 years ago

Closing since @philharrisathome has kindly migrated this PR to the new repository: https://github.com/OpenAstroTech/OpenAstroTracker-Firmware/pull/8