On 18/12/2019 20.25, Philippe Mathieu-Daudé wrote: > We don't need to explicit this obvious switch fall through. > Stay consistent with the rest of the codebase. > > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> > --- > Cc: Peter Chubb <peter.ch...@nicta.com.au> > Cc: Markus Armbruster <arm...@redhat.com> > Cc: Peter Maydell <peter.mayd...@linaro.org> > Cc: Andrey Smirnov <andrew.smir...@gmail.com> > Cc: qemu-...@nongnu.org > --- > hw/pci-host/designware.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/pci-host/designware.c b/hw/pci-host/designware.c > index 71e9b0d9b5..dd245516dd 100644 > --- a/hw/pci-host/designware.c > +++ b/hw/pci-host/designware.c > @@ -182,7 +182,7 @@ designware_pcie_root_config_read(PCIDevice *d, uint32_t > address, int len) > break; > > case DESIGNWARE_PCIE_ATU_CR1: > - case DESIGNWARE_PCIE_ATU_CR2: /* FALLTHROUGH */ > + case DESIGNWARE_PCIE_ATU_CR2: > val = viewport->cr[(address - DESIGNWARE_PCIE_ATU_CR1) / > sizeof(uint32_t)]; > break; >
Reviewed-by: Thomas Huth <th...@redhat.com>
