Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932358Ab1DGFsL (ORCPT ); Thu, 7 Apr 2011 01:48:11 -0400 Received: from mail-iw0-f174.google.com ([209.85.214.174]:53487 "EHLO mail-iw0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932245Ab1DGFsK (ORCPT ); Thu, 7 Apr 2011 01:48:10 -0400 Date: Wed, 6 Apr 2011 22:48:06 -0700 From: Grant Likely To: "Rafael J. Wysocki" Cc: Kevin Hilman , Magnus Damm , Linux PM mailing list , LKML , Paul Mundt , linux-sh@vger.kernel.org Subject: Re: [RFC][PATCH] PM / Platform: Remove __weak definitions of runtime PM callbacks Message-ID: <20110407054806.GC6427@angua.secretlab.ca> References: <201103270058.41632.rjw@sisk.pl> <87zko3dn4b.fsf@ti.com> <201104070729.45484.rjw@sisk.pl> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201104070729.45484.rjw@sisk.pl> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2146 Lines: 54 On Thu, Apr 07, 2011 at 07:29:45AM +0200, Rafael J. Wysocki wrote: > On Thursday, April 07, 2011, Kevin Hilman wrote: > > Hi Rafael, Magnus, > > > > "Rafael J. Wysocki" writes: > > > > > From: Rafael J. Wysocki > > > > > > Remove the __weak definitions of platform bus type runtime PM > > > callbacks, make platform_dev_pm_ops point to the generic routines > > > as appropriate and allow architectures using platform_dev_pm_ops to > > > replace the runtime PM callbacks in that structure with their own > > > set. > > > > > > Convert architectures providing its own definitions of the platform > > > runtime PM callbacks to use the new mechanism. > > > > > > Signed-off-by: Rafael J. Wysocki > > > > I dont't think we should be adding yet another new interface for setting > > platform-specific runtime PM ops. > > > > We now have 3. Two existing ones: > > > > 1) new device power domains (presumably preferred) > > 2) platform_bus_set_pm_ops() (disliked by many) > > Hmm, I wasn't aware of that one, will have a look. > > > and now the new one you create here > > > > 3) platform_set_runtime_pm_ops() > > > > This new one is basically the same as platform_bus_set_pm_ops(), but > > targetted only at runtime PM ops, and also has all the same problems > > that have been discussed before. Namely, it overrides the pm ops for > > *every* device on the platform_bus, instead of targetting only specific > > devices. > > This is not a problem for this particular use case. We really want to > replace the PM ops for all of the platform devices on that platform. I strongly doubt that you really want to do that. platform_devices can appear anywhere in the system, and many of them will end up being entirely outside the SoC, and hence outside of any SoC specific behaviour. What is the use case for overriding every platform_device's PM ops? g. -- 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/