>-----Original Message----- >From: Qemu-devel [mailto:qemu-devel- >[email protected]] On Behalf Of Laurent Vivier >Sent: Monday, March 23, 2020 10:56 PM >To: Philippe Mathieu-Daudé <[email protected]>; [email protected] >Cc: Fam Zheng <[email protected]>; Peter Maydell <[email protected]>; >Michael S. Tsirkin <[email protected]>; Mark Cave-Ayland <mark.cave- >[email protected]>; [email protected]; [email protected]; >Markus Armbruster <[email protected]>; Hervé Poussineau ><[email protected]>; Joel Stanley <[email protected]>; Michael Tokarev ><[email protected]>; Alistair Francis <[email protected]>; qemu- >[email protected]; Cédric Le Goater <[email protected]>; John Snow ><[email protected]>; David Gibson <[email protected]>; Kevin Wolf ><[email protected]>; Andrew Jeffery <[email protected]>; Max Reitz ><[email protected]>; Igor Mitsyanko <[email protected]>; qemu- >[email protected]; Paolo Bonzini <[email protected]> >Subject: Re: [PATCH-for-5.0 v2 00/11] misc: Trivial static code analyzer fixes > >Le 23/03/2020 à 15:45, Philippe Mathieu-Daudé a écrit : >> On 3/23/20 3:32 PM, Laurent Vivier wrote: >>> Le 21/03/2020 à 15:40, Philippe Mathieu-Daudé a écrit : >>>> Fix trivial warnings reported by the Clang static code analyzer. >>>> >>>> Since v1: >>>> - Addressed Markus/Zoltan/Aleksandar review comments >>>> >>>> Philippe Mathieu-Daudé (11): >>>> block: Avoid dead assignment >>>> blockdev: Remove dead assignment >>>> hw/i2c/pm_smbus: Remove dead assignment >>>> hw/input/adb-kbd: Remove dead assignment >>>> hw/ide/sii3112: Remove dead assignment >>>> hw/isa/i82378: Remove dead assignment >>>> hw/gpio/aspeed_gpio: Remove dead assignment >>>> hw/timer/exynos4210_mct: Remove dead assignments
Same as this: https://patchwork.kernel.org/patch/11415527/ >>>> hw/timer/stm32f2xx_timer: Remove dead assignment >>>> hw/timer/pxa2xx_timer: Add assertion to silent static analyzer >>>> warning >>>> hw/scsi/esp-pci: Remove dead assignment Same as this: https://patchwork.kernel.org/patch/11415529/ >>>> >>>> block.c | 2 +- >>>> blockdev.c | 2 +- >>>> hw/gpio/aspeed_gpio.c | 2 +- >>>> hw/i2c/pm_smbus.c | 1 - >>>> hw/ide/sii3112.c | 5 +++-- >>>> hw/input/adb-kbd.c | 6 +----- >>>> hw/isa/i82378.c | 8 ++++---- >>>> hw/scsi/esp-pci.c | 1 - >>>> hw/timer/exynos4210_mct.c | 3 --- >>>> hw/timer/pxa2xx_timer.c | 1 + >>>> hw/timer/stm32f2xx_timer.c | 1 - >>>> 11 files changed, 12 insertions(+), 20 deletions(-) >>>> >>> >>> I think your series covers cases already covered by: >>> >>> [PATCH v3 00/12] redundant code: Fix warnings reported by Clang >>> static code analyzer >>> https://patchew.org/QEMU/20200302130715.29440-1-kuhn.ch >> >> Unfortunately [for me...] I don't have v3 in my INBOX... *sigh* This >> was 3 weeks ago. *sigh*. >> >> I can see the series in the archives: >> https://lists.gnu.org/archive/html/qemu-devel/2020-03/msg00219.html >> But I can't find the outcome, was it queued in the trivial tree? >> Any idea when this will be merged in the master tree? > >Some patches are already merged via trivial (1, 2 (should go by SCSI >queue) 3, 5, 6, 7, 9, 11 (by USB queue), 12). > >But others needed R-b tags or new version. I didn't check which of your patches >are already covered by this series. > Hi, Laurent and Philippe Yes, there are currently three patches( 4、8 、10) that are not merged. And, only patch4 and patch10 have the same points as this series. I should update it again earlier. Thanks.
