Skip to content

Conversation

@xiulipan
Copy link
Contributor

@xiulipan xiulipan commented Dec 18, 2019

notifier is unregister in dma put. need to register it in get to make
sure we do not lose any notifier.

Signed-off-by: Pan Xiuli xiuli.pan@linux.intel.com

Fix #2227

@paulstelian97
Copy link
Collaborator

I think you may wish to say it's platform specific change? (I'm really watching over the DMA issues because there is an issue in the generic code too, not just in DW)

Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

Can you update commit subject to say dw-dma instead of dma. Everything else good.

@lgirdwood
Copy link
Member

CI is usual suspects.

notifier is unregister in dma put. need to register it in get to make
sure we do not lose any notifier.

Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
@xiulipan xiulipan changed the title dma: register notifier in dma get instead of dma probe dw-dma: register notifier in dma get instead of dma probe Dec 19, 2019
@lgirdwood lgirdwood merged commit c28ecb9 into thesofproject:master Dec 19, 2019
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.

[BUG] [BDW] aplay failed on bdw

4 participants