Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752049AbdDCH6H (ORCPT ); Mon, 3 Apr 2017 03:58:07 -0400 Received: from mail-it0-f66.google.com ([209.85.214.66]:35684 "EHLO mail-it0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751734AbdDCH6F (ORCPT ); Mon, 3 Apr 2017 03:58:05 -0400 MIME-Version: 1.0 X-Originating-IP: [2a02:168:56b5:0:3dcb:681a:5b6e:8025] In-Reply-To: <1491046529-4310-8-git-send-email-jeffy.chen@rock-chips.com> References: <1491046529-4310-1-git-send-email-jeffy.chen@rock-chips.com> <1491046529-4310-8-git-send-email-jeffy.chen@rock-chips.com> From: Daniel Vetter Date: Mon, 3 Apr 2017 09:58:04 +0200 X-Google-Sender-Auth: pxtpW_hO1ld3J8wyz5d5N5mRZB8 Message-ID: Subject: Re: [PATCH v2 7/9] drm/rockchip: Force disable all crtc when unload To: Jeffy Chen Cc: Linux Kernel Mailing List , Brian Norris , Douglas Anderson , dri-devel , Tomasz Figa , "open list:ARM/Rockchip SoC..." , Chris Zhong , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 808 Lines: 19 On Sat, Apr 1, 2017 at 1:35 PM, Jeffy Chen wrote: > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > index a5d83cb..5dbf011 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > @@ -246,6 +246,7 @@ static void rockchip_drm_unbind(struct device *dev) > rockchip_drm_fbdev_fini(drm_dev); > drm_kms_helper_poll_fini(drm_dev); > > + drm_crtc_force_disable_all(drm_dev); This will result in a WARN_ON in latest drm-next because rockchip is atomic, and this helper is for legacy kms drivers. You want to use drm_atomic_helper_shutdown here. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch