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

Adding USB-audio feedback and mute command to usb_sound_card.c #53

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sh1ura
Copy link

@sh1ura sh1ura commented Jan 1, 2025

USB-audio feedback (USB Audio Streaming Isochronous Feedback) is indispensable for stable sound playback without clicking noise. I implemented a simple feedback function by monitoring the buffer's vacancy. To avoid modifying other sources (such as audio.cpp in pico-extras), the number of free buffers is checked directly within this source code.

USB audio also includes a mute function (mute without changing the volume value). I enhanced this functionality by adding just three lines of code, as the original source code was almost complete.

Many DIY USB-DAC enthusiasts have been eagerly anticipating the implementation of the feedback function. The modified parts are isolated using switches named USB_FEEDBACK and MUTE_CMD. Please check it out.

USB-audio feedback (USB AudioStreaming Isochronous Feedback) is indispensable for stable sound playback without clicking noise. I added simple feedback function by checking the vacancy of the buffer. To avoid modifying other sources (such as audio.cpp in pico-extras), number of free buffers is checked in this source code.

USB audio also has mute function (mute without changing volume value). I also added this function (in the original source code, it is almost done, so I added just three lines).
@sh1ura sh1ura closed this Jan 7, 2025
@sh1ura sh1ura deleted the patch-1 branch January 7, 2025 07:09
@sh1ura
Copy link
Author

sh1ura commented Jan 7, 2025

I noticed something should be improved more, so I will check throughly and redo it.

@sh1ura sh1ura restored the patch-1 branch January 7, 2025 11:47
@sh1ura sh1ura reopened this Jan 7, 2025
Reviewed and tested again. Feedback is properly working.
Copy link
Author

@sh1ura sh1ura left a comment

Choose a reason for hiding this comment

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

Reviewed and tested.

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.

1 participant