RE: [RFC][PATCH v2 4/4] OMAP4: PMIC: Update TWL mfd driver to create twl6030 regulators

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux