[coreboot-gerrit] Change in ...coreboot[master]: sb/intel/i82801gx: Autodisable functions based on devicetree

[Arthur Heymans (Code Review)]

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

Change subject: sb/intel/i82801gx: Autodisable functions based on devicetree
......................................................................

Patch Set 12:

(1 comment)

https://review.coreboot.org/#/c/30244/12/src/southbridge/intel/i82801gx/i828... 
File src/southbridge/intel/i82801gx/i82801gx.c:

https://review.coreboot.org/#/c/30244/12/src/southbridge/intel/i82801gx/i828... 
PS12, Line 25: 	switch (devfn) {
> why is internal lan missing ?

The internal LAN is on the bus behind the PCI bridge afaict from the datasheets. No coreboot device is using it, so it was easier to still disable it in the romstage.