[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v2 4/9] hw/block/pflash_cfi01: Start state machine
From: |
John Snow |
Subject: |
Re: [Qemu-ppc] [PATCH v2 4/9] hw/block/pflash_cfi01: Start state machine as READY to accept commands |
Date: |
Mon, 1 Jul 2019 23:04:49 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.2 |
On 7/1/19 8:12 PM, Philippe Mathieu-Daudé wrote:
> When the state machine is ready to accept command, the bit 7 of
> the status register (SR) is set to 1.
> The guest polls the status register and check this bit before
> writting command to the internal 'Write State Machine' (WSM).
writing
>
> Set SR.7 bit to 1 when the device is created.
>
Right, ok -- it looks like we don't set this on realize otherwise.
(Or if we do, it's not obvious where or how.)
> Reference: Read Array Flowchart
> "Common Flash Interface (CFI) and Command Sets"
> (Intel Application Note 646)
> Appendix B "Basic Command Set"
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: John Snow <address@hidden>
> ---
> hw/block/pflash_cfi01.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/block/pflash_cfi01.c b/hw/block/pflash_cfi01.c
> index 33c77f6569..dd1dfd266b 100644
> --- a/hw/block/pflash_cfi01.c
> +++ b/hw/block/pflash_cfi01.c
> @@ -764,7 +764,7 @@ static void pflash_cfi01_realize(DeviceState *dev, Error
> **errp)
> }
>
> pflash_mode_read_array(pfl);
> - pfl->status = 0;
> + pfl->status = 0x80; /* WSM ready */
> /* Hardcoded CFI table */
> /* Standard "QRY" string */
> pfl->cfi_table[0x10] = 'Q';
>
- [Qemu-ppc] [PATCH v2 0/9] hw/block/pflash_cfi01: Add DeviceReset() handler, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-ppc] [PATCH v2 1/9] hw/block/pflash_cfi01: Removed an unused timer, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-ppc] [PATCH v2 2/9] hw/block/pflash_cfi01: Use the correct READ_ARRAY value, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-ppc] [PATCH v2 3/9] hw/block/pflash_cfi01: Extract pflash_mode_read_array(), Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-ppc] [PATCH v2 4/9] hw/block/pflash_cfi01: Start state machine as READY to accept commands, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-ppc] [PATCH v2 5/9] hw/block/pflash_cfi01: Add the DeviceReset() handler, Philippe Mathieu-Daudé, 2019/07/01
- Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 5/9] hw/block/pflash_cfi01: Add the DeviceReset() handler, Laszlo Ersek, 2019/07/02
[Qemu-ppc] [PATCH v2 6/9] hw/block/pflash_cfi01: Simplify CFI_QUERY processing, Philippe Mathieu-Daudé, 2019/07/01