On Wed, 27 Sept 2023 at 07:34, Matthias Schiffer <matthias.schif...@ew.tq-group.com> wrote: > > 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.schif...@ew.tq-group.com> > --- > drivers/core/fdtaddr.c | 4 ++-- > include/dm/fdtaddr.h | 4 ++-- > 2 files changed, 4 insertions(+), 4 deletions(-) > Reviewed-by: Simon Glass <s...@chromium.org>
- [PATCH 1/5] core: fix doc comments of dev_read_addr*() a... Matthias Schiffer
- [PATCH 2/5] core: return FDT_ADDR_T_NONE from devfd... Matthias Schiffer
- Re: [PATCH 2/5] core: return FDT_ADDR_T_NONE fr... Simon Glass
- Re: [PATCH 2/5] core: return FDT_ADDR_T_NONE fr... Simon Glass
- [PATCH 3/5] core: introduce dev_read_addr_name[_siz... Matthias Schiffer
- Re: [PATCH 3/5] core: introduce dev_read_addr_n... Simon Glass
- Re: [PATCH 3/5] core: introduce dev_read_addr_n... Simon Glass
- [PATCH 5/5] treewide: use dev_read_addr_*_ptr() whe... Matthias Schiffer
- Re: [PATCH 5/5] treewide: use dev_read_addr_*_p... Simon Glass
- Re: [PATCH 5/5] treewide: use dev_read_addr_*_p... Simon Glass
- [PATCH 4/5] pinctrl: single: fix compile warnings w... Matthias Schiffer
- Re: [PATCH 4/5] pinctrl: single: fix compile wa... Simon Glass
- Re: [PATCH 4/5] pinctrl: single: fix compile wa... Simon Glass