From: Yalagada Pavan Kumar pavankumaryalagada@gmail.com

Remove the error message, as tty_alloc_driver() returns -ENOMEM when it fails, so printing the message is unnecessary.

Signed-off-by: Yalagada Pavan Kumar pavankumaryalagada@gmail.com Reviewed-by: Alex Elder elder@riscstar.com --- v6: - Remove the redundant error message, as suggested by Johan Hovold

v5: - Drop arche-apb-ctrl.c and arche-platform.c changes because the files were removed. - keep only the uart.c change, as the fw-core.c changes are already present in staging/staging-next.

v4: - Return error codes directly instead of storing them in ret, as suggested by Julia Lawall. - Remove the unused ret variable.

v3: - Extend the %pe conversion throughout Greybus, as suggested by Alex Elder.

v2: - Use %pe in the firmware management case, as suggested by Jeremy Clifton, to match the other cases.

v1: https://lore.kernel.org/outreachy/as0nhVbagZa8Birm@user/T/#t --- drivers/staging/greybus/uart.c | 1 - 1 file changed, 1 deletion(-)

diff --git a/drivers/staging/greybus/uart.c b/drivers/staging/greybus/uart.c index d47f00751e7a..4de61fca6725 100644 --- a/drivers/staging/greybus/uart.c +++ b/drivers/staging/greybus/uart.c @@ -956,7 +956,6 @@ static int gb_tty_init(void) gb_tty_driver = tty_alloc_driver(GB_NUM_MINORS, TTY_DRIVER_REAL_RAW | TTY_DRIVER_DYNAMIC_DEV); if (IS_ERR(gb_tty_driver)) { - pr_err("Can not allocate tty driver\n"); retval = PTR_ERR(gb_tty_driver); goto fail_unregister_dev; }