Hi, On 2018-01-12 16:07, Benjamin Gaignard wrote: > 2018-01-12 14:21 GMT+01:00 Peter Ujfalusi <peter.ujfalusi@xxxxxx>: >> Instead of re-implementing the drm_atomic_helper_check() locally with just >> adding drm_atomic_normalize_zpos() into it, set the >> drm_device->normalize_zpos. >> > > Acked-by: Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxx> Thanks Benjamin! I just noticed that I forgot to update the commit message with s/drm_device/drm_mode_config I will resend with updated commit message on Monday. > >> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@xxxxxx> >> CC: Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxx> >> CC: Vincent Abriou <vincent.abriou@xxxxxx> >> Acked-by: Daniel Vetter <daniel.vetter@xxxxxxxx> >> --- >> drivers/gpu/drm/sti/sti_drv.c | 24 +++--------------------- >> 1 file changed, 3 insertions(+), 21 deletions(-) >> >> diff --git a/drivers/gpu/drm/sti/sti_drv.c b/drivers/gpu/drm/sti/sti_drv.c >> index 55b6967d27e1..90c46b49c931 100644 >> --- a/drivers/gpu/drm/sti/sti_drv.c >> +++ b/drivers/gpu/drm/sti/sti_drv.c >> @@ -119,30 +119,10 @@ static int sti_drm_dbg_init(struct drm_minor *minor) >> return ret; >> } >> >> -static int sti_atomic_check(struct drm_device *dev, >> - struct drm_atomic_state *state) >> -{ >> - int ret; >> - >> - ret = drm_atomic_helper_check_modeset(dev, state); >> - if (ret) >> - return ret; >> - >> - ret = drm_atomic_normalize_zpos(dev, state); >> - if (ret) >> - return ret; >> - >> - ret = drm_atomic_helper_check_planes(dev, state); >> - if (ret) >> - return ret; >> - >> - return ret; >> -} >> - >> static const struct drm_mode_config_funcs sti_mode_config_funcs = { >> .fb_create = drm_gem_fb_create, >> .output_poll_changed = drm_fb_helper_output_poll_changed, >> - .atomic_check = sti_atomic_check, >> + .atomic_check = drm_atomic_helper_check, >> .atomic_commit = drm_atomic_helper_commit, >> }; >> >> @@ -160,6 +140,8 @@ static void sti_mode_config_init(struct drm_device *dev) >> dev->mode_config.max_height = STI_MAX_FB_HEIGHT; >> >> dev->mode_config.funcs = &sti_mode_config_funcs; >> + >> + dev->mode_config.normalize_zpos = true; >> } >> >> DEFINE_DRM_GEM_CMA_FOPS(sti_driver_fops); >> -- >> Peter >> >> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. >> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki - Péter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel