Description
I just tried to half-assedly port the S3 code I need to arduino-esp 3.0.0-alpha2, which uses a refactored timer API (espressif/arduino-esp32#7904 and espressif/arduino-esp32#7906), to get it to somewhat work, I mostly only had to substitute a few calls and numbers, but one change stood out to me, and I think it might be a bug in the current version too, so I'd like to bring it to someones attention ^^'.
I'm talking about _PM_esp32commonTimerInit
, specifically the last line. Shouldn't this be core->timer? Because if the if
statement above gets executed, core->timer
is overwritten, but not the local timer
variable, which still contains the old core->timer
from before the timerBegin
call.
When I changed the call in line 94 to use core->timer
, my matrix started to light up, whereas something crashed along the way if I compile without that change. All on arduino-esp 3.0.0-alpha2.
I'm happy to provide more information if necessary.
Cheers :)