Nico Huber has submitted this change and it was merged. ( https://review.coreboot.org/c/flashrom/+/34234 )
Change subject: flashchips: Add Macronix MX25L51245G as known chip ......................................................................
flashchips: Add Macronix MX25L51245G as known chip
MX25L51245G is identical to handling of MX66L51235F.
Change-Id: I964e630197e33d69b199fdfb8816f18e3112bbb1 Signed-off-by: Hemanth Guruva Reddy meethemanth@gmail.com Reviewed-on: https://review.coreboot.org/c/flashrom/+/34234 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Nico Huber nico.h@gmx.de --- M flashchips.c M flashchips.h 2 files changed, 2 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, approved
diff --git a/flashchips.c b/flashchips.c index 7f31c30..166af6a 100644 --- a/flashchips.c +++ b/flashchips.c @@ -9441,7 +9441,7 @@
{ .vendor = "Macronix", - .name = "MX66L51235F", + .name = "MX66L51235F/MX25L51245G", .bustype = BUS_SPI, .manufacture_id = MACRONIX_ID, .model_id = MACRONIX_MX66L51235F, diff --git a/flashchips.h b/flashchips.h index 1c7ba99..006b95e 100644 --- a/flashchips.h +++ b/flashchips.h @@ -490,7 +490,7 @@ #define MACRONIX_MX25L25635F 0x2019 /* Same as MX25L25639F, but the latter seems to not support REMS */ #define MACRONIX_MX25L1635D 0x2415 #define MACRONIX_MX25L1635E 0x2515 /* MX25L1635{E} */ -#define MACRONIX_MX66L51235F 0x201a /* MX66L51235F */ +#define MACRONIX_MX66L51235F 0x201a /* MX66L51235F, MX25L51245G */ #define MACRONIX_MX25U8032E 0x2534 #define MACRONIX_MX25U1635E 0x2535 #define MACRONIX_MX25U3235E 0x2536 /* Same as MX25U6435F */