[PATCHv2 2/5] efi_dt_fixup: use fdtdec_get_bool

[Ilias Apalodimas]

Please carry over the r-b tags from older versions

On Thu, 9 Apr 2026 at 22:22, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> Am 2. April 2026 22:50:35 MESZ schrieb rs at ti.com:
> >From: Randolph Sapp <rs at ti.com>
> >
> >Use the more straightforward fdtdec_get_bool instead of fdt_getprop and
> >a return code check.
> >
> >Signed-off-by: Randolph Sapp <rs at ti.com>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>

>
> Reviewed-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
>
> >---
> > lib/efi_loader/efi_dt_fixup.c | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
> >diff --git a/lib/efi_loader/efi_dt_fixup.c b/lib/efi_loader/efi_dt_fixup.c
> >index 544e1aa9808..333711b9957 100644
> >--- a/lib/efi_loader/efi_dt_fixup.c
> >+++ b/lib/efi_loader/efi_dt_fixup.c
> >@@ -123,8 +123,7 @@ void efi_carve_out_dt_rsv(void *fdt)
> >                           fdtdec_get_is_enabled(fdt, subnode)) {
> >                               bool nomap;
> >
> >-                              nomap = !!fdt_getprop(fdt, subnode, "no-map",
> >-                                                    NULL);
> >+                              nomap = fdtdec_get_bool(fdt, subnode, "no-map");
> >                               efi_reserve_memory(fdt_addr, fdt_size, nomap);
> >                       }
> >                       subnode = fdt_next_subnode(fdt, subnode);
>