[coreboot-gerrit] Change in coreboot[master]: drivers/spi/spi_flash: organize spi flash by sector topology

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38379 )

Change subject: drivers/spi/spi_flash: organize spi flash by sector topology
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38379/3/src/drivers/spi/spi_flash_i... 
File src/drivers/spi/spi_flash_internal.h:

https://review.coreboot.org/c/coreboot/+/38379/3/src/drivers/spi/spi_flash_i... 
PS3, Line 78: 	uint32_t id;
> nit: Could consider splitting this into two uint16_t which would allow for tighter packing in an arr […]

Good call out. I'll swap that out when I get around to removing the name field.