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

Audio: modify audio driver satisfy nuttx alsa #15657

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yangyalei
Copy link

Summary

Add vela alsa lib interface

Impact

nop

Testing

Use alsa lib interface play music in allwinnertech&BES platform

@github-actions github-actions bot added Area: Audio Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces. labels Jan 23, 2025
@yangyalei
Copy link
Author

This PR is related to apache/nuttx-apps#2970

Copy link
Contributor

@cederom cederom left a comment

Choose a reason for hiding this comment

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

Thank you @yangyalei amazing work porting ALSA to NuttX! :-)

@@ -277,4 +277,10 @@ config AUDIO_FONT

endif

config AUDIO_MAX_APPS
int "Number of applications that can use upperhalf driver meanwhile"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
int "Number of applications that can use upperhalf driver meanwhile"
int "Number of applications that can use upperhalf driver simultaneously"

int "Number of applications that can use upperhalf driver meanwhile"
default 3
---help---
Specifies the number of applications can play audio data meanwhile.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Specifies the number of applications can play audio data meanwhile.
Specifies the number of applications can play audio data simultaneously.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Audio Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants