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

Support for active-low buzzer logic #14

Open
thankthemaker opened this issue Aug 27, 2021 · 2 comments
Open

Support for active-low buzzer logic #14

thankthemaker opened this issue Aug 27, 2021 · 2 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@thankthemaker
Copy link
Owner

The current implementation uses active-high logic which needs a NPN transistor.
Enhance the firmware to also support active-low logic for PNP transistors

@thankthemaker thankthemaker added enhancement New feature or request bug Something isn't working labels Aug 27, 2021
@kp128
Copy link

kp128 commented Mar 18, 2024

Hello. Can you tell me what part of the code needs to be changed for the buzzer to work on an NPN transistor? The option via a PNP transistor just doesn’t suit me.

@thankthemaker
Copy link
Owner Author

Hello,
I haven't used an active buzzer yet, but I guess the wiring needs to be setup different and the Melody-Library needs to be instantiated in a different manner. See
https://github.com/thankthemaker/rESCue/blob/master/src/Buzzer.cpp
and
https://github.com/fabianoriccardi/melody-player
for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants