On Fri, Apr 07, 2017 at 02:59:08PM +0800, weiqi wrote: > From: Wei Qi <[email protected]> > > It is unnecessary to assign variable 'size' here, because 'size' unused after > assignment. > > Signed-off-by: Wei Qi <[email protected]> > --- > hw/scsi/esp-pci.c | 1 - > 1 file changed, 1 deletion(-)
Reviewed-By: Sahid Orentino Ferdjaoui <[email protected]> > > diff --git a/hw/scsi/esp-pci.c b/hw/scsi/esp-pci.c > index e295d88..1f5980e 100644 > --- a/hw/scsi/esp-pci.c > +++ b/hw/scsi/esp-pci.c > @@ -192,7 +192,6 @@ static void esp_pci_io_write(void *opaque, hwaddr addr, > val <<= shift; > val |= current & ~(mask << shift); > addr &= ~3; > - size = 4; I verified that 'size' is passed by value to 'esp_pci_io_write(...)', no loop, no recursivity. > } > > if (addr < 0x40) { > -- > 1.8.3.1 > >
