staging: gasket: Return EBUSY on mapping create when already in use
authorTodd Poynor <toddpoynor@google.com>
Tue, 17 Jul 2018 20:56:47 +0000 (13:56 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 Jul 2018 09:39:47 +0000 (11:39 +0200)
gasket_sysfs_create_mapping() return EBUSY if sysfs mapping already in
use, as a more appropriate error code than the current return of EINVAL,
which would indicate invalid parameters.

Signed-off-by: Zhongze Hu <frankhu@chromium.org>
Signed-off-by: Todd Poynor <toddpoynor@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/gasket/gasket_sysfs.c

index e3d7706..dd4d3aa 100644 (file)
@@ -194,7 +194,7 @@ int gasket_sysfs_create_mapping(
                        "0x%p.", device);
                put_mapping(mapping);
                mutex_unlock(&function_mutex);
-               return -EINVAL;
+               return -EBUSY;
        }
 
        /* Find the first empty entry in the array. */