2024-11-13 - coreboot Leadership Meeting Minutes

# 2024-11-13 - coreboot Leadership Meeting

## Attendees
Andre, Felix Held, Jay Talbott, Felix Singer, Julius Werner, Werner Zeh, Alicja Michalska, Mina
Asante, Jon Murphy, Matt DeVillier, Maximilian Brune, Ron Minnich, Karthik R.

## Open Action Items
  * 2024-10-30
    *  Add clarification to docs, “do not use gerrit change-id or CB: format in reference to
already-merged patches”.
  * 2024-10-16
    * Matt: Set up a meeting to discuss board status alternatives and send out invites. 
  * 2024-09-18
    * Jon: Schedule a dedicated meeting to discuss the Coverity defects and action plan.
      * Werner: Send out an invite for the meeting. 
        Sent out a poll to find a time slot:https://rallly.co/invite/1c8J3azXAcje
  * 2024-05-01
      * Nick Van Der Harst volunteered for Dutch. "gogo gogo" would like to translate to Russian (?)
  * 2024-01-10
      * Nico: (https://review.coreboot.org/q/topic:enforce_region_api)
    * Daniel: Look at how we want to localize (non console) strings for coreboot. Long term project.

## Minutes

### [Elyes] Any update on OpenSBI case …. ? (See 2024-07-24 - coreboot Leadership Meeting and
(https://review.coreboot.org/c/coreboot/+/84979) ).
  * [Mat]: It was planned to replace the submodule by a binary blob (see patch from Max:
https://review.coreboot.org/c/coreboot/+/83284). This patch is ready to be merged, we do not need
the 3rdparty git repo anymore for OpenSBI.
    * Matt will rebase Max’ patch and merge it in.
  * [Julius]: Shall we keep the old 3rdparty openSBI submodule in the tree or shall we get rid of it
after Max’s patch was merged?
    * [Max]: A lot of users have their own OpenSBI port, so having this submodule in the tree does not
bring us that much benefits. It is hard to update OpenSBI submodule currently (can’t build it with
our toolchain, there might be a missing linker (PIE: *Makefile:193: *** Your linker does not
support creating PIEs, opensbi requires this..  Stop.*) feature).  Not clear currently why our
linker is missing.
    * [Julius]: Looks like it's not actually querying the linker for that, you just need to set an
option?(https://github.com/riscv-software-src/opensbi/blob/master/Makefile#L192)
      * FelixS will give it a try!
      * If we are going to remove it from 3rdparty, we need to document the dependency to the users to
make them aware of the change.

* Conclusion: Keep it as long in the tree as somebody is going to fix the issue.

### [Karthik] Is it ok to enable FW_CONFIG in SMM stage
(https://review.coreboot.org/c/coreboot/+/85107/11/src/lib/Makefile.mk)
  * Are there any security concerns?
  * FW_CONFIG can use either EC or CBFS
    * [Julius]: Is the EC communication driver linked to SMM? –> Yes
    * [Karthik]: Since EC code is linked to SMM there is no issue to add fw_config to SMM from Google’s
point of view.
  * [Matt]:Isn’t fw_config cached on the first read? → Yes, but per stage only.
  * [Mat, FelixH]: Why not toggle the GPIOs even if the WWAN module is not equipped on the board? The
pins should be configured as GPIOs anyway, switching them unconditionally should not harm and will
enable us to not link fw_config into SMM. → Karthik will propose that to the ODMs, looks doable.

# Next Leadership Meeting
  November 27, 2024.
[coreboot Calendar](https://coreboot.org/calendar.html).

# Notice
Decisions shown here are not necessarily final, and are based
on the current information available. If there are questions or comments
about decisions made, or additional information to present, please put
it on the leadership meeting agenda and show up if possible to discuss
it.

Of course items may also be discussed on the mailing list, but as it's
difficult to interpret tone over email, controversial topics frequently
do not have good progress in those discussions.  For particularly
difficult issues, it may be best to try to schedule another meeting.

# coreboot leadership meeting minutes
https://docs.google.com/document/d/1NRXqXcLBp5pFkHiJbrLdv3Spqh1Hu086HYkKrgKj...