[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <74CDBE0F657A3D45AFBB94109FB122FF049EBDED36@HQMAIL01.nvidia.com>
Date: Tue, 19 Jul 2011 16:15:38 -0700
From: Stephen Warren <swarren@...dia.com>
To: Andrew Chew <AChew@...dia.com>,
"grant.likely@...retlab.ca" <grant.likely@...retlab.ca>,
"olof@...om.net" <olof@...om.net>,
Dan Willemsen <dwillemsen@...dia.com>,
Rhyland Klein <rklein@...dia.com>,
Venkat Moganty <vmoganty@...dia.com>
CC: "devicetree-discuss@...ts.ozlabs.org"
<devicetree-discuss@...ts.ozlabs.org>,
"linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 2/3] dt: tegra20: Add ehci host controller nodes.
Stephen Warren wrote at Tuesday, July 19, 2011 5:00 PM:
> achew@...dia.com wrote at Tuesday, July 19, 2011 4:47 PM:
> > From: Andrew Chew <achew@...dia.com>
> >
> > These values were derived from various headers in arch/arm/mach-tegra.
...
> > diff --git a/arch/arm/boot/dts/tegra20.dtsi b/arch/arm/boot/dts/tegra20.dtsi
> > index 83fedf3..a0353bb 100644
> > --- a/arch/arm/boot/dts/tegra20.dtsi
> > +++ b/arch/arm/boot/dts/tegra20.dtsi
> > @@ -160,5 +160,57 @@
> > interrupts = < 63 >;
> > status = "disabled";
> > };
> > +
> > + ehci@...00000 {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + compatible = "nvidia,tegra20-ehci";
> > + reg = <0xc5000000 0x4000>;
> > + interrupts = < 52 >;
> > + mode = < 1 >;
> > + power_down_on_bus_suspend;
> > + type = "utmi";
> > + hssync_start_delay = < 0 >;
> > + idle_wait_delay = < 17 >;
> > + elastic_limit = < 16 >;
> > + term_range_adj = < 6 >;
> > + xcvr_setup = < 15 >;
> > + xcvr_lsfslew = < 2 >;
> > + xcvr_lsrslew = < 2 >;
> > + status = "disabled";
>
> In the latest devicetree/test, tegra20.dtsi doesn't specify status for
> any nodes, but leaves everything enabled. Instead, board-specific files
> disable any unused devices.
Andrew pointed out that isn't actually true right now. However, Grant
did state that he was *going* to make Tegra work that way, so the
comment basically still stands.
--
nvpublic
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists