diff options
| author | Peng Fan <[email protected]> | 2026-05-26 14:48:01 +0800 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2026-06-09 14:53:34 -0600 |
| commit | 305c014fdb7208ced179e47dc3634e7faa6a2502 (patch) | |
| tree | 62e2647cc392c401f4da3e1fa89fa9dbb18e49f5 /drivers | |
| parent | d26cbf4667c9b8193b06c6f8e9fa7334866c94ac (diff) | |
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 <[email protected]>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/reset/reset-ast2500.c | 4 | ||||
| -rw-r--r-- | 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; |