From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754175AbcEYLTv (ORCPT ); Wed, 25 May 2016 07:19:51 -0400 Received: from bear.ext.ti.com ([198.47.19.11]:35403 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751638AbcEYLTt (ORCPT ); Wed, 25 May 2016 07:19:49 -0400 Subject: Re: [PATCH v5 0/5] da8xx USB PHY platform devices and clocks (was "da8xx UBS clocks") To: David Lechner , References: <1462837612-7803-1-git-send-email-david@lechnology.com> <57431E73.5030304@lechnology.com> CC: , , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Kishon Vijay Abraham I , Greg Kroah-Hartman , Alan Stern , Bin Liu , Lee Jones , , , From: Sekhar Nori Message-ID: <57458A0A.5010100@ti.com> Date: Wed, 25 May 2016 16:48:34 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-Version: 1.0 In-Reply-To: <57431E73.5030304@lechnology.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Monday 23 May 2016 08:44 PM, David Lechner wrote: > On 05/09/2016 06:46 PM, David Lechner wrote: >> v5 changes: renamed "usbphy" to "usb_phy" or "usb-phy" as appropriate >> >> David Lechner (5): >> ARM: davinci: da8xx: add usb phy clocks >> ARM: davinci: da8xx: Add CFGCHIP syscon platform declaration. >> ARM: davinci: da8xx: Add USB PHY platform declaration >> ARM: DTS: da850: Add cfgchip syscon node >> ARM: DTS: da850: Add usb phy node >> >> arch/arm/boot/dts/da850.dtsi | 9 ++ >> arch/arm/mach-davinci/board-da830-evm.c | 52 +++---- >> arch/arm/mach-davinci/board-da850-evm.c | 4 + >> arch/arm/mach-davinci/board-mityomapl138.c | 4 + >> arch/arm/mach-davinci/board-omapl138-hawk.c | 23 ++- >> arch/arm/mach-davinci/devices-da8xx.c | 28 ++++ >> arch/arm/mach-davinci/include/mach/da8xx.h | 6 + >> arch/arm/mach-davinci/usb-da8xx.c | 230 >> +++++++++++++++++++++++++++- >> 8 files changed, 314 insertions(+), 42 deletions(-) >> > > What should I be doing to keep this moving along? We need the related driver changes to be applied first. I could then use an immutable branch to push the platform changes against. I did take a look at the patches and they look good to me. Except the one comment from Sergei which I just now indicated that I agree with. Thanks, Sekhar From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sekhar Nori Subject: Re: [PATCH v5 0/5] da8xx USB PHY platform devices and clocks (was "da8xx UBS clocks") Date: Wed, 25 May 2016 16:48:34 +0530 Message-ID: <57458A0A.5010100@ti.com> References: <1462837612-7803-1-git-send-email-david@lechnology.com> <57431E73.5030304@lechnology.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <57431E73.5030304@lechnology.com> Sender: linux-kernel-owner@vger.kernel.org To: David Lechner , khilman@kernel.org Cc: petr@barix.com, sergei.shtylyov@cogentembedded.com, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Kishon Vijay Abraham I , Greg Kroah-Hartman , Alan Stern , Bin Liu , Lee Jones , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org List-Id: devicetree@vger.kernel.org On Monday 23 May 2016 08:44 PM, David Lechner wrote: > On 05/09/2016 06:46 PM, David Lechner wrote: >> v5 changes: renamed "usbphy" to "usb_phy" or "usb-phy" as appropriate >> >> David Lechner (5): >> ARM: davinci: da8xx: add usb phy clocks >> ARM: davinci: da8xx: Add CFGCHIP syscon platform declaration. >> ARM: davinci: da8xx: Add USB PHY platform declaration >> ARM: DTS: da850: Add cfgchip syscon node >> ARM: DTS: da850: Add usb phy node >> >> arch/arm/boot/dts/da850.dtsi | 9 ++ >> arch/arm/mach-davinci/board-da830-evm.c | 52 +++---- >> arch/arm/mach-davinci/board-da850-evm.c | 4 + >> arch/arm/mach-davinci/board-mityomapl138.c | 4 + >> arch/arm/mach-davinci/board-omapl138-hawk.c | 23 ++- >> arch/arm/mach-davinci/devices-da8xx.c | 28 ++++ >> arch/arm/mach-davinci/include/mach/da8xx.h | 6 + >> arch/arm/mach-davinci/usb-da8xx.c | 230 >> +++++++++++++++++++++++++++- >> 8 files changed, 314 insertions(+), 42 deletions(-) >> > > What should I be doing to keep this moving along? We need the related driver changes to be applied first. I could then use an immutable branch to push the platform changes against. I did take a look at the patches and they look good to me. Except the one comment from Sergei which I just now indicated that I agree with. Thanks, Sekhar