Paul Menzel has uploaded this change for review.

View Change

drivers/spi: Instrument erase operation

Verify console output contains the new string.

BS: BS_DEV_INIT times (us): entry 0 run 72771 exit 0
Finalize devices...
Devices finalized

APIC 00: ** Enter AmdInitLate [00020004]
SB800: sb_Late_Post
AmdInitLate() returned AGESA_SUCCESS
APIC 00: Heap in SystemMem (4) at 0x10000014
APIC 00: ** Exit AmdInitLate [00020004]

APIC 00: ** Enter AmdS3Save [0002000b]
Manufacturer: ef
SF: Detected W25Q32_V with sector size 0x1000, total 0x400000
SF: Successfully erased 4096 bytes @ 0xffff1000 in 97754 usecs
Manufacturer: ef
SF: Detected W25Q32_V with sector size 0x1000, total 0x400000
SF: Successfully erased 4096 bytes @ 0xffff0000 in 121777 usecs
AmdS3Save() returned AGESA_SUCCESS
APIC 00: Heap in SystemMem (4) at 0x10000014
APIC 00: ** Exit AmdS3Save [0002000b]
BS: BS_POST_DEVICE times (us): entry 0 run 3 exit 228371

Change-Id: I4eb8a850d7263bc3c800d97655f0b37623b78562
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
M src/drivers/spi/spi_flash.c
1 file changed, 5 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/31793/1
diff --git a/src/drivers/spi/spi_flash.c b/src/drivers/spi/spi_flash.c
index d1f5227..7f64968 100644
--- a/src/drivers/spi/spi_flash.c
+++ b/src/drivers/spi/spi_flash.c
@@ -198,6 +198,9 @@
u32 start, end, erase_size;
int ret = -1;
u8 cmd[4];
+ struct stopwatch sw;
+
+ stopwatch_init(&sw);

erase_size = flash->sector_size;
if (offset % erase_size || len % erase_size) {
@@ -234,7 +237,8 @@
goto out;
}

- printk(BIOS_DEBUG, "SF: Successfully erased %zu bytes @ %#x\n", len, start);
+ printk(BIOS_DEBUG, "SF: Successfully erased %zu bytes @ %#x in %ld usecs\n",
+ len, start, stopwatch_duration_usecs(&sw));

out:
return ret;

To view, visit change 31793. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4eb8a850d7263bc3c800d97655f0b37623b78562
Gerrit-Change-Number: 31793
Gerrit-PatchSet: 1
Gerrit-Owner: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: newchange