From: Sameer Wadgaonkar <sameer.wadgaon...@unisys.com>

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 <gre...@linuxfoundation.org>
Signed-off-by: Sameer Wadgaonkar <sameer.wadgaon...@unisys.com>
Signed-off-by: David Kershner <david.kersh...@unisys.com>
Reviewed-by: Tim Sell <timothy.s...@unisys.com>
---
 drivers/staging/unisys/visorbus/visorbus_main.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c 
b/drivers/staging/unisys/visorbus/visorbus_main.c
index d9b0a8b..05b632e 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;
-- 
git-series 0.9.1
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to