Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965080AbbBJOn7 (ORCPT ); Tue, 10 Feb 2015 09:43:59 -0500 Received: from mx-guillaumet.finsecur.com ([91.217.234.131]:58903 "EHLO guillaumet.finsecur.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750872AbbBJOn5 (ORCPT ); Tue, 10 Feb 2015 09:43:57 -0500 Date: Tue, 10 Feb 2015 15:43:51 +0100 From: Sylvain Rochet To: Boris Brezillon Cc: David Airlie , dri-devel@lists.freedesktop.org, Nicolas Ferre , Jean-Christophe Plagniol-Villard , Alexandre Belloni , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Daniel Vetter Message-ID: <20150210144351.GA14171@gradator.net> References: <1423575646-3273-1-git-send-email-sylvain.rochet@finsecur.com> <1423575646-3273-2-git-send-email-sylvain.rochet@finsecur.com> <20150210150557.235fa139@bbrezillon> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20150210150557.235fa139@bbrezillon> User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: 172.16.8.13 X-SA-Exim-Mail-From: sylvain.rochet@finsecur.com Subject: Re: [PATCH 1/2] drm: atmel-hlcdc: Add PM suspend/resume support X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on guillaumet.finsecur.com) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1294 Lines: 32 Hello Boris, On Tue, Feb 10, 2015 at 03:05:57PM +0100, Boris Brezillon wrote: > On Tue, 10 Feb 2015 14:40:45 +0100 > Sylvain Rochet wrote: > > + > > +static SIMPLE_DEV_PM_OPS(atmel_hlcdc_dc_drm_pm_ops, > > + atmel_hlcdc_dc_drm_suspend, atmel_hlcdc_dc_drm_resume); > > + > > Do we really need to register both SIMPLE_DEV_PM_OPS and drm_driver > suspend/resume functions. > I thought the suspend/resume callbacks were called by DRM core code > as part of the PM class specific operations. > However, exynos driver seems to do the same, so this might well be > required. Looks like we need, if I comment out SIMPLE_DEV_PM_OPS ops registration we don't suspend or resume anymore. I guess drm_driver suspend/resume functions are only used for PM runtime support, which we don't support yet, at least we added the basic requirements to add PM runtime support at some point in the future. Anyway, even without PM runtime support we need to suspend and resume this device, other DRM drivers do this as well. Sylvain -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/