Skip to content

Conversation

@XinfengZhang
Copy link
Contributor

@ceyusa
Copy link

ceyusa commented Jun 23, 2021

ping?

libva 1.12 is out with this intel/libva#513 merged, but without support in media-driver :)

@XinfengZhang XinfengZhang force-pushed the display_caps branch 2 times, most recently from 0aad94e to cffe5a8 Compare August 16, 2021 07:59
#define DDI_CODEC_GEN_MAX_DISPLAY_ATTRIBUTES 4
#else
#define DDI_CODEC_GEN_MAX_DISPLAY_ATTRIBUTES 0 // set it to zero, unsupported.
#define DDI_CODEC_GEN_MAX_DISPLAY_ATTRIBUTES 1
Copy link
Contributor

Choose a reason for hiding this comment

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

How about change to DDI_MEDIA_GEN_MAX_DISPLAY_ATTRIBUTES

Copy link
Contributor Author

Choose a reason for hiding this comment

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

accept

return VA_STATUS_SUCCESS;
}

VAStatus MediaLibvaCaps::SetDisplayAttributes(
Copy link
Contributor

Choose a reason for hiding this comment

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

pls remove it if media doesn't need it.

{
switch(attribList->type)
{
case VADisplayAttribSubDevice:
Copy link
Contributor

Choose a reason for hiding this comment

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

Does it for VADisplayAttribCopy?

@XinfengZhang XinfengZhang force-pushed the display_caps branch 2 times, most recently from 1b75ec9 to 72711ad Compare September 3, 2021 02:25
@XinfengZhang XinfengZhang added the verifying PR: fix ready and verifying with build/test label Sep 3, 2021
move the implementation of display attributes into MediaLibvaCaps class
inherit display attributes from MediaLibvaCaps for Gen12

Signed-off-by: XinfengZhang <[email protected]>
@intel-mediadev intel-mediadev merged commit 47654fc into intel:master Sep 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

verifying PR: fix ready and verifying with build/test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants