From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 01/18] drm/omap: fix up pdev_remove Date: Wed, 16 Apr 2014 10:15:03 +0200 Message-ID: <20140416081503.GG10722@phenom.ffwll.local> References: <1397252175-14227-1-git-send-email-daniel.vetter@ffwll.ch> <1397252175-14227-2-git-send-email-daniel.vetter@ffwll.ch> <534B7F86.1040104@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ee0-f44.google.com (mail-ee0-f44.google.com [74.125.83.44]) by gabe.freedesktop.org (Postfix) with ESMTP id 87BF66EA1E for ; Wed, 16 Apr 2014 01:15:08 -0700 (PDT) Received: by mail-ee0-f44.google.com with SMTP id e49so8513068eek.31 for ; Wed, 16 Apr 2014 01:15:07 -0700 (PDT) Content-Disposition: inline In-Reply-To: <534B7F86.1040104@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Tomi Valkeinen Cc: Daniel Vetter , DRI Development , Dave Airlie List-Id: dri-devel@lists.freedesktop.org On Mon, Apr 14, 2014 at 09:26:14AM +0300, Tomi Valkeinen wrote: > Hi, > > On 12/04/14 00:35, Daniel Vetter wrote: > > Dave accidentally merged the wrong version of the patch in > > > > commit fd3c02531461924853db65f2664db361b53a70d3 > > Author: Daniel Vetter > > Date: Wed Dec 11 11:34:26 2013 +0100 > > > > drm/omap: call drm_put_dev directly in ->remove > > > > which did not include the fix from Rob's review: omapdrm is split into > > the drm driver and the dmm driver (yeah, I've complained about that > > almost-impossible-to-spot difference, too). We need to unregister the > > dmm driver in the pdev_remove hook. > > > > Cc: Dave Airlie > > Cc: Rob Clark > > Signed-off-by: Daniel Vetter > > --- > > drivers/gpu/drm/omapdrm/omap_drv.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/gpu/drm/omapdrm/omap_drv.c b/drivers/gpu/drm/omapdrm/omap_drv.c > > index bf39fcc49e0f..89557989737d 100644 > > --- a/drivers/gpu/drm/omapdrm/omap_drv.c > > +++ b/drivers/gpu/drm/omapdrm/omap_drv.c > > @@ -699,6 +699,8 @@ static int pdev_remove(struct platform_device *device) > > omap_disconnect_dssdevs(); > > omap_crtc_pre_uninit(); > > > > + platform_driver_unregister(&omap_dmm_driver); > > + > > drm_put_dev(platform_get_drvdata(device)); > > return 0; > > } > > Your patch does fix the "drm/omap: call drm_put_dev directly in > ->remove" patch, but I think the unregistering is not right if done in > pdev_remove. > > I sent this patch a few weeks ago: > > http://www.spinics.net/lists/dri-devel/msg56464.html > > It unregisters the dmm driver at the module exit function, which is the > proper place for it as the dmm driver is registered in the module init > function. I don't mind how this is getting fixed, I just don't like when regression caused by my patches are left lingering too long ;-) Have you merged your patch for 3.14-rc already so that I can drop this one here? Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch