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

ESP32 Dev Module stops working after including Mozzi.h #297

Open
sensorium opened this issue Dec 11, 2024 · 4 comments · May be fixed by #301
Open

ESP32 Dev Module stops working after including Mozzi.h #297

sensorium opened this issue Dec 11, 2024 · 4 comments · May be fixed by #301
Labels

Comments

@sensorium
Copy link
Owner

This sketch works:

    //#include <Mozzi.h>
    
    void setup(){
      pinMode(22, OUTPUT);
      digitalWrite(22, HIGH);
    }
    
    void loop(){
    }

But this doesn't:

    #include <Mozzi.h>
    
    void setup(){
      pinMode(22, OUTPUT);
      digitalWrite(22, HIGH);
    }
    
    void loop(){
    }
@tomcombriat
Copy link
Collaborator

Just received some ESP32! Will try here if I can reproduce.

@tomcombriat
Copy link
Collaborator

Ok, I can confirm this. I also get a lot of "Deprecated" warnings on compilation. Looks like the ESP32 port needs an update!

@tomcombriat tomcombriat added the bug label Jan 5, 2025
@tomcombriat
Copy link
Collaborator

tomcombriat commented Jan 5, 2025

Ok, this seems to be exactly the same than #261 : a conflict because we are using the legacy i2s implementation. I do not really understand why the conflict as we are only using the legacy of all drivers but…

Probably worth moving everything to the new API, I'll give it a try in the branch fix/esp32, but if @tfry-git or @dietervandoren which I think are more accustomed with the platform want to give it a try, they should feel welcome :)

Edit:

The driver API also changed and will need to be updated at the same time…

@cammycam
Copy link

Ahh thank you, very much appreciate you updating for esp32!

@tomcombriat tomcombriat linked a pull request Jan 13, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants