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 X-Spam-Level: X-Spam-Status: No, score=-7.1 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 844BBC282CE for ; Tue, 4 Jun 2019 13:55:59 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 5AEDD23A6C for ; Tue, 4 Jun 2019 13:55:59 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=st.com header.i=@st.com header.b="rv9CHvkX" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727493AbfFDNz6 (ORCPT ); Tue, 4 Jun 2019 09:55:58 -0400 Received: from mx08-00178001.pphosted.com ([91.207.212.93]:38892 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727033AbfFDNz6 (ORCPT ); Tue, 4 Jun 2019 09:55:58 -0400 Received: from pps.filterd (m0046661.ppops.net [127.0.0.1]) by mx08-00178001.pphosted.com (8.16.0.27/8.16.0.27) with SMTP id x54DlLtO019052; Tue, 4 Jun 2019 15:55:44 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=st.com; h=from : to : subject : date : message-id : references : in-reply-to : content-type : content-id : content-transfer-encoding : mime-version; s=STMicroelectronics; bh=/l+I9/VrslMxF9koCf1mice5DTAFdPweFnW/HGgh+yI=; b=rv9CHvkXyrmpcAoDio1Pfc/MNx3qZ51DB0Q5F3hkUzvJKrd3xH1K+w4/qRbYM0SylD6d xN76Ltc2s9LmsGqKNES984we0hKcBkDGCGwoPd8aQCyUgW9Xfbo7N8FtEEItSeX1vNbz z/6HWuw55C/2E+LnHwa97k/59GASjfVM3WrEkh+D0scfNsIKRU0A1vhPdJvj4FJ0Mhnj F/kRxHd8fdO0TWjewwgHhWcUcMgyoJiDwKq8p5TVwPQDmw9ryk8/HjjzTfqtuJgfL4LZ N2CdQImk8Qx2n2ALqAFfLNMkD3E8jLRpDiFy1Es+aejds6vvLamHuljewox7UD+SLEkU AA== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx08-00178001.pphosted.com with ESMTP id 2sunds184k-1 (version=TLSv1 cipher=ECDHE-RSA-AES256-SHA bits=256 verify=NOT); Tue, 04 Jun 2019 15:55:44 +0200 Received: from zeta.dmz-eu.st.com (zeta.dmz-eu.st.com [164.129.230.9]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 88BD738; Tue, 4 Jun 2019 13:55:43 +0000 (GMT) Received: from Webmail-eu.st.com (sfhdag3node3.st.com [10.75.127.9]) by zeta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 666A12B3B; Tue, 4 Jun 2019 13:55:43 +0000 (GMT) Received: from SFHDAG3NODE1.st.com (10.75.127.7) by SFHDAG3NODE3.st.com (10.75.127.9) with Microsoft SMTP Server (TLS) id 15.0.1347.2; Tue, 4 Jun 2019 15:55:43 +0200 Received: from SFHDAG3NODE1.st.com ([fe80::1166:1abb:aad4:5f86]) by SFHDAG3NODE1.st.com ([fe80::1166:1abb:aad4:5f86%20]) with mapi id 15.00.1347.000; Tue, 4 Jun 2019 15:55:43 +0200 From: Erwan LE RAY To: Borut Seljak , Maxime Coquelin , Alexandre TORGUE , "Greg Kroah-Hartman" , "linux-kernel@vger.kernel.org" , "linux-serial@vger.kernel.org" , Jiri Slaby , "linux-stm32@st-md-mailman.stormreply.com" , "linux-arm-kernel@lists.infradead.org" Subject: Re: Fwd: [PATCH] serial: stm32: fix a recursive locking in stm32_config_rs485 Thread-Topic: Fwd: [PATCH] serial: stm32: fix a recursive locking in stm32_config_rs485 Thread-Index: AQHVGr0egfGBr+I5bUuvhxEmcTN9eaaLX30AgAAD5wA= Date: Tue, 4 Jun 2019 13:55:42 +0000 Message-ID: <33271a7e-644b-70e3-f84c-d019b394ce77@st.com> References: <20190604095452.6360-1-borut.seljak@t-2.net> <41dddd5f-5c1c-3346-890a-8018f26ebd49@st.com> In-Reply-To: <41dddd5f-5c1c-3346-890a-8018f26ebd49@st.com> Accept-Language: en-US, fr-FR Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.46] Content-Type: text/plain; charset="Windows-1252" Content-ID: Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:,, definitions=2019-06-04_09:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > Hi Borut, > > Please add the following line in the commit message (before your=20 > sign-off) in a V2 of your patch: > > fixes: 1bcda09d291081 ("serial: stm32: add support for RS485 hardware=20 > control mode") > > I'm OK with the patch itself. > > Erwan. > > > Subject: [PATCH] serial: stm32: fix a recursive locking in >> stm32_config_rs485 >> Date: Tue,=A0 4 Jun 2019 11:54:51 +0200 >> From: Borut Seljak >> CC: Maxime Coquelin , Alexandre Torgue >> , Greg Kroah-Hartman >> , linux-kernel@vger.kernel.org, >> borut.seljak@t-2.net, linux-serial@vger.kernel.org, Jiri Slaby >> , linux-stm32@st-md-mailman.stormreply.com, >> linux-arm-kernel@lists.infradead.org >> >> Remove spin_lock_irqsave in stm32_config_rs485, it cause recursive=20 >> locking. >> Already locked in uart_set_rs485_config. >> >> Signed-off-by: Borut Seljak >> --- >> =A0 drivers/tty/serial/stm32-usart.c | 2 -- >> =A0 1 file changed, 2 deletions(-) >> >> diff --git a/drivers/tty/serial/stm32-usart.c >> b/drivers/tty/serial/stm32-usart.c >> index e8d7a7bb4339..da373a465f51 100644 >> --- a/drivers/tty/serial/stm32-usart.c >> +++ b/drivers/tty/serial/stm32-usart.c >> @@ -107,7 +107,6 @@ static int stm32_config_rs485(struct uart_port=20 >> *port, >> =A0=A0=A0=A0=A0 bool over8; >> =A0=A0=A0=A0=A0 unsigned long flags; >> =A0 -=A0=A0=A0 spin_lock_irqsave(&port->lock, flags); >> =A0=A0=A0=A0=A0 stm32_clr_bits(port, ofs->cr1, BIT(cfg->uart_enable_bit)= ); >> =A0=A0=A0=A0=A0=A0 port->rs485 =3D *rs485conf; >> @@ -147,7 +146,6 @@ static int stm32_config_rs485(struct uart_port=20 >> *port, >> =A0=A0=A0=A0=A0 } >> =A0=A0=A0=A0=A0=A0 stm32_set_bits(port, ofs->cr1, BIT(cfg->uart_enable_b= it)); >> -=A0=A0=A0 spin_unlock_irqrestore(&port->lock, flags); >> =A0=A0=A0=A0=A0=A0 return 0; >> =A0 }= From mboxrd@z Thu Jan 1 00:00:00 1970 From: Erwan LE RAY Subject: Re: Fwd: [PATCH] serial: stm32: fix a recursive locking in stm32_config_rs485 Date: Tue, 4 Jun 2019 13:55:42 +0000 Message-ID: <33271a7e-644b-70e3-f84c-d019b394ce77@st.com> References: <20190604095452.6360-1-borut.seljak@t-2.net> <41dddd5f-5c1c-3346-890a-8018f26ebd49@st.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <41dddd5f-5c1c-3346-890a-8018f26ebd49@st.com> Content-Language: en-US Content-ID: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Borut Seljak , Maxime Coquelin , Alexandre TORGUE , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , "linux-serial@vger.kernel.org" , Jiri Slaby , "linux-stm32@st-md-mailman.stormreply.com" , "linux-arm-kernel@lists.infradead.org" List-Id: linux-serial@vger.kernel.org > Hi Borut, > > Please add the following line in the commit message (before your = > sign-off) in a V2 of your patch: > > fixes: 1bcda09d291081 ("serial: stm32: add support for RS485 hardware = > control mode") > > I'm OK with the patch itself. > > Erwan. > > > Subject: [PATCH] serial: stm32: fix a recursive locking in >> stm32_config_rs485 >> Date: Tue,=A0 4 Jun 2019 11:54:51 +0200 >> From: Borut Seljak >> CC: Maxime Coquelin , Alexandre Torgue >> , Greg Kroah-Hartman >> , linux-kernel@vger.kernel.org, >> borut.seljak@t-2.net, linux-serial@vger.kernel.org, Jiri Slaby >> , linux-stm32@st-md-mailman.stormreply.com, >> linux-arm-kernel@lists.infradead.org >> >> Remove spin_lock_irqsave in stm32_config_rs485, it cause recursive = >> locking. >> Already locked in uart_set_rs485_config. >> >> Signed-off-by: Borut Seljak >> --- >> =A0 drivers/tty/serial/stm32-usart.c | 2 -- >> =A0 1 file changed, 2 deletions(-) >> >> diff --git a/drivers/tty/serial/stm32-usart.c >> b/drivers/tty/serial/stm32-usart.c >> index e8d7a7bb4339..da373a465f51 100644 >> --- a/drivers/tty/serial/stm32-usart.c >> +++ b/drivers/tty/serial/stm32-usart.c >> @@ -107,7 +107,6 @@ static int stm32_config_rs485(struct uart_port = >> *port, >> =A0=A0=A0=A0=A0 bool over8; >> =A0=A0=A0=A0=A0 unsigned long flags; >> =A0 -=A0=A0=A0 spin_lock_irqsave(&port->lock, flags); >> =A0=A0=A0=A0=A0 stm32_clr_bits(port, ofs->cr1, BIT(cfg->uart_enable_bit)= ); >> =A0=A0=A0=A0=A0=A0 port->rs485 =3D *rs485conf; >> @@ -147,7 +146,6 @@ static int stm32_config_rs485(struct uart_port = >> *port, >> =A0=A0=A0=A0=A0 } >> =A0=A0=A0=A0=A0=A0 stm32_set_bits(port, ofs->cr1, BIT(cfg->uart_enable_b= it)); >> -=A0=A0=A0 spin_unlock_irqrestore(&port->lock, flags); >> =A0=A0=A0=A0=A0=A0 return 0; >> =A0 } 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 X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,T_DKIMWL_WL_HIGH,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id EAF89C282CE for ; Tue, 4 Jun 2019 13:55:52 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id BEE1123A6C for ; Tue, 4 Jun 2019 13:55:52 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="ePQfQPcc"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=st.com header.i=@st.com header.b="rv9CHvkX" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org BEE1123A6C Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=st.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To: References:Message-ID:Date:Subject:To:From:Reply-To:Cc:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=j5ZOyX0yPoa4YijJx1ryXaZ8u2g1UGYTkycgCK5TY3Y=; b=ePQfQPccG2rfV0 NTZGKBnfZY9kjH4y1lgAyOsXaMgCLD0uBM91Zeta23CQzB5NeFA7zzAs9mEXqnTzDC06IYR4DPTOZ BwYRyHb9zkS350uS8+rp0rs4ZOtCsFnW2JVRB4o84yZra/7JGfMNy8e/1WZX8Wrg3l+q2Rb6e54xZ w+b04tFBOWRkDBghgttdNBzDNFB0kHQvK/wvK2Zsco2exHy4kAnSf8GeIIwosKlNscObK5pPNTb2M zTMizgLCG5FpqTgMfx62R1v8Jf2YV6sG384YPRx0xh1AX3J7Smac0akRndykEcxakX0RsC6ev/az8 /PS1pxumgW/uqsBGgk6Q==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hY9up-00014R-54; Tue, 04 Jun 2019 13:55:51 +0000 Received: from mx08-00178001.pphosted.com ([91.207.212.93] helo=mx07-00178001.pphosted.com) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hY9ul-00013r-IM for linux-arm-kernel@lists.infradead.org; Tue, 04 Jun 2019 13:55:49 +0000 Received: from pps.filterd (m0046661.ppops.net [127.0.0.1]) by mx08-00178001.pphosted.com (8.16.0.27/8.16.0.27) with SMTP id x54DlLtO019052; Tue, 4 Jun 2019 15:55:44 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=st.com; h=from : to : subject : date : message-id : references : in-reply-to : content-type : content-id : content-transfer-encoding : mime-version; s=STMicroelectronics; bh=/l+I9/VrslMxF9koCf1mice5DTAFdPweFnW/HGgh+yI=; b=rv9CHvkXyrmpcAoDio1Pfc/MNx3qZ51DB0Q5F3hkUzvJKrd3xH1K+w4/qRbYM0SylD6d xN76Ltc2s9LmsGqKNES984we0hKcBkDGCGwoPd8aQCyUgW9Xfbo7N8FtEEItSeX1vNbz z/6HWuw55C/2E+LnHwa97k/59GASjfVM3WrEkh+D0scfNsIKRU0A1vhPdJvj4FJ0Mhnj F/kRxHd8fdO0TWjewwgHhWcUcMgyoJiDwKq8p5TVwPQDmw9ryk8/HjjzTfqtuJgfL4LZ N2CdQImk8Qx2n2ALqAFfLNMkD3E8jLRpDiFy1Es+aejds6vvLamHuljewox7UD+SLEkU AA== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx08-00178001.pphosted.com with ESMTP id 2sunds184k-1 (version=TLSv1 cipher=ECDHE-RSA-AES256-SHA bits=256 verify=NOT); Tue, 04 Jun 2019 15:55:44 +0200 Received: from zeta.dmz-eu.st.com (zeta.dmz-eu.st.com [164.129.230.9]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 88BD738; Tue, 4 Jun 2019 13:55:43 +0000 (GMT) Received: from Webmail-eu.st.com (sfhdag3node3.st.com [10.75.127.9]) by zeta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 666A12B3B; Tue, 4 Jun 2019 13:55:43 +0000 (GMT) Received: from SFHDAG3NODE1.st.com (10.75.127.7) by SFHDAG3NODE3.st.com (10.75.127.9) with Microsoft SMTP Server (TLS) id 15.0.1347.2; Tue, 4 Jun 2019 15:55:43 +0200 Received: from SFHDAG3NODE1.st.com ([fe80::1166:1abb:aad4:5f86]) by SFHDAG3NODE1.st.com ([fe80::1166:1abb:aad4:5f86%20]) with mapi id 15.00.1347.000; Tue, 4 Jun 2019 15:55:43 +0200 From: Erwan LE RAY To: Borut Seljak , Maxime Coquelin , Alexandre TORGUE , "Greg Kroah-Hartman" , "linux-kernel@vger.kernel.org" , "linux-serial@vger.kernel.org" , Jiri Slaby , "linux-stm32@st-md-mailman.stormreply.com" , "linux-arm-kernel@lists.infradead.org" Subject: Re: Fwd: [PATCH] serial: stm32: fix a recursive locking in stm32_config_rs485 Thread-Topic: Fwd: [PATCH] serial: stm32: fix a recursive locking in stm32_config_rs485 Thread-Index: AQHVGr0egfGBr+I5bUuvhxEmcTN9eaaLX30AgAAD5wA= Date: Tue, 4 Jun 2019 13:55:42 +0000 Message-ID: <33271a7e-644b-70e3-f84c-d019b394ce77@st.com> References: <20190604095452.6360-1-borut.seljak@t-2.net> <41dddd5f-5c1c-3346-890a-8018f26ebd49@st.com> In-Reply-To: <41dddd5f-5c1c-3346-890a-8018f26ebd49@st.com> Accept-Language: en-US, fr-FR Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.46] Content-ID: MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:, , definitions=2019-06-04_09:, , signatures=0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190604_065547_887659_F4BF5E15 X-CRM114-Status: GOOD ( 16.69 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org > Hi Borut, > > Please add the following line in the commit message (before your = > sign-off) in a V2 of your patch: > > fixes: 1bcda09d291081 ("serial: stm32: add support for RS485 hardware = > control mode") > > I'm OK with the patch itself. > > Erwan. > > > Subject: [PATCH] serial: stm32: fix a recursive locking in >> stm32_config_rs485 >> Date: Tue,=A0 4 Jun 2019 11:54:51 +0200 >> From: Borut Seljak >> CC: Maxime Coquelin , Alexandre Torgue >> , Greg Kroah-Hartman >> , linux-kernel@vger.kernel.org, >> borut.seljak@t-2.net, linux-serial@vger.kernel.org, Jiri Slaby >> , linux-stm32@st-md-mailman.stormreply.com, >> linux-arm-kernel@lists.infradead.org >> >> Remove spin_lock_irqsave in stm32_config_rs485, it cause recursive = >> locking. >> Already locked in uart_set_rs485_config. >> >> Signed-off-by: Borut Seljak >> --- >> =A0 drivers/tty/serial/stm32-usart.c | 2 -- >> =A0 1 file changed, 2 deletions(-) >> >> diff --git a/drivers/tty/serial/stm32-usart.c >> b/drivers/tty/serial/stm32-usart.c >> index e8d7a7bb4339..da373a465f51 100644 >> --- a/drivers/tty/serial/stm32-usart.c >> +++ b/drivers/tty/serial/stm32-usart.c >> @@ -107,7 +107,6 @@ static int stm32_config_rs485(struct uart_port = >> *port, >> =A0=A0=A0=A0=A0 bool over8; >> =A0=A0=A0=A0=A0 unsigned long flags; >> =A0 -=A0=A0=A0 spin_lock_irqsave(&port->lock, flags); >> =A0=A0=A0=A0=A0 stm32_clr_bits(port, ofs->cr1, BIT(cfg->uart_enable_bit)= ); >> =A0=A0=A0=A0=A0=A0 port->rs485 =3D *rs485conf; >> @@ -147,7 +146,6 @@ static int stm32_config_rs485(struct uart_port = >> *port, >> =A0=A0=A0=A0=A0 } >> =A0=A0=A0=A0=A0=A0 stm32_set_bits(port, ofs->cr1, BIT(cfg->uart_enable_b= it)); >> -=A0=A0=A0 spin_unlock_irqrestore(&port->lock, flags); >> =A0=A0=A0=A0=A0=A0 return 0; >> =A0 } _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel