mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-11-26 21:54:11 +08:00
mfd: arizona: Remove duplicate set of ret variable
arizona_poll_reg already returns ETIMEDOUT if we don't see the expected register changes before the time out, so remove pointless local setting of ETIMEDOUT. Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com> Signed-off-by: Lee Jones <lee.jones@linaro.org>
This commit is contained in:
parent
81d30eda98
commit
de4ea10ad7
@ -342,10 +342,8 @@ static int arizona_enable_freerun_sysclk(struct arizona *arizona,
|
|||||||
ret = arizona_poll_reg(arizona, 25, ARIZONA_INTERRUPT_RAW_STATUS_5,
|
ret = arizona_poll_reg(arizona, 25, ARIZONA_INTERRUPT_RAW_STATUS_5,
|
||||||
ARIZONA_FLL1_CLOCK_OK_STS,
|
ARIZONA_FLL1_CLOCK_OK_STS,
|
||||||
ARIZONA_FLL1_CLOCK_OK_STS);
|
ARIZONA_FLL1_CLOCK_OK_STS);
|
||||||
if (ret) {
|
if (ret)
|
||||||
ret = -ETIMEDOUT;
|
|
||||||
goto err_fll;
|
goto err_fll;
|
||||||
}
|
|
||||||
|
|
||||||
ret = regmap_write(arizona->regmap, ARIZONA_SYSTEM_CLOCK_1, 0x0144);
|
ret = regmap_write(arizona->regmap, ARIZONA_SYSTEM_CLOCK_1, 0x0144);
|
||||||
if (ret) {
|
if (ret) {
|
||||||
@ -407,11 +405,9 @@ static int wm5102_apply_hardware_patch(struct arizona *arizona)
|
|||||||
|
|
||||||
ret = arizona_poll_reg(arizona, 5, ARIZONA_WRITE_SEQUENCER_CTRL_1,
|
ret = arizona_poll_reg(arizona, 5, ARIZONA_WRITE_SEQUENCER_CTRL_1,
|
||||||
ARIZONA_WSEQ_BUSY, 0);
|
ARIZONA_WSEQ_BUSY, 0);
|
||||||
if (ret) {
|
if (ret)
|
||||||
regmap_write(arizona->regmap, ARIZONA_WRITE_SEQUENCER_CTRL_0,
|
regmap_write(arizona->regmap, ARIZONA_WRITE_SEQUENCER_CTRL_0,
|
||||||
ARIZONA_WSEQ_ABORT);
|
ARIZONA_WSEQ_ABORT);
|
||||||
ret = -ETIMEDOUT;
|
|
||||||
}
|
|
||||||
|
|
||||||
err:
|
err:
|
||||||
err = arizona_disable_freerun_sysclk(arizona, &state);
|
err = arizona_disable_freerun_sysclk(arizona, &state);
|
||||||
|
Loading…
Reference in New Issue
Block a user