Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36328 )
Change subject: [RFC] Documentation/fsp: Discuss FSP-S issues ......................................................................
Patch Set 3:
(1 comment)
Patch Set 2: Code-Review-2
(1 comment)
Setting as -2 until the legal section is either removed or vetted by actual lawyers; If we're gonna pour this much gas on the fire we should have a very solid reason for doing so.
Ok, I've dropped it. Not that I would agree with you by a single percent. Just because it gives me a much too high blood pressure to discuss it with hotheads. There was no accusation whatsoever, not even a statement (beside the heading which was already true if a single person in the world has doubts). That the mere finding that somebody could be uncertain makes you uncomfortable makes me wonder what world you live in. If things are that bad in the US, sorry, you have my condolence.
https://review.coreboot.org/c/coreboot/+/36328/2/Documentation/fsp/fsp-s_dis... File Documentation/fsp/fsp-s_discussion.md:
https://review.coreboot.org/c/coreboot/+/36328/2/Documentation/fsp/fsp-s_dis... PS2, Line 173: FSP Creates Legal Uncertainty
Where do you see any speculation? […]
I am not a lawyer either, but kind of a language lawyer. I took great care not to speculate and not to provide any statement (beside the heading) in this section. So please help me with my English here:
* If a single person is uncertain because of FSP, then FSP creates uncertainty, no? * `(to) seem` is subjective. How can the sentence not be true? Isn't it enough if we (or I) don't know any preceding instant? * The last sentence provides a comparison and is written subjunctively. How the hell can it accuse of anything?
My intention was not to discuss any legal matters anyway. All I wanted was to create awareness of these concerns.