Commit c6b4ee9e authored by Dan Carpenter's avatar Dan Carpenter Committed by David S. Miller

sparc64: vcc: Check for IS_ERR() instead of NULL

The tty_alloc_driver() function never returns NULL, it returns error
pointers on error.

Fixes: ce808b74 ("sparc64: vcc: TTY driver initialization and cleanup")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent cc324d1a
...@@ -1086,9 +1086,9 @@ static int vcc_tty_init(void) ...@@ -1086,9 +1086,9 @@ static int vcc_tty_init(void)
pr_info("VCC: %s\n", version); pr_info("VCC: %s\n", version);
vcc_tty_driver = tty_alloc_driver(VCC_MAX_PORTS, VCC_TTY_FLAGS); vcc_tty_driver = tty_alloc_driver(VCC_MAX_PORTS, VCC_TTY_FLAGS);
if (!vcc_tty_driver) { if (IS_ERR(vcc_tty_driver)) {
pr_err("VCC: TTY driver alloc failed\n"); pr_err("VCC: TTY driver alloc failed\n");
return -ENOMEM; return PTR_ERR(vcc_tty_driver);
} }
vcc_tty_driver->driver_name = vcc_driver_name; vcc_tty_driver->driver_name = vcc_driver_name;
......
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