On Tue, Feb 20, 2018 at 9:25 AM, Ilia Mirkin <imirkin@xxxxxxxxxxxx> wrote: > On Tue, Feb 20, 2018 at 8:48 AM, Ville Syrjala > <ville.syrjala@xxxxxxxxxxxxxxx> wrote: >> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> >> >> Replace the ad-hoc iturbt_709 property with the new standard >> COLOR_ENCODING property. Compiles, but not tested. >> >> Cc: Daniel Vetter <daniel@xxxxxxxx> >> Cc: nouveau@xxxxxxxxxxxxxxxxxxxxx >> Cc: Ben Skeggs <bskeggs@xxxxxxxxxx> >> Cc: Ilia Mirkin <imirkin@xxxxxxxxxxxx> >> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > 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 <imirkin@xxxxxxxxxxxx> _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
