* [PATCH 1/3] phy: omap-usb2: Adapt phy-omap-usb2 for AM437x
2014-03-03 10:10 [PATCH 0/3] phy-omap-usb2 driver enhancements George Cherian
@ 2014-03-03 10:10 ` George Cherian
[not found] ` <1393841424-27656-2-git-send-email-george.cherian-l0cyMroinI0@public.gmane.org>
2014-03-03 10:22 ` [PATCH 2/3] phy: omap-usb2: Provide workaround for USB2PHY false disconnect George Cherian
2014-03-06 9:59 ` [PATCH v2 " George Cherian
2 siblings, 1 reply; 8+ messages in thread
From: George Cherian @ 2014-03-03 10:10 UTC (permalink / raw)
To: linux-usb, linux-kernel, linux-doc, devicetree
Cc: s.nawrocki, tony, george.cherian, gregkh, kishon, rob, galak,
ijc+devicetree, mark.rutland, pawel.moll, robh+dt
Adapt phy-omap-usb2 driver for AM437x.
- Add new comaptible "ti,am437x-usb2" for AM437x
- Pass proper data to differentiate AM437x and others.
- AM437x doesnot support set_vbus and start_srp.
- Also update the Documentation to add new compatible.
Signed-off-by: George Cherian <george.cherian@ti.com>
---
Documentation/devicetree/bindings/usb/usb-phy.txt | 4 +-
drivers/phy/phy-omap-usb2.c | 49 +++++++++++++++++------
include/linux/usb/omap_usb.h | 9 +++++
3 files changed, 49 insertions(+), 13 deletions(-)
diff --git a/Documentation/devicetree/bindings/usb/usb-phy.txt b/Documentation/devicetree/bindings/usb/usb-phy.txt
index c0245c8..b3fa409 100644
--- a/Documentation/devicetree/bindings/usb/usb-phy.txt
+++ b/Documentation/devicetree/bindings/usb/usb-phy.txt
@@ -3,7 +3,9 @@ USB PHY
OMAP USB2 PHY
Required properties:
- - compatible: Should be "ti,omap-usb2"
+ - compatible: Should be either of
+ * "ti,omap-usb2" for OMAP4, OMAP5, DRA7
+ * "ti,am437x-usb2" for AM437x
- reg : Address and length of the register set for the device.
- #phy-cells: determine the number of cells that should be given in the
phandle while referencing this phy.
diff --git a/drivers/phy/phy-omap-usb2.c b/drivers/phy/phy-omap-usb2.c
index 7699752..d54f24b 100644
--- a/drivers/phy/phy-omap-usb2.c
+++ b/drivers/phy/phy-omap-usb2.c
@@ -144,6 +144,31 @@ static struct phy_ops ops = {
.owner = THIS_MODULE,
};
+#ifdef CONFIG_OF
+static const struct usb_phy_data omap_usb2_data = {
+ .label = "omap_usb2",
+ .flags = OMAP_USB2_HAS_START_SRP | OMAP_USB2_HAS_SET_VBUS,
+};
+
+static const struct usb_phy_data am437x_usb2_data = {
+ .label = "am437x_usb2",
+ .flags = 0,
+};
+
+static const struct of_device_id omap_usb2_id_table[] = {
+ {
+ .compatible = "ti,omap-usb2",
+ .data = &omap_usb2_data,
+ },
+ {
+ .compatible = "ti,am437x-usb2",
+ .data = &am437x_usb2_data,
+ },
+ {},
+};
+MODULE_DEVICE_TABLE(of, omap_usb2_id_table);
+#endif
+
static int omap_usb2_probe(struct platform_device *pdev)
{
struct omap_usb *phy;
@@ -153,10 +178,16 @@ static int omap_usb2_probe(struct platform_device *pdev)
struct device_node *node = pdev->dev.of_node;
struct device_node *control_node;
struct platform_device *control_pdev;
+ const struct of_device_id *of_id;
+ struct usb_phy_data *phy_data;
+
+ of_id = of_match_device(of_match_ptr(omap_usb2_id_table), &pdev->dev);
- if (!node)
+ if (!of_id)
return -EINVAL;
+ phy_data = (struct usb_phy_data *)of_id->data;
+
phy = devm_kzalloc(&pdev->dev, sizeof(*phy), GFP_KERNEL);
if (!phy) {
dev_err(&pdev->dev, "unable to allocate memory for USB2 PHY\n");
@@ -172,7 +203,7 @@ static int omap_usb2_probe(struct platform_device *pdev)
phy->dev = &pdev->dev;
phy->phy.dev = phy->dev;
- phy->phy.label = "omap-usb2";
+ phy->phy.label = phy_data->label;
phy->phy.set_suspend = omap_usb2_suspend;
phy->phy.otg = otg;
phy->phy.type = USB_PHY_TYPE_USB2;
@@ -196,8 +227,10 @@ static int omap_usb2_probe(struct platform_device *pdev)
otg->set_host = omap_usb_set_host;
otg->set_peripheral = omap_usb_set_peripheral;
- otg->set_vbus = omap_usb_set_vbus;
- otg->start_srp = omap_usb_start_srp;
+ if (phy_data->flags & OMAP_USB2_HAS_SET_VBUS)
+ otg->set_vbus = omap_usb_set_vbus;
+ if (phy_data->flags & OMAP_USB2_HAS_START_SRP)
+ otg->start_srp = omap_usb_start_srp;
otg->phy = &phy->phy;
platform_set_drvdata(pdev, phy);
@@ -297,14 +330,6 @@ static const struct dev_pm_ops omap_usb2_pm_ops = {
#define DEV_PM_OPS NULL
#endif
-#ifdef CONFIG_OF
-static const struct of_device_id omap_usb2_id_table[] = {
- { .compatible = "ti,omap-usb2" },
- {}
-};
-MODULE_DEVICE_TABLE(of, omap_usb2_id_table);
-#endif
-
static struct platform_driver omap_usb2_driver = {
.probe = omap_usb2_probe,
.remove = omap_usb2_remove,
diff --git a/include/linux/usb/omap_usb.h b/include/linux/usb/omap_usb.h
index 6ae2936..034161d 100644
--- a/include/linux/usb/omap_usb.h
+++ b/include/linux/usb/omap_usb.h
@@ -42,6 +42,15 @@ struct omap_usb {
u8 is_suspended:1;
};
+struct usb_phy_data {
+ const char *label;
+ u32 flags;
+};
+
+/* Driver Flags */
+#define OMAP_USB2_HAS_START_SRP (1 << 0)
+#define OMAP_USB2_HAS_SET_VBUS (1 << 1)
+
#define phy_to_omapusb(x) container_of((x), struct omap_usb, phy)
#if defined(CONFIG_OMAP_USB2) || defined(CONFIG_OMAP_USB2_MODULE)
--
1.8.3.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 2/3] phy: omap-usb2: Provide workaround for USB2PHY false disconnect
2014-03-03 10:10 [PATCH 0/3] phy-omap-usb2 driver enhancements George Cherian
2014-03-03 10:10 ` [PATCH 1/3] phy: omap-usb2: Adapt phy-omap-usb2 for AM437x George Cherian
@ 2014-03-03 10:22 ` George Cherian
[not found] ` <1393842174-29868-1-git-send-email-george.cherian-l0cyMroinI0@public.gmane.org>
2014-03-05 7:07 ` [PATCH 2/3] phy: omap-usb2: Provide workaround for USB2PHY false disconnect Kishon Vijay Abraham I
2014-03-06 9:59 ` [PATCH v2 " George Cherian
2 siblings, 2 replies; 8+ messages in thread
From: George Cherian @ 2014-03-03 10:22 UTC (permalink / raw)
To: linux-usb, linux-kernel, linux-doc, devicetree
Cc: s.nawrocki, tony, george.cherian, gregkh, kishon, rob, galak,
ijc+devicetree, mark.rutland, pawel.moll, robh+dt
From: Austin Beam <austinbeam@ti.com>
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.
Update the Documentation also to add new comaptible.
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>
---
Documentation/devicetree/bindings/usb/usb-phy.txt | 3 +-
drivers/phy/phy-omap-usb2.c | 48 +++++++++++++++++++++++
include/linux/usb/omap_usb.h | 3 ++
3 files changed, 53 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/usb/usb-phy.txt b/Documentation/devicetree/bindings/usb/usb-phy.txt
index b3fa409..03de61a5 100644
--- a/Documentation/devicetree/bindings/usb/usb-phy.txt
+++ b/Documentation/devicetree/bindings/usb/usb-phy.txt
@@ -4,7 +4,8 @@ OMAP USB2 PHY
Required properties:
- compatible: Should be either of
- * "ti,omap-usb2" for OMAP4, OMAP5, DRA7
+ * "ti,omap-usb2" for OMAP4 and OMAP5
+ * "ti,dra7x-usb2" for DRA7
* "ti,am437x-usb2" for AM437x
- reg : Address and length of the register set for the device.
- #phy-cells: determine the number of cells that should be given in the
diff --git a/drivers/phy/phy-omap-usb2.c b/drivers/phy/phy-omap-usb2.c
index d54f24b..80ba7f0 100644
--- a/drivers/phy/phy-omap-usb2.c
+++ b/drivers/phy/phy-omap-usb2.c
@@ -31,6 +31,9 @@
#include <linux/phy/phy.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
* this phy
@@ -138,7 +141,30 @@ static int omap_usb_power_on(struct phy *x)
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 = {
+ .init = omap_usb_init,
.power_on = omap_usb_power_on,
.power_off = omap_usb_power_off,
.owner = THIS_MODULE,
@@ -150,6 +176,11 @@ static const struct usb_phy_data omap_usb2_data = {
.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 = {
.label = "am437x_usb2",
.flags = 0,
@@ -161,6 +192,10 @@ static const struct of_device_id omap_usb2_id_table[] = {
.data = &omap_usb2_data,
},
{
+ .compatible = "ti,dra7x-usb2",
+ .data = &dra7x_usb2_data,
+ },
+ {
.compatible = "ti,am437x-usb2",
.data = &am437x_usb2_data,
},
@@ -173,6 +208,7 @@ static int omap_usb2_probe(struct platform_device *pdev)
{
struct omap_usb *phy;
struct phy *generic_phy;
+ struct resource *res;
struct phy_provider *phy_provider;
struct usb_otg *otg;
struct device_node *node = pdev->dev.of_node;
@@ -208,6 +244,18 @@ static int omap_usb2_probe(struct platform_device *pdev)
phy->phy.otg = otg;
phy->phy.type = USB_PHY_TYPE_USB2;
+ if (phy_data->flags & OMAP_USB2_CALIBRATE_FALSE_DISCONNECT) {
+ res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ if (!res) {
+ dev_err(&pdev->dev, "memory resource not available\n");
+ return -ENODEV;
+ }
+ phy->phy_base = devm_request_and_ioremap(&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);
if (!control_node) {
dev_err(&pdev->dev, "Failed to get control device phandle\n");
diff --git a/include/linux/usb/omap_usb.h b/include/linux/usb/omap_usb.h
index 034161d..248a603 100644
--- a/include/linux/usb/omap_usb.h
+++ b/include/linux/usb/omap_usb.h
@@ -34,11 +34,13 @@ struct omap_usb {
struct usb_phy phy;
struct phy_companion *comparator;
void __iomem *pll_ctrl_base;
+ void __iomem *phy_base;
struct device *dev;
struct device *control_dev;
struct clk *wkupclk;
struct clk *sys_clk;
struct clk *optclk;
+ u8 flags;
u8 is_suspended:1;
};
@@ -50,6 +52,7 @@ struct usb_phy_data {
/* Driver Flags */
#define OMAP_USB2_HAS_START_SRP (1 << 0)
#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)
--
1.8.3.1
^ permalink raw reply related [flat|nested] 8+ messages in thread[parent not found: <1393842174-29868-1-git-send-email-george.cherian-l0cyMroinI0@public.gmane.org>]
* [PATCH 3/3] phy: omap-usb2: Add different compatible for OMAP5
[not found] ` <1393842174-29868-1-git-send-email-george.cherian-l0cyMroinI0@public.gmane.org>
@ 2014-03-03 10:22 ` George Cherian
0 siblings, 0 replies; 8+ messages in thread
From: George Cherian @ 2014-03-03 10:22 UTC (permalink / raw)
To: linux-usb-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-doc-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ, tony-4v6yS6AI5VpBDgjK7y7TUQ,
george.cherian-l0cyMroinI0,
gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r, kishon-l0cyMroinI0,
rob-VoJi6FS/r0vR7s880joybQ, galak-sgV2jX0FEOL9JmXXK+q4OQ,
ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, mark.rutland-5wv7dgnIgG8,
pawel.moll-5wv7dgnIgG8, robh+dt-DgEjT+Ai2ygdnm+yROfE0A
Add a new compatible for OMAP5 since it does not use any of the
OTG operations as of now.
HAS_SRP and SET_VBUS functionalities are used only for OMAP4.
Update the Documentation also to add new comaptible.
Signed-off-by: George Cherian <george.cherian-l0cyMroinI0@public.gmane.org>
---
Documentation/devicetree/bindings/usb/usb-phy.txt | 3 ++-
drivers/phy/phy-omap-usb2.c | 9 +++++++++
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/usb/usb-phy.txt b/Documentation/devicetree/bindings/usb/usb-phy.txt
index 03de61a5..ec199a5 100644
--- a/Documentation/devicetree/bindings/usb/usb-phy.txt
+++ b/Documentation/devicetree/bindings/usb/usb-phy.txt
@@ -4,7 +4,8 @@ OMAP USB2 PHY
Required properties:
- compatible: Should be either of
- * "ti,omap-usb2" for OMAP4 and OMAP5
+ * "ti,omap-usb2" for OMAP4
+ * "ti,omap5-usb2" for OMAP5
* "ti,dra7x-usb2" for DRA7
* "ti,am437x-usb2" for AM437x
- reg : Address and length of the register set for the device.
diff --git a/drivers/phy/phy-omap-usb2.c b/drivers/phy/phy-omap-usb2.c
index 80ba7f0..4af778f 100644
--- a/drivers/phy/phy-omap-usb2.c
+++ b/drivers/phy/phy-omap-usb2.c
@@ -176,6 +176,11 @@ static const struct usb_phy_data omap_usb2_data = {
.flags = OMAP_USB2_HAS_START_SRP | OMAP_USB2_HAS_SET_VBUS,
};
+static const struct usb_phy_data omap5_usb2_data = {
+ .label = "omap5_usb2",
+ .flags = 0,
+};
+
static const struct usb_phy_data dra7x_usb2_data = {
.label = "dra7x_usb2",
.flags = OMAP_USB2_CALIBRATE_FALSE_DISCONNECT,
@@ -192,6 +197,10 @@ static const struct of_device_id omap_usb2_id_table[] = {
.data = &omap_usb2_data,
},
{
+ .compatible = "ti,omap5-usb2",
+ .data = &omap5_usb2_data,
+ },
+ {
.compatible = "ti,dra7x-usb2",
.data = &dra7x_usb2_data,
},
--
1.8.3.1
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 2/3] phy: omap-usb2: Provide workaround for USB2PHY false disconnect
2014-03-03 10:22 ` [PATCH 2/3] phy: omap-usb2: Provide workaround for USB2PHY false disconnect George Cherian
[not found] ` <1393842174-29868-1-git-send-email-george.cherian-l0cyMroinI0@public.gmane.org>
@ 2014-03-05 7:07 ` Kishon Vijay Abraham I
1 sibling, 0 replies; 8+ messages in thread
From: Kishon Vijay Abraham I @ 2014-03-05 7:07 UTC (permalink / raw)
To: George Cherian
Cc: linux-usb, linux-kernel, linux-doc, devicetree, s.nawrocki, tony,
gregkh, rob, galak, ijc+devicetree, mark.rutland, pawel.moll,
robh+dt
Hi,
On Monday 03 March 2014 03:52 PM, George Cherian wrote:
> From: Austin Beam <austinbeam@ti.com>
>
> 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.
>
> Update the Documentation also to add new comaptible.
>
> 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>
> ---
> Documentation/devicetree/bindings/usb/usb-phy.txt | 3 +-
> drivers/phy/phy-omap-usb2.c | 48 +++++++++++++++++++++++
> include/linux/usb/omap_usb.h | 3 ++
> 3 files changed, 53 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/usb/usb-phy.txt b/Documentation/devicetree/bindings/usb/usb-phy.txt
> index b3fa409..03de61a5 100644
> --- a/Documentation/devicetree/bindings/usb/usb-phy.txt
> +++ b/Documentation/devicetree/bindings/usb/usb-phy.txt
> @@ -4,7 +4,8 @@ OMAP USB2 PHY
>
> Required properties:
> - compatible: Should be either of
> - * "ti,omap-usb2" for OMAP4, OMAP5, DRA7
> + * "ti,omap-usb2" for OMAP4 and OMAP5
> + * "ti,dra7x-usb2" for DRA7
> * "ti,am437x-usb2" for AM437x
> - reg : Address and length of the register set for the device.
> - #phy-cells: determine the number of cells that should be given in the
> diff --git a/drivers/phy/phy-omap-usb2.c b/drivers/phy/phy-omap-usb2.c
> index d54f24b..80ba7f0 100644
> --- a/drivers/phy/phy-omap-usb2.c
> +++ b/drivers/phy/phy-omap-usb2.c
> @@ -31,6 +31,9 @@
> #include <linux/phy/phy.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
> * this phy
> @@ -138,7 +141,30 @@ static int omap_usb_power_on(struct phy *x)
> 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 = {
> + .init = omap_usb_init,
> .power_on = omap_usb_power_on,
> .power_off = omap_usb_power_off,
> .owner = THIS_MODULE,
> @@ -150,6 +176,11 @@ static const struct usb_phy_data omap_usb2_data = {
> .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 = {
> .label = "am437x_usb2",
> .flags = 0,
> @@ -161,6 +192,10 @@ static const struct of_device_id omap_usb2_id_table[] = {
> .data = &omap_usb2_data,
> },
> {
> + .compatible = "ti,dra7x-usb2",
> + .data = &dra7x_usb2_data,
> + },
> + {
> .compatible = "ti,am437x-usb2",
> .data = &am437x_usb2_data,
> },
> @@ -173,6 +208,7 @@ static int omap_usb2_probe(struct platform_device *pdev)
> {
> struct omap_usb *phy;
> struct phy *generic_phy;
> + struct resource *res;
> struct phy_provider *phy_provider;
> struct usb_otg *otg;
> struct device_node *node = pdev->dev.of_node;
> @@ -208,6 +244,18 @@ static int omap_usb2_probe(struct platform_device *pdev)
> phy->phy.otg = otg;
> phy->phy.type = USB_PHY_TYPE_USB2;
>
> + if (phy_data->flags & OMAP_USB2_CALIBRATE_FALSE_DISCONNECT) {
> + res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> + if (!res) {
> + dev_err(&pdev->dev, "memory resource not available\n");
> + return -ENODEV;
> + }
You can remove the error check here and..
> + phy->phy_base = devm_request_and_ioremap(&pdev->dev, res);
use devm_ioremap_resource instead.
Thanks
Kishon
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 2/3] phy: omap-usb2: Provide workaround for USB2PHY false disconnect
2014-03-03 10:10 [PATCH 0/3] phy-omap-usb2 driver enhancements George Cherian
2014-03-03 10:10 ` [PATCH 1/3] phy: omap-usb2: Adapt phy-omap-usb2 for AM437x George Cherian
2014-03-03 10:22 ` [PATCH 2/3] phy: omap-usb2: Provide workaround for USB2PHY false disconnect George Cherian
@ 2014-03-06 9:59 ` George Cherian
2 siblings, 0 replies; 8+ messages in thread
From: George Cherian @ 2014-03-06 9:59 UTC (permalink / raw)
To: linux-usb, linux-kernel, linux-doc, devicetree
Cc: s.nawrocki, tony, george.cherian, gregkh, kishon, rob, galak,
ijc+devicetree, mark.rutland, pawel.moll, robh+dt
From: Austin Beam <austinbeam@ti.com>
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.
Update the Documentation also to add new comaptible.
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>
---
v1 -> v2
changed devm_request_and ioremap to devm_ioremap_resource
drivers/phy/phy-omap-usb2.c | 44 ++++++++++++++++++++++++++++++++++++++++++++
include/linux/usb/omap_usb.h | 3 +++
2 files changed, 47 insertions(+)
diff --git a/drivers/phy/phy-omap-usb2.c b/drivers/phy/phy-omap-usb2.c
index d54f24b..0ae4129 100644
--- a/drivers/phy/phy-omap-usb2.c
+++ b/drivers/phy/phy-omap-usb2.c
@@ -31,6 +31,9 @@
#include <linux/phy/phy.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
* this phy
@@ -138,7 +141,30 @@ static int omap_usb_power_on(struct phy *x)
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 = {
+ .init = omap_usb_init,
.power_on = omap_usb_power_on,
.power_off = omap_usb_power_off,
.owner = THIS_MODULE,
@@ -150,6 +176,11 @@ static const struct usb_phy_data omap_usb2_data = {
.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 = {
.label = "am437x_usb2",
.flags = 0,
@@ -161,6 +192,10 @@ static const struct of_device_id omap_usb2_id_table[] = {
.data = &omap_usb2_data,
},
{
+ .compatible = "ti,dra7x-usb2",
+ .data = &dra7x_usb2_data,
+ },
+ {
.compatible = "ti,am437x-usb2",
.data = &am437x_usb2_data,
},
@@ -173,6 +208,7 @@ static int omap_usb2_probe(struct platform_device *pdev)
{
struct omap_usb *phy;
struct phy *generic_phy;
+ struct resource *res;
struct phy_provider *phy_provider;
struct usb_otg *otg;
struct device_node *node = pdev->dev.of_node;
@@ -208,6 +244,14 @@ static int omap_usb2_probe(struct platform_device *pdev)
phy->phy.otg = otg;
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);
if (!control_node) {
dev_err(&pdev->dev, "Failed to get control device phandle\n");
diff --git a/include/linux/usb/omap_usb.h b/include/linux/usb/omap_usb.h
index 034161d..248a603 100644
--- a/include/linux/usb/omap_usb.h
+++ b/include/linux/usb/omap_usb.h
@@ -34,11 +34,13 @@ struct omap_usb {
struct usb_phy phy;
struct phy_companion *comparator;
void __iomem *pll_ctrl_base;
+ void __iomem *phy_base;
struct device *dev;
struct device *control_dev;
struct clk *wkupclk;
struct clk *sys_clk;
struct clk *optclk;
+ u8 flags;
u8 is_suspended:1;
};
@@ -50,6 +52,7 @@ struct usb_phy_data {
/* Driver Flags */
#define OMAP_USB2_HAS_START_SRP (1 << 0)
#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)
--
1.8.3.1
^ permalink raw reply related [flat|nested] 8+ messages in thread