UPSTREAM: i2c: qcom-geni: Use goto for clearer exit path

Refactor the code by using goto statements to reduce duplication
and make the exit path clearer.

Signed-off-by: Andi Shyti <andi.shyti@kernel.org>

Bug: 370629635
Change-Id: I6e7152bd47d76bc7dd0eb98ffd7906eb52b91478
(cherry picked from commit 23cc961a08591976eb17ca8552a602425b1a780d)
Signed-off-by: Praveen Talari <quic_ptalari@quicinc.com>
This commit is contained in:
Andi Shyti
2024-08-12 21:40:29 +02:00
committed by John Stultz
parent 80c036f7f2
commit c24feb930a

View File

@@ -985,21 +985,24 @@ static int __maybe_unused geni_i2c_runtime_resume(struct device *dev)
return ret; return ret;
ret = clk_prepare_enable(gi2c->core_clk); ret = clk_prepare_enable(gi2c->core_clk);
if (ret) { if (ret)
geni_icc_disable(&gi2c->se); goto out_icc_disable;
return ret;
}
ret = geni_se_resources_on(&gi2c->se); ret = geni_se_resources_on(&gi2c->se);
if (ret) { if (ret)
clk_disable_unprepare(gi2c->core_clk); goto out_clk_disable;
geni_icc_disable(&gi2c->se);
return ret;
}
enable_irq(gi2c->irq); enable_irq(gi2c->irq);
gi2c->suspended = 0; gi2c->suspended = 0;
return 0; return 0;
out_clk_disable:
clk_disable_unprepare(gi2c->core_clk);
out_icc_disable:
geni_icc_disable(&gi2c->se);
return ret;
} }
static int __maybe_unused geni_i2c_suspend_noirq(struct device *dev) static int __maybe_unused geni_i2c_suspend_noirq(struct device *dev)