New patch to review for coreboot: 6bf01cb Reduce default verbosity of SPI flash drivers

[Stefan Reinauer]

Stefan Reinauer ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1045

-gerrit

commit 6bf01cb29d9a59090affae34724b3220fd19d8cb
Author: Stefan Reinauer 
Date:   Wed May 23 11:03:29 2012 -0700

Reduce default verbosity of SPI flash drivers

Only print PP: lines if CONFIG_DEBUG_SPI_FLASH is enabled.

Change-Id: If25e916ecb585f37c90d42980e933a6cd1a3d956
    Signed-off-by: Stefan Reinauer 
---
 src/drivers/spi/eon.c      |    2 ++
 src/drivers/spi/macronix.c |    2 ++
 src/drivers/spi/spansion.c |    2 ++
 src/drivers/spi/stmicro.c  |    2 ++
 src/drivers/spi/winbond.c  |    2 ++
 5 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/src/drivers/spi/eon.c b/src/drivers/spi/eon.c
index b7c0630..7d9bc44 100644
--- a/src/drivers/spi/eon.c
+++ b/src/drivers/spi/eon.c
@@ -85,9 +85,11 @@ static int eon_write(struct spi_flash *flash,
    	cmd[2] = page_addr;
    	cmd[3] = byte_addr;

+#if CONFIG_DEBUG_SPI_FLASH
    	printk(BIOS_SPEW,
    	    "PP: 0x%p => cmd = { 0x%02x 0x%02x%02x%02x } chunk_len = %zu\n",
    	     buf + actual, cmd[0], cmd[1], cmd[2], cmd[3], chunk_len);
+#endif

ret = spi_flash_cmd(flash->spi, CMD_EN25Q128_WREN, NULL, 0);
    	if (ret < 0) {
diff --git a/src/drivers/spi/macronix.c b/src/drivers/spi/macronix.c
index 84e79ef..f74eb4f 100644
--- a/src/drivers/spi/macronix.c
+++ b/src/drivers/spi/macronix.c
@@ -141,9 +141,11 @@ static int macronix_write(struct spi_flash *flash,
    	cmd[2] = (offset >> 8) & 0xff;
    	cmd[3] = offset & 0xff;

+#if CONFIG_DEBUG_SPI_FLASH
    	printk(BIOS_SPEW, "PP: 0x%p => cmd = { 0x%02x 0x%02x%02x%02x }"
    	     " chunk_len = %zu\n",
    	     buf + actual, cmd[0], cmd[1], cmd[2], cmd[3], chunk_len);
+#endif

ret = spi_flash_cmd(flash->spi, CMD_MX25XX_WREN, NULL, 0);
    	if (ret < 0) {
diff --git a/src/drivers/spi/spansion.c b/src/drivers/spi/spansion.c
index 5479cae..511091b 100644
--- a/src/drivers/spi/spansion.c
+++ b/src/drivers/spi/spansion.c
@@ -162,9 +162,11 @@ static int spansion_write(struct spi_flash *flash,
    	cmd[2] = page_addr;
    	cmd[3] = byte_addr;

+#if CONFIG_DEBUG_SPI_FLASH
    	printk(BIOS_SPEW, "PP: 0x%p => cmd = { 0x%02x 0x%02x%02x%02x }"
    	     " chunk_len = %zu\n",
    	     buf + actual, cmd[0], cmd[1], cmd[2], cmd[3], chunk_len);
+#endif

ret = spi_flash_cmd(flash->spi, CMD_S25FLXX_WREN, NULL, 0);
    	if (ret < 0) {
diff --git a/src/drivers/spi/stmicro.c b/src/drivers/spi/stmicro.c
index 9322c06..ccdfa00 100644
--- a/src/drivers/spi/stmicro.c
+++ b/src/drivers/spi/stmicro.c
@@ -163,9 +163,11 @@ static int stmicro_write(struct spi_flash *flash,
    	cmd[2] = page_addr;
    	cmd[3] = byte_addr;

+#if CONFIG_DEBUG_SPI_FLASH
    	printk(BIOS_SPEW, "PP: 0x%p => cmd = { 0x%02x 0x%02x%02x%02x }"
    	     " chunk_len = %zu\n",
    	     buf + actual, cmd[0], cmd[1], cmd[2], cmd[3], chunk_len);
+#endif

ret = spi_flash_cmd(flash->spi, CMD_M25PXX_WREN, NULL, 0);
    	if (ret < 0) {
diff --git a/src/drivers/spi/winbond.c b/src/drivers/spi/winbond.c
index 7b6d105..01644b0 100644
--- a/src/drivers/spi/winbond.c
+++ b/src/drivers/spi/winbond.c
@@ -130,9 +130,11 @@ static int winbond_write(struct spi_flash *flash,
    	cmd[1] = (offset >> 16) & 0xff;
    	cmd[2] = (offset >> 8) & 0xff;
    	cmd[3] = offset & 0xff;
+#if CONFIG_DEBUG_SPI_FLASH
    	printk(BIOS_SPEW, "PP: 0x%p => cmd = { 0x%02x 0x%02x%02x%02x }"
    	        " chunk_len = %zu\n", buf + actual,
    		cmd[0], cmd[1], cmd[2], cmd[3], chunk_len);
+#endif

ret = spi_flash_cmd(flash->spi, CMD_W25_WREN, NULL, 0);
    	if (ret < 0) {