From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751532AbdJERga (ORCPT ); Thu, 5 Oct 2017 13:36:30 -0400 Received: from mx07-00178001.pphosted.com ([62.209.51.94]:8626 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751286AbdJERg3 (ORCPT ); Thu, 5 Oct 2017 13:36:29 -0400 From: Vikas MANOCHA To: Alexandre TORGUE , "linux-kernel@vger.kernel.org" CC: Patrice CHOTARD Subject: RE: [PATCH 0/3] Arm: dts: stm32: remove extra compatible uart string Thread-Topic: [PATCH 0/3] Arm: dts: stm32: remove extra compatible uart string Thread-Index: AQHTOK/DEalJiwRjsUupd+T2D1PVPqLVU5gAgAA53LA= Date: Thu, 5 Oct 2017 17:36:26 +0000 Message-ID: <75c4d593a1b1402b9afafb5ee646c1a9@SFHDAG7NODE3.st.com> References: <1506639114-16913-1-git-send-email-vikas.manocha@st.com> <71b2711c-40cd-d09f-a879-218f688a3a51@st.com> In-Reply-To: <71b2711c-40cd-d09f-a879-218f688a3a51@st.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.50] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-10-05_08:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id v95HaYhN011148 Hi Alex, > -----Original Message----- > From: Alexandre TORGUE > Sent: Thursday, October 05, 2017 9:02 AM > To: Vikas MANOCHA ; linux-kernel@vger.kernel.org > Cc: Patrice CHOTARD > Subject: Re: [PATCH 0/3] Arm: dts: stm32: remove extra compatible uart string > > Hi Vikas > > > On 09/29/2017 12:51 AM, Vikas Manocha wrote: > > stm32 uart driver is using two compatible strings "st,stm32-usart" > > & "st,stm32-uart". One can be removed safely to save some space & time. > > > > Vikas Manocha (3): > > Arm: dts: stm32: remove extra compatible string for uart > > Arm: dts: stm32: remove extra compatible string from DT & driver > > ARM: dts: stm32h7: correct uart nodes compatible string > > > > Documentation/devicetree/bindings/dma/stm32-dma.txt | 2 +- > > Documentation/devicetree/bindings/serial/st,stm32-usart.txt | 10 +++------- > > arch/arm/boot/dts/stm32f429.dtsi | 12 ++++++------ > > arch/arm/boot/dts/stm32f746.dtsi | 12 ++++++------ > > arch/arm/boot/dts/stm32h743.dtsi | 4 ++-- > > drivers/tty/serial/stm32-usart.c | 3 --- > > 6 files changed, 18 insertions(+), 25 deletions(-) > > > > I think it is better to put people in CC of patch 1, 2, 3 also in CC of cover-letter. I am not sure how to add relevant people in cover-letter, Get_maintainers script works for the patches only. > I think you should split patches differently. Maybe one > for drivers and bindings (doc) updates. And others for DT updates. Yes, it's like this only. First patch is DT binding & second is its implementation in the driver & stm32 device tree. > > Conerning DT patches, all headers should start by "ARM: dts: stm32: blabla" Yes, Same headers is used for the patch. You mean "arm" should be "ARM" ? Cheers, Vikas > > Regards > Alex