On Tue, Apr 04, 2017 at 12:44:26PM +0200, Maarten Lankhorst wrote:
> Op 03-04-17 om 10:32 schreef Daniel Vetter:
> > We do set DRIVER_ATOMIC now.
> >
> > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
> > ---
> > drivers/gpu/drm/i915/intel_display.c | 44 +-----------------------------------
> > 1 file changed, 1 insertion(+), 43 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> > index ba6687e31cbd..779ab46200c2 100644
> > --- a/drivers/gpu/drm/i915/intel_display.c
> > +++ b/drivers/gpu/drm/i915/intel_display.c
> > @@ -13113,50 +13113,8 @@ void intel_crtc_restore_mode(struct drm_crtc *crtc)
> > drm_atomic_state_put(state);
> > }
> >
> > -/*
> > - * FIXME: Remove this once i915 is fully DRIVER_ATOMIC by calling
> > - * drm_atomic_helper_legacy_gamma_set() directly.
> > - */
> > -static int intel_atomic_legacy_gamma_set(struct drm_crtc *crtc,
> > - u16 *red, u16 *green, u16 *blue,
> > - uint32_t size)
> > -{
> > - struct drm_device *dev = crtc->dev;
> > - struct drm_mode_config *config = &dev->mode_config;
> > - struct drm_crtc_state *state;
> > - int ret;
> > -
> > - ret = drm_atomic_helper_legacy_gamma_set(crtc, red, green, blue, size);
> > - if (ret)
> > - return ret;
> > -
> > - /*
> > - * Make sure we update the legacy properties so this works when
> > - * atomic is not enabled.
> > - */
> > -
> > - state = crtc->state;
> > -
> > - drm_object_property_set_value(&crtc->base,
> > - config->degamma_lut_property,
> > - (state->degamma_lut) ?
> > - state->degamma_lut->base.id : 0);
> > -
> > - drm_object_property_set_value(&crtc->base,
> > - config->ctm_property,
> > - (state->ctm) ?
> > - state->ctm->base.id : 0);
> > -
> > - drm_object_property_set_value(&crtc->base,
> > - config->gamma_lut_property,
> > - (state->gamma_lut) ?
> > - state->gamma_lut->base.id : 0);
> > -
> > - return 0;
> > -}
> > -
> > static const struct drm_crtc_funcs intel_crtc_funcs = {
> > - .gamma_set = intel_atomic_legacy_gamma_set,
> > + .gamma_set = drm_atomic_helper_legacy_gamma_set,
> > .set_config = drm_atomic_helper_set_config,
> > .set_property = drm_atomic_helper_crtc_set_property,
> > .destroy = intel_crtc_destroy,
>
> I guess the comment is out of date with drm_object_property_get_value now checking for
> drm_drv_uses_atomic_modeset instead.
Good point, I added a note about that to the commit message before
applying.
>
> All patches in this series look good, I've seen nothing to comment on. :)
>
> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
Thanks, Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
