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

Fixed Sonde RSSI "Beep" #2012

Merged
merged 3 commits into from
Mar 19, 2024
Merged

Conversation

NotherNgineer
Copy link
Contributor

@NotherNgineer NotherNgineer commented Mar 19, 2024

  1. Added audio output beep support to the baseband audio_dma module.
  2. Cleaned up and fixed RSSI Audio Beep in Sonde app, whereas before it was silent.
  3. Produce a beep sound when enabling Beep (on packet reception) in the Sonde app.
  4. Cleaned up dead code in the background tone_gen module.

I haven't actually verified that the tone frequencies are suitable for various RSSI levels, but at least it's making a beep sound now. Will check beep tone frequency later. Note that the volume (in upper right corner) may need to be increased from the default.

Fixes #1973

Test version available here: https://discord.com/channels/719669764804444213/722101917135798312/1219549250145357845

@gullradriel
Copy link
Member

Wooooo nice !
I should add an option to activate it on Level app !

Copy link
Member

@gullradriel gullradriel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't wait to play with this tonight. Thanks Mark :-)

@gullradriel gullradriel merged commit 74442f1 into portapack-mayhem:next Mar 19, 2024
3 checks passed
@htotoo
Copy link
Member

htotoo commented Mar 19, 2024

Will love to use this in the BLE Foxhunt app :) Thanks!

@NotherNgineer
Copy link
Contributor Author

FYI, I am planning to also add a time parameter to this (in the next PR) so the background app doesn't stall while the beep is occurring.

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

Successfully merging this pull request may close these issues.

Add audio "beep" support
3 participants