Skip to content

drm/i915: apply AUD_FREQ_CNTRL state save to all GEN9+ platforms #1881

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

Closed
wants to merge 1 commit into from

Conversation

kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Mar 10, 2020

Replace the TGL/ICL specific platform checks with a more generic
check using INTEL_GEN().

An initial version of state save and restore of AUD_FREQ_CNTRL
register was added for subset of platforms in commit 87c1694
("drm/i915: save AUD_FREQ_CNTRL state at audio domain suspend").
The state save has proven to work well and it is needed
in newer platforms, so needs to be extended. Although the logic is not
strictly needed on GEN9/10, align change to hardware specification and
apply state and restore to all platforms since GEN9.

Bspec: 49281
Signed-off-by: Kai Vehmanen [email protected]

Fixes: #1719

Replace the TGL/ICL specific platform checks with a more generic
check using INTEL_GEN().

An initial version of state save and restore of AUD_FREQ_CNTRL
register was added for subset of platforms in commit 87c1694
("drm/i915: save AUD_FREQ_CNTRL state at audio domain suspend").
The state save has proven to work well and it is needed
in newer platforms, so needs to be extended. Although the logic is not
strictly needed on GEN9/10, align change to hardware specification and
apply state and restore to all platforms since GEN9.

Bspec: 49281
Signed-off-by: Kai Vehmanen <[email protected]>
@kv2019i kv2019i added upstream Patch has been sent to upstream (e.g. alsa-devel) GFX GFX driver restriction labels Mar 10, 2020
@kv2019i kv2019i requested a review from emilchudzik March 10, 2020 13:00
@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 26, 2020

Review still ongoing on intel-gfx... keeping open

@kv2019i
Copy link
Collaborator Author

kv2019i commented Apr 20, 2020

Merged upstream and version merge to sof-dev as well (#1881).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GFX GFX driver restriction upstream Patch has been sent to upstream (e.g. alsa-devel)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG][JSL] SOF doesn't work after S3
2 participants