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

generic: add support for BT2020 #7414

Closed
wants to merge 1 commit into from
Closed

Conversation

CvH
Copy link
Member

@CvH CvH commented Jan 22, 2023

This adds the BT2020 patch for generic, we already carry that patch for RPi and afaik it was long term at the SMP HDR builds.

I have no HW to test it, @smp79 could you test it ?

@smp79
Copy link
Contributor

smp79 commented Jan 23, 2023

This is not the correct patch.
IMO it would make sense to include xbmc/xbmc#21845 in LE 11.0. This does make the TV switch to BT2020 when playing HDR video on Intel HW.

@lrusak
Copy link
Member

lrusak commented Jan 23, 2023

We shouldn't be adding functionality that doesn't exist outside vanilla Kodi.

Just because it's PR'd doesn't mean it's going to be merged.

@atoulmin
Copy link

This is not the correct patch.
IMO it would make sense to include xbmc/xbmc#21845 in LE 11.0. This does make the TV switch to BT2020 when playing HDR video on Intel HW.

Not that my opinion is worth much, but this should definitely be included.

@allixx
Copy link

allixx commented Feb 23, 2023

The patch from the pull request won't apply on Nexus, it seems to be dependant on some other patch. It also only covers DRM Prime.

xbmc/xbmc#21845 this one works fine on Generic/GBM Intel; and while it is understandable that it is not a complete solution and doesn't cover all possible corner cases, imo it is better than nothing. If there is interest in test run results material, logs etc, I'm happy to provide it.

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

Successfully merging this pull request may close these issues.

None yet

5 participants