I'll just roll this into the "b43: LED triggers support" patch. Thanks, John On Sun, Sep 30, 2007 at 12:04:36AM +0200, Michael Buesch wrote: > This was missing an address operator. > > Signed-off-by: Michael Buesch <mb@xxxxxxxxx> > > Index: wireless-2.6/drivers/net/wireless/b43/leds.c > =================================================================== > --- wireless-2.6.orig/drivers/net/wireless/b43/leds.c 2007-09-29 12:44:08.000000000 +0200 > +++ wireless-2.6/drivers/net/wireless/b43/leds.c 2007-09-30 00:02:18.000000000 +0200 > @@ -37,14 +37,14 @@ static void b43_led_turn_on(struct b43_w > unsigned long flags; > u16 ctl; > > - spin_lock_irqsave(wl->leds_lock, flags); > + spin_lock_irqsave(&wl->leds_lock, flags); > ctl = b43_read16(dev, B43_MMIO_GPIO_CONTROL); > if (activelow) > ctl &= ~(1 << led_index); > else > ctl |= (1 << led_index); > b43_write16(dev, B43_MMIO_GPIO_CONTROL, ctl); > - spin_unlock_irqrestore(wl->leds_lock, flags); > + spin_unlock_irqrestore(&wl->leds_lock, flags); > } > > static void b43_led_turn_off(struct b43_wldev *dev, u8 led_index, > @@ -54,14 +54,14 @@ static void b43_led_turn_off(struct b43_ > unsigned long flags; > u16 ctl; > > - spin_lock_irqsave(wl->leds_lock, flags); > + spin_lock_irqsave(&wl->leds_lock, flags); > ctl = b43_read16(dev, B43_MMIO_GPIO_CONTROL); > if (activelow) > ctl |= (1 << led_index); > else > ctl &= ~(1 << led_index); > b43_write16(dev, B43_MMIO_GPIO_CONTROL, ctl); > - spin_unlock_irqrestore(wl->leds_lock, flags); > + spin_unlock_irqrestore(&wl->leds_lock, flags); > } > > /* Callback from the LED subsystem. */ -- John W. Linville linville@xxxxxxxxxxxxx - 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
