-
Notifications
You must be signed in to change notification settings - Fork 366
Pcm capibility clean up #3324
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
Pcm capibility clean up #3324
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -57,6 +57,15 @@ define(`COMP_FORMAT_VALUE', | |
| $1, `s32le', `0x02', | ||
| )') | ||
|
|
||
| dnl CAPABILITY_FORMAT_NAME(BUF_FMT) | ||
| define(`CAPABILITY_FORMAT_NAME', | ||
| `ifelse( | ||
| $1, `s16le', `S16_LE', | ||
| $1, `s24le', ``S24_LE,S16_LE'', | ||
| $1, `s32le', ``S32_LE,S24_LE,S16_LE'', | ||
| $1, `float', `FLOAT_LE', | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What's the rational here, do you want to mask the format names so one name can be used to describe more than one format ? i.e.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No, I am not gonna change the naming here, instead, as written in the commit message, it adds helper to generate supported formats of the PCM capability from the pipeline format (buffer allocated format). `S32_LE,S24_LE,S16_LE' is the format we used to denote the supported format of PCM capability in our m4. So here, it means for:
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. can we rename the s32le names here, it's confusing with S32LE uppercase name as they are NOT the same thing.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
They are existed for component format, and the uppercase names are for the PCM format. If want to change, we need another PR for to clean them up as there are hundreds of place where using them.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok, we need to live with this atm until topology 2.0 is ready. |
||
| )') | ||
|
|
||
| # note: only support number < 256 at the moment | ||
| dnl DEC2HEX(dec_num) | ||
| define(`DEC2HEX', | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.