On Tue, 2011-04-12 at 13:41 +0200, Johannes Berg wrote: > > + * static struct regulator_consumer_supply pcap_regulator_V6_consumers [] = { > > + * { .dev_name = "rfkill-regulator.0", supply = "vrfkill" }, > > + * }; > > It's a comment, but it should be .supply = (missing the .) > > > + if (pdata->name == NULL || pdata->type == 0) { > > + dev_err(&pdev->dev, "invalid name or type in platform data\n"); > > + return -EINVAL; > > + } > > + > > + vcc = regulator_get_exclusive(&pdev->dev, "vrfkill"); > > Wasn't that supposed to use pdata->supply? Actually, there's no member > "supply" in the struct? Oh wait, I think I just misunderstood how this works. But if the name is "vrfkill" how does that really work with multiple instances? johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html