From patchwork Thu Oct 26 10:41:17 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Th=C3=A9o_Lebrun?= X-Patchwork-Id: 738621 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 0FAC9C25B48 for ; Thu, 26 Oct 2023 10:41:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231239AbjJZKl4 (ORCPT ); Thu, 26 Oct 2023 06:41:56 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48088 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234902AbjJZKlz (ORCPT ); Thu, 26 Oct 2023 06:41:55 -0400 Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [IPv6:2001:4b98:dc4:8::221]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D4DF91B2; Thu, 26 Oct 2023 03:41:49 -0700 (PDT) Received: by mail.gandi.net (Postfix) with ESMTPSA id F20F824000B; Thu, 26 Oct 2023 10:41:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1698316908; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=7FO3bXg4jaYqWVw7SUw9GVHbeVNPGlx4JKNogiekb4w=; b=nuNiebBs72L1ynsy2Kldz2/h12GOsQYrI8XbztCaRZlHCO+74uJ1qjaiLXX9heZSnEvVf3 AJPBvI8zVv0oqF/xzmyHPW0kpZSW+x9Jvv/CuOn5DBTMpDsR5IuLjazzEHqMQvsQgD/QM1 BE97roaofc9OeBsJXP537pvNtmv8WSVgGb5rjcBi9kBuZqZ/AD/I6SVe6MLMXexaLybEuM v+jl67/WdATJFIwE1fWNF8INxAltZF6dHD+4bGkATvgYGzDoP+y7oig8xRl4sFMv8i8vvF o/OpRYGoY8ADus4Q2dCxDuNn57XTLgpnaGp2exuR60E7A1e6wYTkwDqmWiqwRA== From: =?utf-8?q?Th=C3=A9o_Lebrun?= Subject: [PATCH 0/6] Cleanup AMBA PL011 driver Date: Thu, 26 Oct 2023 12:41:17 +0200 Message-Id: <20231026-mbly-uart-v1-0-9258eea297d3@bootlin.com> MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAE1COmUC/6tWKk4tykwtVrJSqFYqSi3LLM7MzwNyDHUUlJIzE vPSU3UzU4B8JSMDI2NDINbNTcqp1C1NLCrRTUxLTkxOSrK0SLNIUgKqLyhKTcusAJsVHVtbCwB 74cndWwAAAA== To: Russell King , Greg Kroah-Hartman , Jiri Slaby Cc: linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, Linus Walleij , Gregory CLEMENT , Alexandre Belloni , Thomas Petazzoni , Vladimir Kondratiev , Tawfik Bayouk , =?utf-8?q?Th=C3=A9o_Lebrun?= X-Mailer: b4 0.12.3 X-GND-Sasl: theo.lebrun@bootlin.com Precedence: bulk List-ID: X-Mailing-List: linux-serial@vger.kernel.org Hi, While adding upstream support to a new platform (Mobileye EyeQ5[1]) that uses the AMBA PL011 driver, I took some time to look at the PL011 driver and ended up with a few patches that cleanup parts of it. The line-diff is big mostly because of the checkpatch-fixing commits. The driver hadn't received any love for quite some time. A single commit changes the code's behavior: see "tty: serial: amba-pl011: Parse bits option as 5, 6, 7 or 8 in _get_options". See commit messages for more information. [1]: https://lore.kernel.org/all/202310050726.GDpZbMDO-lkp@intel.com/T/ Have a nice day, Théo Lebrun Signed-off-by: Théo Lebrun --- Théo Lebrun (6): tty: serial: amba: cleanup whitespace tty: serial: amba: Use BIT() macro for constant declarations tty: serial: amba-pl011: cleanup driver tty: serial: amba-pl011: replace TIOCMBIT macros by static functions tty: serial: amba-pl011: unindent pl011_console_get_options function body tty: serial: amba-pl011: Parse bits option as 5, 6, 7 or 8 in _get_options drivers/tty/serial/amba-pl011.c | 238 ++++++++++++++++++++-------------------- include/linux/amba/serial.h | 192 ++++++++++++++++---------------- 2 files changed, 214 insertions(+), 216 deletions(-) --- base-commit: ad582615776e62e365ab2dfa7a7a3806ada28b30 change-id: 20231023-mbly-uart-afcacbb98f8b Best regards,