staging: unisys: visorhba change -1 return value
authorErik Arfvidson <erik.arfvidson@unisys.com>
Sat, 14 May 2016 03:17:18 +0000 (23:17 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Jun 2016 05:58:16 +0000 (22:58 -0700)
This patch changes the vague -1 return value to -EBUSY

Signed-off-by: Erik Arfvidson <erik.arfvidson@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/unisys/visorhba/visorhba_main.c

index 48551fe..120ba20 100644 (file)
@@ -184,7 +184,7 @@ static void visor_thread_stop(struct task_struct *task)
  *     Partition so that it can be handled when it completes. If new is
  *     NULL it is assumed the entry refers only to the cmdrsp.
  *     Returns insert_location where entry was added,
- *     SCSI_MLQUEUE_DEVICE_BUSY if it can't
+ *     -EBUSY if it can't
  */
 static int add_scsipending_entry(struct visorhba_devdata *devdata,
                                 char cmdtype, void *new)
@@ -199,7 +199,7 @@ static int add_scsipending_entry(struct visorhba_devdata *devdata,
                insert_location = (insert_location + 1) % MAX_PENDING_REQUESTS;
                if (insert_location == (int)devdata->nextinsert) {
                        spin_unlock_irqrestore(&devdata->privlock, flags);
-                       return -1;
+                       return -EBUSY;
                }
        }