Commit cf1ad8f9 authored Oct 25, 2013 by Solomon Peachy Committed by John W. Linville Dec 05, 2013

wireless: cw1200: Use consistent internal locking conventions



The cw1200_irq_handler() function expects the hwbus lock to be held when
it is called.  On the SDIO platform, this lock is implemented in terms
of sdio_claim_host/sdio_release_host.

This trivial patch makes it explicit that we are performing the hwbus
lock rather than something SDIO-specific.

Signed-off-by: Solomon Peachy <pizza@shaftnet.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>

parent 809c5255

drivers/net/wireless/cw1200/cw1200_sdio.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -108,9 +108,9 @@ static irqreturn_t cw1200_gpio_irq(int irq, void *dev_id)
		struct hwbus_priv *self = dev_id;

		if (self->core) {
		sdio_claim_host(self->func);
		cw1200_sdio_lock(self);
		cw1200_irq_handler(self->core);
		sdio_release_host(self->func);
		cw1200_sdio_unlock(self);
		return IRQ_HANDLED;
		} else {
		return IRQ_NONE;