[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Call pci_dev_request_bus_master() from .final ops

[Angel Pons (Code Review)]

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

Change subject: soc/intel/common: Call pci_dev_request_bus_master() from .final ops
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/44512/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/44512/2//COMMIT_MSG@15 
PS2, Line 15: access
> Great plan. […]

Something that would be good to know is whether Bus Master needs to be enabled *while* initializing the device. AFAIK, the only device for which BWGs explicitly state that Bus Master needs to be set during initialization is the iGPU.

If we don't need to enable Bus Master at all until we're ready to jump to the payload, we can postpone setting it as much as possible to mitigate DMA attacks