123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128 |
- From 5f0c864c1f207dba1db587747a58ec6b362dadf8 Mon Sep 17 00:00:00 2001
- From: Hans de Goede <hdegoede@redhat.com>
- Date: Sun, 29 Nov 2015 18:44:14 +0100
- Subject: [PATCH] ohci-platform: Add support for controllers with multiple
- reset lines
- At least the EHCI/OHCI found on the Allwinnner H3 SoC needs multiple
- reset lines, the controller will not initialize while the reset for
- its companion is still asserted, which means we need to de-assert
- 2 resets for the controller to work.
- Signed-off-by: Hans de Goede <hdegoede@redhat.com>
- ---
- Changes in v2:
- -New patch in v2 of this patch-set, to complement the identical patch for
- the ehci-platform code
- ---
- Documentation/devicetree/bindings/usb/usb-ohci.txt | 2 +-
- drivers/usb/host/ohci-platform.c | 49 +++++++++++++---------
- 2 files changed, 30 insertions(+), 21 deletions(-)
- --- a/Documentation/devicetree/bindings/usb/usb-ohci.txt
- +++ b/Documentation/devicetree/bindings/usb/usb-ohci.txt
- @@ -14,7 +14,7 @@ Optional properties:
- - clocks : a list of phandle + clock specifier pairs
- - phys : phandle + phy specifier pair
- - phy-names : "usb"
- -- resets : phandle + reset specifier pair
- +- resets : a list of phandle + reset specifier pairs
-
- Example:
-
- --- a/drivers/usb/host/ohci-platform.c
- +++ b/drivers/usb/host/ohci-platform.c
- @@ -33,11 +33,12 @@
-
- #define DRIVER_DESC "OHCI generic platform driver"
- #define OHCI_MAX_CLKS 3
- +#define OHCI_MAX_RESETS 2
- #define hcd_to_ohci_priv(h) ((struct ohci_platform_priv *)hcd_to_ohci(h)->priv)
-
- struct ohci_platform_priv {
- struct clk *clks[OHCI_MAX_CLKS];
- - struct reset_control *rst;
- + struct reset_control *resets[OHCI_MAX_RESETS];
- struct phy **phys;
- int num_phys;
- };
- @@ -117,7 +118,7 @@ static int ohci_platform_probe(struct pl
- struct usb_ohci_pdata *pdata = dev_get_platdata(&dev->dev);
- struct ohci_platform_priv *priv;
- struct ohci_hcd *ohci;
- - int err, irq, phy_num, clk = 0;
- + int err, irq, phy_num, clk = 0, rst = 0;
-
- if (usb_disabled())
- return -ENODEV;
- @@ -195,19 +196,23 @@ static int ohci_platform_probe(struct pl
- break;
- }
- }
- -
- - }
- -
- - priv->rst = devm_reset_control_get_optional(&dev->dev, NULL);
- - if (IS_ERR(priv->rst)) {
- - err = PTR_ERR(priv->rst);
- - if (err == -EPROBE_DEFER)
- - goto err_put_clks;
- - priv->rst = NULL;
- - } else {
- - err = reset_control_deassert(priv->rst);
- - if (err)
- - goto err_put_clks;
- + for (rst = 0; rst < OHCI_MAX_RESETS; rst++) {
- + priv->resets[rst] =
- + of_reset_control_get_by_index(dev->dev.of_node,
- + rst);
- + if (IS_ERR(priv->resets[rst])) {
- + err = PTR_ERR(priv->resets[rst]);
- + if (err == -EPROBE_DEFER)
- + goto err_reset;
- + priv->resets[rst] = NULL;
- + break;
- + }
- + err = reset_control_deassert_shared(priv->resets[rst]);
- + if (err) {
- + reset_control_put(priv->resets[rst]);
- + goto err_reset;
- + }
- + }
- }
-
- if (pdata->big_endian_desc)
- @@ -265,8 +270,10 @@ err_power:
- if (pdata->power_off)
- pdata->power_off(dev);
- err_reset:
- - if (priv->rst)
- - reset_control_assert(priv->rst);
- + while (--rst >= 0) {
- + reset_control_assert_shared(priv->resets[rst]);
- + reset_control_put(priv->resets[rst]);
- + }
- err_put_clks:
- while (--clk >= 0)
- clk_put(priv->clks[clk]);
- @@ -284,15 +291,17 @@ static int ohci_platform_remove(struct p
- struct usb_hcd *hcd = platform_get_drvdata(dev);
- struct usb_ohci_pdata *pdata = dev_get_platdata(&dev->dev);
- struct ohci_platform_priv *priv = hcd_to_ohci_priv(hcd);
- - int clk;
- + int clk, rst;
-
- usb_remove_hcd(hcd);
-
- if (pdata->power_off)
- pdata->power_off(dev);
-
- - if (priv->rst)
- - reset_control_assert(priv->rst);
- + for (rst = 0; rst < OHCI_MAX_RESETS && priv->resets[rst]; rst++) {
- + reset_control_assert_shared(priv->resets[rst]);
- + reset_control_put(priv->resets[rst]);
- + }
-
- for (clk = 0; clk < OHCI_MAX_CLKS && priv->clks[clk]; clk++)
- clk_put(priv->clks[clk]);
|