[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Implement __weak smihandler_soc_disable_busmaster

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58093 )

Change subject: soc/intel/common: Implement __weak smihandler_soc_disable_busmaster
......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS2:
> hi Tim,  […]

Thanks for always looking out for opportunities for improvement Kane, it is appreciated! You're right that it does still access the PMC I/O space, I am just wondering where the requirement for BME (which is supposed to be for DMA) comes from. coreboot's strategy is to use BME as little as possible; there are very few devices that require it to be set while coreboot is running.