Skip to content

dma: link timeout - #804

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
slawblauciak:hda_dma_tx_timeout
Jan 17, 2019
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
slawblauciak:hda_dma_tx_timeout

Conversation

@slawblauciak

Copy link
Copy Markdown
Collaborator

Signed-off-by: Slawomir Blauciak slawomir.blauciak@linux.intel.com

Comment thread src/drivers/intel/cavs/hda-dma.c Outdated
Comment thread src/drivers/intel/cavs/hda-dma.c Outdated
@slawblauciak

Copy link
Copy Markdown
Collaborator Author

Changes applied

Comment thread src/drivers/intel/cavs/hda-dma.c Outdated
@slawblauciak

Copy link
Copy Markdown
Collaborator Author

Done

@lgirdwood

Copy link
Copy Markdown
Member

@slawblauciak was about to merge but cant see QB in the CI ?

This is to prevent FW from getting stuck in case of anomalous
situations such as DMA taking too long to copy the data.

Signed-off-by: Slawomir Blauciak <slawomir.blauciak@linux.intel.com>
@slawblauciak

slawblauciak commented Jan 17, 2019 •

Copy link
Copy Markdown
Collaborator Author

Force pushed an amended commit, let's see if it appears now

@lgirdwood

Copy link
Copy Markdown
Member

@slawblauciak @zrombel I have a suspicion that internal CI is running but not notifying github for some reason. i.e. I'd expect to see pending at this point after the force push.

@slawblauciak

Copy link
Copy Markdown
Collaborator Author

@lgirdwood yeah, there are some issues with QB currently. However, the build has passed, I checked manually.

@lgirdwood
lgirdwood merged commit f9640aa into thesofproject:master Jan 17, 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.

2 participants