[PATCH qemu 08/12] m25p80: add mx25l25635f chip
Cédric Le Goater
clg at kaod.org
Mon May 30 07:20:01 AEST 2016
The Macronix chip mx25l25635f used on some OpenPower boxes is very
similar to the mx25l25635e. They share the same JEDEC identifier but
the WRSR instruction requires 2 bytes in the mx25l25635f case.
This patch introduce a new chip to prevent some warnings on guests
identifying JEDEC 0xc22019 as a MX25L25635F chip. OpenBMC is one them.
Signed-off-by: Cédric Le Goater <clg at kaod.org>
---
This is may not be the best way to distinguish these chips. Adding a
flag in the FlashPartInfo would be better ?
hw/block/m25p80.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c
index 02f77012c21b..58607786c11c 100644
--- a/hw/block/m25p80.c
+++ b/hw/block/m25p80.c
@@ -157,6 +157,7 @@ static const FlashPartInfo known_devices[] = {
{ INFO("mx25l12805d", 0xc22018, 0, 64 << 10, 256, 0) },
{ INFO("mx25l12855e", 0xc22618, 0, 64 << 10, 256, 0) },
{ INFO("mx25l25635e", 0xc22019, 0, 64 << 10, 512, 0) },
+ { INFO("mx25l25635f", 0xc22019, 0, 64 << 10, 512, 0) },
{ INFO("mx25l25655e", 0xc22619, 0, 64 << 10, 512, 0) },
/* Micron */
@@ -692,6 +693,9 @@ static void decode_new_cmd(Flash *s, uint32_t value)
s->pos = 0;
s->len = 0;
s->state = STATE_COLLECTING_DATA;
+ if (!strcmp(s->pi->part_name, "mx25l25635f")) {
+ s->needed_bytes = 2;
+ }
}
break;
--
2.1.4
More information about the openbmc
mailing list