Commit 3799c401 authored by Ben Dooks's avatar Ben Dooks Committed by Greg Kroah-Hartman

[PATCH] USB: S3C2410: use clk_enable() to ensure 48MHz to OHCI core

Get the "usb-bus" clock and ensure it is enabled
when the OHCI core is in use.

It seems that a few bootloaders do not enable the
UPLL at startup, which stops the OHCI core having
a 48MHz bus clock. The improvements to the clock
framework for the s3c24xx now allow the USB PLL
to be started and stopped when being used.
Signed-off-by: default avatarBen Dooks <ben-linux@fluff.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent bfb25849
...@@ -30,6 +30,7 @@ ...@@ -30,6 +30,7 @@
/* clock device associated with the hcd */ /* clock device associated with the hcd */
static struct clk *clk; static struct clk *clk;
static struct clk *usb_clk;
/* forward definitions */ /* forward definitions */
...@@ -47,6 +48,10 @@ static void s3c2410_start_hc(struct platform_device *dev, struct usb_hcd *hcd) ...@@ -47,6 +48,10 @@ static void s3c2410_start_hc(struct platform_device *dev, struct usb_hcd *hcd)
struct s3c2410_hcd_info *info = dev->dev.platform_data; struct s3c2410_hcd_info *info = dev->dev.platform_data;
dev_dbg(&dev->dev, "s3c2410_start_hc:\n"); dev_dbg(&dev->dev, "s3c2410_start_hc:\n");
clk_enable(usb_clk);
mdelay(2); /* let the bus clock stabilise */
clk_enable(clk); clk_enable(clk);
if (info != NULL) { if (info != NULL) {
...@@ -75,6 +80,7 @@ static void s3c2410_stop_hc(struct platform_device *dev) ...@@ -75,6 +80,7 @@ static void s3c2410_stop_hc(struct platform_device *dev)
} }
clk_disable(clk); clk_disable(clk);
clk_disable(usb_clk);
} }
/* ohci_s3c2410_hub_status_data /* ohci_s3c2410_hub_status_data
...@@ -354,14 +360,21 @@ static int usb_hcd_s3c2410_probe (const struct hc_driver *driver, ...@@ -354,14 +360,21 @@ static int usb_hcd_s3c2410_probe (const struct hc_driver *driver,
if (!request_mem_region(hcd->rsrc_start, hcd->rsrc_len, hcd_name)) { if (!request_mem_region(hcd->rsrc_start, hcd->rsrc_len, hcd_name)) {
dev_err(&dev->dev, "request_mem_region failed"); dev_err(&dev->dev, "request_mem_region failed");
retval = -EBUSY; retval = -EBUSY;
goto err0; goto err_put;
} }
clk = clk_get(NULL, "usb-host"); clk = clk_get(&dev->dev, "usb-host");
if (IS_ERR(clk)) { if (IS_ERR(clk)) {
dev_err(&dev->dev, "cannot get usb-host clock\n"); dev_err(&dev->dev, "cannot get usb-host clock\n");
retval = -ENOENT; retval = -ENOENT;
goto err1; goto err_mem;
}
usb_clk = clk_get(&dev->dev, "upll");
if (IS_ERR(usb_clk)) {
dev_err(&dev->dev, "cannot get usb-host clock\n");
retval = -ENOENT;
goto err_clk;
} }
s3c2410_start_hc(dev, hcd); s3c2410_start_hc(dev, hcd);
...@@ -370,26 +383,29 @@ static int usb_hcd_s3c2410_probe (const struct hc_driver *driver, ...@@ -370,26 +383,29 @@ static int usb_hcd_s3c2410_probe (const struct hc_driver *driver,
if (!hcd->regs) { if (!hcd->regs) {
dev_err(&dev->dev, "ioremap failed\n"); dev_err(&dev->dev, "ioremap failed\n");
retval = -ENOMEM; retval = -ENOMEM;
goto err2; goto err_ioremap;
} }
ohci_hcd_init(hcd_to_ohci(hcd)); ohci_hcd_init(hcd_to_ohci(hcd));
retval = usb_add_hcd(hcd, dev->resource[1].start, SA_INTERRUPT); retval = usb_add_hcd(hcd, dev->resource[1].start, SA_INTERRUPT);
if (retval != 0) if (retval != 0)
goto err2; goto err_ioremap;
return 0; return 0;
err2: err_ioremap:
s3c2410_stop_hc(dev); s3c2410_stop_hc(dev);
iounmap(hcd->regs); iounmap(hcd->regs);
clk_put(usb_clk);
err_clk:
clk_put(clk); clk_put(clk);
err1: err_mem:
release_mem_region(hcd->rsrc_start, hcd->rsrc_len); release_mem_region(hcd->rsrc_start, hcd->rsrc_len);
err0: err_put:
usb_put_hcd(hcd); usb_put_hcd(hcd);
return retval; return retval;
} }
......
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