Messages in this thread | | | Date | Wed, 25 Jun 2014 16:02:48 -0700 | Subject | Re: [PATCH v1 5/9] of: Add NVIDIA Tegra XHCI controller binding | From | Andrew Bresticker <> |
| |
On Wed, Jun 25, 2014 at 2:54 PM, Stephen Warren <swarren@wwwdotorg.org> wrote: > On 06/18/2014 12:16 AM, Andrew Bresticker wrote: >> Add device-tree binding documentation for the XHCI controller present >> on Tegra124 and later SoCs. > >> diff --git a/Documentation/devicetree/bindings/usb/nvidia,tegra124-xhci.txt b/Documentation/devicetree/bindings/usb/nvidia,tegra124-xhci.txt > >> + - clock-names: Must include the following entries: >> + - xusb_host >> + - xusb_falcon_src >> + - xusb_ss >> + - xusb_ss_src >> + - xusb_hs_src >> + - xusb_fs_src > > Looking at include/dt-bindings/clock/tegra124-car.h I see a few entries > potentially missing here: > > #define TEGRA124_CLK_XUSB_HOST_SRC 252 > #define TEGRA124_CLK_XUSB_DEV_SRC 256 > #define TEGRA124_CLK_XUSB_DEV 257 > #define TEGRA124_CLK_XUSB_SS_DIV2 312
The driver doesn't use them, so I didn't put them in the binding.
>> + - pll_u_480m > > Not just pll_u?
We specifically want pll_u_480M as that's what we use as the parent of xusb_ss_src when scaling it to 120Mhz.
| |