From patchwork Tue Mar 3 22:59:59 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Brownell X-Patchwork-Id: 9768 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n23N07Aa008952 for ; Tue, 3 Mar 2009 23:00:07 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755306AbZCCXAG (ORCPT ); Tue, 3 Mar 2009 18:00:06 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754945AbZCCXAG (ORCPT ); Tue, 3 Mar 2009 18:00:06 -0500 Received: from n23.bullet.mail.mud.yahoo.com ([68.142.206.162]:28191 "HELO n23.bullet.mail.mud.yahoo.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1754794AbZCCXAF (ORCPT ); Tue, 3 Mar 2009 18:00:05 -0500 Received: from [68.142.194.243] by n23.bullet.mail.mud.yahoo.com with NNFMP; 03 Mar 2009 23:00:02 -0000 Received: from [68.142.201.250] by t1.bullet.mud.yahoo.com with NNFMP; 03 Mar 2009 23:00:02 -0000 Received: from [127.0.0.1] by omp411.mail.mud.yahoo.com with NNFMP; 03 Mar 2009 23:00:02 -0000 X-Yahoo-Newman-Id: 747777.72130.bm@omp411.mail.mud.yahoo.com Received: (qmail 21502 invoked from network); 3 Mar 2009 23:00:02 -0000 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=s1024; d=pacbell.net; h=Received:X-YMail-OSG:X-Yahoo-Newman-Property:From:To:Subject:Date:User-Agent:Cc:References:In-Reply-To:MIME-Version:Content-Type:Content-Transfer-Encoding:Content-Disposition:Message-Id; b=j1gL8FX7MDvmXRDryNTMC4Xu1i75/w2td3Rb12yGt3HFQF+lxXaoKrglVwzeekIxjXlyxz++DOA6e/SkTRWz1TLTF6MmgqPbn/A+9D0RZ886xeh48BKzetNv+jAe2BoQBpXM9TBSr9s8pHj7ECD3yPc/1KSi3t3ivLU2CEWVEYE= ; Received: from unknown (HELO pogo) (david-b@69.226.224.20 with plain) by smtp101.sbc.mail.sp1.yahoo.com with SMTP; 3 Mar 2009 23:00:01 -0000 X-YMail-OSG: OX5tHfgVM1l4tPKM17O7MFiHNqMdepNxPuhX1qNQwf9IBpaGvmzU1VYJbVjpn.jvIOJKhA93Xk.WiV2tR6WRiSnkQVWh0RxmQnu5khaKciMkag8LA18vbnQlOE7AZAm8RKpkPkEJE6NBU7y2s1c4Ipsz8PU._g_IJWEPIFESI6UHIaM5tOYGpI0cAL_yhv09iw5C4R1ThtksPM0d_l5iiO.Y5mnKuTnitmR2sUmyOowkzu5rxLxDNTvcQRux8sSMfR0LrWatwyaq5VxM1jtvxnCT3aykL8L2uBxNXw-- X-Yahoo-Newman-Property: ymail-3 From: David Brownell To: Liam Girdwood Subject: Re: [patch/rfc 2.6.29-rc6 2/2] regulator: twl4030 voltage enumeration (v2) Date: Tue, 3 Mar 2009 14:59:59 -0800 User-Agent: KMail/1.9.10 Cc: Mark Brown , lkml , OMAP References: <200902081037.06645.david-b@pacbell.net> <200902261602.58832.david-b@pacbell.net> <1235737930.31223.117.camel@vega.slimlogic.co.uk> In-Reply-To: <1235737930.31223.117.camel@vega.slimlogic.co.uk> MIME-Version: 1.0 Content-Disposition: inline Message-Id: <200903031500.00166.david-b@pacbell.net> Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org On Friday 27 February 2009, Liam Girdwood wrote: > Sorry, this didn't apply. It looks like I'm missing an earlier patch(s) > here. Could you regenerate this and your core patch against latest > for-next. Here you go. - Dave --- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ========== CUT HERE From: David Brownell Minor cleanups to the twl403 regulator driver, mostly enabled by other recent changes: comments, shrink memory usage, add definition for one bit. Signed-off-by: David Brownell --- drivers/regulator/twl4030-regulator.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) --- a/drivers/regulator/twl4030-regulator.c +++ b/drivers/regulator/twl4030-regulator.c @@ -36,13 +36,13 @@ struct twlreg_info { /* twl4030 resource ID, for resource control state machine */ u8 id; + /* FIXED_LDO voltage */ + u8 deciV; + /* voltage in mV = table[VSEL]; table_len must be a power-of-two */ u8 table_len; const u16 *table; - /* chip constraints on regulator behavior */ - u16 min_mV; - /* used by regulator core */ struct regulator_desc desc; }; @@ -93,6 +93,7 @@ static int twl4030reg_grp(struct regulat #define P3_GRP BIT(7) /* "peripherals" */ #define P2_GRP BIT(6) /* secondary processor, modem, etc */ #define P1_GRP BIT(5) /* CPU/Linux */ +#define WARM_CFG BIT(4) static int twl4030reg_is_enabled(struct regulator_dev *rdev) { @@ -325,14 +326,14 @@ static int twl4030fixed_list_voltage(str { struct twlreg_info *info = rdev_get_drvdata(rdev); - return info->min_mV * 1000; + return info->deciV * 100 * 1000; } static int twl4030fixed_get_voltage(struct regulator_dev *rdev) { struct twlreg_info *info = rdev_get_drvdata(rdev); - return info->min_mV * 1000; + return info->deciV * 100 * 1000; } static struct regulator_ops twl4030fixed_ops = { @@ -369,7 +370,7 @@ static struct regulator_ops twl4030fixed #define TWL_FIXED_LDO(label, offset, mVolts, num) { \ .base = offset, \ .id = num, \ - .min_mV = mVolts, \ + .deciV = mVolts / 100 , \ .desc = { \ .name = #label, \ .id = TWL4030_REG_##label, \ @@ -381,7 +382,7 @@ static struct regulator_ops twl4030fixed } /* - * We list regulators here if systems need some level of + * We expose regulators here if systems need some level of * software control over them after boot. */ static struct twlreg_info twl4030_regs[] = { @@ -435,6 +436,7 @@ static int twl4030reg_probe(struct platf /* Constrain board-specific capabilities according to what * this driver and the chip itself can actually do. + * (Regulator core now does this for voltage constraints.) */ c = &initdata->constraints; c->valid_modes_mask &= REGULATOR_MODE_NORMAL | REGULATOR_MODE_STANDBY;