Skip to content
Snippets Groups Projects
Commit fca10453 authored by Heinrich Schuchardt's avatar Heinrich Schuchardt Committed by Philipp Tomsich
Browse files

video: rk3399_mipi: correct error checking


Pointers are never negative. Use macro IS_ERR() for checking.
cf. rk3288_mipi.c

Signed-off-by: default avatarHeinrich Schuchardt <xypron.glpk@gmx.de>
Acked-by: default avatarPhilipp Tomsich <philipp.tomsich@theobroma-systems.com>
Reviewed-by: default avatarPhilipp Tomsich <philipp.tomsich@theobroma-systems.com>
parent e56a713e
No related branches found
No related tags found
No related merge requests found
...@@ -128,7 +128,7 @@ static int rk_mipi_ofdata_to_platdata(struct udevice *dev) ...@@ -128,7 +128,7 @@ static int rk_mipi_ofdata_to_platdata(struct udevice *dev)
struct rk_mipi_priv *priv = dev_get_priv(dev); struct rk_mipi_priv *priv = dev_get_priv(dev);
priv->grf = syscon_get_first_range(ROCKCHIP_SYSCON_GRF); priv->grf = syscon_get_first_range(ROCKCHIP_SYSCON_GRF);
if (priv->grf <= 0) { if (!priv->grf || IS_ERR(priv->grf)) {
debug("%s: Get syscon grf failed (ret=%p)\n", debug("%s: Get syscon grf failed (ret=%p)\n",
__func__, priv->grf); __func__, priv->grf);
return -ENXIO; return -ENXIO;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment