[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v5 16/28] hw/block/pflash_cfi02: Ho
From: |
Alistair Francis |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v5 16/28] hw/block/pflash_cfi02: Hold the PRI table offset in a variable |
Date: |
Fri, 28 Jun 2019 09:34:13 -0700 |
On Thu, Jun 27, 2019 at 1:34 PM Philippe Mathieu-Daudé
<address@hidden> wrote:
>
> Manufacturers are allowed to move the PRI table, this is why the
> offset is queryable via fixed offsets 0x15/0x16.
> Add a variable to hold the offset, so it will be easier to later
> move the PRI table.
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> ---
> hw/block/pflash_cfi02.c | 31 +++++++++++++++++--------------
> 1 file changed, 17 insertions(+), 14 deletions(-)
>
> diff --git a/hw/block/pflash_cfi02.c b/hw/block/pflash_cfi02.c
> index f1bac480f5..23d05a6308 100644
> --- a/hw/block/pflash_cfi02.c
> +++ b/hw/block/pflash_cfi02.c
> @@ -552,6 +552,7 @@ static void pflash_cfi02_realize(DeviceState *dev, Error
> **errp)
> pfl->status = 0;
>
> /* Hardcoded CFI table (mostly from SG29 Spansion flash) */
> + const uint16_t pri_ofs = 0x31;
> /* Standard "QRY" string */
> pfl->cfi_table[0x10] = 'Q';
> pfl->cfi_table[0x11] = 'R';
> @@ -560,8 +561,8 @@ static void pflash_cfi02_realize(DeviceState *dev, Error
> **errp)
> pfl->cfi_table[0x13] = 0x02;
> pfl->cfi_table[0x14] = 0x00;
> /* Primary extended table address */
> - pfl->cfi_table[0x15] = 0x31;
> - pfl->cfi_table[0x16] = 0x00;
> + pfl->cfi_table[0x15] = pri_ofs;
> + pfl->cfi_table[0x16] = pri_ofs >> 8;
> /* Alternate command set (none) */
> pfl->cfi_table[0x17] = 0x00;
> pfl->cfi_table[0x18] = 0x00;
> @@ -609,32 +610,34 @@ static void pflash_cfi02_realize(DeviceState *dev,
> Error **errp)
> pfl->cfi_table[0x2E] = (pfl->nb_blocs - 1) >> 8;
> pfl->cfi_table[0x2F] = pfl->sector_len >> 8;
> pfl->cfi_table[0x30] = pfl->sector_len >> 16;
> + assert(0x30 < pri_ofs);
>
> /* Extended */
> - pfl->cfi_table[0x31] = 'P';
> - pfl->cfi_table[0x32] = 'R';
> - pfl->cfi_table[0x33] = 'I';
> + pfl->cfi_table[0x00 + pri_ofs] = 'P';
> + pfl->cfi_table[0x01 + pri_ofs] = 'R';
> + pfl->cfi_table[0x02 + pri_ofs] = 'I';
>
> /* Extended version 1.0 */
> - pfl->cfi_table[0x34] = '1';
> - pfl->cfi_table[0x35] = '0';
> + pfl->cfi_table[0x03 + pri_ofs] = '1';
> + pfl->cfi_table[0x04 + pri_ofs] = '0';
>
> /* Address sensitive unlock required. */
> - pfl->cfi_table[0x36] = 0x00;
> + pfl->cfi_table[0x05 + pri_ofs] = 0x00;
> /* Erase suspend not supported. */
> - pfl->cfi_table[0x37] = 0x00;
> + pfl->cfi_table[0x06 + pri_ofs] = 0x00;
> /* Sector protect not supported. */
> - pfl->cfi_table[0x38] = 0x00;
> + pfl->cfi_table[0x07 + pri_ofs] = 0x00;
> /* Temporary sector unprotect not supported. */
> - pfl->cfi_table[0x39] = 0x00;
> + pfl->cfi_table[0x08 + pri_ofs] = 0x00;
>
> /* Sector protect/unprotect scheme. */
> - pfl->cfi_table[0x3a] = 0x00;
> + pfl->cfi_table[0x09 + pri_ofs] = 0x00;
>
> /* Simultaneous operation not supported. */
> - pfl->cfi_table[0x3b] = 0x00;
> + pfl->cfi_table[0x0a + pri_ofs] = 0x00;
> /* Burst mode not supported. */
> - pfl->cfi_table[0x3c] = 0x00;
> + pfl->cfi_table[0x0b + pri_ofs] = 0x00;
> + assert(0x0b + pri_ofs < ARRAY_SIZE(pfl->cfi_table));
> }
>
> static Property pflash_cfi02_properties[] = {
> --
> 2.20.1
>
>
- Re: [Qemu-block] [Qemu-devel] [PATCH v5 10/28] hw/block/pflash_cfi02: Extract the pflash_data_read() function, (continued)
- [Qemu-block] [PATCH v5 12/28] hw/block/pflash_cfi02: Fix command address comparison, Philippe Mathieu-Daudé, 2019/06/27
- [Qemu-block] [PATCH v5 13/28] tests/pflash-cfi02: Refactor to support testing multiple configurations, Philippe Mathieu-Daudé, 2019/06/27
- [Qemu-block] [PATCH v5 14/28] hw/block/pflash_cfi02: Remove pointless local variable, Philippe Mathieu-Daudé, 2019/06/27
- [Qemu-block] [PATCH v5 15/28] hw/block/pflash_cfi02: Document the current CFI values, Philippe Mathieu-Daudé, 2019/06/27
- [Qemu-block] [PATCH v5 16/28] hw/block/pflash_cfi02: Hold the PRI table offset in a variable, Philippe Mathieu-Daudé, 2019/06/27
- Re: [Qemu-block] [Qemu-devel] [PATCH v5 16/28] hw/block/pflash_cfi02: Hold the PRI table offset in a variable,
Alistair Francis <=
- [Qemu-block] [PATCH v5 17/28] hw/block/pflash_cfi02: Document 'Page Mode' operations are not supported, Philippe Mathieu-Daudé, 2019/06/27
- [Qemu-block] [PATCH v5 18/28] hw/block/pflash_cfi02: Implement nonuniform sector sizes, Philippe Mathieu-Daudé, 2019/06/27
- [Qemu-block] [PATCH v5 20/28] hw/block/pflash_cfi02: Split if() condition, Philippe Mathieu-Daudé, 2019/06/27
- [Qemu-block] [PATCH v5 19/28] hw/block/pflash_cfi02: Extract pflash_regions_count(), Philippe Mathieu-Daudé, 2019/06/27
- [Qemu-block] [PATCH v5 22/28] hw/block/pflash_cfi02: Fix reset command not ignored during erase, Philippe Mathieu-Daudé, 2019/06/27