From patchwork Wed Jul 19 05:15:23 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tony Lindgren X-Patchwork-Id: 13318139 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C60F9C001DC for ; Wed, 19 Jul 2023 05:15:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229459AbjGSFPf (ORCPT ); Wed, 19 Jul 2023 01:15:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50426 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229454AbjGSFPf (ORCPT ); Wed, 19 Jul 2023 01:15:35 -0400 Received: from muru.com (muru.com [72.249.23.125]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id EEDA01BF2; Tue, 18 Jul 2023 22:15:33 -0700 (PDT) Received: from hillo.muru.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTP id 98A9280AA; Wed, 19 Jul 2023 05:15:31 +0000 (UTC) From: Tony Lindgren To: Greg Kroah-Hartman , Jiri Slaby Cc: Andy Shevchenko , Dhruva Gole , =?utf-8?q?Ilpo_J=C3=A4rvinen?= , John Ogness , Johan Hovold , Sebastian Andrzej Siewior , Vignesh Raghavendra , linux-omap@vger.kernel.org, linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] serial: core: Add support for dev_name:0.0 naming for kernel console Date: Wed, 19 Jul 2023 08:15:23 +0300 Message-ID: <20230719051525.46494-1-tony@atomide.com> X-Mailer: git-send-email 2.41.0 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org With the serial core controller related changes we can now start addressing serial ports with dev_name:0.0 naming. The names are something like 00:04.0:0.0 on qemu, and 2800000.serial.0:0.0 on ARM for example. The dev_name is unique serial port hardware controller device name, also known as port->dev, and 0.0 are the serial core controller id and port id. Typically 0.0 are used for each controller and port instance unless the serial port hardware controller has multiple controllers or ports. Signed-off-by: Tony Lindgren --- Note that this depends on fix for serial core port ids patch "[PATCH] serial: core: Fix serial core port id to not use port->line" --- drivers/tty/serial/serial_core.c | 47 ++++++++++++++++++++++++++++++++ 1 file changed, 47 insertions(+) diff --git a/drivers/tty/serial/serial_core.c b/drivers/tty/serial/serial_core.c --- a/drivers/tty/serial/serial_core.c +++ b/drivers/tty/serial/serial_core.c @@ -3322,6 +3322,49 @@ static int serial_core_port_device_add(struct serial_ctrl_device *ctrl_dev, return 0; } +/* + * Add preferred console if configured on kernel command line with naming + * "console=dev_name:0.0". + */ +static int serial_core_add_preferred_console(struct uart_driver *drv, + struct uart_port *port) +{ + char *port_match, *opt, *name; + int len, ret = 0; + + port_match = kasprintf(GFP_KERNEL, "console=%s:%i.%i", + dev_name(port->dev), port->ctrl_id, + port->port_id); + if (!port_match) + return -ENOMEM; + + opt = strstr(saved_command_line, port_match); + if (!opt) + goto free_port_match; + + len = strlen(port_match); + + if (strlen(opt) > len + 1 && opt[len] == ',') + opt += len + 1; + else + opt = NULL; + + name = kstrdup(drv->dev_name, GFP_KERNEL); + if (!name) { + ret = -ENOMEM; + goto free_port_match; + } + + add_preferred_console(name, port->line, opt); + + kfree(name); + +free_port_match: + kfree(port_match); + + return ret; +} + /* * Initialize a serial core port device, and a controller device if needed. */ @@ -3358,6 +3401,10 @@ int serial_core_register_port(struct uart_driver *drv, struct uart_port *port) if (ret) goto err_unregister_ctrl_dev; + ret = serial_core_add_preferred_console(drv, port); + if (ret) + goto err_unregister_port_dev; + ret = serial_core_add_one_port(drv, port); if (ret) goto err_unregister_port_dev;