Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable-v2.9] ipc4: fw_reg: downgrade FW_REGS version to 0 to disable delay reporting #8941

Merged

Conversation

kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Mar 13, 2024

Disable delay reporting as a workaround to bug
thesofproject/linux#4781 SOF driver checks the ABI version and if it's not 1, delay reporting is disabled.

Disable delay reporting as a workaround to bug
thesofproject/linux#4781
SOF driver checks the ABI version and if it's not 1, delay reporting
is disabled.

Signed-off-by: Kai Vehmanen <[email protected]>
@kv2019i kv2019i changed the title ipc4: fw_reg: downgrade FW_REGS version to 0 to disable delay reporting [stable-v2.9] ipc4: fw_reg: downgrade FW_REGS version to 0 to disable delay reporting Mar 13, 2024
@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 13, 2024

@ujfalusi @plbossart Need your input whether to merge this or not. I know thesofproject/linux#4851 is still in progress. We had this PR in SOF2.8.1 release, so we need to decide whether we renable in SOF2.9 or wait until SOF2.10 to give more time for the kernel-side fixes to get deployed.

@kv2019i kv2019i requested a review from plbossart March 13, 2024 12:26
Copy link
Contributor

@ujfalusi ujfalusi left a comment

Choose a reason for hiding this comment

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

@kv2019i, the kernel fix is unlikely to be in distro kernels in time for the 2.9, I would proceed with a merge.
We can revert this when we know that the release is not going to break user machines.

@kv2019i kv2019i merged commit c0b0e4c into thesofproject:stable-v2.9 Mar 15, 2024
38 of 44 checks passed
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