Skip to content

wait: print register values if poll_for_register_delay fails - #4251

Merged
lgirdwood merged 1 commit into
thesofproject:mainfrom
juimonen:timeout_print
May 26, 2021
Merged

lgirdwood merged 1 commit into
thesofproject:mainfrom
juimonen:timeout_print

Conversation

@juimonen

Copy link
Copy Markdown

Print register and its values if poll_for_register_delay fails. Also
check the return value and print warning in sai.c

Signed-off-by: Jaska Uimonen jaska.uimonen@intel.com

@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.

Btw, I think there are other users of the poll() api who are not checking the result. Hence the error log in the bug.

Print register and its values if poll_for_register_delay fails. Also
check the return value and print warning in sai.c and dw/dma.c.

Signed-off-by: Jaska Uimonen <jaska.uimonen@intel.com>

@paulstelian97 paulstelian97 left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, still need to check if there are other instances where this is useful/needed.

@juimonen
juimonen requested a review from lyakh as a code owner May 26, 2021 11:00
@juimonen

Copy link
Copy Markdown
Author

@lgirdwood @paulstelian97 added 1 warn to dw_dma, but I could not just find more without error checking...

@lgirdwood
lgirdwood merged commit 643db1d into thesofproject:main May 26, 2021
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.

4 participants