Ilia Mirkin
2018-Feb-27 05:54 UTC
[Nouveau] [PATCH] drm/nouveau: Replace the iturbt_709 prop with the standarad COLOR_ENCODNIG prop
On Tue, Feb 20, 2018 at 9:25 AM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:> On Tue, Feb 20, 2018 at 8:48 AM, Ville Syrjala > <ville.syrjala at linux.intel.com> wrote: >> From: Ville Syrjälä <ville.syrjala at linux.intel.com> >> >> Replace the ad-hoc iturbt_709 property with the new standard >> COLOR_ENCODING property. Compiles, but not tested. >> >> Cc: Daniel Vetter <daniel at ffwll.ch> >> Cc: nouveau at lists.freedesktop.org >> Cc: Ben Skeggs <bskeggs at redhat.com> >> Cc: Ilia Mirkin <imirkin at alum.mit.edu> >> Signed-off-by: Ville Syrjälä <ville.syrjala at 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 <imirkin at alum.mit.edu>
Ville Syrjälä
2018-Feb-27 08:49 UTC
[Nouveau] [PATCH] drm/nouveau: Replace the iturbt_709 prop with the standarad COLOR_ENCODNIG prop
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 at alum.mit.edu> wrote: > > On Tue, Feb 20, 2018 at 8:48 AM, Ville Syrjala > > <ville.syrjala at linux.intel.com> wrote: > >> From: Ville Syrjälä <ville.syrjala at linux.intel.com> > >> > >> Replace the ad-hoc iturbt_709 property with the new standard > >> COLOR_ENCODING property. Compiles, but not tested. > >> > >> Cc: Daniel Vetter <daniel at ffwll.ch> > >> Cc: nouveau at lists.freedesktop.org > >> Cc: Ben Skeggs <bskeggs at redhat.com> > >> Cc: Ilia Mirkin <imirkin at alum.mit.edu> > >> Signed-off-by: Ville Syrjälä <ville.syrjala at 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.)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 at alum.mit.edu>Cool. Thanks. -- Ville Syrjälä Intel OTC
Ville Syrjälä
2018-Mar-08 16:28 UTC
[Nouveau] [PATCH] drm/nouveau: Replace the iturbt_709 prop with the standarad COLOR_ENCODNIG prop
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 at alum.mit.edu> wrote: > > > On Tue, Feb 20, 2018 at 8:48 AM, Ville Syrjala > > > <ville.syrjala at linux.intel.com> wrote: > > >> From: Ville Syrjälä <ville.syrjala at linux.intel.com> > > >> > > >> Replace the ad-hoc iturbt_709 property with the new standard > > >> COLOR_ENCODING property. Compiles, but not tested. > > >> > > >> Cc: Daniel Vetter <daniel at ffwll.ch> > > >> Cc: nouveau at lists.freedesktop.org > > >> Cc: Ben Skeggs <bskeggs at redhat.com> > > >> Cc: Ilia Mirkin <imirkin at alum.mit.edu> > > >> Signed-off-by: Ville Syrjälä <ville.syrjala at 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.) > > 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 at alum.mit.edu> > > 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
Possibly Parallel Threads
- [PATCH] drm/nouveau: Replace the iturbt_709 prop with the standarad COLOR_ENCODNIG prop
- [PATCH] drm/nouveau: Replace the iturbt_709 prop with the standarad COLOR_ENCODNIG prop
- [PATCH] drm/nouveau: Replace the iturbt_709 prop with the standarad COLOR_ENCODNIG prop
- [PATCH 1/2] drm/nouveau: Disable atomic support on a per-device basis
- [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions