Commit e134201b authored by Sudip Mukherjee's avatar Sudip Mukherjee Committed by Greg Kroah-Hartman

staging: panel: return register value

we were returning success even if the module failed to register.
now we are returning the actual return value, success or error.
Signed-off-by: default avatarSudip Mukherjee <sudip@vectorindia.org>
Acked-by: default avatarWilly Tarreau <w@1wt.eu>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent bb046fef
...@@ -2282,7 +2282,7 @@ static struct parport_driver panel_driver = { ...@@ -2282,7 +2282,7 @@ static struct parport_driver panel_driver = {
/* init function */ /* init function */
static int __init panel_init_module(void) static int __init panel_init_module(void)
{ {
int selected_keypad_type = NOT_SET; int selected_keypad_type = NOT_SET, err;
/* take care of an eventual profile */ /* take care of an eventual profile */
switch (profile) { switch (profile) {
...@@ -2388,9 +2388,10 @@ static int __init panel_init_module(void) ...@@ -2388,9 +2388,10 @@ static int __init panel_init_module(void)
return -ENODEV; return -ENODEV;
} }
if (parport_register_driver(&panel_driver)) { err = parport_register_driver(&panel_driver);
if (err) {
pr_err("could not register with parport. Aborting.\n"); pr_err("could not register with parport. Aborting.\n");
return -EIO; return err;
} }
if (pprt) if (pprt)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment