-
Notifications
You must be signed in to change notification settings - Fork 346
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
[Encode] report sub features for HEVC VDENC #1121
base: master
Are you sure you want to change the base?
Conversation
8f276a9
to
2680ab4
Compare
The latest version of the attributes remove The rest of this looks good to me, thank you. (I suppose the attributes will need to have more versions for each of the hardware platforms supported by the driver - Gen12 with shaders, Gen11 in both modes, Gen9.) |
fc8e94e
to
fbf8e09
Compare
} | ||
break; | ||
} | ||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May need to call GetPlatformSpecificAttrib with type VAConfigAttribEncHEVCFeatures/VAConfigAttribEncHEVCBlockSizes in CreateEncAttributes to register them in attribList.
Also could you help to add the two attribute setting for non-low-power mode?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Registering them in attribList is feasible, will update this PR after verify them.
Supporting these two attributes setting for non-low-power mode, we need setup a meeting to discuss.
Fill in VAConfigAttribValEncHEVCFeatures and VAConfigAttribValEncHEVCBlockSizes with the approriate values for HEVC VDENC.
fbf8e09
to
8675528
Compare
Fill in VAConfigAttribValEncHEVCFeatures and VAConfigAttribValEncHEVCBlockSizes
with the approriate values for HEVC VDENC.