mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-05 18:41:58 +09:00
media: sun8i-di: Fix power on/off sequences
[ Upstream commit cff104e33bad38f4b2c8d58816a7accfaa2879f9 ]
According to user manual, reset line should be deasserted before clocks
are enabled. Also fix power down sequence to be reverse of that.
Fixes: a4260ea495 ("media: sun4i: Add H3 deinterlace driver")
Signed-off-by: Jernej Skrabec <jernej.skrabec@gmail.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Sasha Levin
parent
4bdc1b352e
commit
de09db4e5d
@@ -933,11 +933,18 @@ static int deinterlace_runtime_resume(struct device *device)
|
|||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
ret = reset_control_deassert(dev->rstc);
|
||||||
|
if (ret) {
|
||||||
|
dev_err(dev->dev, "Failed to apply reset\n");
|
||||||
|
|
||||||
|
goto err_exclusive_rate;
|
||||||
|
}
|
||||||
|
|
||||||
ret = clk_prepare_enable(dev->bus_clk);
|
ret = clk_prepare_enable(dev->bus_clk);
|
||||||
if (ret) {
|
if (ret) {
|
||||||
dev_err(dev->dev, "Failed to enable bus clock\n");
|
dev_err(dev->dev, "Failed to enable bus clock\n");
|
||||||
|
|
||||||
goto err_exclusive_rate;
|
goto err_rst;
|
||||||
}
|
}
|
||||||
|
|
||||||
ret = clk_prepare_enable(dev->mod_clk);
|
ret = clk_prepare_enable(dev->mod_clk);
|
||||||
@@ -954,23 +961,16 @@ static int deinterlace_runtime_resume(struct device *device)
|
|||||||
goto err_mod_clk;
|
goto err_mod_clk;
|
||||||
}
|
}
|
||||||
|
|
||||||
ret = reset_control_deassert(dev->rstc);
|
|
||||||
if (ret) {
|
|
||||||
dev_err(dev->dev, "Failed to apply reset\n");
|
|
||||||
|
|
||||||
goto err_ram_clk;
|
|
||||||
}
|
|
||||||
|
|
||||||
deinterlace_init(dev);
|
deinterlace_init(dev);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
err_ram_clk:
|
|
||||||
clk_disable_unprepare(dev->ram_clk);
|
|
||||||
err_mod_clk:
|
err_mod_clk:
|
||||||
clk_disable_unprepare(dev->mod_clk);
|
clk_disable_unprepare(dev->mod_clk);
|
||||||
err_bus_clk:
|
err_bus_clk:
|
||||||
clk_disable_unprepare(dev->bus_clk);
|
clk_disable_unprepare(dev->bus_clk);
|
||||||
|
err_rst:
|
||||||
|
reset_control_assert(dev->rstc);
|
||||||
err_exclusive_rate:
|
err_exclusive_rate:
|
||||||
clk_rate_exclusive_put(dev->mod_clk);
|
clk_rate_exclusive_put(dev->mod_clk);
|
||||||
|
|
||||||
@@ -981,11 +981,12 @@ static int deinterlace_runtime_suspend(struct device *device)
|
|||||||
{
|
{
|
||||||
struct deinterlace_dev *dev = dev_get_drvdata(device);
|
struct deinterlace_dev *dev = dev_get_drvdata(device);
|
||||||
|
|
||||||
reset_control_assert(dev->rstc);
|
|
||||||
|
|
||||||
clk_disable_unprepare(dev->ram_clk);
|
clk_disable_unprepare(dev->ram_clk);
|
||||||
clk_disable_unprepare(dev->mod_clk);
|
clk_disable_unprepare(dev->mod_clk);
|
||||||
clk_disable_unprepare(dev->bus_clk);
|
clk_disable_unprepare(dev->bus_clk);
|
||||||
|
|
||||||
|
reset_control_assert(dev->rstc);
|
||||||
|
|
||||||
clk_rate_exclusive_put(dev->mod_clk);
|
clk_rate_exclusive_put(dev->mod_clk);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|||||||
Reference in New Issue
Block a user