Skip to content

dw-dma: register notifier in dma get instead of dma probe - #2229

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
xiulipan:pr/fixdmanotify
Dec 19, 2019
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
xiulipan:pr/fixdmanotify

Conversation

@xiulipan

@xiulipan xiulipan commented Dec 18, 2019 •

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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)

@lgirdwood lgirdwood left a comment

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.

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

@lgirdwood

Copy link
Copy Markdown
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