> -----Original Message----- > From: Premi, Sanjeev > Sent: Tuesday, August 04, 2009 6:54 PM > To: Shilimkar, Santosh; Tony Lindgren; Mark Brown > Cc: Krishnamoorthy, Balaji T; linux-omap@xxxxxxxxxxxxxxx; > khilman@xxxxxxxxxxxxxxxxxxx; david-b@xxxxxxxxxxx; linux- > i2c@xxxxxxxxxxxxxxx; sameo@xxxxxxxxxxxxxx; wim@xxxxxxxxx; > timo.t.kokkonen@xxxxxxxxx; ben-linux@xxxxxxxxx; lrg@xxxxxxxxxxxxxxx; Nayak, > Rajendra > Subject: RE: [RFC][PATCH v2 4/4] OMAP4: PMIC: Update TWL mfd driver to > create twl6030 regulators > > -----Original Message----- > From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap- > owner@xxxxxxxxxxxxxxx] On Behalf Of Shilimkar, Santosh > Sent: Tuesday, August 04, 2009 1:27 PM > To: Tony Lindgren; Mark Brown > Cc: Krishnamoorthy, Balaji T; linux-omap@xxxxxxxxxxxxxxx; > khilman@xxxxxxxxxxxxxxxxxxx; david-b@xxxxxxxxxxx; linux- > i2c@xxxxxxxxxxxxxxx; sameo@xxxxxxxxxxxxxx; wim@xxxxxxxxx; > timo.t.kokkonen@xxxxxxxxx; ben-linux@xxxxxxxxx; lrg@xxxxxxxxxxxxxxx; Nayak, > Rajendra > Subject: RE: [RFC][PATCH v2 4/4] OMAP4: PMIC: Update TWL mfd driver to > create twl6030 regulators > > > -----Original Message----- > > From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap- > > owner@xxxxxxxxxxxxxxx] On Behalf Of Tony Lindgren > > Sent: Tuesday, August 04, 2009 1:16 PM > > To: Mark Brown > > Cc: Krishnamoorthy, Balaji T; linux-omap@xxxxxxxxxxxxxxx; > > khilman@xxxxxxxxxxxxxxxxxxx; david-b@xxxxxxxxxxx; linux- > > i2c@xxxxxxxxxxxxxxx; sameo@xxxxxxxxxxxxxx; wim@xxxxxxxxx; > > timo.t.kokkonen@xxxxxxxxx; ben-linux@xxxxxxxxx; lrg@xxxxxxxxxxxxxxx; > Nayak, > > Rajendra > > Subject: Re: [RFC][PATCH v2 4/4] OMAP4: PMIC: Update TWL mfd driver to > > create twl6030 regulators > > > > * Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> [090803 17:23]: > > > On Wed, Jul 29, 2009 at 10:06:13AM +0530, balajitk@xxxxxx wrote: > > > > > > > - if (twl_has_regulator()) { > > > > + if (twl_has_regulator() && is_class_twl4030()) { > > > > > > is_class_twl4030() feels like it should have better namespacing, > though > > > having the part name in there means it's not actually an issue. > > > > How about twl_is_4030()? > The idea was to have a class of twl4030 IC's type IC's. There are many of > them with just different name (TPS *) > ... twl_class_is_4030().... What you say ? > > Can I suggest: pmic_class_is_xxx() ? Even better Regards, Santosh -- 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