summaryrefslogtreecommitdiffstats
path: root/drivers/usb
diff options
context:
space:
mode:
authorStephen Boyd <sboyd@codeaurora.org>2016-02-22 11:12:47 -0800
committerStephen Boyd <sboyd@codeaurora.org>2016-02-26 16:01:32 -0800
commit3d755dcc20dd452b52532eca17da40ebbd12aee9 (patch)
tree8535241ccbf44c6a5124915954d3416e3f5ffd69 /drivers/usb
parent921bacfa34d48c019bb068257c0947d728662bf3 (diff)
downloadop-kernel-dev-3d755dcc20dd452b52532eca17da40ebbd12aee9.zip
op-kernel-dev-3d755dcc20dd452b52532eca17da40ebbd12aee9.tar.gz
usb: dwc3: Remove impossible check for of_clk_get_parent_count() < 0
The check for < 0 is impossible now that of_clk_get_parent_count() returns an unsigned int. Simplify the code and update the types. Acked-by: Felipe Balbi <balbi@kernel.org> Cc: <linux-usb@vger.kernel.org> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Diffstat (limited to 'drivers/usb')
-rw-r--r--drivers/usb/dwc3/dwc3-of-simple.c9
1 files changed, 5 insertions, 4 deletions
diff --git a/drivers/usb/dwc3/dwc3-of-simple.c b/drivers/usb/dwc3/dwc3-of-simple.c
index 9c9f741..9743353 100644
--- a/drivers/usb/dwc3/dwc3-of-simple.c
+++ b/drivers/usb/dwc3/dwc3-of-simple.c
@@ -42,6 +42,7 @@ static int dwc3_of_simple_probe(struct platform_device *pdev)
struct device *dev = &pdev->dev;
struct device_node *np = dev->of_node;
+ unsigned int count;
int ret;
int i;
@@ -49,11 +50,11 @@ static int dwc3_of_simple_probe(struct platform_device *pdev)
if (!simple)
return -ENOMEM;
- ret = of_clk_get_parent_count(np);
- if (ret < 0)
- return ret;
+ count = of_clk_get_parent_count(np);
+ if (!count)
+ return -ENOENT;
- simple->num_clocks = ret;
+ simple->num_clocks = count;
simple->clks = devm_kcalloc(dev, simple->num_clocks,
sizeof(struct clk *), GFP_KERNEL);
OpenPOWER on IntegriCloud