Skip to content

report the capability of vaCopy support #513

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

Merged
merged 1 commit into from
Jun 16, 2021

Conversation

XinfengZhang
Copy link
Contributor

Signed-off-by: Carl Zhang carl.zhang@intel.com

va/va.h Outdated
* adaptor / platform attributes. read only. specify whether vaCopy support on current HW
* the value of each bit should equal to 1 << VA_EXEC_MODE_XXX
*/
VADisplayAttibCopy = 19,
Copy link
Contributor

Choose a reason for hiding this comment

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

Fix: s/Attib/Attrib/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, thanks

va/va.h Outdated
* App could call vaQueryDisplayAttributes/vaGetDisplayAttributes to query these attributes
* at anytime after vaInitialize, but only could call vaSetDisplayAttributes for these attributes
* after vaInitialize and before any other function call if the attributes is settable.
* To distinguish these two types display attributes, this display adaptor related attributres
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: "attributes"

@XinfengZhang XinfengZhang force-pushed the mcp_attrib branch 2 times, most recently from 88701a9 to e448c30 Compare April 12, 2021 14:11
XinfengZhang added a commit to XinfengZhang/media-driver that referenced this pull request Apr 25, 2021
it is against intel/libva#513

Signed-off-by: XinfengZhang <carl.zhang@intel.com>
XinfengZhang added a commit to XinfengZhang/media-driver that referenced this pull request Apr 25, 2021
it is against intel/libva#513

Signed-off-by: XinfengZhang <carl.zhang@intel.com>
va/va.h Outdated
@@ -4835,6 +4835,15 @@ VAStatus vaDeassociateSubpicture (
* brightness etc. in the rendering process. The application can query what
* attributes are supported by the driver, and then set the appropriate attributes
* before calling vaPutSurface()
*
* Display attributes also are used to query/set display adaptor (vaDisplay) related information.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest to rewrite:

 * Display attributes can also be used to query/set platform or display adaptor (vaDisplay)
 * related information. These attributes do not depend on vaConfig, and could not be used
 * for vaPutSurface. Application can use vaQueryDisplayAttributes/vaGetDisplayAttributes
 * at anytime after vaInitialize, but (for settable attributes) vaSetDisplayAttributes should be
 * called after vaInitialize and before any other function call.
 *
 * To distinguish these two types of display attributes, display adaptor related attributes
 * should be marked as "HW attribute" in the description.

Report vaCopy support, it is a standalone feature , which is related with vaDisplay. not depend on VPP , vaConfig etc.

Signed-off-by: Carl Zhang <carl.zhang@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants