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

[VideoPlayer] d3d11va improvements. #9198

Merged
merged 3 commits into from Feb 24, 2016
Merged

[VideoPlayer] d3d11va improvements. #9198

merged 3 commits into from Feb 24, 2016

Conversation

afedchin
Copy link
Member

  1. dropped unneeded dxva->d3d11va wrapper. Now we build ffmpeg with d3d11va support and can use it directly.
  2. Added support of hevc main10 profile to our dxva decoder. Still needs to updated ffmpeg.
  3. Added support 10/16 bit video output through dxva renderer.

@FernetMenta @fritsch ping.

@afedchin afedchin added Type: Improvement non-breaking change which improves existing functionality v17 Krypton Component: Video Platform: Windows labels Feb 24, 2016
@afedchin
Copy link
Member Author

jenkins build this please

@fritsch
Copy link
Member

fritsch commented Feb 24, 2016

Thank you very much, I pushed a the 3.0-xbmc branch to kodi's ffmpeg fork - if you want you can provide testbuilds by just changing our ffmpeg version as we build from branch already for master: https://github.com/xbmc/FFmpeg/commits/release/3.0-xbmc

Btw. we have a whole lot of warnings to fix with the upcoming 3.0 version

@fritsch
Copy link
Member

fritsch commented Feb 24, 2016

If it works correctly, you can pick: fritsch@1cb028d

@afedchin
Copy link
Member Author

@fritsch properly ffmpeg bump afedchin@a46a3b6

@fritsch
Copy link
Member

fritsch commented Feb 24, 2016

Jep, that's better :-) After those commits are properly discussed just add that one on top.

@afedchin
Copy link
Member Author

hm.. interesting fail. lets' try again
jenkins build this please

@FernetMenta
Copy link
Contributor

I don't want to see ffmpeg bumped with this change. Please bump it separately and take the time to look at the mentioned warning.

@afedchin
Copy link
Member Author

as I said in slack this may go in without ffmpeg bump. if @FernetMenta has no objections I will merge this in next window.

@fritsch
Copy link
Member

fritsch commented Feb 24, 2016

@FernetMenta jep, good decision. The warnings are deprecation warnings. We currently have ~ 10 warnings in our code with 2.8, that can also be solved / silenced / ported that way.

@FernetMenta
Copy link
Contributor

I don't have any objections to this pr. go ahead.

afedchin added a commit that referenced this pull request Feb 24, 2016
[VideoPlayer] d3d11va improvements.
@afedchin afedchin merged commit 9d2b1fc into xbmc:master Feb 24, 2016
@afedchin afedchin deleted the d3d11va branch February 24, 2016 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Video Platform: Windows Type: Improvement non-breaking change which improves existing functionality v17 Krypton
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants