Re: [PATCH v2] hw/block: m25p80: Support fast read for SST flashes
On Sat, Mar 6, 2021 at 1:02 AM Bin Meng wrote: > > From: Bin Meng > > Per SST25VF016B datasheet [1], SST flash requires a dummy byte after > the address bytes. Note only SPI mode is supported by SST flashes. > > [1] http://ww1.microchip.com/downloads/en/devicedoc/s71271_04.pdf > > Signed-off-by: Bin Meng > Acked-by: Alistair Francis Thanks! Applied to riscv-to-apply.next Alistair > > --- > > Changes in v2: > - rebase on qemu/master > > hw/block/m25p80.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c > index 5f9471d83c..183d3f44c2 100644 > --- a/hw/block/m25p80.c > +++ b/hw/block/m25p80.c > @@ -895,6 +895,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.25.1 > >
Re: [PATCH v2] hw/block: m25p80: Support fast read for SST flashes
On Tue, Mar 16, 2021 at 9:39 AM Bin Meng wrote: > > On Thu, Mar 11, 2021 at 4:18 PM Bin Meng wrote: > > > > On Sat, Mar 6, 2021 at 2:01 PM Bin Meng wrote: > > > > > > From: Bin Meng > > > > > > Per SST25VF016B datasheet [1], SST flash requires a dummy byte after > > > the address bytes. Note only SPI mode is supported by SST flashes. > > > > > > [1] http://ww1.microchip.com/downloads/en/devicedoc/s71271_04.pdf > > > > > > Signed-off-by: Bin Meng > > > Acked-by: Alistair Francis > > > > > > --- > > > > > > Changes in v2: > > > - rebase on qemu/master > > > > > > hw/block/m25p80.c | 3 +++ > > > 1 file changed, 3 insertions(+) > > > > > > > Ping? > > Ping? Ping?
Re: [PATCH v2] hw/block: m25p80: Support fast read for SST flashes
On Thu, Mar 11, 2021 at 4:18 PM Bin Meng wrote: > > On Sat, Mar 6, 2021 at 2:01 PM Bin Meng wrote: > > > > From: Bin Meng > > > > Per SST25VF016B datasheet [1], SST flash requires a dummy byte after > > the address bytes. Note only SPI mode is supported by SST flashes. > > > > [1] http://ww1.microchip.com/downloads/en/devicedoc/s71271_04.pdf > > > > Signed-off-by: Bin Meng > > Acked-by: Alistair Francis > > > > --- > > > > Changes in v2: > > - rebase on qemu/master > > > > hw/block/m25p80.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > Ping? Ping?
Re: [PATCH v2] hw/block: m25p80: Support fast read for SST flashes
On Sat, Mar 6, 2021 at 2:01 PM Bin Meng wrote: > > From: Bin Meng > > Per SST25VF016B datasheet [1], SST flash requires a dummy byte after > the address bytes. Note only SPI mode is supported by SST flashes. > > [1] http://ww1.microchip.com/downloads/en/devicedoc/s71271_04.pdf > > Signed-off-by: Bin Meng > Acked-by: Alistair Francis > > --- > > Changes in v2: > - rebase on qemu/master > > hw/block/m25p80.c | 3 +++ > 1 file changed, 3 insertions(+) > Ping?
[PATCH v2] hw/block: m25p80: Support fast read for SST flashes
From: Bin Meng Per SST25VF016B datasheet [1], SST flash requires a dummy byte after the address bytes. Note only SPI mode is supported by SST flashes. [1] http://ww1.microchip.com/downloads/en/devicedoc/s71271_04.pdf Signed-off-by: Bin Meng Acked-by: Alistair Francis --- Changes in v2: - rebase on qemu/master hw/block/m25p80.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c index 5f9471d83c..183d3f44c2 100644 --- a/hw/block/m25p80.c +++ b/hw/block/m25p80.c @@ -895,6 +895,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.25.1