diff options
| author | Matthias Schiffer <matthias.schiffer@ew.tq-group.com> | 2023-09-27 15:33:31 +0200 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2023-10-13 10:15:41 -0700 |
| commit | e367305769d091bc8a0026d69136eb77fd055784 (patch) | |
| tree | 5e351b8bdffb66a00ca923c6d62b3c7b8463b80b /drivers/core | |
| parent | e42c4d6d90c60f6b7fe01e88f458782c42cee8c9 (diff) | |
| download | u-boot-e367305769d091bc8a0026d69136eb77fd055784.zip u-boot-e367305769d091bc8a0026d69136eb77fd055784.tar.gz u-boot-e367305769d091bc8a0026d69136eb77fd055784.tar.bz2 | |
core: return FDT_ADDR_T_NONE from devfdt_get_addr_[size_]name() on errors
Checking for the error cast to fdt_addr_t is rather awkward - IS_ERR()
can be used, but it's not really made to be used on fdt_addr_t, which
may not even be the same size as a native pointer.
Most places in U-Boot only check for FDT_ADDR_T_NONE; let's adjust the
error return to match the expectation.
Signed-off-by: Matthias Schiffer <matthias.schiffer@ew.tq-group.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'drivers/core')
| -rw-r--r-- | drivers/core/fdtaddr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/core/fdtaddr.c b/drivers/core/fdtaddr.c index b79d138..0f4eff6 100644 --- a/drivers/core/fdtaddr.c +++ b/drivers/core/fdtaddr.c @@ -145,7 +145,7 @@ fdt_addr_t devfdt_get_addr_name(const struct udevice *dev, const char *name) index = fdt_stringlist_search(gd->fdt_blob, dev_of_offset(dev), "reg-names", name); if (index < 0) - return index; + return FDT_ADDR_T_NONE; return devfdt_get_addr_index(dev, index); #else @@ -162,7 +162,7 @@ fdt_addr_t devfdt_get_addr_size_name(const struct udevice *dev, index = fdt_stringlist_search(gd->fdt_blob, dev_of_offset(dev), "reg-names", name); if (index < 0) - return index; + return FDT_ADDR_T_NONE; return devfdt_get_addr_size_index(dev, index, size); #else |