Clean up a25.c, at25.c, spi25.c
- introduce spi_prettyprint_status_register_atmel_at25_wpen()
- use spi_prettyprint_status_register_bit() where possible
- generify spi_prettyprint_status_register_bp3210 and use it in at25.c too
Corresponding to flashrom svn r1560.
Signed-off-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
Acked-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
diff --git a/spi25.c b/spi25.c
index a6e7e21..3693783 100644
--- a/spi25.c
+++ b/spi25.c
@@ -327,21 +327,24 @@
}
/* Prettyprint the status register. Common definitions. */
-void spi_prettyprint_status_register_bp3210(uint8_t status, int bp)
+void spi_prettyprint_status_register_bp(uint8_t status, int bp)
{
switch (bp) {
/* Fall through. */
+ case 4:
+ msg_cdbg("Chip status register: Block Protect 4 (BP4) "
+ "is %sset\n", (status & (1 << 5)) ? "" : "not ");
case 3:
- msg_cdbg("Chip status register: Bit 5 / Block Protect 3 (BP3) "
+ msg_cdbg("Chip status register: Block Protect 3 (BP3) "
"is %sset\n", (status & (1 << 5)) ? "" : "not ");
case 2:
- msg_cdbg("Chip status register: Bit 4 / Block Protect 2 (BP2) "
+ msg_cdbg("Chip status register: Block Protect 2 (BP2) "
"is %sset\n", (status & (1 << 4)) ? "" : "not ");
case 1:
- msg_cdbg("Chip status register: Bit 3 / Block Protect 1 (BP1) "
+ msg_cdbg("Chip status register: Block Protect 1 (BP1) "
"is %sset\n", (status & (1 << 3)) ? "" : "not ");
case 0:
- msg_cdbg("Chip status register: Bit 2 / Block Protect 0 (BP0) "
+ msg_cdbg("Chip status register: Block Protect 0 (BP0) "
"is %sset\n", (status & (1 << 2)) ? "" : "not ");
}
}
@@ -355,7 +358,7 @@
static void spi_prettyprint_status_register_common(uint8_t status)
{
- spi_prettyprint_status_register_bp3210(status, 3);
+ spi_prettyprint_status_register_bp(status, 3);
spi_prettyprint_status_register_welwip(status);
}