[PATCH] image: fdt: check "status" of "/reserved-memory" subnodes

Simon Goldschmidt simon.k.r.goldschmidt at gmail.com
Tue Jan 7 09:33:54 CET 2020


On Tue, Jan 7, 2020 at 7:21 AM Thirupathaiah Annapureddy
<thiruan at linux.microsoft.com> wrote:
>
> boot_fdt_add_mem_rsv_regions() scans the subnodes of
> "/reserved-memory" and adds them to reserved lmb regions.
> Currently this scanning does not take into "status" property.
> Even if the subnode is disabled, it gets added to the
> reserved lmb regions.
>
> This patch checks the "status" property before adding it
> to reserved lmb regions.
>
> Signed-off-by: Thirupathaiah Annapureddy <thiruan at linux.microsoft.com>

Reviewed-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>

> ---
>  common/image-fdt.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/common/image-fdt.c b/common/image-fdt.c
> index 48388488d9..cf13d655c0 100644
> --- a/common/image-fdt.c
> +++ b/common/image-fdt.c
> @@ -122,7 +122,7 @@ void boot_fdt_add_mem_rsv_regions(struct lmb *lmb, void *fdt_blob)
>                         /* check if this subnode has a reg property */
>                         ret = fdt_get_resource(fdt_blob, subnode, "reg", 0,
>                                                &res);
> -                       if (!ret) {
> +                       if (!ret && fdtdec_get_is_enabled(fdt_blob, subnode)) {
>                                 addr = res.start;
>                                 size = res.end - res.start + 1;
>                                 boot_fdt_reserve_region(lmb, addr, size);
> --
> 2.24.1
>


More information about the U-Boot mailing list