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

VAAPI: add vp8 support #13843

Merged
merged 2 commits into from
May 5, 2018
Merged

VAAPI: add vp8 support #13843

merged 2 commits into from
May 5, 2018

Conversation

lrusak
Copy link
Contributor

@lrusak lrusak commented May 2, 2018

used on Gemini lake hardware

Kodi log: http://ix.io/19fI

12:33:32.155 T:139942141953792   DEBUG: CDVDVideoCodecFFmpeg - Updated codec: ff-vp8-vaapi

vainfo relevance:

      VAProfileVP8Version0_3          :	VAEntrypointVLD

@FernetMenta
Copy link
Contributor

does this work reliable for all vp8 files?

@lrusak
Copy link
Contributor Author

lrusak commented May 3, 2018

I haven't had any issues yet, most of my vp8 files are just videos I've downloaded from youtube using youtube-dl

@FernetMenta
Copy link
Contributor

The background of my question: we have settings to disable codecs supported by vaapi (all but h.264). IMO it would make sense making vp8 configurable too.

@lrusak
Copy link
Contributor Author

lrusak commented May 4, 2018

should I do it for vp9 also?

@lrusak lrusak force-pushed the vp8-vaapi branch 2 times, most recently from 384efac to df3ef2c Compare May 4, 2018 21:36
@lrusak
Copy link
Contributor Author

lrusak commented May 4, 2018

I can PR the vp9 commit separately if desired

@@ -121,6 +121,8 @@ class CSettings : public CSettingsBase, public CSettingCreator, public CSettingC
static const std::string SETTING_VIDEOPLAYER_USEVAAPIMPEG2;
static const std::string SETTING_VIDEOPLAYER_USEVAAPIMPEG4;
static const std::string SETTING_VIDEOPLAYER_USEVAAPIVC1;
static const std::string SETTING_VIDEOPLAYER_USEVAAPIVP8;

This comment was marked as spam.

@lrusak
Copy link
Contributor Author

lrusak commented May 5, 2018

tested both settings and they are working properly on gemini lake.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants