[PATCH] reset: ast: Use dev_read_addr_ptr()
peng.fan at nxp.com
peng.fan at nxp.com
Tue May 26 08:48:01 CEST 2026
From: Peng Fan <peng.fan at nxp.com>
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 at nxp.com>
---
drivers/reset/reset-ast2500.c | 4 ++--
drivers/reset/reset-ast2600.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
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;
--
2.51.0
More information about the U-Boot
mailing list