Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754010Ab0HWOyB (ORCPT ); Mon, 23 Aug 2010 10:54:01 -0400 Received: from mail-pv0-f174.google.com ([74.125.83.174]:42233 "EHLO mail-pv0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753961Ab0HWOx5 convert rfc822-to-8bit (ORCPT ); Mon, 23 Aug 2010 10:53:57 -0400 From: Kevin Hilman To: Grant Likely Cc: "Moffett\, Kyle D" , Patrick Pannuto , "linux-kernel\@vger.kernel.org" , "linux-arm-msm\@vger.kernel.org" , "magnus.damm\@gmail.com" , "gregkh\@suse.de" , Paul Mundt , Magnus Damm , "Rafael J. Wysocki" , Eric Miao , Dmitry Torokhov , "netdev\@vger.kernel.org" , Kyle D Moffett Subject: Re: [PATCH 2/2] platform: Facilitate the creation of pseudo-platform buses Organization: Deep Root Systems, LLC References: <1281484174-32174-1-git-send-email-ppannuto@codeaurora.org> <1281484174-32174-3-git-send-email-ppannuto@codeaurora.org> <3F978429-F916-42E5-8B36-6AC02DAC8CA2@boeing.com> <87hbiql89d.fsf@deeprootsystems.com> <87hbip9kxx.fsf@deeprootsystems.com> <87y6c03jxj.fsf@deeprootsystems.com> Date: Mon, 23 Aug 2010 07:53:40 -0700 In-Reply-To: (Grant Likely's message of "Sat, 21 Aug 2010 01:10:05 -0600") Message-ID: <87tyml1itn.fsf@deeprootsystems.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1756 Lines: 40 Grant Likely writes: >>> If the override is global to the platform_bus_type, then the model >>> will not scale. >> >> It will not scale any more (or less) than the current functionality of >> the driver model which handles this globally. ?Again, if scalabilty >> becomes a problem down the road, lets fix it then. > > Alright, I can agree to that. I do agree that the runtime override is > far and away better than the weak symbol approach. However, there > needs to be some very clear rules in place for users of the override, > namely that users must understand that they are stewards of the > platform_bus_type, and must take care to preserve the default > behaviour for "uninteresting" devices. Completely agree here. Any overrides of the dev_pm_ops functions that do not also call the pm_generic_* functions (or their equivalents) should be suspect. I'll post a proposal for a runtime override shortly. > Also, the expectation should be that it is a temporary measure until a > better abstraction is implemented. It might be that a separate > bus_type is the way to go (if the multiple driver registration problem > can be solved) or it might be a way to differentiate pm_ops either > per-device or per-parent. I'm not sure, but I'm starting on an OMAP > project soon, so I may very well end up working on this. Yes, I'll be glad to work on this too, but first I need to get through reviewing the backlog of all the OMAP drivers we're converting to runtime PM. Kevin -- 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/