MIME-Version: 1.0
From: Joerie de Gram <j.de.gram@gmail.com>
Date: Mon, 8 Nov 2010 23:14:18 +0100
Message-ID: <AANLkTikVZT-hQvr83bTkuVENe8jPSG=LNMw5NTwUXCp1@mail.gmail.com>
Subject: wl1271 sdio timeouts on omap3
To: linux-wireless@vger.kernel.org
Content-Type: text/plain; charset=ISO-8859-1
Sender: linux-wireless-owner@vger.kernel.org

Hi,

I'm trying to get the wl1271 (sdio) driver running on an
OMAP3430-based board. I've based my board support on the patches by
Ohad Ben-Cohen for zoom2 and Luciano Coelho's beagleboard patches.
However, I'm experiencing trouble bringing the interface up, getting a
lockup in mmc_wait_for_req (called from wl1271_sdio_power_on). In 10%
of my attempts the driver gets as far as uploading the firmware after
which I'm getting similar SDIO timeouts.

I'm mostly unsure about my regulator config (relevant snippets below),
having no wl1271 documentation to work with. The wl1271 is on mmc3,
powered by twl4030 vsim. I assume the fixed regulator in [1] is just
to be able to control the PMENA GPIO?

Pin muxing aside, all changes are equal to those displayed in [1]. Is
there anything I might be doing wrong wrt. the power-up sequence?

[1]: https://patchwork.kernel.org/patch/201572/

Joerie

----------

static struct regulator_consumer_supply nowplus_vsim_supply = {
	.supply			= "vmmc",
};


static struct regulator_consumer_supply nowplus_wl1271_supply = {
	.supply			= "vmmc_aux",
	.dev_name		= "mmci-omap-hs.2",
};


/* VSIM for WiFi SDIO */
static struct regulator_init_data nowplus_vsim = {
	.constraints = {
		.name			= "VSIM",
		.min_uV			= 1800000,
		.max_uV			= 1800000,
		.apply_uV		= true,
		.valid_modes_mask	= REGULATOR_MODE_NORMAL
					| REGULATOR_MODE_STANDBY,
		.valid_ops_mask		= REGULATOR_CHANGE_MODE
					| REGULATOR_CHANGE_STATUS,
	},
	.num_consumer_supplies	= 1,
	.consumer_supplies	= &nowplus_vsim_supply,
};

static struct regulator_init_data nowplus_wlgpio = {
	.constraints = {
		.valid_ops_mask = REGULATOR_CHANGE_STATUS,
	},
	.num_consumer_supplies	= 1,
	.consumer_supplies	= &nowplus_wl1271_supply,
};