Commit f1207ba1 authored by John W. Linville's avatar John W. Linville

[PATCH] bcm43xx: fix-up build breakage from merging patches out of order

Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 27be44ff
...@@ -331,7 +331,8 @@ static ssize_t restart_write_file(struct file *file, const char __user *user_buf ...@@ -331,7 +331,8 @@ static ssize_t restart_write_file(struct file *file, const char __user *user_buf
res = -EFAULT; res = -EFAULT;
goto out_up; goto out_up;
} }
bcm43xx_lock_irqsafe(bcm, flags); mutex_lock(&(bcm)->mutex);
spin_lock_irqsave(&(bcm)->irq_lock, flags);
if (bcm43xx_status(bcm) != BCM43xx_STAT_INITIALIZED) { if (bcm43xx_status(bcm) != BCM43xx_STAT_INITIALIZED) {
printk(KERN_INFO PFX "debugfs: Board not initialized.\n"); printk(KERN_INFO PFX "debugfs: Board not initialized.\n");
res = -EFAULT; res = -EFAULT;
...@@ -344,7 +345,8 @@ static ssize_t restart_write_file(struct file *file, const char __user *user_buf ...@@ -344,7 +345,8 @@ static ssize_t restart_write_file(struct file *file, const char __user *user_buf
res = -EINVAL; res = -EINVAL;
out_unlock: out_unlock:
bcm43xx_unlock_irqsafe(bcm, flags); spin_unlock_irqrestore(&(bcm)->irq_lock, flags);
mutex_unlock(&(bcm)->mutex);
out_up: out_up:
up(&big_buffer_sem); up(&big_buffer_sem);
return res; return res;
......
...@@ -3199,9 +3199,9 @@ static int bcm43xx_rng_read(struct hwrng *rng, u32 *data) ...@@ -3199,9 +3199,9 @@ static int bcm43xx_rng_read(struct hwrng *rng, u32 *data)
struct bcm43xx_private *bcm = (struct bcm43xx_private *)rng->priv; struct bcm43xx_private *bcm = (struct bcm43xx_private *)rng->priv;
unsigned long flags; unsigned long flags;
bcm43xx_lock_irqonly(bcm, flags); spin_lock_irqsave(&(bcm)->irq_lock, flags);
*data = bcm43xx_read16(bcm, BCM43xx_MMIO_RNG); *data = bcm43xx_read16(bcm, BCM43xx_MMIO_RNG);
bcm43xx_unlock_irqonly(bcm, flags); spin_unlock_irqrestore(&(bcm)->irq_lock, flags);
return (sizeof(u16)); return (sizeof(u16));
} }
...@@ -3264,10 +3264,10 @@ static void bcm43xx_free_board(struct bcm43xx_private *bcm) ...@@ -3264,10 +3264,10 @@ static void bcm43xx_free_board(struct bcm43xx_private *bcm)
bcm43xx_sysfs_unregister(bcm); bcm43xx_sysfs_unregister(bcm);
bcm43xx_periodic_tasks_delete(bcm); bcm43xx_periodic_tasks_delete(bcm);
bcm43xx_lock_noirq(bcm); mutex_lock(&(bcm)->mutex);
bcm43xx_shutdown_all_wireless_cores(bcm); bcm43xx_shutdown_all_wireless_cores(bcm);
bcm43xx_pctl_set_crystal(bcm, 0); bcm43xx_pctl_set_crystal(bcm, 0);
bcm43xx_unlock_noirq(bcm); mutex_unlock(&(bcm)->mutex);
} }
static void prepare_phydata_for_init(struct bcm43xx_phyinfo *phy) static void prepare_phydata_for_init(struct bcm43xx_phyinfo *phy)
...@@ -3511,7 +3511,7 @@ static int bcm43xx_init_board(struct bcm43xx_private *bcm) ...@@ -3511,7 +3511,7 @@ static int bcm43xx_init_board(struct bcm43xx_private *bcm)
{ {
int err; int err;
bcm43xx_lock_noirq(bcm); mutex_lock(&(bcm)->mutex);
tasklet_enable(&bcm->isr_tasklet); tasklet_enable(&bcm->isr_tasklet);
err = bcm43xx_pctl_set_crystal(bcm, 1); err = bcm43xx_pctl_set_crystal(bcm, 1);
...@@ -3533,7 +3533,7 @@ static int bcm43xx_init_board(struct bcm43xx_private *bcm) ...@@ -3533,7 +3533,7 @@ static int bcm43xx_init_board(struct bcm43xx_private *bcm)
schedule_work(&bcm->softmac->associnfo.work); schedule_work(&bcm->softmac->associnfo.work);
out: out:
bcm43xx_unlock_noirq(bcm); mutex_unlock(&(bcm)->mutex);
return err; return err;
...@@ -4097,10 +4097,10 @@ static void bcm43xx_chip_reset(void *_bcm) ...@@ -4097,10 +4097,10 @@ static void bcm43xx_chip_reset(void *_bcm)
struct bcm43xx_phyinfo *phy; struct bcm43xx_phyinfo *phy;
int err; int err;
bcm43xx_lock_noirq(bcm); mutex_lock(&(bcm)->mutex);
phy = bcm43xx_current_phy(bcm); phy = bcm43xx_current_phy(bcm);
err = bcm43xx_select_wireless_core(bcm, phy->type); err = bcm43xx_select_wireless_core(bcm, phy->type);
bcm43xx_unlock_noirq(bcm); mutex_unlock(&(bcm)->mutex);
printk(KERN_ERR PFX "Controller restart%s\n", printk(KERN_ERR PFX "Controller restart%s\n",
(err == 0) ? "ed" : " failed"); (err == 0) ? "ed" : " failed");
......
...@@ -333,9 +333,9 @@ static ssize_t bcm43xx_attr_phymode_store(struct device *dev, ...@@ -333,9 +333,9 @@ static ssize_t bcm43xx_attr_phymode_store(struct device *dev,
goto out; goto out;
} }
bcm43xx_lock_noirq(bcm); mutex_lock(&(bcm)->mutex);
err = bcm43xx_select_wireless_core(bcm, phytype); err = bcm43xx_select_wireless_core(bcm, phytype);
bcm43xx_unlock_noirq(bcm); mutex_unlock(&(bcm)->mutex);
if (err == -ESRCH) if (err == -ESRCH)
err = -ENODEV; err = -ENODEV;
...@@ -350,7 +350,7 @@ static ssize_t bcm43xx_attr_phymode_show(struct device *dev, ...@@ -350,7 +350,7 @@ static ssize_t bcm43xx_attr_phymode_show(struct device *dev,
struct bcm43xx_private *bcm = dev_to_bcm(dev); struct bcm43xx_private *bcm = dev_to_bcm(dev);
ssize_t count = 0; ssize_t count = 0;
bcm43xx_lock_noirq(bcm); mutex_lock(&(bcm)->mutex);
switch (bcm43xx_current_phy(bcm)->type) { switch (bcm43xx_current_phy(bcm)->type) {
case BCM43xx_PHYTYPE_A: case BCM43xx_PHYTYPE_A:
snprintf(buf, PAGE_SIZE, "A"); snprintf(buf, PAGE_SIZE, "A");
...@@ -364,7 +364,7 @@ static ssize_t bcm43xx_attr_phymode_show(struct device *dev, ...@@ -364,7 +364,7 @@ static ssize_t bcm43xx_attr_phymode_show(struct device *dev,
default: default:
assert(0); assert(0);
} }
bcm43xx_unlock_noirq(bcm); mutex_unlock(&(bcm)->mutex);
return count; return count;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment