Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760644Ab3GaSS4 (ORCPT ); Wed, 31 Jul 2013 14:18:56 -0400 Received: from mail-lb0-f179.google.com ([209.85.217.179]:58631 "EHLO mail-lb0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757055Ab3GaSSx (ORCPT ); Wed, 31 Jul 2013 14:18:53 -0400 Message-ID: <51F9550F.3080503@cogentembedded.com> Date: Wed, 31 Jul 2013 22:18:55 +0400 From: Sergei Shtylyov Organization: Cogent Embedded User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130620 Thunderbird/17.0.7 MIME-Version: 1.0 To: Tuomas Tynkkynen CC: swarren@wwwdotorg.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, Mikko Perttunen Subject: Re: [PATCH 2/2] ARM: dts: USB for Tegra114 Dalmore References: <1375292543-7896-1-git-send-email-ttynkkynen@nvidia.com> <1375292543-7896-3-git-send-email-ttynkkynen@nvidia.com> In-Reply-To: <1375292543-7896-3-git-send-email-ttynkkynen@nvidia.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1914 Lines: 67 Hello. On 07/31/2013 09:42 PM, Tuomas Tynkkynen wrote: > From: Mikko Perttunen > Device tree entries for the three EHCI controllers on Tegra114. > Enables the the third controller (USB host) on Dalmore. I would have done the board patch separately from the SoC one. > Signed-off-by: Mikko Perttunen [...] > diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi > index abf6c40..2905145 100644 > --- a/arch/arm/boot/dts/tegra114.dtsi > +++ b/arch/arm/boot/dts/tegra114.dtsi > @@ -430,6 +430,68 @@ > status = "disable"; > }; > > + usb@7d000000 { > + compatible = "nvidia,tegra30-ehci", "usb-ehci"; > + reg = <0x7d000000 0x4000>; > + interrupts = ; > + phy_type = "utmi"; > + clocks = <&tegra_car TEGRA114_CLK_USBD>; > + nvidia,phy = <&phy1>; > + status = "disabled"; > + }; > + > + phy1: usb-phy@7d000000 { At the same address as the previous node? > + compatible = "nvidia,tegra30-usb-phy"; > + reg = <0x7d000000 0x4000 0x7d000000 0x4000>; Hm, there must be some mistake: two similar register ranges. > + usb@7d008000 { > + compatible = "nvidia,tegra30-ehci", "usb-ehci"; > + reg = <0x7d008000 0x4000>; > + interrupts = ; > + phy_type = "utmi"; > + clocks = <&tegra_car TEGRA114_CLK_USB3>; > + nvidia,phy = <&phy3>; > + status = "disabled"; > + }; > + > + phy3: usb-phy@7d008000 { Again at the same address as previous node? > + compatible = "nvidia,tegra30-usb-phy"; > + reg = <0x7d008000 0x4000 0x7d000000 0x4000>; Second range conflicts with previous nodes. WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/