diff options
| author | Ondrej Jirman <megi@xff.cz> | 2023-05-22 23:47:04 +0200 |
|---|---|---|
| committer | Anatolij Gustschin <agust@denx.de> | 2023-07-14 18:23:40 +0200 |
| commit | 14dd77fdc4d5f2d55cf2dfcd97891e436a0aaf99 (patch) | |
| tree | af4bd95690f92dec6dabbdada584d5fabef3335a | |
| parent | b7d8d40346f29036e0b3a4e666e9c6ed14c14c4b (diff) | |
video: rockchip: dw_mipi_dsi: Fix error path checks in probe function
Wrong return codes were checked in several places. Check the proper ones.
Signed-off-by: Ondrej Jirman <megi@xff.cz>
| -rw-r--r-- | drivers/video/rockchip/dw_mipi_dsi_rockchip.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/video/rockchip/dw_mipi_dsi_rockchip.c b/drivers/video/rockchip/dw_mipi_dsi_rockchip.c index b1b5328595e..b7d6b51703c 100644 --- a/drivers/video/rockchip/dw_mipi_dsi_rockchip.c +++ b/drivers/video/rockchip/dw_mipi_dsi_rockchip.c @@ -822,6 +822,7 @@ static int dw_mipi_dsi_rockchip_probe(struct udevice *dev) priv->pclk = devm_clk_get(dev, "pclk"); if (IS_ERR(priv->pclk)) { + ret = PTR_ERR(priv->pclk); dev_err(dev, "peripheral clock get error %d\n", ret); return ret; } @@ -833,7 +834,8 @@ static int dw_mipi_dsi_rockchip_probe(struct udevice *dev) } else { priv->ref = devm_clk_get(dev, "ref"); - if (ret) { + if (IS_ERR(priv->ref)) { + ret = PTR_ERR(priv->ref); dev_err(dev, "pll reference clock get error %d\n", ret); return ret; } @@ -841,7 +843,8 @@ static int dw_mipi_dsi_rockchip_probe(struct udevice *dev) priv->rst = devm_reset_control_get_by_index(device->dev, 0); if (IS_ERR(priv->rst)) { - dev_err(dev, "missing dsi hardware reset\n"); + ret = PTR_ERR(priv->rst); + dev_err(dev, "missing dsi hardware reset %d\n", ret); return ret; } |