Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42460 )
Change subject: treewide: Configure PCI Bus Master based on Kconfig option ......................................................................
Patch Set 13:
(2 comments)
https://review.coreboot.org/c/coreboot/+/42460/13/src/soc/intel/broadwell/hd... File src/soc/intel/broadwell/hda.c:
https://review.coreboot.org/c/coreboot/+/42460/13/src/soc/intel/broadwell/hd... PS13, Line 96: pci_dev_request_bus_master(dev);
AFAIUI, HDA init sequence sets bus master. […]
No idea. We can't do tests for all platforms. But at this moment this change doesn't have any effect since the related option is enabled by default.
https://review.coreboot.org/c/coreboot/+/42460/13/src/southbridge/intel/lynx... File src/southbridge/intel/lynxpoint/azalia.c:
https://review.coreboot.org/c/coreboot/+/42460/13/src/southbridge/intel/lynx... PS13, Line 115: pci_dev_request_bus_master(dev);
AFAIUI, Azalia init sequence sets bus master. […]
No idea. We can't do tests for all platforms. But at this moment this change doesn't have any effect since the related option is enabled by default.