Re: [PATCH v7 14/17] hw/sd/sdcard: Make iolen unsigned
On Tue, Jun 30, 2020 at 6:51 AM Philippe Mathieu-Daudé wrote: > > From: Philippe Mathieu-Daudé > > I/O request length can not be negative. > > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alistair Francis 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 > >
Re: [PATCH v7 14/17] hw/sd/sdcard: Make iolen unsigned
On Tue, 30 Jun 2020 at 14:39, Philippe Mathieu-Daudé wrote: > > From: Philippe Mathieu-Daudé > > I/O request length can not be negative. > > Signed-off-by: Philippe Mathieu-Daudé > --- Reviewed-by: Peter Maydell thanks -- PMM
[PATCH v7 14/17] hw/sd/sdcard: Make iolen unsigned
From: Philippe Mathieu-Daudé I/O request length can not be negative. Signed-off-by: Philippe Mathieu-Daudé --- 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