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. > @@ -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-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html