democloid / picoTracker

BSD 3-Clause "New" or "Revised" License
187 stars 17 forks source link

Enable USB on by default #198

Closed maks closed 1 month ago

democloid commented 1 month ago

So I was thinking about this. I think we should enable the USB stack by default, because we want it for MIDI in prod, eventually. But not entirely sure if I'd like stdio enabled by default, I'd imagine it could have some performance issues. Thoughts?

maks commented 1 month ago

So I was thinking about this. I think we should enable the USB stack by default, because we want it for MIDI in prod, eventually. But not entirely sure if I'd like stdio enabled by default, I'd imagine it could have some performance issues. Thoughts?

The reason I wanted to have stdio in USB on was to support the remote display protocol. But I didn't enable that as well because I want to remove it and instead have it's as a setting once we have a device settings screen. So I'd be happy to make stdio on USB a device setting as well if that's possible, I'll look up the Pico SDK docs to see if that is doable.

maks commented 1 month ago

@democloid ok cool, so I found this post which covers how to enable/disable use uart and usb for stdio printf and then a couple of post further donw how to just write to serial over USB directly.

So what I'll do is leave the CMakeList.txt config default off and then instead switch the remote UI to instead write directly to the USB serial output directly.