diff options
| author |   Jan Glauber <jang@linux.vnet.ibm.com> | 2009-03-26 15:24:25 +0100 |
|---|---|---|
| committer |   Martin Schwidefsky <schwidefsky@de.ibm.com> | 2009-03-26 15:24:20 +0100 |
| commit | e4c14e2085cd32f61e9ffc47d5b20d4f5f7639f3 (patch) | |
| tree | d20047f2f8424dfc15238e562402e931fc49ba07 /drivers | |
| parent | [S390] qdio: add missing tiq_list locking (diff) | |
| download | linux-dev-e4c14e2085cd32f61e9ffc47d5b20d4f5f7639f3.tar.xz linux-dev-e4c14e2085cd32f61e9ffc47d5b20d4f5f7639f3.zip | |
[S390] qdio: Dont call qdio_shutdown in case qdio_activate fails
Remove the call to qdio_shutdown from qdio_activate since the upper-layer
drivers are responsible to call qdio_shutdown when qdio_activate returns
with an error.
Signed-off-by: Jan Glauber <jang@linux.vnet.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/s390/cio/qdio_main.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/s390/cio/qdio_main.c b/drivers/s390/cio/qdio_main.c index 5100996201d1..fa902703996c 100644 --- a/drivers/s390/cio/qdio_main.c +++ b/drivers/s390/cio/qdio_main.c @@ -1404,9 +1404,8 @@ int qdio_activate(struct ccw_device *cdev) switch (irq_ptr->state) { case QDIO_IRQ_STATE_STOPPED: case QDIO_IRQ_STATE_ERR: - mutex_unlock(&irq_ptr->setup_mutex); - qdio_shutdown(cdev, QDIO_FLAG_CLEANUP_USING_CLEAR); - return -EIO; + rc = -EIO; + break; default: qdio_set_state(irq_ptr, QDIO_IRQ_STATE_ACTIVE); rc = 0; |