Change in flashrom[master]: layout: Use linked list for `struct romentry`

[Nico Huber (Code Review)]

Attention is currently required from: Edward O'Callaghan, Angel Pons, Daniel Campello, Anastasia Klimchuk.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/33521 )

Change subject: layout: Use linked list for `struct romentry`
......................................................................

Patch Set 8:

(2 comments)

File layout.c:

https://review.coreboot.org/c/flashrom/+/33521/comment/0111516e_643ae007 
PS8, Line 388: 	*layout = malloc(sizeof(**layout));
> Given that you memset() the allocated space afterwards, use calloc() instead?

I like to see it explicitly if something is cleared explicitly. I can try
something else, though. Maybe you'll like it better.

https://review.coreboot.org/c/flashrom/+/33521/comment/00d12983_54967aad 
PS8, Line 458: if (layout == global_layout)
             : 		return;
             : 
             : 	while (layout && layout->head) {
             : 		struct romentry *const entry = layout->head;
             : 		layout->head = entry->next;
             : 		free(entry->file);
             : 		free(entry->name);
             : 		free(entry);
             : 	}
             : 	free(layout);
> Given that `layout` is not modified in the while loop, how about evaluating it once? […]

Ack. Might make it less odd to read.