Olof Johansson wrote at Thursday, November 03, 2011 9:26 PM:
> Rely on platform_data being passed through auxdata for now; more elaborate
> bindings for phy config and tunings to be added.
...
> diff --git a/arch/arm/mach-tegra/board-dt.c b/arch/arm/mach-tegra/board-dt.c
...
> static __initdata struct tegra_clk_init_table tegra_dt_clk_init_table[] = {
> /* name parent rate enabled */
> { "uartd", "pll_p", 216000000, true },
> + { "usbd", "clk_m", 12000000, false },
> + { "usb3", "clk_m", 12000000, false },
> { NULL, NULL, 0, 0},
> };
As woglinde mentioned on IRC, I think you do want to add "usb2" to the
table here; it's in board-paz00.c at least. It shouldn't hurt other boards.
> diff --git a/drivers/usb/host/ehci-tegra.c b/drivers/usb/host/ehci-tegra.c
...
> @@ -590,6 +617,13 @@ static int tegra_ehci_probe(struct platform_device *pdev)
> return -EINVAL;
> }
>
> + if (!pdev->dev.dma_mask)
> + pdev->dev.dma_mask = &tegra_ehci_dma_mask;
Should this come from DT, or is it some more system-level/internal thing
that doesn't make sense to represent there?
> +
> + vbus_gpio = of_get_named_gpio(pdev->dev.of_node, "nvidia,vbus-gpio", 0);
of_get_named_gpio() does check for NULL node pointer in practice, but is
it defined to? I wonder if this needs to be conditional of of_node!=NULL
or not?
> + if (vbus_gpio > 0)
Should that use gpio_is_valid()?
> + setup_vbus_gpio(pdev, vbus_gpio);
--
nvpublic
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
