Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35891 )
Change subject: mb/pcengines/apu2: Change default SATA mode to AHCI
......................................................................
Patch Set 3:
Patch Set 3:
As AHCI is nowadays more common than IDE, why not make this the chipset default?
Right, why not. It also speeds up the interface a little bit. But touching something that was unchanged for years leaves an uneasy feel of breaking something. I can give a try and change it globally for chipset, also hoping for more opinions on that change.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/35891
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1b0322392712d797dd5a8931150c8d0ff1b60940
Gerrit-Change-Number: 35891
Gerrit-PatchSet: 3
Gerrit-Owner: Piotr Kleinschmidt
piotr.kleinschmidt@3mdeb.com
Gerrit-Reviewer: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Michał Żygowski
michal.zygowski@3mdeb.com
Gerrit-Reviewer: Patrick Rudolph
siro@das-labor.org
Gerrit-Reviewer: Piotr Kleinschmidt
piotr.kleinschmidt@3mdeb.com
Gerrit-Reviewer: Piotr Król
piotr.krol@3mdeb.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Wed, 08 Jan 2020 16:42:14 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment