Chris

Results 22 comments of Chris

Ok, after taking another look at this i've managed to simplify the constexpr `get_mpll_config` function so it no longer causes the error. I haven't yet tested this tho. On paper...

> I haven't yet tested this tho. On paper this should work, but i'll test later today and comment back. Ok, just tested it and it does indeed work as...

Hi @thisiskeithb , is there anything preventing this PR from getting merged? Anything i can do?

@thinkyhead is there anything preventhing the PR from getting merged?

Hi @ivankravets, is there anything preventing this PR from getting merged? judging by the linked issue, it seems like people are beign recommended to patch their platformio installations to include...

hope the `sd_card` stuff is ok. if not, it should be easy to remove again.

> It should be split out to a separate PR. ok, i removed all the local playback stuff now.

> Please fix the CI issues @jesserockz CI should be fixed now

hi @jesserockz , anything missing on this PR?

note: on a real-world use-case, this PR increases the flash usage by about 220 KiB and ram usage by about 1 KiB. however, this comparison may not be quite fair...