From: Peng Fan <peng.fan@nxp.com>
Date: Tue, 26 May 2026 06:48:01 +0000 (+0800)
Subject: reset: ast: Use dev_read_addr_ptr()
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=305c014fdb7208ced179e47dc3634e7faa6a2502;p=thirdparty%2Fu-boot.git

reset: ast: Use dev_read_addr_ptr()

Use dev_read_addr_ptr() which supports both live device tree and flat DT
backends, avoiding direct dependency on devfdt_* helpers.

While at here, correct error return value, when priv->scu is NULL,
PTR_ERR(priv->scu) is 0 which implies success. Change to return '-EINVAL'.

No functional changes.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---

diff --git a/drivers/reset/reset-ast2500.c b/drivers/reset/reset-ast2500.c
index 39b3d025713..c85906bbeb5 100644
--- a/drivers/reset/reset-ast2500.c
+++ b/drivers/reset/reset-ast2500.c
@@ -77,10 +77,10 @@ static int ast2500_reset_probe(struct udevice *dev)
 		return rc;
 	}
 
-	priv->scu = devfdt_get_addr_ptr(scu_dev);
+	priv->scu = dev_read_addr_ptr(scu_dev);
 	if (IS_ERR_OR_NULL(priv->scu)) {
 		debug("%s: invalid SCU base pointer\n", __func__);
-		return PTR_ERR(priv->scu);
+		return -EINVAL;
 	}
 
 	return 0;
diff --git a/drivers/reset/reset-ast2600.c b/drivers/reset/reset-ast2600.c
index 9b77f6c2b71..71b6220225a 100644
--- a/drivers/reset/reset-ast2600.c
+++ b/drivers/reset/reset-ast2600.c
@@ -76,10 +76,10 @@ static int ast2600_reset_probe(struct udevice *dev)
 		return rc;
 	}
 
-	priv->scu = devfdt_get_addr_ptr(scu_dev);
+	priv->scu = dev_read_addr_ptr(scu_dev);
 	if (IS_ERR_OR_NULL(priv->scu)) {
 		debug("%s: invalid SCU base pointer\n", __func__);
-		return PTR_ERR(priv->scu);
+		return -EINVAL;
 	}
 
 	return 0;