From mboxrd@z Thu Jan 1 00:00:00 1970 From: Samuel Ortiz Subject: Re: [PM8921 MFD V5 0/6] pm8921 core and subdevices Date: Tue, 26 Apr 2011 17:40:46 +0200 Message-ID: <20110426154045.GG3175@sortiz-mobl> References: <1302039657-19395-1-git-send-email-adharmap@codeaurora.org> <20110426150746.GF3175@sortiz-mobl> <20110426153737.GZ31131@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mga11.intel.com ([192.55.52.93]:57576 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751049Ab1DZPlH (ORCPT ); Tue, 26 Apr 2011 11:41:07 -0400 Content-Disposition: inline In-Reply-To: <20110426153737.GZ31131@pengutronix.de> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= Cc: Abhijeet Dharmapurikar , Greg Kroah-Hartman , Linus Walleij , linux-arm-msm@vger.kernel.org, Mark Brown , linux-kernel@vger.kernel.org, Thomas Glexiner , Andres Salomon , dwalker@fifo99.com, davidb@codeaurora.org, linux-arm-kernel@lists.infradead.org Hi Uwe, On Tue, Apr 26, 2011 at 05:37:37PM +0200, Uwe Kleine-K=F6nig wrote: > Hi Samuel, >=20 > On Tue, Apr 26, 2011 at 05:07:47PM +0200, Samuel Ortiz wrote: > > > - mfd_cell does not have platfrom_data anymore - use mfd data ins= tead. > > I'm currently moving that away from the mfd_cell structure, in my f= or-next > > branch. mfd_cell now has a platform_data pointer back, and you shou= ld use > > mfd_get_cell() if you really need to reach the mfd_cell pointer (wh= ich you > > apparently don't).=20 > > Could you please generate patchs 3 and 4 against this branch ? > I just saw commit >=20 > 5828fdb (mfd: Use mfd cell platform_data for mc13xxx cells platform = bits) >=20 > in next. It has a Cc: for me, but I didn't got it by mail. >=20 > Did you send a mail and it didn't reach me? Or maybe you just forgot = to > send the mail out? I haven't sent the patchset to lkml yet. > Anyhow, Acked-by: Uwe Kleine-K=F6nig = for > 5828fdb,=20 Thanks a lot. > but I wonder if it's worth to add a new member to struct > platform_device for that to work. This has been discussed at length already, and it's probably the best s= olution for now to clearly separate mfd cell access from the platform data bits= =2E Cheers, Samuel. --=20 Intel Open Source Technology Centre http://oss.intel.com/