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

2020-02-05 Thread Simon Glass
On Tue, 4 Feb 2020 at 17:16, Simon Glass wrote: > > ]Hi Thirupathaiah, > > On Tue, 4 Feb 2020 at 10:09, Thirupathaiah Annapureddy > wrote: > > > > Thank You Simon for the review. > > > > May I know what are the next steps in making forward progress on this? > > The patch is in my queue but I've

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

2020-02-04 Thread Simon Glass
]Hi Thirupathaiah, On Tue, 4 Feb 2020 at 10:09, Thirupathaiah Annapureddy wrote: > > Thank You Simon for the review. > > May I know what are the next steps in making forward progress on this? The patch is in my queue but I've had some test failures. Assuming it is not the culprit I expect it

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

2020-02-04 Thread Thirupathaiah Annapureddy
Thank You Simon for the review. May I know what are the next steps in making forward progress on this? Best Regards, Thiru On 1/7/2020 12:33 AM, Simon Goldschmidt wrote: > On Tue, Jan 7, 2020 at 7:21 AM Thirupathaiah Annapureddy > wrote: >> >> boot_fdt_add_mem_rsv_regions() scans the

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

2020-01-07 Thread Thirupathaiah Annapureddy
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

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

2020-01-07 Thread Simon Goldschmidt
On Tue, Jan 7, 2020 at 7:21 AM Thirupathaiah Annapureddy 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