On Mon, Aug 06, 2012 at 11:10:51PM +0900, Dongjin Kim wrote: > +static struct regulator_consumer_supply __initdata max77686_buck4_consumer[] = { > + REGULATOR_SUPPLY("vdd_g3d", NULL), > +}; Not that it matters much since new board drivers won't get merged but all your regulator setup here is problematic. These look like names from the board schematic as opposed to mappings of the supplies onto the consumer devices. > +static struct regulator_init_data __initdata max77686_buck1_data = { > + .constraints = { > + .name = "vdd_mif range", This would normally just be the name of the supply for display purposes. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html