On Fri, 2010-02-05 at 10:45 +0200, Eduardo Valentin wrote: > Hello Anuj, > > First of all, I liked the idea of unifying the regulator definitions > for boards that use twl regulators. > > But I guess we need to improve a little bit. > > See following comments. > > On Tue, Jan 12, 2010 at 10:13:52AM +0100, ext Anuj Aggarwal wrote: > > A new file for TWL4030/TPS65950 is created which has common supplies > > and regulator init data structures. They will be referenced from the > > various board-evm files depending upon the EVM requirements, using > > the twl4030-pmic.h header file. > > > > Signed-off-by: Anuj Aggarwal <anuj.aggarwal@xxxxxx> > > --- > > arch/arm/mach-omap2/twl4030-pmic.c | 175 ++++++++++++++++++++++++++++++++++++ > > arch/arm/mach-omap2/twl4030-pmic.h | 34 +++++++ > > 2 files changed, 209 insertions(+), 0 deletions(-) > > create mode 100644 arch/arm/mach-omap2/twl4030-pmic.c > > create mode 100644 arch/arm/mach-omap2/twl4030-pmic.h > > > > diff --git a/arch/arm/mach-omap2/twl4030-pmic.c b/arch/arm/mach-omap2/twl4030-pmic.c > > new file mode 100644 > > index 0000000..0c0a860 > > --- /dev/null > > +++ b/arch/arm/mach-omap2/twl4030-pmic.c > > @@ -0,0 +1,175 @@ > > +/* > > + * twl4030-pmic.c > > + * > > + * Common regulator supplies and init data structs for TWL4030/TPS65950 > > + * PMIC for OMAP3 based EVMs. They can be used in various board-evm > > + * files for OMAP3 based platforms using TWL4030. > > + * > > + * Copyright (C) 2010 Texas Instrument Incorporated - http://www.ti.com/ > > + * > > + * This program is free software; you can redistribute it and/or > > + * modify it under the terms of the GNU General Public License as > > + * published by the Free Software Foundation version 2. > > + * > > + * This program is distributed "as is" WITHOUT ANY WARRANTY of any kind, > > + * whether express or implied; without even the implied warranty of > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > + * General Public License for more details. > > + */ > > + > > +#include <linux/regulator/machine.h> > > + > > +/* VDAC */ > > +struct regulator_consumer_supply twl4030_vdac_supply = { > > + .supply = "vdac", > > +}; > > + > > +/* VMMC1 */ > > +struct regulator_consumer_supply twl4030_vmmc1_supply = { > > + .supply = "vmmc", > > +}; > > + > > +/* VMMC2 */ > > +struct regulator_consumer_supply twl4030_vmmc2_supply = { > > + .supply = "vmmc", > > +}; > > + > > +/* VSIM */ > > +struct regulator_consumer_supply twl4030_vsim_supply = { > > + .supply = "vmmc_aux", > > +}; > > + > > +/* VPLL2 for digital video outputs */ > > +struct regulator_consumer_supply twl4030_vpll2_supply = { > > + .supply = "vdvi", > > +}; > > > Ohh no! This is not good! Defining here the supply list would make things harder. > Because from board to board the supply list will change! That's the > whole point of having the supply list. We can't share them for all boards > definitions. OK, my understanding here was that this change was only to support a set of similar OMAP + TWL4030 boards and not *all* OMAP + TWL4030 boards. Anuj, what were your intentions here ? Thanks Liam -- Freelance Developer, SlimLogic Ltd ASoC and Voltage Regulator Maintainer. http://www.slimlogic.co.uk -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
