Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MIDI Not working #52

Closed
meltdown03 opened this issue Sep 9, 2021 · 9 comments
Closed

MIDI Not working #52

meltdown03 opened this issue Sep 9, 2021 · 9 comments

Comments

@meltdown03
Copy link
Contributor

When I try the MIDI example, it builds and runs ok, but I dont see the MIDI device on my system. Running lsusb -v shows that the Espressif Device is there and has some MIDI properties, but no program that uses MIDI cannot see or use it. I tried it in Windows and Linux.

@chegewara
Copy link
Owner

yes, i found it recently too. I am not working with midi and i dont know it at all, thats why i didnt found it earlier.

The problem is with USB descriptor, but dont know what yet. I am busy in next few weeks, so it has to wait.

@meltdown03
Copy link
Contributor Author

meltdown03 commented Sep 9, 2021 via email

@chegewara
Copy link
Owner

In this library descriptor didnt change. It may be some issue in tinyusb, which will be harder to track down.

@meltdown03
Copy link
Contributor Author

I found the fix:

#define EPNUM_MIDI 0x06

Change EPNUM_MIDI to 0x01 and it works

@meltdown03
Copy link
Contributor Author

I found it by looking at the TinyUSB midi-test example

@chegewara
Copy link
Owner

Good catch. esp32 S2 does not have that many endpoints.

@meltdown03
Copy link
Contributor Author

meltdown03 commented Sep 9, 2021

Good catch. esp32 S2 does not have that many endpoints.

Yeah, i guess it works with 0x05 too. Is there a specific one? or doesnt it matter?

Nevermind, it shows up but does not play anything with 0x05, lol

@chegewara
Copy link
Owner

chegewara commented Sep 9, 2021

As long as you dont want to mix it with other USB class it does not matter. 4 or 5 would be good.

@meltdown03
Copy link
Contributor Author

As long as you dont want to mix it with other USB class it does not matter. 4 or 5 would be good.

Ok, 4 works, but 5 doesn't, weird because that's what the esp32-hal-tinyusb.h file has for USB_INTERFACE_MIDI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants