From nobody Fri Jan 2 13:46:21 2026 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 7779BCDB470 for ; Wed, 11 Oct 2023 17:39:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232996AbjJKRjj (ORCPT ); Wed, 11 Oct 2023 13:39:39 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55248 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233183AbjJKRj0 (ORCPT ); Wed, 11 Oct 2023 13:39:26 -0400 Received: from EUR02-DB5-obe.outbound.protection.outlook.com (mail-db5eur02on2088.outbound.protection.outlook.com [40.107.249.88]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 82388CC; Wed, 11 Oct 2023 10:39:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XAh2drSLMhIDGnpgmbeR5QxKNfoLgrTmzxZj3wqU08yayWnBU4t3w03PaPJG/8GPpIhBSSSjtzrVAZIhlsGd1y1qzYbyJU/XEvj90rbrQyZSGhqq5e/X5PaYYIq96gyZYd2zCeC3Obxh1wk4O7fxfGv/UAs5XKrZ1Qu3SKZHRXj0B25LpNvRhgCPJa2b/bPqrA8gh0yRU2UfCOJts7ifoVqT24T3ROAqIFGT2oU4txiwI3HcuenRiaFr+RheOEWhtwlMl1IbLhCJiUkrY4sGj/nXB4sqWawIF5Swf9t49O9SnNESAt+n6j/Pbk1qS3NA7Vkg3ytrWuTFxBmjQ6rwTA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=g9VNnrRh+93b9tiZ7+fqQ29BWG2jfMJ+okhxSduWaaw=; b=bBeAblgk4q/AM/hZYIdL5SHKWSx35276mQ/idQ9keKNchocVM+RCfsW4IH8pAeb1U9nxV9CTA3Xy+jG9kH19mGjSjpoVNO1bq8oORQM/VxgZCceBR8+2t0Jz51jlXMvZkNqmvWxcGXyDIK3Ls0dp1KSGd196U+F+soorjGPJeDOvyMT8e3W9ESyw/lHF17hAZHw0ZSVWVBrxhzx7vHyYUeujv6cvmzNNKBMCyiXtO06vbmC103NAfpm38e/bmr2GXsu2EE2rgTL3GQ4WCssnDqTEKThE0T9s4gWXeA+dofGXBUjBNoUI94FfI8vtpJ5hFJdBDjSTXwrhAT/4toGgpg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=kunbus.com; dmarc=pass action=none header.from=kunbus.com; dkim=pass header.d=kunbus.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kunbus.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=g9VNnrRh+93b9tiZ7+fqQ29BWG2jfMJ+okhxSduWaaw=; b=aAO/cdrvJeQ7KrAch7thEVyxuqGaSN5jU0IxrAIquJXnO8kRwSMYGd7C6IAOGlWthsjeQjg4CaLu066BgAS94rUJV6pf3jb+0BBp9vmkc0lAeWoP5zeu0RG8nlJ0Hbvj4GWsdubrWf1HrcEW41rfoYNxzKwY2nfCpixV+2vM9KM= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=kunbus.com; Received: from VI1P193MB0413.EURP193.PROD.OUTLOOK.COM (2603:10a6:803:4e::14) by AM8P193MB1187.EURP193.PROD.OUTLOOK.COM (2603:10a6:20b:362::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6863.41; Wed, 11 Oct 2023 17:39:08 +0000 Received: from VI1P193MB0413.EURP193.PROD.OUTLOOK.COM ([fe80::550d:2425:c0ed:3e59]) by VI1P193MB0413.EURP193.PROD.OUTLOOK.COM ([fe80::550d:2425:c0ed:3e59%3]) with mapi id 15.20.6863.032; Wed, 11 Oct 2023 17:39:08 +0000 From: Lino Sanfilippo To: gregkh@linuxfoundation.org, jirislaby@kernel.org, ilpo.jarvinen@linux.intel.com Cc: shawnguo@kernel.org, s.hauer@pengutronix.de, mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com, cniedermaier@dh-electronics.com, linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, LinoSanfilippo@gmx.de, lukas@wunner.de, p.rosenberger@kunbus.com, Lino Sanfilippo , stable@vger.kernel.org, =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= Subject: [PATCH v2 6/7] serial: core, imx: do not set RS485 enabled if it is not supported Date: Wed, 11 Oct 2023 19:38:28 +0200 Message-Id: <20231011173829.10508-7-l.sanfilippo@kunbus.com> X-Mailer: git-send-email 2.40.1 In-Reply-To: <20231011173829.10508-1-l.sanfilippo@kunbus.com> References: <20231011173829.10508-1-l.sanfilippo@kunbus.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: FR2P281CA0093.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:9b::18) To VI1P193MB0413.EURP193.PROD.OUTLOOK.COM (2603:10a6:803:4e::14) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VI1P193MB0413:EE_|AM8P193MB1187:EE_ X-MS-Office365-Filtering-Correlation-Id: 7563fd0f-131b-4ca1-b310-08dbca80f8a2 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 58WR2e9qytcd68wapZlInQB0RgQtzyhh+s/A+CUeJqJIyQS5PDGb8QgP/UiSxn1AEwkKSBVfKkCRiuOPoMuGKL/ImF+7QakoAuPjZQ6Tqxh2h3DB3CsmMIzfPJ3kzFv8YIHmt6gBwyOZOy0HZSGXCzKDtJpYMvMhVdeV9br2L1j2ZO5XI44Sg15Dy3i/tTkvtt6MhGirONesdrUfTQMF6dh5HIPMD/uRuRjZS7rwCwWYaYmr4L9TKrLJ505rcp7tS/66/h6CiEKc9o1hNumk13Y6MJQ0RUR51cjqxwIJSHXWYJGOOy2oR7iJPg7zHWrSKWNxSSH5bTdYSk5a4yugE2Cy8R+5Aoigufm258uoiFFZx+vG7of2XhQiLj4vvgLoVyossx52PvcLWjbxhzOAaL6s8qNEo0c9fRyQ4cZz5fciwv1IEmQDSUqMB5+FjiNoMjTKCtZmbiSkaJDoOj9HnVdPqXzHDgu6f1tnvGU6CXTLIctzDZOBhUojrKTBWiubJjfXYZSg4kciB1fb3rVz/ELrYeymehic3t+gi8k9JOnMizht/BHHp9uCgPNYLTHQ X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1P193MB0413.EURP193.PROD.OUTLOOK.COM;PTR:;CAT:NONE;SFS:(13230031)(136003)(39840400004)(376002)(366004)(346002)(396003)(230922051799003)(186009)(64100799003)(451199024)(1800799009)(6666004)(2616005)(83380400001)(6486002)(478600001)(1076003)(52116002)(38100700002)(6506007)(6512007)(66476007)(66946007)(66556008)(4326008)(8676002)(8936002)(41300700001)(316002)(86362001)(7416002)(2906002)(54906003)(36756003)(5660300002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?M1hMSmg0aXRGaW5sdWVUdWRqQ0p4VjJOd0F3dVcrOXVubnJrN04wK005Mks1?= =?utf-8?B?cGtoT0dUbHRyT0dEOEtMZENjUDVNOGQrNnNRNHRkMmZjaXhsSGZJSjBlWVpx?= =?utf-8?B?QlRGSlpQdmVodC9lZkk2U0V4M0psZkI5NTJNVU9ZUldnSzZQejZRdlRYMkNS?= =?utf-8?B?YTJ5NTV0TDlsa2ViVmY2QnJIcXVyKy80UHoxQWlOTHlCVGova0xRdm9IYVpx?= =?utf-8?B?QlhUajVtUG85c2crT0k5YTlydTNUL3JzR2lqM25FQ0w4K1hTQ0xjQ3NMaTNG?= =?utf-8?B?aEtleTAwQXRJMDZxOVBEcThaZDEvdmJrcTY2cndGT0dTd3B4eEx2ejhZc1Aw?= =?utf-8?B?SXRvRFV3S251dTg1ZDJ5NDVkeDhqK1FBMFpISlpSckFsNC9yUHRGNUVTRU4x?= =?utf-8?B?TXhHMFV5b0FqUGNwdS9mZ2NXK1RDL0lhQk5LNnBwUDJidnVXRXpyM016TUVX?= =?utf-8?B?NTNWYk50OGxPOUNXeC8rTC9xNXY1TnJQRDkrcGlvU2crNlVZa0kzRzRDTUdR?= =?utf-8?B?S1ZSUUtkVUVHSTBoV01maFlqVWhlVHYyT1o1Ui9IK3lYWFlsWWFXanlmNFFH?= =?utf-8?B?cjFUNFpmN3RkdHN1TmNvamV2WmxleHdOeDV5OGJZcUxXaEN0ZkJQTU5ZdVc3?= =?utf-8?B?WVJCcUQ4Z1ZaQ1BRQXRjS2JXSFNwOGNKZ3dBZzBUUHBVOTV1RXpHUytQMUhF?= =?utf-8?B?b2hkUFM1dS9TOVFtYVloR0I3MndZOEg5S2xDenYrQ0p2YVBsb2pIYmF2M0hK?= =?utf-8?B?eXJuckI2T0gzenpJUGtxSFBHNitjaHVmQUxxVElpdUhBQWZMTUFDSzlVZm1Y?= =?utf-8?B?R0llSG1WYTRaSWRuanhmOEhOaXRTV3o2US9ObXhWNEdiTCsrNTZDR2pIM1I0?= =?utf-8?B?YmlzaExHYllKZG4wOTZ0ZnJCU0pIOEUvNzNPMjJJTUF1YXBzUlpXZjJlZGJG?= =?utf-8?B?YU91YmFaa0VnOUsxeGlkUmZnWlRPdGJ2dWZwTEFaWGptQnIyajJDcWhpcDRy?= =?utf-8?B?SkRybFF5b0Zvay9XbituV24zVVFUUGh4bUEzUEpQdFZrTEdPd0NFOEVwR3ZB?= =?utf-8?B?NVp6ZmJVMk9FSzl0Nk9CSllDdVRnMkVMemZPVXJ5U3F2SXBMbXdyaFp5V0Nr?= =?utf-8?B?OU00aTA5VlFybFF2blJmUDlkdnVoTkg3Rjl3a1JnN0FxeEpNalFzK1R2dnIr?= =?utf-8?B?ZFVlNjJISWNUSmpraVRTak1xUVd1cmZSMFYwZGdpck92dUdkaW10TFY0MlRY?= =?utf-8?B?cGFjV2lNWHdMRGVCMzEyclZFb0xYS2hvZm9oMWF1SHVyRFRYYkRDMnNNN0t6?= =?utf-8?B?MFZFRytmcDFmYlAwenlQVHc3VlYzOWx5TWRXeUFUTENZK2k2UVlMdmtqcEl1?= =?utf-8?B?bFNrMjZqNGJGL2lwYng0UXQ2anhMeXRNZjY2TSt4NFNpbDM2OU5rT2owWXRX?= =?utf-8?B?OFFWWjZiUU5Zb2NxeFlVcEpTd1lLTFZkYjJKMExoOXFwMmJRcDJONVMwSWFi?= =?utf-8?B?ZmZFS2MxTHZVa0dxcThWcDZIdDE5S0RWdERIcDRBcERPUkZNbGloN25QOWI5?= =?utf-8?B?L3RWR1RiMTBxT2RoRng5MWcxVWFNMTRyWkNHWXVvd0JqVlFvZkkwOExSbnZC?= =?utf-8?B?L3lNdVRiTWgvd2dGNHp5bXN2KzI4L3F4L05ENTZ6eGpoZUVFZEhxdUp5a0t0?= =?utf-8?B?Ulk2UFJ1YTN5Vm9MN2hObjV1NUc5V0hUUElsWXZmOTdBams1SlZyTG1tSGRZ?= =?utf-8?B?UGM4c21xSitHbWtZTEpJNDFEVm5lY2Vsa1dzakJWem5QTWJjY3E5UUd0ZU14?= =?utf-8?B?eVN6blphbGNreU8vVWtGKysrakVNSlRQWHJLd000WWN5WWVPMGlKeDlHczV1?= =?utf-8?B?VStJdnhnRjRKdjZ4L3NrL0FnOVRBb1VHbE9xbDJKcGdCeC90TkRraUtRMFB2?= =?utf-8?B?cFBaUVVML0RDK2lpWEJPZ0dxMjhVUkQvWXJmdEJYMXFOcjJWUmZkcXUvZkNY?= =?utf-8?B?ZHZNS1k2bGNHVHNqZ2N3K3g0cExwVnlDemxlbmJLUm1pQUhFLytYcExscDRH?= =?utf-8?B?N284Vi9aeEdGNEhIdU9QdmVXMTlTZk5ld0lORFB4bm8vYVJRZGtuc2N3eThu?= =?utf-8?B?VWV6dXdlZDFDOFVpVUpPcjQvWkJkajlVSkpKUE9ORXlOQktLNnRZM2NPZ1FE?= =?utf-8?Q?caBeGvRCazl5t2cme45FSmvfuLvBaj9OlxOvaoQ1p8Gf?= X-OriginatorOrg: kunbus.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7563fd0f-131b-4ca1-b310-08dbca80f8a2 X-MS-Exchange-CrossTenant-AuthSource: VI1P193MB0413.EURP193.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Oct 2023 17:39:08.0536 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: aaa4d814-e659-4b0a-9698-1c671f11520b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 3G/BcBG+MR0SKgB0v9OH9fWXUk4Z96ezoa242iKnjMLz/idaDYLIuT7767KHxp4W8c/vim2rxhhbVa9f0eLhMA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM8P193MB1187 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" If the imx driver cannot support RS485 it sets the ports rs485_supported structure to NULL. But it still calls uart_get_rs485_mode() which may set the RS485_ENABLED flag nevertheless. This may lead to an attempt to configure RS485 even if it is not supported when the flag is evaluated in uart_configure_port() at port startup. Avoid this by bailing out of uart_get_rs485_mode() if the RS485_ENABLED flag is not supported by the caller. With this fix a check for RTS availability is now obsolete in the imx driver, since it can not evaluate to true any more. Remove this check, too. Fixes: 00d7a00e2a6f ("serial: imx: Fill in rs485_supported") Cc: stable@vger.kernel.org Suggested-by: Uwe Kleine-K=C3=B6nig Signed-off-by: Lino Sanfilippo --- drivers/tty/serial/imx.c | 4 ---- drivers/tty/serial/serial_core.c | 3 +++ 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.c index edb2ec6a5567..c8c19bf8585d 100644 --- a/drivers/tty/serial/imx.c +++ b/drivers/tty/serial/imx.c @@ -2332,10 +2332,6 @@ static int imx_uart_probe(struct platform_device *pd= ev) return ret; } =20 - if (sport->port.rs485.flags & SER_RS485_ENABLED && - (!sport->have_rtscts && !sport->have_rtsgpio)) - dev_err(&pdev->dev, "no RTS control, disabling rs485\n"); - /* * If using the i.MX UART RTS/CTS control then the RTS (CTS_B) * signal cannot be set low during transmission in case the diff --git a/drivers/tty/serial/serial_core.c b/drivers/tty/serial/serial_c= ore.c index dca09877fabc..95cec2343b35 100644 --- a/drivers/tty/serial/serial_core.c +++ b/drivers/tty/serial/serial_core.c @@ -3576,6 +3576,9 @@ int uart_get_rs485_mode(struct uart_port *port) int ret; int rx_during_tx_gpio_flag; =20 + if (!(port->rs485_supported.flags & SER_RS485_ENABLED)) + return 0; + ret =3D device_property_read_u32_array(dev, "rs485-rts-delay", rs485_delay, 2); if (!ret) { --=20 2.40.1