[L] Change in coreboot[master]: allocator_v4: Use memranges only for toplevel

[Nico Huber (Code Review)]

Attention is currently required from: Arthur Heymans, Bill XIE, Tim Wawrzynczak, Eric Lai, Werner Zeh, Kyösti Mälkki.

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/65420 )

Change subject: allocator_v4: Use memranges only for toplevel
......................................................................

Patch Set 8:

(1 comment)

File src/device/resource_allocator_v4.c:

https://review.coreboot.org/c/coreboot/+/65420/comment/3a7f1ce5_9d1f5b2c 
PS7, Line 437: void assign_resource_cb(void *param, struct device *dev, struct resource *res)
             : 	{
             : 		/* We have to filter the same resources as update_bridge_resource(). */
             : 		if (!res->size || !res->limit)
             : 			return;
             : 
             : 		assign_resource(res, *(const resource_t *)param + res->base, dev);
             : 	}
> It's like a "lambda" function from FP, I get what you're trying to do 😊 Just doesn't seem a very co […]

More or less common practice in Ada ;) with very similar semantics, now that
I think about it (no closure, the nested function is only valid while the
outer one executes).

Do you think the meaning of the code could be misunderstood? Then I'll
change it.