[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