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]> _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
