On Tue, Jun 30, 2020 at 6:51 AM Philippe Mathieu-Daudé <f4...@amsat.org> wrote: > > From: Philippe Mathieu-Daudé <phi...@redhat.com> > > I/O request length can not be negative. > > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> Alistair > --- > v4: Use uint32_t (pm215) > --- > hw/sd/sd.c | 2 +- > hw/sd/trace-events | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/sd/sd.c b/hw/sd/sd.c > index 364a6d1fcd..3e9faa8add 100644 > --- a/hw/sd/sd.c > +++ b/hw/sd/sd.c > @@ -1981,7 +1981,7 @@ uint8_t sd_read_data(SDState *sd) > { > /* TODO: Append CRCs */ > uint8_t ret; > - int io_len; > + uint32_t io_len; > > if (!sd->blk || !blk_is_inserted(sd->blk) || !sd->enable) > return 0x00; > diff --git a/hw/sd/trace-events b/hw/sd/trace-events > index 5f09d32eb2..d0cd7c6ec4 100644 > --- a/hw/sd/trace-events > +++ b/hw/sd/trace-events > @@ -52,7 +52,7 @@ sdcard_unlock(void) "" > sdcard_read_block(uint64_t addr, uint32_t len) "addr 0x%" PRIx64 " size 0x%x" > sdcard_write_block(uint64_t addr, uint32_t len) "addr 0x%" PRIx64 " size > 0x%x" > sdcard_write_data(const char *proto, const char *cmd_desc, uint8_t cmd, > uint8_t value) "%s %20s/ CMD%02d value 0x%02x" > -sdcard_read_data(const char *proto, const char *cmd_desc, uint8_t cmd, int > length) "%s %20s/ CMD%02d len %d" > +sdcard_read_data(const char *proto, const char *cmd_desc, uint8_t cmd, > uint32_t length) "%s %20s/ CMD%02d len %" PRIu32 > sdcard_set_voltage(uint16_t millivolts) "%u mV" > > # milkymist-memcard.c > -- > 2.21.3 > >