Skip to content

Conversation

@RanderWang
Copy link
Collaborator

Fix capture issue with hda codecs. Now the data captured by
hda codecs is full of zero. It is caused by mismatch of link
dma channel between host and FW.

Now host allocates link dma channel at dai config loading stage,
and sends it to FW to allocate link dma channel at dai_config
function

it for bug:#433 (comment)
kernel patch: thesofproject/linux#291

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

@RanderWang
Copy link
Collaborator Author

Please refer to this commit: b30658d#diff-9f8de02ec82000817dd1a9bbccac158c

@RanderWang RanderWang force-pushed the link-dma-config branch 2 times, most recently from c8d1384 to 9bebdbf Compare November 27, 2018 08:57
Fix capture issue with hda codecs. Now the data captured by
hda codecs is full of zero. It is caused by mismatch of link
dma channel between host and FW.

Now host allocates link dma channel at dai config loading stage,
and sends it to FW to allocate link dma channel at dai_config
function

Signed-off-by: Rander Wang <rander.wang@linux.intel.com>
@lgirdwood lgirdwood merged commit 3b84200 into thesofproject:master Nov 27, 2018
@mengdonglin
Copy link
Collaborator

@lgirdwood @plbossart Missing of kernel patch: thesofproject/linux#291 has caused regression topology loading failure on APL and GLL: thesofproject/linux#322

@mengdonglin
Copy link
Collaborator

@lgirdwood Could you help to revert this PR?
Without the peer kernel PR (thesofproject/linux#291), this patch caused topology loading failure on GLK and APL: thesofproject/linux#322

We need to make FW master branch work with sof-dev kernel. Next GLK release will be branch out from master next Monday and we need time for more testing on sof-dev kernel and FW master.

@RanderWang
Copy link
Collaborator Author

@lgirdwood @mengdonglin 291 is merged, no need to revert 322

@mengdonglin
Copy link
Collaborator

@RanderWang @lgirdwood Thank you! The merged kernel kernel PR (thesofproject/linux#291) solved the regression.

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.

3 participants