diff options
author | eric.gao@rock-chips.com <eric.gao@rock-chips.com> | 2017-04-20 12:45:53 +0800 |
---|---|---|
committer | Philipp Tomsich <philipp.tomsich@theobroma-systems.com> | 2017-05-09 12:41:46 +0200 |
commit | ae06287c0af3fd4fb26190456e960dcd94d9fe7e (patch) | |
tree | 2590c3c4e35bf7344f0f5346d1ced1d86a9b6db4 | |
parent | 925bd60fd45e6eceaabd221a7056d55ae845c38f (diff) |
rockchip: video: vop: Fix rk_display_init() return error
The function clk_set_rate() will return it's input parameter, so it's return
value in normal condition is nonzero. In this case, we should report error
when it return zero rather than return a nonzero value.
Signed-off-by: Eric Gao <eric.gao@rock-chips.com>
-rw-r--r-- | drivers/video/rockchip/rk_vop.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/drivers/video/rockchip/rk_vop.c b/drivers/video/rockchip/rk_vop.c index bc02f800dc..0691fdf8cd 100644 --- a/drivers/video/rockchip/rk_vop.c +++ b/drivers/video/rockchip/rk_vop.c @@ -242,11 +242,13 @@ int rk_display_init(struct udevice *dev, ulong fbbase, } ret = clk_get_by_index(dev, 1, &clk); - if (!ret) + if (!ret) { ret = clk_set_rate(&clk, timing.pixelclock.typ); - if (ret) { - debug("%s: Failed to set pixel clock: ret=%d\n", __func__, ret); - return ret; + if (!ret) { + debug("%s: Failed to set pixel clock: ret=%d\n", + __func__, ret); + return ret; + } } rkvop_mode_set(regs, &timing, vop_id); |