From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751415AbdJEQCI (ORCPT ); Thu, 5 Oct 2017 12:02:08 -0400 Received: from mx07-00178001.pphosted.com ([62.209.51.94]:15901 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751144AbdJEQCG (ORCPT ); Thu, 5 Oct 2017 12:02:06 -0400 Subject: Re: [PATCH 0/3] Arm: dts: stm32: remove extra compatible uart string To: Vikas Manocha , CC: References: <1506639114-16913-1-git-send-email-vikas.manocha@st.com> From: Alexandre Torgue Message-ID: <71b2711c-40cd-d09f-a879-218f688a3a51@st.com> Date: Thu, 5 Oct 2017 18:01:48 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <1506639114-16913-1-git-send-email-vikas.manocha@st.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.75.127.46] X-ClientProxiedBy: SFHDAG8NODE2.st.com (10.75.127.23) To SFHDAG3NODE2.st.com (10.75.127.8) X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-10-05_07:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 think you should split patches differently. Maybe one for drivers and bindings (doc) updates. And others for DT updates. Conerning DT patches, all headers should start by "ARM: dts: stm32: blabla" Regards Alex