Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35622 )
Change subject: mainboard/supermicro/x11-lga1151-series: Rename to x11-lga1151-skykbl ......................................................................
Patch Set 6:
Patch Set 6: Code-Review-1
I like the current name. Everyone is calling the Coffee Lake socket LGA1151v2 (or r2) anyway.
I am not 100% sure, but I guess Cannon Lake also uses this socket. Might be better to split this up as well or?
Cannon Lake doesn't have a socket at all.
Huh?
option 1)We could use x11ss- as prefix (and ignore that maybe the two exceptions X11SAT and X11SAE get into this group some day); The full name would be x11ss-lga1151-series and x11sc-lga1151-series, then.
Please don't. No "you have to know Supermicro's encoding", please.
You don't need to know anything about Supermicro's encoding... when I have a board X11SS<whatever>, I would look in the tree if there is a folder X11SS<blah>... it's that simple ;-)
option 2) keep x11s- as prefix. Then we have to separate v5/v6 (skl/kbl), v7/v8 (cnl/cfl) by another tag. Examples: x11s-v5v6- and x11s-v7v8- or x11s-sklkbl- and x11s-cnlcfl- Then we have 29 Boards with Socket 1151v1, and some with some FCBGA-*.
Sounds all much more confusing than the current name, to me.
However, if there are indeed BGA versions, then the chipset would be the next best thing to identify the line. So I throw `x11-sunrisepoint` into the ring.
That isn't sufficient, as there are many sp board that are absolutely NOT compatible with the two we have currently.