[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 7/9] hw/block/pflash_cfi01: Improve command c
From: |
Alistair Francis |
Subject: |
Re: [Qemu-block] [PATCH v2 7/9] hw/block/pflash_cfi01: Improve command comments |
Date: |
Tue, 2 Jul 2019 09:13:02 -0700 |
On Mon, Jul 1, 2019 at 5:15 PM Philippe Mathieu-Daudé <address@hidden> wrote:
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> ---
> hw/block/pflash_cfi01.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/hw/block/pflash_cfi01.c b/hw/block/pflash_cfi01.c
> index c1b02219b2..f50d0a9d37 100644
> --- a/hw/block/pflash_cfi01.c
> +++ b/hw/block/pflash_cfi01.c
> @@ -208,11 +208,11 @@ static uint32_t pflash_devid_query(PFlashCFI01 *pfl,
> hwaddr offset)
> * Offsets 2/3 are block lock status, is not emulated.
> */
> switch (boff & 0xFF) {
> - case 0:
> + case 0: /* Manufacturer ID */
> resp = pfl->ident0;
> trace_pflash_manufacturer_id(resp);
> break;
> - case 1:
> + case 1: /* Device ID */
> resp = pfl->ident1;
> trace_pflash_device_id(resp);
> break;
> @@ -456,11 +456,11 @@ static void pflash_write(PFlashCFI01 *pfl, hwaddr
> offset,
> case 0:
> /* read mode */
> switch (cmd) {
> - case 0x10: /* Single Byte Program */
> - case 0x40: /* Single Byte Program */
> + case 0x10: /* Single Byte Program (setup) */
> + case 0x40: /* Single Byte Program (setup) [Intel] */
> DPRINTF("%s: Single Byte Program\n", __func__);
> break;
> - case 0x20: /* Block erase */
> + case 0x20: /* Block erase (setup) */
> p = pfl->storage;
> offset &= ~(pfl->sector_len - 1);
>
> @@ -516,8 +516,8 @@ static void pflash_write(PFlashCFI01 *pfl, hwaddr offset,
> break;
> case 1:
> switch (pfl->cmd) {
> - case 0x10: /* Single Byte Program */
> - case 0x40: /* Single Byte Program */
> + case 0x10: /* Single Byte Program (confirm) */
> + case 0x40: /* Single Byte Program (confirm) [Intel] */
> DPRINTF("%s: Single Byte Program\n", __func__);
> if (!pfl->ro) {
> pflash_data_write(pfl, offset, value, width, be);
> @@ -528,7 +528,7 @@ static void pflash_write(PFlashCFI01 *pfl, hwaddr offset,
> pfl->status |= 0x80; /* Ready! */
> pfl->wcycle = 0;
> break;
> - case 0x20: /* Block erase */
> + case 0x20: /* Block erase (confirm) */
> case 0x28:
> if (cmd == 0xd0) { /* confirm */
> pfl->wcycle = 0;
> --
> 2.20.1
>
- [Qemu-block] [PATCH v2 5/9] hw/block/pflash_cfi01: Add the DeviceReset() handler, (continued)
[Qemu-block] [PATCH v2 6/9] hw/block/pflash_cfi01: Simplify CFI_QUERY processing, Philippe Mathieu-Daudé, 2019/07/01
[Qemu-block] [PATCH v2 7/9] hw/block/pflash_cfi01: Improve command comments, Philippe Mathieu-Daudé, 2019/07/01
- Re: [Qemu-block] [PATCH v2 7/9] hw/block/pflash_cfi01: Improve command comments,
Alistair Francis <=
[Qemu-block] [PATCH v2 9/9] hw/block/pflash_cfi01: Hold the PRI table offset in a variable, Philippe Mathieu-Daudé, 2019/07/01
[Qemu-block] [PATCH v2 8/9] hw/block/pflash_cfi01: Replace DPRINTF by qemu_log_mask(GUEST_ERROR), Philippe Mathieu-Daudé, 2019/07/01
Re: [Qemu-block] [Qemu-devel] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler, no-reply, 2019/07/02
Re: [Qemu-block] [Qemu-devel] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler, Laszlo Ersek, 2019/07/02