On Tue, Feb 20, 2018 at 9:25 AM, Ilia Mirkin <imir...@alum.mit.edu> wrote: > On Tue, Feb 20, 2018 at 8:48 AM, Ville Syrjala > <ville.syrj...@linux.intel.com> wrote: >> From: Ville Syrjälä <ville.syrj...@linux.intel.com> >> >> Replace the ad-hoc iturbt_709 property with the new standard >> COLOR_ENCODING property. Compiles, but not tested. >> >> Cc: Daniel Vetter <dan...@ffwll.ch> >> Cc: nouv...@lists.freedesktop.org >> Cc: Ben Skeggs <bske...@redhat.com> >> Cc: Ilia Mirkin <imir...@alum.mit.edu> >> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com> > > s/standarad/standard/ in subject > > I'd like the opportunity to test this out on real hardware, but I > don't have any pre-NV41 boards plugged in right now. I should be able > to attend to it within 7 days. If you don't hear back from me by then, > I'd appreciate a ping, as I do let things (hopefully occasionally) > slip through.
Tested this out on a NV34. Seems to work - at least the green SMPTE bar looks different in the two different modes (had to hack modetest to use the SMPTE pattern on planes). I do wonder about the new color range property... that's more of a connector thing than a plane thing no? Presumably over S-Video it's a partial range, and over VGA it's full? I'd flip it to always full here tbh. (Or just not provide it at all.) Anyways, with the subject typo(s! noticed ENCODNIG -> ENCODING just now) fixed, this is Reviewed-by: Ilia Mirkin <imir...@alum.mit.edu> _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
