On Wed, Nov 10, 2010 at 5:01 PM, Joerie de Gram <j.de.gram@xxxxxxxxx> wrote:
> On Wed, Nov 10, 2010 at 6:35 AM, Ohad Ben-Cohen <ohad@xxxxxxxxxx> wrote:
>> Snippets below are not complete. E.g., where do you actually indicate
>> the gpio number ?
>> It's better to send your complete patch so we can have a look.
>
> Sorry, I've attached a full diff of the board changes below.
Your vsim configuration looks wrong.
<snip>
>
> @@ -213,6 +224,16 @@ static struct regulator_consumer_supply
> nowplus_vpll2_supply = {
> .dev = &nowplus_dss_device.dev,
> };
>
> +static struct regulator_consumer_supply nowplus_vsim_supply = {
> + .supply = "vmmc",
> +};
You need to set .dev or .dev_name here, best just do
+ .dev_name = "mmci-omap-hs.2",
> +
> +
> +static struct regulator_consumer_supply nowplus_wl1271_supply = {
> + .supply = "vmmc_aux",
> + .dev_name = "mmci-omap-hs.2",
> +};
> +
> /* VMMC1 for MMC1 card */
> static struct regulator_init_data nowplus_vmmc1 = {
> .constraints = {
> @@ -311,6 +332,53 @@ static struct regulator_init_data nowplus_vpll2 = {
> .consumer_supplies = &nowplus_vpll2_supply,
> };
>
> +/* VSIM for WiFi SDIO */
> +static struct regulator_init_data nowplus_vsim = {
doesn't seem to be referenced anywhere? You need to add it to
twl4030_platform_data I think.
--
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
