[coreboot-gerrit] Change in coreboot[master]: sb/intel/x/smbus.c: Factor out common code

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48364 )

Change subject: sb/intel/x/smbus.c: Factor out common code
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/48364/1/src/southbridge/intel/commo... 
File src/southbridge/intel/common/smbus_ops.c:

https://review.coreboot.org/c/coreboot/+/48364/1/src/southbridge/intel/commo... 
PS1, Line 77: 	/* The memory BAR does not exist for ICH7 and earlier */
            : 	if (CONFIG(SOUTHBRIDGE_INTEL_I82801GX))
            : 		return;
            : 
            : 	/* Also add MMIO resource */
            : 	res = pci_get_resource(dev, PCI_BASE_ADDRESS_0);
> Could we instead call the generic pci_dev_read_resources() and then patch […]

Done in CB:48538