Skip to content

Possible _PM_esp32commonTimerInit bug? #69

Description

@jmrtsh

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 :)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions