RP2040 Connect + PlatformIO + RP2040_RTC build failure

efpe · November 1, 2021, 4:30pm

Wow, amazing, it works. Nice debugging I get that it’s a dirty hack but at least it’s a good start.

What is a mystery to me is that why does it work with the Arduino IDE and not with pio?

maxgerhardt · November 1, 2021, 4:33pm

Slight differences in linking order (which there are in comparison to the Arduino IDE), compiler flags or compiler version can shuffle the stack around enough to cause a difference in stack memory and set the value of dotw to a value outside the 0-6 range and thus fail verification. The base problem is the code not initializing the datatstructure’s dotw field, leading to undefined behavior.

I can open an issue or directly a PR about this with a fixed version. Setting it to zero doesn’t seem correct, especially since the rtc_set_datetime function writes it in the RTC registers. But then, the entire time in the Arduino IDE , correctness was also more based on luck…

efpe · November 1, 2021, 4:36pm

Thanks for the explanation, I am more than happy to open an issue, don’t worry about that.

As before, many thanks for debugging and looking into this super quickly

maxgerhardt · November 1, 2021, 4:45pm

I was faster already, opened at Library converts datetime_t incorrectly, causing example to fail · Issue #4 · khoih-prog/RP2040_RTC · GitHub

No problems

maxgerhardt · November 1, 2021, 6:45pm

The bug has been properly fixed in v1.0.7 to fix bug in rtc_set_datetime() · khoih-prog/RP2040_RTC@de19cbf · GitHub. The new 1.0.7 version should be visible in https://platformio.org/lib/show/12433/RP2040_RTC/installation soon, or directly via referencing https://github.com/khoih-prog/RP2040_RTC.git.

efpe · November 2, 2021, 9:28am

Yeah, thanks to both of you