diff options
author | Mark Haverkamp <markh@osdl.org> | 2006-09-19 11:59:23 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2006-09-23 21:08:26 -0400 |
commit | 65101355450df2d935f8d56ac3abef279f28a0e2 (patch) | |
tree | deea581c99159e8d7f13d0f92c6972d9a5ab9b74 /drivers | |
parent | 9cbb889786548c1212fb77a9df8d09ed883a3480 (diff) |
[SCSI] aacraid: misc cleanup
Received from Mark Salyzyn:
Basically cleanup, nothing here will have an affect. Adjusting some
error codes, removing superfluous definitions and code fragments.
Signed-off-by: Mark Haverkamp <markh@osdl.org>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/scsi/aacraid/aachba.c | 8 | ||||
-rw-r--r-- | drivers/scsi/aacraid/aacraid.h | 1 | ||||
-rw-r--r-- | drivers/scsi/aacraid/commsup.c | 2 |
3 files changed, 5 insertions, 6 deletions
diff --git a/drivers/scsi/aacraid/aachba.c b/drivers/scsi/aacraid/aachba.c index 37c55ddce214..b14f7cac30e9 100644 --- a/drivers/scsi/aacraid/aachba.c +++ b/drivers/scsi/aacraid/aachba.c | |||
@@ -1803,7 +1803,7 @@ static int query_disk(struct aac_dev *dev, void __user *arg) | |||
1803 | 1803 | ||
1804 | fsa_dev_ptr = dev->fsa_dev; | 1804 | fsa_dev_ptr = dev->fsa_dev; |
1805 | if (!fsa_dev_ptr) | 1805 | if (!fsa_dev_ptr) |
1806 | return -ENODEV; | 1806 | return -EBUSY; |
1807 | if (copy_from_user(&qd, arg, sizeof (struct aac_query_disk))) | 1807 | if (copy_from_user(&qd, arg, sizeof (struct aac_query_disk))) |
1808 | return -EFAULT; | 1808 | return -EFAULT; |
1809 | if (qd.cnum == -1) | 1809 | if (qd.cnum == -1) |
@@ -1842,6 +1842,8 @@ static int force_delete_disk(struct aac_dev *dev, void __user *arg) | |||
1842 | struct fsa_dev_info *fsa_dev_ptr; | 1842 | struct fsa_dev_info *fsa_dev_ptr; |
1843 | 1843 | ||
1844 | fsa_dev_ptr = dev->fsa_dev; | 1844 | fsa_dev_ptr = dev->fsa_dev; |
1845 | if (!fsa_dev_ptr) | ||
1846 | return -EBUSY; | ||
1845 | 1847 | ||
1846 | if (copy_from_user(&dd, arg, sizeof (struct aac_delete_disk))) | 1848 | if (copy_from_user(&dd, arg, sizeof (struct aac_delete_disk))) |
1847 | return -EFAULT; | 1849 | return -EFAULT; |
@@ -1866,9 +1868,7 @@ static int delete_disk(struct aac_dev *dev, void __user *arg) | |||
1866 | 1868 | ||
1867 | fsa_dev_ptr = dev->fsa_dev; | 1869 | fsa_dev_ptr = dev->fsa_dev; |
1868 | if (!fsa_dev_ptr) | 1870 | if (!fsa_dev_ptr) |
1869 | return -ENODEV; | 1871 | return -EBUSY; |
1870 | if (!fsa_dev_ptr) | ||
1871 | return -ENODEV; | ||
1872 | 1872 | ||
1873 | if (copy_from_user(&dd, arg, sizeof (struct aac_delete_disk))) | 1873 | if (copy_from_user(&dd, arg, sizeof (struct aac_delete_disk))) |
1874 | return -EFAULT; | 1874 | return -EFAULT; |
diff --git a/drivers/scsi/aacraid/aacraid.h b/drivers/scsi/aacraid/aacraid.h index 8924c183d9c3..e5f7be6f3cd6 100644 --- a/drivers/scsi/aacraid/aacraid.h +++ b/drivers/scsi/aacraid/aacraid.h | |||
@@ -1525,7 +1525,6 @@ struct aac_get_name { | |||
1525 | __le32 count; /* sizeof(((struct aac_get_name_resp *)NULL)->data) */ | 1525 | __le32 count; /* sizeof(((struct aac_get_name_resp *)NULL)->data) */ |
1526 | }; | 1526 | }; |
1527 | 1527 | ||
1528 | #define CT_OK 218 | ||
1529 | struct aac_get_name_resp { | 1528 | struct aac_get_name_resp { |
1530 | __le32 dummy0; | 1529 | __le32 dummy0; |
1531 | __le32 dummy1; | 1530 | __le32 dummy1; |
diff --git a/drivers/scsi/aacraid/commsup.c b/drivers/scsi/aacraid/commsup.c index 53add53be0bd..907161d6e92c 100644 --- a/drivers/scsi/aacraid/commsup.c +++ b/drivers/scsi/aacraid/commsup.c | |||
@@ -467,7 +467,7 @@ int aac_fib_send(u16 command, struct fib *fibptr, unsigned long size, | |||
467 | dprintk((KERN_DEBUG " fib being sent=%p\n",fibptr)); | 467 | dprintk((KERN_DEBUG " fib being sent=%p\n",fibptr)); |
468 | 468 | ||
469 | if (!dev->queues) | 469 | if (!dev->queues) |
470 | return -ENODEV; | 470 | return -EBUSY; |
471 | q = &dev->queues->queue[AdapNormCmdQueue]; | 471 | q = &dev->queues->queue[AdapNormCmdQueue]; |
472 | 472 | ||
473 | if(wait) | 473 | if(wait) |