On Sat, Mar 21, 2020 at 7:46 AM Philippe Mathieu-Daudé <phi...@redhat.com> wrote: > > Rename the unique variable assigned as 'pit' which better > represents what it holds, to fix a warning reported by the > Clang static code analyzer: > > CC hw/isa/i82378.o > hw/isa/i82378.c:108:5: warning: Value stored to 'isa' is never read > isa = isa_create_simple(isabus, "i82374"); > ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > Reported-by: Clang Static Analyzer > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> Alistair > --- > hw/isa/i82378.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/hw/isa/i82378.c b/hw/isa/i82378.c > index dcb6b479ea..d9e6c7fa00 100644 > --- a/hw/isa/i82378.c > +++ b/hw/isa/i82378.c > @@ -67,7 +67,7 @@ static void i82378_realize(PCIDevice *pci, Error **errp) > I82378State *s = I82378(dev); > uint8_t *pci_conf; > ISABus *isabus; > - ISADevice *isa; > + ISADevice *pit; > > pci_conf = pci->config; > pci_set_word(pci_conf + PCI_COMMAND, > @@ -99,13 +99,13 @@ static void i82378_realize(PCIDevice *pci, Error **errp) > isa_bus_irqs(isabus, s->i8259); > > /* 1 82C54 (pit) */ > - isa = i8254_pit_init(isabus, 0x40, 0, NULL); > + pit = i8254_pit_init(isabus, 0x40, 0, NULL); > > /* speaker */ > - pcspk_init(isabus, isa); > + pcspk_init(isabus, pit); > > /* 2 82C37 (dma) */ > - isa = isa_create_simple(isabus, "i82374"); > + isa_create_simple(isabus, "i82374"); > } > > static void i82378_init(Object *obj) > -- > 2.21.1 > >
