[coreboot-gerrit] Change in coreboot[master]: device: Add method to configure bus mastering based on Kconfig

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42459 )

Change subject: device: Add method to configure bus mastering based on Kconfig
......................................................................

Patch Set 16:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42459/16/src/device/Kconfig 
File src/device/Kconfig:

https://review.coreboot.org/c/coreboot/+/42459/16/src/device/Kconfig@529 
PS16, Line 529: 	  and is enabled by default to keep the behaviour for now.
It only has to be done in the payload for devices that are used by the
payload. I'd phrase it as follows:

For security reasons, bus mastering should be enabled as late as
    possible. In coreboot, it's usually not necessary and payloads
    should only enable it for devices they use. Since not all payloads
    enable bus mastering properly yet, this option gives some sort of
    "backwards compatibility" and is enabled by default to keep the
    traditional behaviour for now. This is currently necessary, for
    instance, for libpayload based payloads as the drivers don't enable
    bus mastering for PCI bridges.