From: "Philippe Mathieu-Daudé" <philmd@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Kevin Wolf" <kwolf@redhat.com>,
"Laurent Vivier" <lvivier@redhat.com>,
"Thomas Huth" <thuth@redhat.com>,
"Stephen Checkoway" <stephen.checkoway@oberlin.edu>,
qemu-block@nongnu.org, "Max Reitz" <mreitz@redhat.com>,
"Alistair Francis" <alistair.francis@wdc.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@redhat.com>
Subject: [Qemu-devel] [PULL 22/27] hw/block/pflash_cfi02: Fix reset command not ignored during erase
Date: Mon, 1 Jul 2019 21:59:07 -0300 [thread overview]
Message-ID: <20190702005912.15905-23-philmd@redhat.com> (raw)
In-Reply-To: <20190702005912.15905-1-philmd@redhat.com>
From: Stephen Checkoway <stephen.checkoway@oberlin.edu>
When the flash device is performing a chip erase, all commands are
ignored. When it is performing a sector erase, only the erase suspend
command is valid, which is currently not supported.
In particular, the reset command should not cause the device to reset to
read array mode while programming is on going.
Signed-off-by: Stephen Checkoway <stephen.checkoway@oberlin.edu>
Message-Id: <20190426162624.55977-8-stephen.checkoway@oberlin.edu>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
hw/block/pflash_cfi02.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/block/pflash_cfi02.c b/hw/block/pflash_cfi02.c
index 13f76fa71d..39daa95833 100644
--- a/hw/block/pflash_cfi02.c
+++ b/hw/block/pflash_cfi02.c
@@ -311,7 +311,8 @@ static void pflash_write(void *opaque, hwaddr offset, uint64_t value,
trace_pflash_io_write(offset, width, width << 1, value, pfl->wcycle);
cmd = value;
if (pfl->cmd != 0xA0) {
- if (cmd == 0xF0) {
+ /* Reset does nothing during chip erase and sector erase. */
+ if (cmd == 0xF0 && pfl->cmd != 0x10 && pfl->cmd != 0x30) {
if (pfl->wcycle == WCYCLE_AUTOSELECT_CFI) {
/* Return to autoselect mode. */
pfl->wcycle = 3;
--
2.20.1
next prev parent reply other threads:[~2019-07-02 3:31 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-02 0:58 [Qemu-devel] [PULL 00/27] pflash-next patches Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 01/27] tests/pflash-cfi02: Add test for supported CFI commands Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 02/27] hw/block/pflash: Simplify trace_pflash_io_read/write() Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 03/27] hw/block/pflash: Simplify trace_pflash_data_read/write() Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 04/27] hw/block/pflash_cfi02: Fix debug format string Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 05/27] hw/block/pflash_cfi02: Add an enum to define the write cycles Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 06/27] hw/block/pflash_cfi02: Add helpers to manipulate the status bits Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 07/27] hw/block/pflash_cfi02: Simplify a statement using fall through Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 08/27] hw/block/pflash_cfi02: Use the ldst API in pflash_write() Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 09/27] hw/block/pflash_cfi02: Use the ldst API in pflash_read() Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 10/27] hw/block/pflash_cfi02: Extract the pflash_data_read() function Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 11/27] hw/block/pflash_cfi02: Unify the MemoryRegionOps Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 12/27] hw/block/pflash_cfi02: Fix command address comparison Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 13/27] tests/pflash-cfi02: Refactor to support testing multiple configurations Philippe Mathieu-Daudé
2019-07-02 0:58 ` [Qemu-devel] [PULL 14/27] hw/block/pflash_cfi02: Remove pointless local variable Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 15/27] hw/block/pflash_cfi02: Document the current CFI values Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 16/27] hw/block/pflash_cfi02: Hold the PRI table offset in a variable Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 17/27] hw/block/pflash_cfi02: Document 'Page Mode' operations are not supported Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 18/27] hw/block/pflash_cfi02: Implement nonuniform sector sizes Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 19/27] hw/block/pflash_cfi02: Extract pflash_regions_count() Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 20/27] hw/block/pflash_cfi02: Split if() condition Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 21/27] hw/block/pflash_cfi02: Fix CFI in autoselect mode Philippe Mathieu-Daudé
2019-07-02 0:59 ` Philippe Mathieu-Daudé [this message]
2019-07-02 0:59 ` [Qemu-devel] [PULL 23/27] hw/block/pflash_cfi02: Implement multi-sector erase Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 24/27] hw/block/pflash_cfi02: Implement erase suspend/resume Philippe Mathieu-Daudé
2019-07-11 12:35 ` Peter Maydell
2019-07-11 12:51 ` Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 25/27] hw/block/pflash_cfi02: Use chip erase time specified in the CFI table Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 26/27] hw/block/pflash_cfi02: Document commands Philippe Mathieu-Daudé
2019-07-02 0:59 ` [Qemu-devel] [PULL 27/27] hw/block/pflash_cfi02: Reduce I/O accesses to 16-bit Philippe Mathieu-Daudé
2019-07-03 15:52 ` Stephen Checkoway
2019-07-03 16:02 ` Philippe Mathieu-Daudé
2019-07-03 16:20 ` Stephen Checkoway
2019-07-03 16:36 ` Philippe Mathieu-Daudé
2019-07-04 12:45 ` Philippe Mathieu-Daudé
2019-07-08 17:00 ` Stephen Checkoway
2019-07-02 17:56 ` [Qemu-devel] [PULL 00/27] pflash-next patches Peter Maydell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190702005912.15905-23-philmd@redhat.com \
--to=philmd@redhat.com \
--cc=alistair.francis@wdc.com \
--cc=kwolf@redhat.com \
--cc=lvivier@redhat.com \
--cc=mreitz@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stephen.checkoway@oberlin.edu \
--cc=thuth@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).