On Sun, Nov 29, 2020 at 6:55 PM Bin Meng <bmeng...@gmail.com> wrote:
>
> From: Bin Meng <bin.m...@windriver.com>
>
> SST flashes require a dummy byte after the address bits.
>
> Signed-off-by: Bin Meng <bin.m...@windriver.com>
I couldn't find a datasheet that says this... But the actual code
change looks fine, so:
Acked-by: Alistair Francis <alistair.fran...@wdc.com>
Alistair
> ---
>
> hw/block/m25p80.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c
> index 483925f..9b36762 100644
> --- a/hw/block/m25p80.c
> +++ b/hw/block/m25p80.c
> @@ -825,6 +825,9 @@ static void decode_fast_read_cmd(Flash *s)
> s->needed_bytes = get_addr_length(s);
> switch (get_man(s)) {
> /* Dummy cycles - modeled with bytes writes instead of bits */
> + case MAN_SST:
> + s->needed_bytes += 1;
> + break;
> case MAN_WINBOND:
> s->needed_bytes += 8;
> break;
> --
> 2.7.4
>
>
