[coreboot-gerrit] Change in coreboot[master]: mb/supermicro/x11-lga1151-series: set mainboard_ops.init directly

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48083 )

Change subject: mb/supermicro/x11-lga1151-series: set mainboard_ops.init directly
......................................................................

Patch Set 5: Code-Review-1

(1 comment)

https://review.coreboot.org/c/coreboot/+/48083/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/48083/5//COMMIT_MSG@9 
PS5, Line 9: Set mainboard_ops.init directly instead of using the indirection via a
           : mainboard_enable function.
chip.init is run significantly earlier than dev.init, before resources are allocated. For the things you do, it might not be a problem but it's generally not a good idea to do init stuff before resource allocation.