From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753035AbdJKOi5 (ORCPT ); Wed, 11 Oct 2017 10:38:57 -0400 Received: from mx08-00178001.pphosted.com ([91.207.212.93]:52786 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751712AbdJKOiy (ORCPT ); Wed, 11 Oct 2017 10:38:54 -0400 Subject: Re: [PATCHv2 0/7] Add support for USB OTG on STM32F7 To: Felipe Balbi , Amelie Delaunay , Greg Kroah-Hartman , Rob Herring , Mark Rutland , Russell King , Maxime Coquelin , John Youn CC: , , , , Benjamin Gaignard References: <1503930018-536-1-git-send-email-amelie.delaunay@st.com> <878tgim24i.fsf@linux.intel.com> <04a49088-b615-d7b8-0387-e7bfa32ee19f@st.com> <87vajllx8t.fsf@linux.intel.com> <1c4180ef-2da6-b420-7c6e-41769608384a@st.com> <87sheplsrm.fsf@linux.intel.com> From: Alexandre Torgue Message-ID: <3c36db6d-75fd-1be6-e946-b75a04e3760e@st.com> Date: Wed, 11 Oct 2017 16:38:08 +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: <87sheplsrm.fsf@linux.intel.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: SFHDAG5NODE1.st.com (10.75.127.13) To SFHDAG3NODE2.st.com (10.75.127.8) X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-10-11_05:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/11/2017 03:26 PM, Felipe Balbi wrote: > > Hi, > > Alexandre Torgue writes: >> Hi >> >> On 10/11/2017 01:50 PM, Felipe Balbi wrote: >>> >>> Hi, >>> >>> Alexandre Torgue writes: >>>> Hi Felip >>>> >>>> On 10/11/2017 12:04 PM, Felipe Balbi wrote: >>>>> >>>>> Hi, >>>>> >>>>> Amelie Delaunay writes: >>>>>> The STM32F7 MCU family embeds two DWC2 USB OTG cores. One core is USB >>>>>> OTG FS and the other is USB OTG HS. The USB FS core only works with its >>>>>> internal phy whilst the USB HS core can work in HS with external ULPI phy >>>>>> or in FS/LS with the on-chip FS phy. >>>>>> >>>>>> Amelie Delaunay (7): >>>>>> dt-bindings: usb: Document the STM32F7 DWC2 USB OTG HS core binding >>>>>> usb: dwc2: add support for STM32F7 USB OTG HS >>>>>> ARM: dts: stm32: Add USB HS support for STM32F746 MCU >>>>> >>>>> I have applied these three patches. Should I take the rest? They seems >>>>> like they could go upstream through the ARM maintainers. >>>>> >>>> >>>> I will take other DT patches in my PR. >>>> >>>> Concerning "ARM: dts: stm32: Add USB HS support for STM32F746 MCU" patch >>>> I prefer also to take it. This patch adds some pinctrl groups but stm32 >>>> pinctrl bindings will change in my next PR (we will use a macro to >>>> define pins instead of using defined values). So if you push the DT >>>> patch through your pull request there will be a merge issue. >>>> It is possible that I take also this one ? >>> >>> In that case, it's best if you take them all :-) Here's my Ack: >>> >>> Acked-by: Felipe Balbi >>> >>> I'll drop them from my tree now >> >> Ok perfect, I will take DT patches (3 to 7) and I let you take patch 1&2 >> in your tree. > > Well, I have dropped them from my tree. Please two 1-7 through yours. Hum, ok for this patchset but IMO it is better (next time) that you take driver pacthes in your tree and I take only DT patches in mine. No ? Regards Alex >