Received: by 2002:a05:7412:98c1:b0:fa:551:50a7 with SMTP id kc1csp60768rdb; Fri, 5 Jan 2024 02:46:55 -0800 (PST) X-Google-Smtp-Source: AGHT+IE+JcmfDN0OnA16AIc0xnLoFU+6hvFQ8d6939nxd1t93arnJQD6nj5aB6qxZo4JzHJjTxx2 X-Received: by 2002:a50:d742:0:b0:556:c592:8bf2 with SMTP id i2-20020a50d742000000b00556c5928bf2mr994296edj.40.1704451615477; Fri, 05 Jan 2024 02:46:55 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704451615; cv=none; d=google.com; s=arc-20160816; b=GhMoyjwAvi6RYc024QAqiKlSe/8rK9Hsk0l2uOO+/OzJ/Y2IpXkRIXWmgP/jEqr1X7 6/rii6bXfP/iFso8dieduiZ7CWIUy4Wr6OPSNeyAJzWQ/m+7KPFhHFLWMrgrVNPRDKjj vAEMLbGL1K+eIjo8vZbKHZ7V9qp7VECgphrlXBJh8aqWkZU9bT1TM83we2WsqwVFUYRe wA7i5udOWcniAXxYEC/PLAqcjbgToG6XaRQVbFWc9+UQHqPUGxf0GMk0qVTbyeuczqyz Hn7z7D6p2X/RbKmWiHSBk6hSmDb+ibFaUxX78cO0ZX3d4rQF+V2SVWYx4kPF3SfXaX6T hAqw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=ui-outboundreport:content-transfer-encoding:content-language :in-reply-to:mime-version:list-unsubscribe:list-subscribe:list-id :precedence:user-agent:date:message-id:from:references:cc:to:subject :dkim-signature; bh=VlmhTpPZ6/DLSBoKQjVBM78xTc7tFS9bvVMLHu/gujI=; fh=wDPcTvbU0Pu/LISt2XTz99zYkn7216wnczEnOn4dj+U=; b=rmsNfdxtU06aU0L0pjVK22kYzRgTLGA5w/Fx8IAl7jpdZPWrE7xAqfkOiY7TU8eUzD 83IjPQbGE4SA8XfbOeVCs9StoB+v6OtUhIIKFNDy4XB8p/TYCZh19C6F8M2Tcab8XGr2 rYjqqq54cYyew1lOsgBSXjgJ1cV3RYsbEKs6AAguKuXzGFjpDSfva2jFwbyjX7i+tawW soEy21Z2qnxHg1LRYYjs4HCRsl8RKmMwnRY8A+ONHYgGM6kgH7BeiWmwjQmmK5lK1GvU GHQ+Avs9xGA0txYBZ7t6wvw8DX8G9z9ifoxawAwJyybmm1/fdoyFy9EAPf8msYXYU0Tw U6qQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmx.de header.s=s31663417 header.b=ahcWSTWB; spf=pass (google.com: domain of linux-kernel+bounces-17768-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-17768-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=gmx.de Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id z95-20020a509e68000000b005573cabdb7esi127623ede.126.2024.01.05.02.46.55 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 05 Jan 2024 02:46:55 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-17768-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3; Authentication-Results: mx.google.com; dkim=pass header.i=@gmx.de header.s=s31663417 header.b=ahcWSTWB; spf=pass (google.com: domain of linux-kernel+bounces-17768-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-17768-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=gmx.de Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 3B1C51F2367D for ; Fri, 5 Jan 2024 10:46:55 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 0336D250F9; Fri, 5 Jan 2024 10:46:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmx.de header.i=linosanfilippo@gmx.de header.b="ahcWSTWB" X-Original-To: linux-kernel@vger.kernel.org Received: from mout.gmx.net (mout.gmx.net [212.227.15.15]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E21252C691; Fri, 5 Jan 2024 10:46:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=gmx.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmx.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.de; s=s31663417; t=1704451469; x=1705056269; i=linosanfilippo@gmx.de; bh=VlmhTpPZ6/DLSBoKQjVBM78xTc7tFS9bvVMLHu/gujI=; h=X-UI-Sender-Class:Subject:To:Cc:References:From:Date: In-Reply-To; b=ahcWSTWB4+Gg+VaPKht8owHL9Z/qgtVveN2C1ldaiQP9uyj36CNIkFErlWQY7oC7 BSmHlKl0M1A21a3lxdPce5TyHsz5q9qtnO9z2fgbCMS5fbPyfLQzYDmeGbNW04I1X ZwkwAY/XdxpzHf8z0Gcxl/aowCcRrJwKYgFtkEUWJFMj8IUCHv7p16/nIEPx2uDAn DXAyt5ztSleSWT2FsjIbJcHjVLYN1Ougr/FDMW6W/JLicIbhsjcEcTneU9ou5aGrQ lPlndjPJ1xKqc0NNFSOlhTlLx7a8tsRH06AFI45spBvzqu8it0Dfb/JH/tAib0i1r oZ1G0LJHh5/R1pOi8Q== X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a Received: from [192.168.2.37] ([84.162.15.98]) by mail.gmx.net (mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id 1Mlf0K-1quebv0qaQ-00inRR; Fri, 05 Jan 2024 11:44:29 +0100 Subject: Re: [PATCH v7 1/7] serial: Do not hold the port lock when setting rx-during-tx GPIO To: =?UTF-8?Q?Ilpo_J=c3=a4rvinen?= , Lino Sanfilippo Cc: Greg Kroah-Hartman , Jiri Slaby , u.kleine-koenig@pengutronix.de, shawnguo@kernel.org, s.hauer@pengutronix.de, mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com, cniedermaier@dh-electronics.com, hugo@hugovil.com, m.brock@vanmierlo.com, LKML , linux-serial , Lukas Wunner , p.rosenberger@kunbus.com, stable@vger.kernel.org References: <20240103061818.564-1-l.sanfilippo@kunbus.com> <20240103061818.564-2-l.sanfilippo@kunbus.com> <485fbc21-d099-d316-2146-36fef116f894@linux.intel.com> From: Lino Sanfilippo Message-ID: Date: Fri, 5 Jan 2024 11:44:27 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 In-Reply-To: <485fbc21-d099-d316-2146-36fef116f894@linux.intel.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:cON+uAom9WLCxa7nEVAwB+HD1igo5z8st1aDwqQcTSX3daYFnri hcTsI0tzVWPYUiv/tFud7Foma2G9Utb4yIvQdF8bVLhL22aTEUn9fh++MeWPh+jHhLuQUEe Weiry39N7AuamQ3Qjim6UowZxSs8MIveQkh/mUE76kvcC40tf+VKFXnNNb3O9px+PcBWoIC jmyGqgUxF0hsAqkETFxAA== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:YniKicCEILw=;hNp7S56051Ato2JeMjKo75QDS48 y05JINy3OJGxV2pazbW0V5XPvlOh2Ft5WSQkDChWkHPrst9YDAOXiMDEz4wDOqmkj7YIewEtd GEqRZQR/AaJ+UL+dg5woKXps8jQbxGrwMPaXjsgam8K8k3i4D5vC0/OYhkJwADi1HPImgV/tr f982zggOdB9fAcMtACjH08CHHnNLfOsidjKl7cKmlWHvfrWHWml93g4aJecTiOqdvLY4aRw+t VXdtzN7XxtoJFST+/SfPgr6nr2Z9kGeOuWq9FawMtpRILRXjYHOTzb8ikcU5Ski92QXoyDi7L vyYjooDcLJao+XBZyCd2ptoo/Snp3woTxwnAecz0ypHFZf/qU6WOpKZRwc8AwLKO6PJajEvAA I2Ppmn8DWHUGLsv/UzaUYaWVF2lrc2N4W4wLlh0WDscD0P8oRlOS1OwmQU/UTzzuoFJoENAiN J0RGczLifvWnwkagERiJ06RVk16uRGsfFEujHRfBiFM9eri8mxJXJfYLZEtnzYogAxH0v0r1S RhsWSa6llRr/quTguqD7CIhBzWE+k2TFHZ6cce6ArHuMp771m+Ti9boDhNewLGz/Elt4k6Lkh GotJ8J68zUcLfDl4AYTwYVYmUcI4cqLv9v4H81ZeqnMBZ4cP7lB3Htk+FR1UA84e94f6LGNbK cUNsEvlbvmKEkCbhHccKSD0OnKgPxxaCI0F8RkYPZ6vda0uzZK756jKFsYn8rt8g/VXnNpPwH VHYiaDjqFIaj8eWMvhzsc0aHYbnCRouOCQGXYft0XNTv3ZrluY7KfdvO4pVER9BQNpDcVawwp 6mdxMOb9J+KDi1TeGk8J7+Pufeb5pw0OYSRuR+2p89bN9V9lj9bmf0wApwdS+MdtMMqbJ15EB XFL1ETteq7nLOdn9ww4ry307Q7uWdjEhRHYcUvmoVrGKVK2d9xqrBzHOn99fX9y+zNNZ/MAXu EdfQv4Huxk3RiZE/QLeT5U0LN10= On 03.01.24 at 12:10, Ilpo J=C3=A4rvinen wrote: > On Wed, 3 Jan 2024, Lino Sanfilippo wrote: > >> Also with commit c54d48543689 ("serial: stm32: Add support for rs485 >> RX_DURING_TX output GPIO") the SER_RS485_RX_DURING_TX flag is only set = if a >> rx-during-tx GPIO is _not_ available, which is wrong. Fix this, too. > > This doesn't explain why it's wrong and I could figure it out myself > either. > Ugh, you are right. I confused some things here. This part of the patch is= completely wrong and the concerning code was correct as it was before. I will correct= this patch for v8. Thanks, Lino