On Wed, Mar 28, 2012 at 11:27 PM, Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote: > On Wed, Mar 28, 2012 at 05:21:59PM +0200, Igor Grinberg wrote: >> On 03/09/12 00:06, Paul Gortmaker wrote: > >> > +REGULATOR_CONSUMER(ldo3, "reg-userspace-consumer", "vcc gps"); > >> If you make this: >> REGULATOR_CONSUMER(ldo3, NULL, "vcc gps"); > >> > +REGULATOR_CONSUMER(ldo19, "reg-userspace-consumer", "vcc gprs"); >> >> and this: >> REGULATOR_CONSUMER(ldo19, NULL, "vcc gprs"); > > These don't look like good fixes, you should be specifying the > dev_name() for the consumer device? Presumably it's two separate > consumers and should be .0 and .1 or something? Paul, Could you do a quick update on dev_name() according to Mark's comment? -REGULATOR_CONSUMER(ldo10, &pxa_device_mci.dev, "vcc sdio"); +REGULAOTR_CONSUMER(ldo10, dev_name(&pxa_device_mci.dev), "vcc sdio"); Best Regards Haojian -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html