[coreboot-gerrit] Patch set updated for coreboot: drivers/spi: Measure execution time of `spi_flash_probe()`

[Paul Menzel]

Paul Menzel ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18503

-gerrit

commit 1c161da0148c17a921b0cc7aef6f7f99b4c238d9
Author: Paul Menzel 
Date:   Sun Feb 26 19:58:24 2017 +0100

drivers/spi: Measure execution time of `spi_flash_probe()`

Stop the time of the execution of the method `spi_flash_probe()`. It’s
    useful to analyze the boot process, and to compare different flash ROM
    chips.

TEST=Build and boot on ASRock E350M1, and observe the new messages.

```
    Manufacturer: ef
    SF: Detected W25Q32 with sector size 0x1000, total 0x400000
    spi_flash_probe: Probing the flash chip took 24 usecs
    SF: Successfully erased 4096 bytes @ 0xffff1000
    Manufacturer: ef
    SF: Detected W25Q32 with sector size 0x1000, total 0x400000
    spi_flash_probe: Probing the flash chip took 21 usecs
    SF: Successfully erased 4096 bytes @ 0xffff0000
    ```

Change-Id: I448e12508b09a8796c5351b17495113afe9a50ef
    Signed-off-by: Paul Menzel 
---
 src/drivers/spi/spi_flash.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/drivers/spi/spi_flash.c b/src/drivers/spi/spi_flash.c
index 95362f2..b2e928b 100644
--- a/src/drivers/spi/spi_flash.c
+++ b/src/drivers/spi/spi_flash.c
@@ -331,6 +331,9 @@ struct spi_flash *spi_flash_probe(unsigned int bus, unsigned int cs)
 {
    struct spi_slave spi;
    struct spi_flash *flash;
+	struct stopwatch sw;
+
+	stopwatch_init(&sw);

if (spi_setup_slave(bus, cs, &spi)) {
    	printk(BIOS_WARNING, "SF: Failed to set up slave\n");
@@ -365,6 +368,9 @@ struct spi_flash *spi_flash_probe(unsigned int bus, unsigned int cs)
    	CONFIG_BOOT_DEVICE_SPI_FLASH_BUS == bus && !spi_flash_dev)
    	spi_flash_dev = flash;

+	printk(BIOS_DEBUG, "%s: Probing the flash chip took %ld usecs\n",
+		__func__, stopwatch_duration_usecs(&sw));
+
    return flash;
 }