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

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36328 )

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

Patch Set 5:

(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 185: And due to its complex nature, it is no surprise
             : if memory-controller initialization is kept secret.
> It's an example... but I wouldn't want to write it down here because […]

I'll try rephrasing the paragraph:

However, in the absence of such a firmware-development utopia, there
 may be more realistic approaches. As mentioned earlier, not all of FSP
 has a negative impact on development performance. For instance, FSP-M
 rarely causes trouble. Also, given the complex nature of high-speed
 interfaces, it is no surprise that information about memory controller
 initialization is kept secret. On the other hand, the configuration
 done by FSP-S overlaps or even conflicts with many of coreboot's duties.