As we initialize our own gamma LUT to sane values during initialisation, we should also update the user visible gamma ramp to match our settings. And before touching the storage for the user visible gamma ramp, we also need to check for an allocation failure. v2: set_gamma_size() returns an error code, not success/fail. Also expand the 8bit LUT value to 16bit using bit replication. v3: Just log the error and disable gamma user functionality if we fail to allocate the ramp during initialisation. Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Cc: stable@xxxxxxxxxxxxxxx --- drivers/gpu/drm/i915/intel_display.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 5988bda..22bdf16 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -8983,11 +8983,18 @@ static void intel_crtc_init(struct drm_device *dev, int pipe) drm_crtc_init(dev, &intel_crtc->base, &intel_crtc_funcs); - drm_mode_crtc_set_gamma_size(&intel_crtc->base, 256); + if (drm_mode_crtc_set_gamma_size(&intel_crtc->base, 256)) + DRM_ERROR("failed to allocate ramp for gamma, changing gamma tables will be unavailable on CRTC:%d (pipe %d)\n", intel_crtc->base.base.id, pipe); + for (i = 0; i < 256; i++) { intel_crtc->lut_r[i] = i; intel_crtc->lut_g[i] = i; intel_crtc->lut_b[i] = i; + if (intel_crtc->base.gamma_size) { + intel_crtc->base.gamma_store[i + 0*256] = i << 8 | i; + intel_crtc->base.gamma_store[i + 1*256] = i << 8 | i; + intel_crtc->base.gamma_store[i + 2*256] = i << 8 | i; + } } /* Swap pipes & planes for FBC on pre-965 */ -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html