Commit bbe2028f authored by Chunfeng Yun's avatar Chunfeng Yun Committed by Greg Kroah-Hartman

usb: misc: usb3503: get optional clock by devm_clk_get_optional()

When the driver tries to get optional clock, it ignores all errors except
-EPROBE_DEFER, but if only ignores -ENOENT, it will cover some real errors,
such as -ENOMEM, so use devm_clk_get_optional() to get optional clock.
And remove unnecessary stack variable clk.

Cc: Dongjin Kim <tobetter@gmail.com>
Signed-off-by: default avatarChunfeng Yun <chunfeng.yun@mediatek.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 08048c04
...@@ -172,7 +172,6 @@ static int usb3503_probe(struct usb3503 *hub) ...@@ -172,7 +172,6 @@ static int usb3503_probe(struct usb3503 *hub)
hub->gpio_reset = pdata->gpio_reset; hub->gpio_reset = pdata->gpio_reset;
hub->mode = pdata->initial_mode; hub->mode = pdata->initial_mode;
} else if (np) { } else if (np) {
struct clk *clk;
u32 rate = 0; u32 rate = 0;
hub->port_off_mask = 0; hub->port_off_mask = 0;
...@@ -198,33 +197,28 @@ static int usb3503_probe(struct usb3503 *hub) ...@@ -198,33 +197,28 @@ static int usb3503_probe(struct usb3503 *hub)
} }
} }
clk = devm_clk_get(dev, "refclk"); hub->clk = devm_clk_get_optional(dev, "refclk");
if (IS_ERR(clk) && PTR_ERR(clk) != -ENOENT) { if (IS_ERR(hub->clk)) {
dev_err(dev, "unable to request refclk (%ld)\n", dev_err(dev, "unable to request refclk (%ld)\n",
PTR_ERR(clk)); PTR_ERR(hub->clk));
return PTR_ERR(clk); return PTR_ERR(hub->clk);
} }
if (!IS_ERR(clk)) {
hub->clk = clk;
if (rate != 0) { if (rate != 0) {
err = clk_set_rate(hub->clk, rate); err = clk_set_rate(hub->clk, rate);
if (err) { if (err) {
dev_err(dev, dev_err(dev,
"unable to set reference clock rate to %d\n", "unable to set reference clock rate to %d\n",
(int) rate); (int)rate);
return err; return err;
} }
} }
err = clk_prepare_enable(hub->clk); err = clk_prepare_enable(hub->clk);
if (err) { if (err) {
dev_err(dev, dev_err(dev, "unable to enable reference clock\n");
"unable to enable reference clock\n");
return err; return err;
} }
}
property = of_get_property(np, "disabled-ports", &len); property = of_get_property(np, "disabled-ports", &len);
if (property && (len / sizeof(u32)) > 0) { if (property && (len / sizeof(u32)) > 0) {
...@@ -324,7 +318,6 @@ static int usb3503_i2c_remove(struct i2c_client *i2c) ...@@ -324,7 +318,6 @@ static int usb3503_i2c_remove(struct i2c_client *i2c)
struct usb3503 *hub; struct usb3503 *hub;
hub = i2c_get_clientdata(i2c); hub = i2c_get_clientdata(i2c);
if (hub->clk)
clk_disable_unprepare(hub->clk); clk_disable_unprepare(hub->clk);
return 0; return 0;
...@@ -348,7 +341,6 @@ static int usb3503_platform_remove(struct platform_device *pdev) ...@@ -348,7 +341,6 @@ static int usb3503_platform_remove(struct platform_device *pdev)
struct usb3503 *hub; struct usb3503 *hub;
hub = platform_get_drvdata(pdev); hub = platform_get_drvdata(pdev);
if (hub->clk)
clk_disable_unprepare(hub->clk); clk_disable_unprepare(hub->clk);
return 0; return 0;
...@@ -358,8 +350,6 @@ static int usb3503_platform_remove(struct platform_device *pdev) ...@@ -358,8 +350,6 @@ static int usb3503_platform_remove(struct platform_device *pdev)
static int usb3503_suspend(struct usb3503 *hub) static int usb3503_suspend(struct usb3503 *hub)
{ {
usb3503_switch_mode(hub, USB3503_MODE_STANDBY); usb3503_switch_mode(hub, USB3503_MODE_STANDBY);
if (hub->clk)
clk_disable_unprepare(hub->clk); clk_disable_unprepare(hub->clk);
return 0; return 0;
...@@ -367,9 +357,7 @@ static int usb3503_suspend(struct usb3503 *hub) ...@@ -367,9 +357,7 @@ static int usb3503_suspend(struct usb3503 *hub)
static int usb3503_resume(struct usb3503 *hub) static int usb3503_resume(struct usb3503 *hub)
{ {
if (hub->clk)
clk_prepare_enable(hub->clk); clk_prepare_enable(hub->clk);
usb3503_switch_mode(hub, hub->mode); usb3503_switch_mode(hub, hub->mode);
return 0; return 0;
......
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