> -----Original Message----- > From: Premi, Sanjeev > Sent: Tuesday, August 04, 2009 6:54 PM > To: Shilimkar, Santosh; Tony Lindgren; Mark Brown > Cc: Krishnamoorthy, Balaji T; [email protected]; > [email protected]; [email protected]; linux- > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; Nayak, > Rajendra > Subject: RE: [RFC][PATCH v2 4/4] OMAP4: PMIC: Update TWL mfd driver to > create twl6030 regulators > > -----Original Message----- > From: [email protected] [mailto:linux-omap- > [email protected]] On Behalf Of Shilimkar, Santosh > Sent: Tuesday, August 04, 2009 1:27 PM > To: Tony Lindgren; Mark Brown > Cc: Krishnamoorthy, Balaji T; [email protected]; > [email protected]; [email protected]; linux- > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; Nayak, > Rajendra > Subject: RE: [RFC][PATCH v2 4/4] OMAP4: PMIC: Update TWL mfd driver to > create twl6030 regulators > > > -----Original Message----- > > From: [email protected] [mailto:linux-omap- > > [email protected]] On Behalf Of Tony Lindgren > > Sent: Tuesday, August 04, 2009 1:16 PM > > To: Mark Brown > > Cc: Krishnamoorthy, Balaji T; [email protected]; > > [email protected]; [email protected]; linux- > > [email protected]; [email protected]; [email protected]; > > [email protected]; [email protected]; [email protected]; > Nayak, > > Rajendra > > Subject: Re: [RFC][PATCH v2 4/4] OMAP4: PMIC: Update TWL mfd driver to > > create twl6030 regulators > > > > * Mark Brown <[email protected]> [090803 17:23]: > > > On Wed, Jul 29, 2009 at 10:06:13AM +0530, [email protected] 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
