Change in coreboot[master]: src/southbridge/i82801gx: Include a default SeaBIOS config file - coreboot-gerrit

21 May 2017


      Kevin O'Connor has posted comments on this change. ( https://review.coreboot.org/19690 )
Change subject: src/southbridge/i82801gx: Include a default SeaBIOS config file
......................................................................
Patch Set 1:
...
(1 comment)
...
Why can’t this be enabled by default for coreboot?
Why isn't it the global default in SeaBIOS?
The SeaBIOS ATA DMA setting isn't safe on PATA, nor is it safe on SATA to PATA adapters.  On those devices, a complicated controller specific negotiation must be done to determine the DMA level.  The SeaBIOS ATA DMA support works fine on regular SATA devices, but almost all new machines use an AHCI controller now.
-- 
To view, visit https://review.coreboot.org/19690
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6a6e79ea6cd0a14070b8092aefa477b9e3a0c4b1
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans arthur@aheymans.xyz
Gerrit-Reviewer: Kevin O'Connor kevin@koconnor.net
Gerrit-Reviewer: Nico Huber nico.h@gmx.de
Gerrit-Reviewer: Paul Menzel paulepanter@users.sourceforge.net
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-HasComments: No