[flashrom-gerrit] Change in flashrom[master]: layout: Introduce flashrom_layout_new()

[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/+/33543 )

Change subject: layout: Introduce flashrom_layout_new()
......................................................................

Patch Set 7:

(3 comments)

File flash.h:

https://review.coreboot.org/c/flashrom/+/33543/comment/a2f73282_0bd27cfe 
PS5, Line 268: struct flashrom_layout *default_layout;
> Is single_layout used anywhere else? I see that struct ich_layout is deleted in this patch, but comm […]

Removed, thanks for spotting!

File layout.c:

https://review.coreboot.org/c/flashrom/+/33543/comment/4c64d47c_25a07f3c 
PS2, Line 284:  out of memory.
> That would be implementation specific. I don't think the libflashrom […]

Ping?

File layout.c:

https://review.coreboot.org/c/flashrom/+/33543/comment/578e0e0d_31156dd7 
PS5, Line 397: malloc
> use calloc as it will always zero the heap.

Can you elaborate why it would be useful in this case?