On Tue, Jan 13, 2009 at 01:11:08PM +0530, Manikandan Pillai wrote: > +config REGULATOR_TPS6235X > + bool "TPS6235X Power regulator for OMAP3EVM" > + depends on I2C=y This driver should not be OMAP3EVM specific, I'd expect. > +extern struct regulator_consumer_supply tps62352_core_consumers; > +extern struct regulator_consumer_supply tps62352_mpu_consumers; These should not be required. > + /* Register the regulators */ > + dev_child = device_find_child(client->adapter->dev.parent, > + (void *)regulator_consumer_name[id->driver_data], > + omap_i2c_match_child); > + rdev = regulator_register(®ulators[id->driver_data], > + dev_child, client); I'm not 100% sure what this is intended to do but apart from anything else it depends on specific consumer names which means that dependency on the specific board hasn't been removed. This is also OMAP-specific. You should be using a device which is specific to the regulator as the device that is being registered. -- 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