phy: omap-usb2: Provide workaround for USB2PHY false disconnect

Enable the dra7x errata workaround for false disconnect problem
with USB2PHY. False disconnects were detected with some of the devices.
Reduce the sensitivity of the disconnect logic within the USB2PHY subsystem
to enusre these false disconnects are not registered.

[george.cherian@ti.com]
While at that, pass proper flags for each SoC's. This is a common driver
used across OMAP4,OMAP5,DRA7xx and AM437x USB2PHY.

False disconnect workaround is currently applicable for only DRA7x.

Signed-off-by: Austin Beam <austinbeam@ti.com>
Signed-off-by: George Cherian <george.cherian@ti.com>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
This commit is contained in:
Austin Beam 2014-03-06 18:11:53 +05:30 committed by Kishon Vijay Abraham I
parent 09a0168de1
commit 7e472402ca
2 changed files with 48 additions and 0 deletions

View File

@ -31,6 +31,9 @@
#include <linux/phy/phy.h> #include <linux/phy/phy.h>
#include <linux/of_platform.h> #include <linux/of_platform.h>
#define USB2PHY_DISCON_BYP_LATCH (1 << 31)
#define USB2PHY_ANA_CONFIG1 0x4c
/** /**
* omap_usb2_set_comparator - links the comparator present in the sytem with * omap_usb2_set_comparator - links the comparator present in the sytem with
* this phy * this phy
@ -116,7 +119,30 @@ static int omap_usb_power_on(struct phy *x)
return 0; return 0;
} }
static int omap_usb_init(struct phy *x)
{
struct omap_usb *phy = phy_get_drvdata(x);
u32 val;
if (phy->flags & OMAP_USB2_CALIBRATE_FALSE_DISCONNECT) {
/*
*
* Reduce the sensitivity of internal PHY by enabling the
* DISCON_BYP_LATCH of the USB2PHY_ANA_CONFIG1 register. This
* resolves issues with certain devices which can otherwise
* be prone to false disconnects.
*
*/
val = omap_usb_readl(phy->phy_base, USB2PHY_ANA_CONFIG1);
val |= USB2PHY_DISCON_BYP_LATCH;
omap_usb_writel(phy->phy_base, USB2PHY_ANA_CONFIG1, val);
}
return 0;
}
static struct phy_ops ops = { static struct phy_ops ops = {
.init = omap_usb_init,
.power_on = omap_usb_power_on, .power_on = omap_usb_power_on,
.power_off = omap_usb_power_off, .power_off = omap_usb_power_off,
.owner = THIS_MODULE, .owner = THIS_MODULE,
@ -128,6 +154,11 @@ static const struct usb_phy_data omap_usb2_data = {
.flags = OMAP_USB2_HAS_START_SRP | OMAP_USB2_HAS_SET_VBUS, .flags = OMAP_USB2_HAS_START_SRP | OMAP_USB2_HAS_SET_VBUS,
}; };
static const struct usb_phy_data dra7x_usb2_data = {
.label = "dra7x_usb2",
.flags = OMAP_USB2_CALIBRATE_FALSE_DISCONNECT,
};
static const struct usb_phy_data am437x_usb2_data = { static const struct usb_phy_data am437x_usb2_data = {
.label = "am437x_usb2", .label = "am437x_usb2",
.flags = 0, .flags = 0,
@ -138,6 +169,10 @@ static const struct of_device_id omap_usb2_id_table[] = {
.compatible = "ti,omap-usb2", .compatible = "ti,omap-usb2",
.data = &omap_usb2_data, .data = &omap_usb2_data,
}, },
{
.compatible = "ti,dra7x-usb2",
.data = &dra7x_usb2_data,
},
{ {
.compatible = "ti,am437x-usb2", .compatible = "ti,am437x-usb2",
.data = &am437x_usb2_data, .data = &am437x_usb2_data,
@ -151,6 +186,7 @@ static int omap_usb2_probe(struct platform_device *pdev)
{ {
struct omap_usb *phy; struct omap_usb *phy;
struct phy *generic_phy; struct phy *generic_phy;
struct resource *res;
struct phy_provider *phy_provider; struct phy_provider *phy_provider;
struct usb_otg *otg; struct usb_otg *otg;
struct device_node *node = pdev->dev.of_node; struct device_node *node = pdev->dev.of_node;
@ -185,6 +221,14 @@ static int omap_usb2_probe(struct platform_device *pdev)
phy->phy.otg = otg; phy->phy.otg = otg;
phy->phy.type = USB_PHY_TYPE_USB2; phy->phy.type = USB_PHY_TYPE_USB2;
if (phy_data->flags & OMAP_USB2_CALIBRATE_FALSE_DISCONNECT) {
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
phy->phy_base = devm_ioremap_resource(&pdev->dev, res);
if (!phy->phy_base)
return -ENOMEM;
phy->flags |= OMAP_USB2_CALIBRATE_FALSE_DISCONNECT;
}
control_node = of_parse_phandle(node, "ctrl-module", 0); control_node = of_parse_phandle(node, "ctrl-module", 0);
if (!control_node) { if (!control_node) {
dev_err(&pdev->dev, "Failed to get control device phandle\n"); dev_err(&pdev->dev, "Failed to get control device phandle\n");

View File

@ -33,10 +33,13 @@ struct usb_dpll_params {
struct omap_usb { struct omap_usb {
struct usb_phy phy; struct usb_phy phy;
struct phy_companion *comparator; struct phy_companion *comparator;
void __iomem *pll_ctrl_base;
void __iomem *phy_base;
struct device *dev; struct device *dev;
struct device *control_dev; struct device *control_dev;
struct clk *wkupclk; struct clk *wkupclk;
struct clk *optclk; struct clk *optclk;
u8 flags;
}; };
struct usb_phy_data { struct usb_phy_data {
@ -47,6 +50,7 @@ struct usb_phy_data {
/* Driver Flags */ /* Driver Flags */
#define OMAP_USB2_HAS_START_SRP (1 << 0) #define OMAP_USB2_HAS_START_SRP (1 << 0)
#define OMAP_USB2_HAS_SET_VBUS (1 << 1) #define OMAP_USB2_HAS_SET_VBUS (1 << 1)
#define OMAP_USB2_CALIBRATE_FALSE_DISCONNECT (1 << 2)
#define phy_to_omapusb(x) container_of((x), struct omap_usb, phy) #define phy_to_omapusb(x) container_of((x), struct omap_usb, phy)