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

[Media Common] Set Scanout by default #1822

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

vcheah
Copy link
Contributor

@vcheah vcheah commented Jun 25, 2024

recommend flagging all potential scanout surfaces as SCANOUT

Copy link
Contributor

@XinfengZhang XinfengZhang left a comment

Choose a reason for hiding this comment

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

LGTM, suppose only ddi allocate surface is using "Media" words? + @Jexu @Effiey to review

@XinfengZhang XinfengZhang requested a review from Jexu July 24, 2024 08:17
@vcheah
Copy link
Contributor Author

vcheah commented Aug 1, 2024

LGTM, suppose only ddi allocate surface is using "Media" words? + @Jexu @Effiey to review

I noticed that media_softlet (media_libva_util_next.cpp) ddi have both "MEDIA" and "Media".

@Jexu Jexu added Decode video decode related Encode video encode related VP Video Processing labels Aug 5, 2024
@vcheah
Copy link
Contributor Author

vcheah commented Sep 4, 2024

I've just rebased. @Effieyu0, could you please help to review?

recommend to flag all potential scanout surface as SCNAOUT. This is
required to fix DMA DRM render on XE.

Signed-off-by: Cheah, Vincent Beng Keat <[email protected]>
@Jexu Jexu added the verifying PR: fix ready and verifying with build/test label Sep 9, 2024
@intel-mediadev intel-mediadev merged commit f6d9dd6 into intel:master Sep 19, 2024
12 of 13 checks passed
@vcheah vcheah deleted the force_scanout branch September 19, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Decode video decode related Encode video encode related verifying PR: fix ready and verifying with build/test VP Video Processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants