> From: Shilimkar, Santosh > > > From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap- > > owner@xxxxxxxxxxxxxxx] On Behalf Of Tony Lindgren > > > > * 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 ? twl_class_is_4030 should return TRUE for twl4030 / twl5030 / tps65950 / tps65930 / tps65920 -- 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