diff options
Diffstat (limited to 'drivers/pci/hotplug')
-rw-r--r-- | drivers/pci/hotplug/pciehp_hpc.c | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/drivers/pci/hotplug/pciehp_hpc.c b/drivers/pci/hotplug/pciehp_hpc.c index 6339c638701e..7e3a3d17c334 100644 --- a/drivers/pci/hotplug/pciehp_hpc.c +++ b/drivers/pci/hotplug/pciehp_hpc.c | |||
@@ -268,9 +268,8 @@ completed: | |||
268 | return timeout; | 268 | return timeout; |
269 | } | 269 | } |
270 | 270 | ||
271 | static inline int pcie_wait_cmd(struct controller *ctrl, int poll) | 271 | static inline void pcie_wait_cmd(struct controller *ctrl, int poll) |
272 | { | 272 | { |
273 | int retval = 0; | ||
274 | unsigned int msecs = pciehp_poll_mode ? 2500 : 1000; | 273 | unsigned int msecs = pciehp_poll_mode ? 2500 : 1000; |
275 | unsigned long timeout = msecs_to_jiffies(msecs); | 274 | unsigned long timeout = msecs_to_jiffies(msecs); |
276 | int rc; | 275 | int rc; |
@@ -278,16 +277,9 @@ static inline int pcie_wait_cmd(struct controller *ctrl, int poll) | |||
278 | if (poll) | 277 | if (poll) |
279 | rc = pcie_poll_cmd(ctrl); | 278 | rc = pcie_poll_cmd(ctrl); |
280 | else | 279 | else |
281 | rc = wait_event_interruptible_timeout(ctrl->queue, | 280 | rc = wait_event_timeout(ctrl->queue, !ctrl->cmd_busy, timeout); |
282 | !ctrl->cmd_busy, timeout); | ||
283 | if (!rc) | 281 | if (!rc) |
284 | dbg("Command not completed in 1000 msec\n"); | 282 | dbg("Command not completed in 1000 msec\n"); |
285 | else if (rc < 0) { | ||
286 | retval = -EINTR; | ||
287 | info("Command was interrupted by a signal\n"); | ||
288 | } | ||
289 | |||
290 | return retval; | ||
291 | } | 283 | } |
292 | 284 | ||
293 | /** | 285 | /** |
@@ -365,7 +357,7 @@ static int pcie_write_cmd(struct controller *ctrl, u16 cmd, u16 mask) | |||
365 | if (!(slot_ctrl & HP_INTR_ENABLE) || | 357 | if (!(slot_ctrl & HP_INTR_ENABLE) || |
366 | !(slot_ctrl & CMD_CMPL_INTR_ENABLE)) | 358 | !(slot_ctrl & CMD_CMPL_INTR_ENABLE)) |
367 | poll = 1; | 359 | poll = 1; |
368 | retval = pcie_wait_cmd(ctrl, poll); | 360 | pcie_wait_cmd(ctrl, poll); |
369 | } | 361 | } |
370 | out: | 362 | out: |
371 | mutex_unlock(&ctrl->ctrl_lock); | 363 | mutex_unlock(&ctrl->ctrl_lock); |
@@ -797,7 +789,7 @@ static irqreturn_t pcie_isr(int irq, void *dev_id) | |||
797 | if (intr_loc & CMD_COMPLETED) { | 789 | if (intr_loc & CMD_COMPLETED) { |
798 | ctrl->cmd_busy = 0; | 790 | ctrl->cmd_busy = 0; |
799 | smp_mb(); | 791 | smp_mb(); |
800 | wake_up_interruptible(&ctrl->queue); | 792 | wake_up(&ctrl->queue); |
801 | } | 793 | } |
802 | 794 | ||
803 | if (!(intr_loc & ~CMD_COMPLETED)) | 795 | if (!(intr_loc & ~CMD_COMPLETED)) |