usb: dwc3: of-simple: fix missing clk_disable_unprepare

If of_clk_get() fails, the clean-up of already initialized clocks should be
the same as when clk_prepare_enable() fails. Thus a clk_disable_unprepare()
for each clock should be called before the clk_put().

Found by Linux Driver Verification project (linuxtesting.org).

Fixes: 16adc674d0 ("usb: dwc3: ep0: fix setup_packet_pending initialization")

Signed-off-by: Andreas Platschek <andreas.platschek@opentech.at>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
This commit is contained in:
Andreas Platschek 2017-12-07 11:32:20 +01:00 committed by Felipe Balbi
parent 8722e095f5
commit ded600ea9f
1 changed files with 3 additions and 1 deletions

View File

@ -51,8 +51,10 @@ static int dwc3_of_simple_clk_init(struct dwc3_of_simple *simple, int count)
clk = of_clk_get(np, i); clk = of_clk_get(np, i);
if (IS_ERR(clk)) { if (IS_ERR(clk)) {
while (--i >= 0) while (--i >= 0) {
clk_disable_unprepare(simple->clks[i]);
clk_put(simple->clks[i]); clk_put(simple->clks[i]);
}
return PTR_ERR(clk); return PTR_ERR(clk);
} }