From: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Date: Wed, 30 Aug 2017 17:36:10 +0000 (-0400)
Subject: staging: unisys: visorbus: visorbus_main.c: Fix return values for checks in visorbus_... 
X-Git-Tag: v4.14-rc1~148^2~46
X-Git-Url: https://asedeno.scripts.mit.edu/gitweb/?a=commitdiff_plain;h=3459e83a141e2af284c5980917dfa8a1bb224319;p=linux.git

staging: unisys: visorbus: visorbus_main.c: Fix return values for checks in visorbus_register_visor_driver.

The error return values for the drv->probe, drv->remove, drv->pause
and drv->resume checks should be -EINVAL instead of -ENODEV.

Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: Tim Sell <timothy.sell@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c b/drivers/staging/unisys/visorbus/visorbus_main.c
index d9b0a8bd3c70..05b632e97241 100644
--- a/drivers/staging/unisys/visorbus/visorbus_main.c
+++ b/drivers/staging/unisys/visorbus/visorbus_main.c
@@ -969,16 +969,16 @@ int visorbus_register_visor_driver(struct visor_driver *drv)
 		return -ENODEV;
 
 	if (!drv->probe)
-		return -ENODEV;
+		return -EINVAL;
 
 	if (!drv->remove)
-		return -ENODEV;
+		return -EINVAL;
 
 	if (!drv->pause)
-		return -ENODEV;
+		return -EINVAL;
 
 	if (!drv->resume)
-		return -ENODEV;
+		return -EINVAL;
 
 	drv->driver.name = drv->name;
 	drv->driver.bus = &visorbus_type;