serial: sunsab: Return proper error code from console ->setup() hook
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 18 Jun 2020 16:47:47 +0000 (19:47 +0300)
committerPetr Mladek <pmladek@suse.com>
Thu, 25 Jun 2020 12:21:28 +0000 (14:21 +0200)
For unifying console ->setup() handling, which is poorly documented,
return error code, rather than non-zero arbitrary number.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Petr Mladek <pmladek@suse.com>
Acked-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Petr Mladek <pmladek@suse.com>
Link: https://lore.kernel.org/r/20200618164751.56828-3-andriy.shevchenko@linux.intel.com
drivers/tty/serial/sunsab.c

index 1eb703c980e054b55b41f0134488316fdc6b7f1a..bab551f46963128acbe1f6aaf389c88f9a84281a 100644 (file)
@@ -886,7 +886,7 @@ static int sunsab_console_setup(struct console *con, char *options)
         * though...
         */
        if (up->port.type != PORT_SUNSAB)
-               return -1;
+               return -EINVAL;
 
        printk("Console: ttyS%d (SAB82532)\n",
               (sunsab_reg.minor - 64) + con->index);