search

electro-smith / libDaisy

Hardware Library for the Daisy Audio Platform
https://www.electro-smith.com/daisy
MIT License
312 stars 131 forks source link

SRAM Bootloader Breaks SW1 and SW2 on Daisy Field #534

Closed byronellis closed 4 months ago

byronellis commented 2 years ago

Hi, switching to the SRAM-based boot loader (BOOT_SRAM) appears to break SW_1 and SW_2 on the Daisy Field. Specifically it seems like RisingEdge() is always true. Not sure if this is intentional or if there is a workaround, but the same code will work fine if you load the same code without the custom daisy boot loader.

stephenhensley commented 1 year ago

@byronellis thanks for the bug report, we'll look into this!

byronellis commented 1 year ago

No problem! If you need a minimal reproduction I can make one but it's pretty much "switches don't seem to work" so it's almost trivial 🙂

dromer commented 7 months ago

Oh wow, I've been scratching my head around this for weeks/months!

Had totally missed this issue report and it's exactly what I'm seeing! -> with the bootloader sw1 and sw2 on the Field always present as high.

I'll see if I can test the new bootloader.

dromer commented 7 months ago

Ok can confirm that with this new branch of libDaisy and the bootloader the switches work as expected.

Really hope this gets merged/released soon, but at least I know the cause of the issue and that it will get resolved at some point.

dromer commented 6 months ago

I suppose this ticket can now be counted as resolved. Or should it need a release tag for libDaisy first?

dromer commented 4 months ago

This works now on bootloader v6 and beyond :)

stephenhensley commented 4 months ago

@dromer Thanks for confirming!