lkml.org 
[lkml]   [2017]   [Dec]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 79/95] drm/fb_helper: Disable all crtcs when initial setup fails.
    Date
    4.14-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>

    commit 52dd06506e9bbc2a37b352df7dfc5468f8da21fd upstream.

    Some drivers like i915 start with crtc's enabled, but with deferred
    fbcon setup they were no longer disabled as part of fbdev setup.
    Headless units could no longer enter pc3 state because the crtc was
    still enabled.

    Fix this by calling restore_fbdev_mode when we would have called
    it otherwise once during initial fbdev setup.

    Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
    Fixes: ca91a2758fce ("drm/fb-helper: Support deferred setup")
    Reported-by: Thomas Voegtle <tv@lio96.de>
    Tested-by: Thomas Voegtle <tv@lio96.de>
    Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
    Link: https://patchwork.freedesktop.org/patch/msgid/20171128111603.62757-1-maarten.lankhorst@linux.intel.com
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    drivers/gpu/drm/drm_fb_helper.c | 4 ++++
    1 file changed, 4 insertions(+)

    --- a/drivers/gpu/drm/drm_fb_helper.c
    +++ b/drivers/gpu/drm/drm_fb_helper.c
    @@ -1809,6 +1809,10 @@ static int drm_fb_helper_single_fb_probe

    if (crtc_count == 0 || sizes.fb_width == -1 || sizes.fb_height == -1) {
    DRM_INFO("Cannot find any crtc or sizes\n");
    +
    + /* First time: disable all crtc's.. */
    + if (!fb_helper->deferred_setup && !READ_ONCE(fb_helper->dev->master))
    + restore_fbdev_mode(fb_helper);
    return -EAGAIN;
    }


    \
     
     \ /
      Last update: 2017-12-04 17:08    [W:4.195 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site