[coreboot-gerrit] [L] Change in coreboot[main]: mb/asrock: Add SPR 1S server board ASRock Rack SPC741D8-2L2T/BCM

[Nico Huber (Code Review)]

Attention is currently required from: Angel Pons, Fabian Meyer, Paul Menzel.

Nico Huber has posted comments on this change by Fabian Meyer. ( https://review.coreboot.org/c/coreboot/+/82203?usp=email )

Change subject: mb/asrock: Add SPR 1S server board ASRock Rack SPC741D8-2L2T/BCM
......................................................................

Patch Set 9:

(1 comment)

File src/mainboard/asrock/spc741d8/ramstage.c:

https://review.coreboot.org/c/coreboot/+/82203/comment/51d39674_6fcc0896?usp... :
PS3, Line 15: 	// stub for EDK2
> SOFTWARE_I2C, however, seems to be designed as a developer option, nothing that
> should be selected by a driver (expect bus number conflicts if some other driver
> tries to use a SoC's I2C bus 0).

lol, I just realized that's what the driver already does... so forget what I
said about selecting it in the board Kconfig.