util/inteltool: ahci: rework AHCI

Rework AHCI to align the code with the rest of inteltool.

Change-Id: I37116f8e269d0376e147dd6de7365c45ac90bda0
Signed-off-by: Michael Niewöhner <foss@mniewoehner.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39504
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Michael Niewöhner 2020-03-13 21:15:55 +01:00 committed by Felix Held
parent 3c78445ad9
commit 8676c268a0
1 changed files with 19 additions and 18 deletions

View File

@ -59,7 +59,7 @@ static void print_port(const uint8_t *const mmio, size_t port)
int print_ahci(struct pci_dev *ahci) int print_ahci(struct pci_dev *ahci)
{ {
size_t mmio_size, i; size_t ahci_registers_size = 0, i;
if (!ahci) { if (!ahci) {
puts("No SATA device found"); puts("No SATA device found");
@ -67,15 +67,19 @@ int print_ahci(struct pci_dev *ahci)
} }
printf("\n============= AHCI Registers ==============\n\n"); printf("\n============= AHCI Registers ==============\n\n");
if (ahci->device_id == PCI_DEVICE_ID_INTEL_SUNRISEPOINT_SATA || switch (ahci->device_id) {
ahci->device_id == PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SATA) case PCI_DEVICE_ID_INTEL_SUNRISEPOINT_SATA:
mmio_size = 0x800; case PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SATA:
else ahci_registers_size = 0x800;
mmio_size = 0x400; break;
default:
ahci_registers_size = 0x400;
}
const pciaddr_t mmio_phys = ahci->base_addr[5] & ~0x7ULL; const pciaddr_t ahci_phys = ahci->base_addr[5] & ~0x7ULL;
printf("ABAR = 0x%08llx (MEM)\n\n", (unsigned long long)mmio_phys); printf("\n============= ABAR ==============\n\n");
const uint8_t *const mmio = map_physical(mmio_phys, mmio_size); printf("ABAR = 0x%08llx (MEM)\n\n", (unsigned long long)ahci_phys);
const uint8_t *const mmio = map_physical(ahci_phys, ahci_registers_size);
if (mmio == NULL) { if (mmio == NULL) {
perror("Error mapping MMIO"); perror("Error mapping MMIO");
exit(1); exit(1);
@ -91,21 +95,18 @@ int print_ahci(struct pci_dev *ahci)
} }
} }
const size_t max_ports = (mmio_size - 0x100) / 0x80; const size_t max_ports = (ahci_registers_size - 0x100) / 0x80;
for (i = 0; i < max_ports; i++) { for (i = 0; i < max_ports; i++) {
if (MMIO(0x0c) & 1 << i) if (MMIO(0x0c) & 1 << i)
print_port(mmio, i); print_port(mmio, i);
} }
if (ahci->device_id == PCI_DEVICE_ID_INTEL_SUNRISEPOINT_SATA ||
ahci->device_id == PCI_DEVICE_ID_INTEL_SUNRISEPOINT_LP_SATA) {
puts("\nOther registers:"); puts("\nOther registers:");
for (i = 0x500; i < mmio_size; i += 4) { for (i = 0x500; i < ahci_registers_size; i += 4) {
if (MMIO(i)) if (MMIO(i))
printf("0x%03zx: 0x%08x\n", i, MMIO(i)); printf("0x%03zx: 0x%08x\n", i, MMIO(i));
} }
}
unmap_physical((void *)mmio, mmio_size); unmap_physical((void *)mmio, ahci_registers_size);
return 0; return 0;
} }