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

[Nico Huber (Code Review)]

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

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

Patch Set 7:

(3 comments)

Commit Message:

https://review.coreboot.org/c/flashrom/+/33542/comment/0939b4b4_83a5a2c7 
PS3, Line 7: layout: Introduce layout_next()
           : 
           : Also, a `layout.c` internal version _layout_next()
> Yes, thank you, definitely better! Just commit message left (still old one).

Done

File layout.c:

https://review.coreboot.org/c/flashrom/+/33542/comment/a45179ff_4bf7efa3 
PS3, Line 44: static struct romentry *_layout_next(
> Nico is correct, one is const while the other isn't. […]

Done?

File libflashrom.c:

https://review.coreboot.org/c/flashrom/+/33542/comment/868a29f0_fd3fe0a7 
PS2, Line 351: 			chip_entry = layout_next(&chip_layout->base, chip_entry);
             : 			dump_entry = layout_next(&dump_layout.base, dump_entry);
> Anything to do about it? I could add more pointers, but I don't think […]

Ping