Werner Zeh 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+2
(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 155: Missing Communication and Responsibility
However, we do not have any obligation to provide some sort of FSP help desk. I try to help as I am able, but keep in mind that I have a regular day job as well working on getting the firmware for our upcoming chips ready to ship. It is unreasonable to expect me to be able to answer every question in my very limited spare time, and management is unwilling to formally assign people to something that won't directly impact chip sales.
So on a certain level we agree: Intel does not have the resources to provide a full featured support for the FSP. The people inside Intel taking care of FSP have a lot of different tasks to do so their time is limited. On the other hand there is this already huge grown coreboot community which would love to write code in high quality for every new platform that arrives. So the only thing that hinders the community to help Intel out is the lack of information and the mindset at Intel to hide more and more of the internals of the chipsets. I never got the real reason for this hiding mentality, what is it? - Is it that a competitor may have a more detailed view to the internals? I guess this audience (other CPU vendors) would have much more powerful ways of getting the implementation details and would not rely on open source firmware alone. And, speaking of the FSP-S part, I doubt that there are _the_ most important assets in there (it might be different for FSP-M) - Is it hiding a possibly awkward way of a given implementation because of limited time or resource at design time? This would be nothing new for the community and nobody would really blame Intel for that if the access to the needed information would be open. In contrast, if this will be open, the community would have a much better way to deal with it and the most of us will be happy.
So finally I am sure that opening up more stuff to the community would help both sides and make all of us happy again.