drm/fb_cma_helper: Remove implicit call to disable_unused_functions
authorMaxime Ripard <maxime.ripard@free-electrons.com>
Thu, 14 Jan 2016 15:24:56 +0000 (16:24 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 15 Jan 2016 10:16:15 +0000 (11:16 +0100)
commit4314e19ef4ae0ba8872bd8610f6fef5e8743e236
tree0b8a4e4632dcbe267370d632215d904544f3f989
parentd122cbf1a310d962cef45c2f161b8dc3ec1475a3
drm/fb_cma_helper: Remove implicit call to disable_unused_functions

The drm_fbdev_cma_init function always calls the
drm_helper_disable_unused_functions. Since it's part of the usual probe
process, all the drivers using that helper will end up having their encoder
and CRTC disable functions called at probe if their device has not been
reported as enabled.

This could be fixed by reading out from the registers the current state of
the device if it is enabled, but even that will not handle the case where
the device is actually disabled.

Moreover, the drivers using the atomic modesetting expect that their enable
and disable callback to be called when the device is already enabled or
disabled (respectively).

We can however fix this issue by moving the call to
drm_helper_disable_unused_functions out of drm_fbdev_cma_init and make the
drivers needing it (all the drivers calling drm_fbdev_cma_init and not
using the atomic modesetting) explicitly call it.

Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1452785109-6172-14-git-send-email-maxime.ripard@free-electrons.com
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/drm_fb_cma_helper.c
drivers/gpu/drm/imx/imx-drm-core.c
drivers/gpu/drm/sti/sti_drv.c
drivers/gpu/drm/tilcdc/tilcdc_drv.c