[coreboot-gerrit] Patch set updated for coreboot: ef165f3 spi: Change 'page' to 'sector' in log message

David Imhoff (dimhoff_devel@xs4all.nl) gerrit at coreboot.org
Sun May 3 17:18:11 CEST 2015


David Imhoff (dimhoff_devel at xs4all.nl) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10067

-gerrit

commit ef165f3487af053bb633bdf90084d2382ff2a7ee
Author: David Imhoff <dimhoff_devel at xs4all.nl>
Date:   Sun May 3 13:47:49 2015 +0200

    spi: Change 'page' to 'sector' in log message
    
    The log message says 'page size' while actually the sector size is
    printed. This is confusing since for stmicro page size != sector size.
    Also add '0x' prefix to numbers to make it clear they are in hex.
    
    TEST=Build and booted on Minnowboard Max
    
    Change-Id: I795a4b7c1bc8de2538a87fd4ba56f5a78d9ca2ac
    Signed-off-by: David Imhoff <dimhoff_devel at xs4all.nl>
---
 src/drivers/spi/spi_flash.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/drivers/spi/spi_flash.c b/src/drivers/spi/spi_flash.c
index d40f72a..d2a3c66 100644
--- a/src/drivers/spi/spi_flash.c
+++ b/src/drivers/spi/spi_flash.c
@@ -381,7 +381,7 @@ flash_detected:
 	tseg_relocate((void **)&flash->name);
 #endif
 
-	printk(BIOS_INFO, "SF: Detected %s with page size %x, total %x\n",
+	printk(BIOS_INFO, "SF: Detected %s with sector size 0x%x, total 0x%x\n",
 			flash->name, flash->sector_size, flash->size);
 
 	spi_flash_dev = flash;



More information about the coreboot-gerrit mailing list