From patchwork Mon Jan 21 14:08:37 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [1/2] hw/pflash_cfi01.c: Make read after byte-write or erase return status Date: Mon, 21 Jan 2013 04:08:37 -0000 From: Peter Maydell X-Patchwork-Id: 214173 Message-Id: <1358777318-7579-2-git-send-email-peter.maydell@linaro.org> To: qemu-devel@nongnu.org Cc: Anthony Liguori , patches@linaro.org The Intel flash command set requires that a read operation after doing a 'single byte write' command returns the status register; add this case to pflash_read() so we return the correct information. Similarly, the case for the 0x28 flavour of block erase was missing. Signed-off-by: Peter Maydell Reviewed-by: Peter Crosthwaite Tested-by: Peter Crosthwaite --- hw/pflash_cfi01.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/hw/pflash_cfi01.c b/hw/pflash_cfi01.c index aadedef..310c716 100644 --- a/hw/pflash_cfi01.c +++ b/hw/pflash_cfi01.c @@ -162,7 +162,10 @@ static uint32_t pflash_read (pflash_t *pfl, hwaddr offset, } break; + case 0x10: /* Single byte program */ case 0x20: /* Block erase */ + case 0x28: /* Block erase */ + case 0x40: /* single byte program */ case 0x50: /* Clear status register */ case 0x60: /* Block /un)lock */ case 0x70: /* Status Register */