Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32734 )
Change subject: mb/supermicro/x11ssh: Add Supermicro X11SSH-TF ......................................................................
Patch Set 74:
Patch Set 74:
Patch Set 74:
I had a look on Supermicro's products list, especially on the X11 series, when I bought my X11SSM-F some days ago (I am currently working on porting coreboot to that board, too). I just realized that X11SSM-F and X11SSH-TF are basically the same board with just some minor differences (SSH has an m.2 slot and 10GBase-T).
I am wondering if it may be sensible to group those boards in a different scheme than suggested here (mainboards/x11ssh/tf), because we can support more boards this way more simple.
Many X11SS* boards share the same chipset and are more or less the same boards but have only minimal differences in internal/external interfaces.
My suggestion would be to group all these mainboards by chipset and make the rest configurable by selecting the specific board in Kconfig.
This could result in the following tree:
*snip*
Is this structure used somewhere else as well in the coreboot tree? I dont think we should reinvent something new and mix up chipsets with vendors and board variants.
I believe that renaming this board's folder to "x11ss" would be good enough to add both the X11SSH-TF and the X11SSM-F as variants, as well as any future variants. Looks like the 'X11' in the board names defines the generation (in this case, Intel Skylake), and the 'SS' part means "Single Socket", so all X11SS* board ports could be grouped together.
However, I'd prefer if the "variants without baseboard" Kconfig mechanism (e.g., what gigabyte/ga-h61m-s2pv does) would be used instead. I don't think we have a way of knowing what the baseboard for these supermicro boards would be.
I don't think splitting by chipset makes sense if all the boards were to run the same code, especially considering the additional nesting this would result in. However, I believe the X10 series is an older Intel platform than the X11 series, so code reuse between those series isn't worth implementing.
The resulting paths would be: src/mainboard/supermicro/x11ss/variants/x11ssh-tf/ src/mainboard/supermicro/x11ss/variants/x11ssm-f/
-
Angel Pons (Code Review)