Commit 6558d7ed authored by Tuomas Tynkkynen's avatar Tuomas Tynkkynen Committed by Felipe Balbi

usb: phy: tegra: Use DT helpers for dr_mode

Use the new of_usb_get_dr_mode helper function for parsing dr_mode
from the device tree. Also replace the usage of the custom
tegra_usb_phy_mode enum with the standard enum.
Signed-off-by: default avatarTuomas Tynkkynen <ttynkkynen@nvidia.com>
Reviewed-by: default avatarStephen Warren <swarren@nvidia.com>
Tested-by: default avatarStephen Warren <swarren@nvidia.com>
Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
parent 9fdb07f7
...@@ -377,7 +377,7 @@ static int utmi_phy_power_on(struct tegra_usb_phy *phy) ...@@ -377,7 +377,7 @@ static int utmi_phy_power_on(struct tegra_usb_phy *phy)
UTMIP_PLLU_ENABLE_DLY_COUNT(phy->freq->enable_delay); UTMIP_PLLU_ENABLE_DLY_COUNT(phy->freq->enable_delay);
writel(val, base + UTMIP_PLL_CFG1); writel(val, base + UTMIP_PLL_CFG1);
if (phy->mode == TEGRA_USB_PHY_MODE_DEVICE) { if (phy->mode == USB_DR_MODE_PERIPHERAL) {
val = readl(base + USB_SUSP_CTRL); val = readl(base + USB_SUSP_CTRL);
val &= ~(USB_WAKE_ON_CNNT_EN_DEV | USB_WAKE_ON_DISCON_EN_DEV); val &= ~(USB_WAKE_ON_CNNT_EN_DEV | USB_WAKE_ON_DISCON_EN_DEV);
writel(val, base + USB_SUSP_CTRL); writel(val, base + USB_SUSP_CTRL);
...@@ -412,7 +412,7 @@ static int utmi_phy_power_on(struct tegra_usb_phy *phy) ...@@ -412,7 +412,7 @@ static int utmi_phy_power_on(struct tegra_usb_phy *phy)
if (phy->is_legacy_phy) { if (phy->is_legacy_phy) {
val = readl(base + UTMIP_SPARE_CFG0); val = readl(base + UTMIP_SPARE_CFG0);
if (phy->mode == TEGRA_USB_PHY_MODE_DEVICE) if (phy->mode == USB_DR_MODE_PERIPHERAL)
val &= ~FUSE_SETUP_SEL; val &= ~FUSE_SETUP_SEL;
else else
val |= FUSE_SETUP_SEL; val |= FUSE_SETUP_SEL;
...@@ -453,7 +453,7 @@ static int utmi_phy_power_off(struct tegra_usb_phy *phy) ...@@ -453,7 +453,7 @@ static int utmi_phy_power_off(struct tegra_usb_phy *phy)
utmi_phy_clk_disable(phy); utmi_phy_clk_disable(phy);
if (phy->mode == TEGRA_USB_PHY_MODE_DEVICE) { if (phy->mode == USB_DR_MODE_PERIPHERAL) {
val = readl(base + USB_SUSP_CTRL); val = readl(base + USB_SUSP_CTRL);
val &= ~USB_WAKEUP_DEBOUNCE_COUNT(~0); val &= ~USB_WAKEUP_DEBOUNCE_COUNT(~0);
val |= USB_WAKE_ON_CNNT_EN_DEV | USB_WAKEUP_DEBOUNCE_COUNT(5); val |= USB_WAKE_ON_CNNT_EN_DEV | USB_WAKEUP_DEBOUNCE_COUNT(5);
...@@ -907,15 +907,15 @@ static int tegra_usb_phy_probe(struct platform_device *pdev) ...@@ -907,15 +907,15 @@ static int tegra_usb_phy_probe(struct platform_device *pdev)
return -EINVAL; return -EINVAL;
} }
err = of_property_match_string(np, "dr_mode", "otg"); if (of_find_property(np, "dr_mode", NULL))
if (err < 0) { tegra_phy->mode = of_usb_get_dr_mode(np);
err = of_property_match_string(np, "dr_mode", "peripheral");
if (err < 0)
tegra_phy->mode = TEGRA_USB_PHY_MODE_HOST;
else else
tegra_phy->mode = TEGRA_USB_PHY_MODE_DEVICE; tegra_phy->mode = USB_DR_MODE_HOST;
} else
tegra_phy->mode = TEGRA_USB_PHY_MODE_OTG; if (tegra_phy->mode == USB_DR_MODE_UNKNOWN) {
dev_err(&pdev->dev, "dr_mode is invalid\n");
return -EINVAL;
}
/* On some boards, the VBUS regulator doesn't need to be controlled */ /* On some boards, the VBUS regulator doesn't need to be controlled */
if (of_find_property(np, "vbus-supply", NULL)) { if (of_find_property(np, "vbus-supply", NULL)) {
......
...@@ -34,12 +34,6 @@ enum tegra_usb_phy_port_speed { ...@@ -34,12 +34,6 @@ enum tegra_usb_phy_port_speed {
TEGRA_USB_PHY_PORT_SPEED_HIGH, TEGRA_USB_PHY_PORT_SPEED_HIGH,
}; };
enum tegra_usb_phy_mode {
TEGRA_USB_PHY_MODE_DEVICE,
TEGRA_USB_PHY_MODE_HOST,
TEGRA_USB_PHY_MODE_OTG,
};
struct tegra_xtal_freq; struct tegra_xtal_freq;
struct tegra_usb_phy { struct tegra_usb_phy {
...@@ -51,7 +45,7 @@ struct tegra_usb_phy { ...@@ -51,7 +45,7 @@ struct tegra_usb_phy {
struct clk *pll_u; struct clk *pll_u;
struct clk *pad_clk; struct clk *pad_clk;
struct regulator *vbus; struct regulator *vbus;
enum tegra_usb_phy_mode mode; enum usb_dr_mode mode;
void *config; void *config;
struct usb_phy *ulpi; struct usb_phy *ulpi;
struct usb_phy u_phy; struct usb_phy u_phy;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment