GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

src/southbridge/amd/sb700: Reset SATA controller in AHCI mode during startup 

In AHCI mode SeaBIOS randomly fails to detect disks (AHCI timeouts),
with the probability of a failure increasing with the number of disks
connected to the controller.  Resetting the SATA controller appears to
show the true state of the underlying hardware, allowing the drive
detection code to attempt link renegotiation as needed.

Change-Id: Ib1f7c5f830a0cdba41cb6f5b05d759adee5ce369
Signed-off-by: Timothy Pearson <tpearson@raptorengineeringinc.com>
Reviewed-on: http://review.coreboot.org/11998
Tested-by: Raptor Engineering Automated Test Stand <noreply@raptorengineeringinc.com>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Tested-by: build bot (Jenkins)
This commit is contained in:
Timothy Pearson 2015-06-21 16:27:03 -05:00 committed by Martin Roth
parent 398540f568
commit 14ebf951e4
1 changed files with 35 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
src/southbridge/amd/sb700/sata.c
View File

@ -131,6 +131,8 @@ static void sata_init(struct device *dev)
/* get rev_id */ /* get rev_id */
rev_id = pci_read_config8(sm_dev, 0x08) - 0x28; rev_id = pci_read_config8(sm_dev, 0x08) - 0x28;
printk(BIOS_SPEW, "rev_id=%x\n", rev_id);
if (sata_ahci_mode) { if (sata_ahci_mode) {
/* Enable link latency enhancement on A14 and above */ /* Enable link latency enhancement on A14 and above */
if (rev_id >= 0x14) { if (rev_id >= 0x14) {
@ -241,6 +243,27 @@ static void sata_init(struct device *dev)
write32(sata_bar5 + 0xfc, dword); write32(sata_bar5 + 0xfc, dword);
} }
if (sata_ahci_mode) {
/* FIXME
* SeaBIOS does not know how to spin
* up the drives and therefore hangs
* in AHCI init if this is enabled...
*/
/* Enable staggered spin-up */
dword = read32(sata_bar5 + 0x00);
#if 0
dword |= 0x1 << 27;
#else
dword &= ~(0x1 << 27);
#endif
write32(sata_bar5 + 0x00, dword);
/* Reset the HBA to avoid stuck drives in SeaBIOS */
dword = read32(sata_bar5 + 0x04);
dword |= 0x1;
write32(sata_bar5 + 0x04, dword);
}
/* Write protect Sub-Class Code */ /* Write protect Sub-Class Code */
byte = pci_read_config8(dev, 0x40); byte = pci_read_config8(dev, 0x40);
byte &= ~(1 << 0); byte &= ~(1 << 0);
@ -367,21 +390,21 @@ static void sata_init(struct device *dev)
} }
/* Below is CIM InitSataLateFar */ /* Below is CIM InitSataLateFar */
/* Enable interrupts from the HBA */
byte = read8(sata_bar5 + 0x4);
byte |= 1 << 1;
write8((sata_bar5 + 0x4), byte);
if (!sata_ahci_mode) { if (!sata_ahci_mode) {
/* Clear error status */ /* Enable interrupts from the HBA */
write32((sata_bar5 + 0x130), 0xFFFFFFFF); byte = read8(sata_bar5 + 0x4);
write32((sata_bar5 + 0x1b0), 0xFFFFFFFF); byte |= 1 << 1;
write32((sata_bar5 + 0x230), 0xFFFFFFFF); write8((sata_bar5 + 0x4), byte);
write32((sata_bar5 + 0x2b0), 0xFFFFFFFF);
write32((sata_bar5 + 0x330), 0xFFFFFFFF);
write32((sata_bar5 + 0x3b0), 0xFFFFFFFF);
} }
/* Clear error status */
write32((sata_bar5 + 0x130), 0xFFFFFFFF);
write32((sata_bar5 + 0x1b0), 0xFFFFFFFF);
write32((sata_bar5 + 0x230), 0xFFFFFFFF);
write32((sata_bar5 + 0x2b0), 0xFFFFFFFF);
write32((sata_bar5 + 0x330), 0xFFFFFFFF);
write32((sata_bar5 + 0x3b0), 0xFFFFFFFF);
/* Clear SATA status,Firstly we get the AcpiGpe0BlkAddr */ /* Clear SATA status,Firstly we get the AcpiGpe0BlkAddr */
/* ????? why CIM does not set the AcpiGpe0BlkAddr , but use it??? */ /* ????? why CIM does not set the AcpiGpe0BlkAddr , but use it??? */