From mboxrd@z Thu Jan 1 00:00:00 1970 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 F3214C433F5 for ; Thu, 17 Feb 2022 09:22:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238531AbiBQJWW (ORCPT ); Thu, 17 Feb 2022 04:22:22 -0500 Received: from mxb-00190b01.gslb.pphosted.com ([23.128.96.19]:59760 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238413AbiBQJWV (ORCPT ); Thu, 17 Feb 2022 04:22:21 -0500 Received: from mail-wm1-x334.google.com (mail-wm1-x334.google.com [IPv6:2a00:1450:4864:20::334]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 928516409; Thu, 17 Feb 2022 01:22:06 -0800 (PST) Received: by mail-wm1-x334.google.com with SMTP id d14-20020a05600c34ce00b0037bf4d14dc7so3518054wmq.3; Thu, 17 Feb 2022 01:22:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=uEF/FU/QB3+JPW9i0x2lAGeqkeJMKwTkLaDfp85o4eA=; b=DY+MxSUAazGgw0yYDJltTUdAkc4UgtVq4+ctUknKf21YBPaMDiGfUnVX7BHtJJuqHu 1SYdMuGzCw+h4q8+FwO+M7XjGOkzjXPQXhTwf+I9PpvoHSRBA7ehI1K3BugPXbedrc1p aVCOobSkjGFu+kR91na4ZDELFbcmv+HlgA1AhukRVY4kzK+XIK2j+TyqNlRN/gd+Of8X UoYHrOua6xlip9UDXmcWaMftQmZ4uA2bIKAOPpCj3yS/X50MySCDwh/RP6Pxma3ui19H exH1l/GR9ZT+PrEB9p+ai3Fvt3S68xBiawm9KjnFnYyGYdVy02sqs/MBti/L5jSXVGY9 Dylw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=uEF/FU/QB3+JPW9i0x2lAGeqkeJMKwTkLaDfp85o4eA=; b=KlMT0THTZGIHoJCzbdRD2XN+gFLTRKkC7Cgj9DAeI917O+B9TTb1hvG7Patdz5VoAy tkYaGVAQGfMBXzxyhOlywfeIng1l9BJ9w+2Wd/O/uwOWT+6UVcGUBFvX1rabuQ7FgM44 KanWsJZqQQDyzOUjNlwwcpnNpdc/877X21i2byjmc2Pi8DrlAyVRbiVi5WS9yk2bmiBC /fHMcdApwKMVtkWIPVA5YJCPj19vmqKDoj8YrXPPg8PP/D5oZMaWV+TCJAON+CFhYSSW /zlteqyvleTV06571uXpmhsAh9jbP+hgmOAyuWqVvIiA7N22XKAGphQshTKL6mipHgoA MvjQ== X-Gm-Message-State: AOAM532kBfWdngLhZDgknj82aIZkFY+VhY9MvFjUL8B204klvFIhb4Af xVyUoxQxVvhbiyuYlJzwCxA= X-Google-Smtp-Source: ABdhPJxLAKZpTcdUX9zsbSPT39Xp2a3eU0dgLLZ3ehz2f2hFJv3v5S8CmNO4M+fKKNRAi5NErhBVvg== X-Received: by 2002:a1c:7c0d:0:b0:37d:1e29:5cc6 with SMTP id x13-20020a1c7c0d000000b0037d1e295cc6mr5162767wmc.31.1645089725110; Thu, 17 Feb 2022 01:22:05 -0800 (PST) Received: from ?IPV6:2a01:e0a:181:90d0:6ce1:d6aa:6a23:353b? ([2a01:e0a:181:90d0:6ce1:d6aa:6a23:353b]) by smtp.gmail.com with ESMTPSA id f18sm1184725wre.66.2022.02.17.01.22.04 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 17 Feb 2022 01:22:04 -0800 (PST) Message-ID: Date: Thu, 17 Feb 2022 10:22:03 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.6.0 Subject: Re: [PATCH 2 9/9] serial: atmel: remove redundant assignment in rs485_config Content-Language: fr-FR To: Lino Sanfilippo , gregkh@linuxfoundation.org, jirislaby@kernel.org, u.kleine-koenig@pengutronix.de Cc: linux@armlinux.org.uk, richard.genoud@gmail.com, nicolas.ferre@microchip.com, alexandre.belloni@bootlin.com, ludovic.desroches@microchip.com, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com, linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, lukas@wunner.de References: <20220216001803.637-1-LinoSanfilippo@gmx.de> <20220216001803.637-10-LinoSanfilippo@gmx.de> From: Richard Genoud In-Reply-To: <20220216001803.637-10-LinoSanfilippo@gmx.de> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-serial@vger.kernel.org Le 16/02/2022 à 01:18, Lino Sanfilippo a écrit : > In uart_set_rs485_config() the serial core already assigns the passed > serial_rs485 struct to the uart port. > > So remove the assignment from the drivers rs485_config() function to avoid > redundancy. > > Signed-off-by: Lino Sanfilippo Acked-by: Richard Genoud > --- > drivers/tty/serial/atmel_serial.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/tty/serial/atmel_serial.c b/drivers/tty/serial/atmel_serial.c > index 2d09a89974a2..2ab589a3d86c 100644 > --- a/drivers/tty/serial/atmel_serial.c > +++ b/drivers/tty/serial/atmel_serial.c > @@ -299,11 +299,9 @@ static int atmel_config_rs485(struct uart_port *port, > /* Resetting serial mode to RS232 (0x0) */ > mode &= ~ATMEL_US_USMODE; > > - port->rs485 = *rs485conf; > - > if (rs485conf->flags & SER_RS485_ENABLED) { > dev_dbg(port->dev, "Setting UART to RS485\n"); > - if (port->rs485.flags & SER_RS485_RX_DURING_TX) > + if (rs485conf->flags & SER_RS485_RX_DURING_TX) > atmel_port->tx_done_mask = ATMEL_US_TXRDY; > else > atmel_port->tx_done_mask = ATMEL_US_TXEMPTY; Thanks ! From mboxrd@z Thu Jan 1 00:00:00 1970 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id EDBAAC433F5 for ; Thu, 17 Feb 2022 09:23:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:To:Subject:MIME-Version: Date:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=NwAw7aVfn/Xi8iBDpeN7Kr/4snRZeso/b8K1hLoL/BM=; b=YD1Fn3XNGp8R0kTDsEEut31ehr W61/yRQILmpDSsU7kEIpfZyc/+SyMFxRn2yu8rE5eFkqn2vPkVGoD62RRL89snTYro8WKco68Fnqh LcrBaWrH85Cq/RzX0UkSVaMxzvaNp7PkMRqoPfB8RAdGkyvu9oIljhB/2b5c8ln5GqIixPdlanSBD vx0W/g8vzU9plLM7Ld6/s2w9enBHTmXSB5IB7JPCNMXvNzk1gS4ecbQ4DtMIeCmNYrvuhY71IdfLo kwiD1fFKgyUQoMoxLIXNv0lRrbSY8vncjbgdkAzDYpuaV2U52dVDZpQUnqZ51ZlVbCrgybINuhQz0 rmXXgUhA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nKczL-009hzB-Fl; Thu, 17 Feb 2022 09:22:11 +0000 Received: from mail-wm1-x334.google.com ([2a00:1450:4864:20::334]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nKczH-009hxj-Ig for linux-arm-kernel@lists.infradead.org; Thu, 17 Feb 2022 09:22:09 +0000 Received: by mail-wm1-x334.google.com with SMTP id k3-20020a1ca103000000b0037bdea84f9cso3539253wme.1 for ; Thu, 17 Feb 2022 01:22:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=uEF/FU/QB3+JPW9i0x2lAGeqkeJMKwTkLaDfp85o4eA=; b=DY+MxSUAazGgw0yYDJltTUdAkc4UgtVq4+ctUknKf21YBPaMDiGfUnVX7BHtJJuqHu 1SYdMuGzCw+h4q8+FwO+M7XjGOkzjXPQXhTwf+I9PpvoHSRBA7ehI1K3BugPXbedrc1p aVCOobSkjGFu+kR91na4ZDELFbcmv+HlgA1AhukRVY4kzK+XIK2j+TyqNlRN/gd+Of8X UoYHrOua6xlip9UDXmcWaMftQmZ4uA2bIKAOPpCj3yS/X50MySCDwh/RP6Pxma3ui19H exH1l/GR9ZT+PrEB9p+ai3Fvt3S68xBiawm9KjnFnYyGYdVy02sqs/MBti/L5jSXVGY9 Dylw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=uEF/FU/QB3+JPW9i0x2lAGeqkeJMKwTkLaDfp85o4eA=; b=SFIFT0oO7slrX02mC4VhvGymcRo1/tY+WHFCfsOJtcFHyhgSFkuhPwewIqboQccBe0 uB1wjhwVRtHA9hBmY9HevKsZKv9IRbyKOZ50qF8iwBe8MScv/HY8pXz6Lwm/XhVpF4hu kavh/KtCABl2Sb2OM/ruhewsBC1e12qYp+oXrMQYWkAvCQsVrqKg/OOTUNwxsQF0gaw6 A1PfiqE7n1C7qVHqEbxZz3NM4mCrbr50cLUbjaa6xMMPPgTKhraeyS8V6qBmdE8Myw8C 6nth/cc38gkzZtvhNj1ZHSh1jm+DQHDSxSLlkVRvdq21XFCEamwagIfc41/+LC4sZIby F3GA== X-Gm-Message-State: AOAM530P+QAKJuAlgxLpBtCogW/o89GLRrcUnUIrWxr2QmxWdbavxFss WXJTHqbkZ5f3bDat36lpmPY= X-Google-Smtp-Source: ABdhPJxLAKZpTcdUX9zsbSPT39Xp2a3eU0dgLLZ3ehz2f2hFJv3v5S8CmNO4M+fKKNRAi5NErhBVvg== X-Received: by 2002:a1c:7c0d:0:b0:37d:1e29:5cc6 with SMTP id x13-20020a1c7c0d000000b0037d1e295cc6mr5162767wmc.31.1645089725110; Thu, 17 Feb 2022 01:22:05 -0800 (PST) Received: from ?IPV6:2a01:e0a:181:90d0:6ce1:d6aa:6a23:353b? ([2a01:e0a:181:90d0:6ce1:d6aa:6a23:353b]) by smtp.gmail.com with ESMTPSA id f18sm1184725wre.66.2022.02.17.01.22.04 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 17 Feb 2022 01:22:04 -0800 (PST) Message-ID: Date: Thu, 17 Feb 2022 10:22:03 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.6.0 Subject: Re: [PATCH 2 9/9] serial: atmel: remove redundant assignment in rs485_config Content-Language: fr-FR To: Lino Sanfilippo , gregkh@linuxfoundation.org, jirislaby@kernel.org, u.kleine-koenig@pengutronix.de References: <20220216001803.637-1-LinoSanfilippo@gmx.de> <20220216001803.637-10-LinoSanfilippo@gmx.de> From: Richard Genoud In-Reply-To: <20220216001803.637-10-LinoSanfilippo@gmx.de> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220217_012207_683591_21EC3E83 X-CRM114-Status: GOOD ( 15.29 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-arm-kernel@lists.infradead.org, alexandre.belloni@bootlin.com, mcoquelin.stm32@gmail.com, richard.genoud@gmail.com, festevam@gmail.com, s.hauer@pengutronix.de, linux@armlinux.org.uk, alexandre.torgue@foss.st.com, ludovic.desroches@microchip.com, lukas@wunner.de, linux-imx@nxp.com, kernel@pengutronix.de, linux-serial@vger.kernel.org, shawnguo@kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-kernel@vger.kernel.org Content-Transfer-Encoding: base64 Content-Type: text/plain; charset="utf-8"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org CkxlIDE2LzAyLzIwMjIgw6AgMDE6MTgsIExpbm8gU2FuZmlsaXBwbyBhIMOpY3JpdMKgOgo+IElu IHVhcnRfc2V0X3JzNDg1X2NvbmZpZygpIHRoZSBzZXJpYWwgY29yZSBhbHJlYWR5IGFzc2lnbnMg dGhlIHBhc3NlZAo+IHNlcmlhbF9yczQ4NSBzdHJ1Y3QgdG8gdGhlIHVhcnQgcG9ydC4KPiAKPiBT byByZW1vdmUgdGhlIGFzc2lnbm1lbnQgZnJvbSB0aGUgZHJpdmVycyByczQ4NV9jb25maWcoKSBm dW5jdGlvbiB0byBhdm9pZAo+IHJlZHVuZGFuY3kuCj4gCj4gU2lnbmVkLW9mZi1ieTogTGlubyBT YW5maWxpcHBvIDxMaW5vU2FuZmlsaXBwb0BnbXguZGU+CkFja2VkLWJ5OiBSaWNoYXJkIEdlbm91 ZCA8cmljaGFyZC5nZW5vdWRAZ21haWwuY29tPgoKPiAtLS0KPiAgIGRyaXZlcnMvdHR5L3Nlcmlh bC9hdG1lbF9zZXJpYWwuYyB8IDQgKy0tLQo+ICAgMSBmaWxlIGNoYW5nZWQsIDEgaW5zZXJ0aW9u KCspLCAzIGRlbGV0aW9ucygtKQo+IAo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL3R0eS9zZXJpYWwv YXRtZWxfc2VyaWFsLmMgYi9kcml2ZXJzL3R0eS9zZXJpYWwvYXRtZWxfc2VyaWFsLmMKPiBpbmRl eCAyZDA5YTg5OTc0YTIuLjJhYjU4OWEzZDg2YyAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL3R0eS9z ZXJpYWwvYXRtZWxfc2VyaWFsLmMKPiArKysgYi9kcml2ZXJzL3R0eS9zZXJpYWwvYXRtZWxfc2Vy aWFsLmMKPiBAQCAtMjk5LDExICsyOTksOSBAQCBzdGF0aWMgaW50IGF0bWVsX2NvbmZpZ19yczQ4 NShzdHJ1Y3QgdWFydF9wb3J0ICpwb3J0LAo+ICAgCS8qIFJlc2V0dGluZyBzZXJpYWwgbW9kZSB0 byBSUzIzMiAoMHgwKSAqLwo+ICAgCW1vZGUgJj0gfkFUTUVMX1VTX1VTTU9ERTsKPiAgIAo+IC0J cG9ydC0+cnM0ODUgPSAqcnM0ODVjb25mOwo+IC0KPiAgIAlpZiAocnM0ODVjb25mLT5mbGFncyAm IFNFUl9SUzQ4NV9FTkFCTEVEKSB7Cj4gICAJCWRldl9kYmcocG9ydC0+ZGV2LCAiU2V0dGluZyBV QVJUIHRvIFJTNDg1XG4iKTsKPiAtCQlpZiAocG9ydC0+cnM0ODUuZmxhZ3MgJiBTRVJfUlM0ODVf UlhfRFVSSU5HX1RYKQo+ICsJCWlmIChyczQ4NWNvbmYtPmZsYWdzICYgU0VSX1JTNDg1X1JYX0RV UklOR19UWCkKPiAgIAkJCWF0bWVsX3BvcnQtPnR4X2RvbmVfbWFzayA9IEFUTUVMX1VTX1RYUkRZ Owo+ICAgCQllbHNlCj4gICAJCQlhdG1lbF9wb3J0LT50eF9kb25lX21hc2sgPSBBVE1FTF9VU19U WEVNUFRZOwoKVGhhbmtzICEKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxA bGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xp c3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK