* 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()? > > @@ -306,6 +306,7 @@ int twl_i2c_read(u8 mod_no, u8 *value, u8 reg, unsigned > > num_bytes); > > #define TWL4030_INT_PWR_EDR2 0x6 > > #define TWL4030_INT_PWR_SIH_CTRL 0x7 > > > > + > > /*----------------------------------------------------------------------*/ > > > > /* Power bus message definitions */ > > Random unrelated indentation change? -- 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
