On Tue, May 28, 2019 at 12:51:04PM +0200, Maarten Lankhorst wrote: > Op 13-05-2019 om 15:39 schreef Ville Syrjala: > > From: Ville Syrjälä <[email protected]> > > > > Leave a hint in dmesg when we reject a configuration attempting > > to use C8 planes without the legacy LUT loaded. > > > > Signed-off-by: Ville Syrjälä <[email protected]> > > --- > > drivers/gpu/drm/i915/intel_color.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/i915/intel_color.c > > b/drivers/gpu/drm/i915/intel_color.c > > index 962db1236970..45649904ba5c 100644 > > --- a/drivers/gpu/drm/i915/intel_color.c > > +++ b/drivers/gpu/drm/i915/intel_color.c > > @@ -959,8 +959,10 @@ static int check_luts(const struct intel_crtc_state > > *crtc_state) > > return 0; > > > > /* C8 relies on its palette being stored in the legacy LUT */ > > - if (crtc_state->c8_planes) > > + if (crtc_state->c8_planes) { > > + DRM_DEBUG_KMS("C8 pixelformat requires the legacy LUT\n"); > > return -EINVAL; > > + } > > > > degamma_length = INTEL_INFO(dev_priv)->color.degamma_lut_size; > > gamma_length = INTEL_INFO(dev_priv)->color.gamma_lut_size; > > For the series: > > Reviewed-by: Maarten Lankhorst <[email protected]>
Thanks. Pushed. -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
