Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756535AbZFYCbZ (ORCPT ); Wed, 24 Jun 2009 22:31:25 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751772AbZFYCbJ (ORCPT ); Wed, 24 Jun 2009 22:31:09 -0400 Received: from mail-gx0-f222.google.com ([209.85.217.222]:48530 "EHLO mail-gx0-f222.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753368AbZFYCbH convert rfc822-to-8bit (ORCPT ); Wed, 24 Jun 2009 22:31:07 -0400 X-Greylist: delayed 330 seconds by postgrey-1.27 at vger.kernel.org; Wed, 24 Jun 2009 22:31:07 EDT DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=o1TbkwWcwwsF5s70mNkmWhDT4R1JDCoxhrNZeTynM4YhKKNwbSiPCcGgMYp9xEWktn sheamjRog9bOhMNOMcNXn+PCyfTfsAV4UtuX7yw9rSRsejyoNt6hjJ/iT5OOWgKn9Ocz xxpav0eQDieMTgAWtKneCoZ5/KQ6676fQBRo0= MIME-Version: 1.0 In-Reply-To: <200906242050.57312.rjw@sisk.pl> References: <20090610121659.27937.13560.sendpatchset@rx1.opensource.se> <87r5xhqzr4.fsf@deeprootsystems.com> <200906242050.57312.rjw@sisk.pl> Date: Thu, 25 Jun 2009 11:25:39 +0900 Message-ID: Subject: Re: [PATCH] Driver Core: Add platform device arch data V3 From: Magnus Damm To: "Rafael J. Wysocki" Cc: linux-kernel@vger.kernel.org, paul@pwsan.com, gregkh@suse.de, khilman@deeprootsystems.com, stern@rowland.harvard.edu, linux-pm@lists.linux-foundation.org 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: 3041 Lines: 71 On Thu, Jun 25, 2009 at 3:50 AM, Rafael J. Wysocki wrote: > On Wednesday 24 June 2009, Magnus Damm wrote: >> On Fri, Jun 19, 2009 at 1:21 AM, Kevin >> Hilman wrote: >> > Magnus Damm writes: >> > >> >> From: Magnus Damm >> >> >> >> Allow architecture specific data in struct platform_device V3. >> >> >> >> With this patch struct pdev_archdata is added to struct >> >> platform_device, similar to struct dev_archdata in found in >> >> struct device. Useful for architecture code that needs to >> >> keep extra data associated with each platform device. >> >> >> >> Struct pdev_archdata is different from dev.platform_data, the >> >> convention is that dev.platform_data points to driver-specific >> >> data. It may or may not be required by the driver. The format >> >> of this depends on driver but is the same across architectures. >> >> >> >> The structure pdev_archdata is a place for architecture specific >> >> data. This data is handled by architecture specific code (for >> >> example runtime PM), and since it is architecture specific it >> >> should _never_ be touched by device driver code. Exactly like >> >> struct dev_archdata but for platform devices. >> >> >> >> Signed-off-by: Magnus Damm >> > >> > Since there is no 'Feature-desired-by:' tag, I'll addd >> > >> > Acked-by: Kevin Hilman >> > >> > For PM on ARM in general, and OMAP in particular we definitely need a >> > generic way to handle arch-specific data per platform_device. >> >> Thanks, Kevin! So ARM in general or at least OMAP wants this, and so >> does SuperH. >> >> Rafael, you kindly gave feedback on earlier versions, are you ok with >> this version? > > Yes, I am. ?I'm planning to include it into my linux-next branch for 2.6.32, if > no one objects. Do you have any specific reason for not including this one in 2.6.31? I guess you were thinking of keeping it together with your Runtime PM patches targeted for 2.6.32? IMO, this patch is decoupled from Runtime PM. It will of course be used for Runtime PM on SuperH, but it can for instance also be used together with the clock framework. On top of that, the patch is only adding code so it's very unlikely to cause any breakage. If possible, I'd like this to be merged as early as possible since a lot of processor specific changes will depend on it. With this included in 2.6.31 I can easily build arch specific code for 2.6.32. Anything I can do to make that happen? My top priority is Runtime PM for SuperH on top of your code, and I intend to post a prototype for SuperH before the PM Summit. It would be great to minimize the dependencies though, and including this in 2.6.31 would certainly help. Thanks! / magnus -- 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/