From: Haolin Li <li.hao...@qq.com>

A pointer can not be negative. Use macro IS_ERR_OR_NULL() for checking.

Signed-off-by: Haolin Li <li.hao...@qq.com>
---
 drivers/mmc/rockchip_sdhci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mmc/rockchip_sdhci.c b/drivers/mmc/rockchip_sdhci.c
index f3f9d83ba3..a46895299d 100644
--- a/drivers/mmc/rockchip_sdhci.c
+++ b/drivers/mmc/rockchip_sdhci.c
@@ -227,7 +227,7 @@ static int rk3399_emmc_get_phy(struct udevice *dev)
        }
 
        grf_base = syscon_get_first_range(ROCKCHIP_SYSCON_GRF);
-       if (grf_base < 0) {
+       if (IS_ERR_OR_NULL(grf_base)) {
                printf("%s Get syscon grf failed", __func__);
                return -ENODEV;
        }
-- 
2.25.1

Reply via email to