[coreboot-gerrit] Change in coreboot[master]: util/spd_tools: Add support for exclusive IDs

[Robert Zieba (Code Review)]

Attention is currently required from: Raul Rangel, Reka Norman, Nick Vaccaro, Karthik Ramasubramanian.
Robert Zieba has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62905 )

Change subject: util/spd_tools: Add support for exclusive IDs
......................................................................

Patch Set 7:

(2 comments)

File util/spd_tools/src/part_id_gen/part_id_gen.go:

https://review.coreboot.org/c/coreboot/+/62905/comment/a37c9ee6_e1ae2484 
PS5, Line 332: &&
> From consistency standpoint, can we add Exclusive entries to SPDToIndexMap. […]

Unfortunately this wouldn't result in the right behavior. If an exclusive entry is present in SPDToIndexMap then the auto-assigning code will continue through line 384 resulting in the part at that index getting overwritten. Additionally, attempting to use both Fixed and Exclusive with the same slot is an error.

https://review.coreboot.org/c/coreboot/+/62905/comment/b33bb494_7ee84486 
PS5, Line 359: if index < len(assignedMapping) {
             : 				mapping = assignedMapping[index]
             : 			}
> This check can be removed if you append assignedMapping after line 385: […]

Done