Skip to content
This repository was archived by the owner on Jan 17, 2019. It is now read-only.

topology: whl: refine bit depth setting for ALC700 - #121

Closed
RanderWang wants to merge 1 commit into
thesofproject:masterfrom
RanderWang:alc700
Closed

RanderWang wants to merge 1 commit into
thesofproject:masterfrom
RanderWang:alc700

Conversation

@RanderWang

Copy link
Copy Markdown
Collaborator

Only bit dpeth of 16/20/24 are supported by ALC700. Now change
pass-through pipeline format from 32 to 16 for it.

Signed-off-by: Rander Wang rander.wang@linux.intel.com

Only bit dpeth of 16/20/24 are supported by ALC700. Now change
pass-through pipeline format from 32 to 16 for it.

Signed-off-by: Rander Wang <rander.wang@linux.intel.com>
Comment thread topology/sof-whl-hda.m4
# Schedule 48 frames per 1000us deadline on core 0 with priority 0
PIPELINE_PCM_ADD(sof/pipe-passthrough-playback.m4,
1, 0, 2, s32le,
1, 0, 2, s16le,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we should really have a volume control for all HDMI outputs (this will also convert 32 to 24bit - dont use 16 when 24 is available)

@RanderWang RanderWang Nov 9, 2018 •

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

ok. I will talk with customer whether they need a volume component.
And if they need, sof-hda-generic.m4 is ready for them

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I would reserve this PR, because it is in release note for customer.
And I would send another PR for WHL after discussing with customer.

@RanderWang RanderWang closed this Jan 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants