search

rppicomidi / pico-usb-midi-processor

Use a Raspberry Pi Pico board to process USB MIDI data between a USB MIDI Keyboard or other device and a PC or other USB MIDI Host
MIT License
28 stars 1 forks source link

Rebasing on latest TinyUSB? #2

Closed burtonrodman closed 1 year ago

burtonrodman commented 1 year ago

I know this isn't exactly the right place, but wasn't sure how else to contact you.

Thanks so much for your work on implementing midihost in TinyUSB. My project wouldn't be possible without this effort.

I'm hoping to update my pico-sdk to version 1.5.0, which of course would be a much newer version of TinyUSB.

I'm happy to do the work of rebasing the midihost branch, but wanted to chat with you first about any known issues, anything that's changed from a architectural perspective in TinyUSB that would change how we'd approach implementing the midi host.

I recall reading something about the bulk transfer something something now being implemented that would affect.

Any thoughts or input would be greatly appreciated!

rppicomidi commented 1 year ago

@burtonrodman The pico-sdk uses tinyusb as a submodule. If you don't need to use the tinyusb version that ships with the pico-sdk, you can continue to use the version of tinyusb you are using now. Just use git pull to update the pico-sdk and don't update the tinyusb git submodule. If you are using a Pico W, you will probably want to update the other git submodules.

rppicomidi commented 1 year ago

@burtonrodman I know this is an old issue, but I wanted to mention I am trying to get away from using a fork of TinyUSB. Please see the latest revision of this project for the application usb_midi_host driver and the simple patch to TinyUSB. If you want to maintain a fork, please use the latest code to update your fork. Does this response solve your issue?

burtonrodman commented 1 year ago

great! i’ll take a look. unfortunately my project has been on the back burner and when i pick it back up, i’m sure i’ll be trying to update all the things and this should hopefully make that easier.

rppicomidi commented 1 year ago

OK. I will close this issue. If you see a need to revisit the fork, please re-open it.