-
Notifications
You must be signed in to change notification settings - Fork 21
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
Require an update to DaisyPatchSM audio init to track latest libdaisy changes #462
Comments
Hi, and thanks again for your report. Indeed our version of For this, we need to update the commit on which the @jazamatronic what minimum version of I think I'll take the issue from there. Thanks! |
Ok, that one's my fault, I didn't init the Cheers |
Ah ok. Then could you please:
And tell me if If that's the case, I would close this issue, but create another one to generally update to one of |
G'day @ohmtech-rdi, Yeah - it build's just fine if I init the submodules as documented. It can stay the way it is, I have my local fixes for the version I'm using and we'll know what to change the day you decide to update. Cheers |
FYI, the libDaisy bootloader had a bug which would prevent to use I have a WIP branch to update libDaisy as soon as they merge their fix in |
G'day,
I had to update the _audio.Init() here to compile when using a recent libdaisy.
I can create a pull request if you'd like.
Cheers
The text was updated successfully, but these errors were encountered: