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: 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() ?

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

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