On Tue, Feb 27, 2018 at 10:49:39AM +0200, Ville Syrjälä wrote: > On Tue, Feb 27, 2018 at 12:54:47AM -0500, Ilia Mirkin wrote: > > 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.) > > No, this property is just about the range of the input data. Full range > YCbCr isn't exaclty common so you'd rarely need this. JPEG uses full > range I believe. > > > > > Anyways, with the subject typo(s! noticed ENCODNIG -> ENCODING just > > now) fixed, this is > > > > Reviewed-by: Ilia Mirkin <imirkin@xxxxxxxxxxxx> > > Cool. Thanks. Pushed to drm-misc-next with Ben's ack. 17:27 <imirkin_> <imirkin_> friendly reminder about https://patchwork.freedesktop.org/patch/205620/ -- ville was wondering whether you'd take it, or if he should push via drm-misc. 17:27 <imirkin_> <skeggsb> ack, i'd say ville should take it in drm-misc for now -- Ville Syrjälä Intel OTC _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
