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

[David Hendricks (Code Review)]

David Hendricks 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: Code-Review-1

(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 194: or raise legal concerns
> > > Thanks for pointing this out. As mentioned somewhere above, there is a lack of documentation. […]

I was originally thinking the words here were too vague to really matter... However I tend to agree with Nate that we should err on the side of caution and avoid any potentially confusing references to legal matters. Nico might know exactly what he's referring to, but another observer (a manager, lawyer, etc) who does not immediately understand the nuances can quickly shut down discussion.

These few words are simply not worth the hassle, and there is practically nothing to be gained by having them here IMO. We can find other ways to discuss underlying concerns, just not here.