[coreboot-gerrit] Change in coreboot[master]: [RFC] Documentation/fsp: Discuss FSP-S issues

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36328 )

Change subject: [RFC] Documentation/fsp: Discuss FSP-S issues
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36328/4/Documentation/fsp/fsp-s_dis... 
File Documentation/fsp/fsp-s_discussion.md:

https://review.coreboot.org/c/coreboot/+/36328/4/Documentation/fsp/fsp-s_dis... 
PS4, Line 89: * One either has to assume that FSP doesn't perform any locking
            :   and do it redundantly in the firmware framework, or expensively
            :   test what is locked and keep doing so for every new FSP revision.
> Well, about the "when": How about last? Also, having it documented at least […]

@Nathaniel Well, why not default to locking an leave the possibility to disable it?