Re: [PATCH 07/26] hw/usb/hcd-dwc2: Restrict some headers to source

2020-07-06 Thread Paul Zimmerman
On Sat, Jul 4, 2020 at 7:50 AM Philippe Mathieu-Daudé wrote: > The header "usb/hcd-dwc2.h" doesn't need to include "qemu/timer.h", > "sysemu/dma.h", "hw/irq.h" (the types required are forward declared). > Include them in the source file which is the only one requiring the > function

Re: [PATCH 07/26] hw/usb/hcd-dwc2: Restrict some headers to source

2020-07-06 Thread Alistair Francis
On Sat, Jul 4, 2020 at 7:52 AM Philippe Mathieu-Daudé wrote: > > The header "usb/hcd-dwc2.h" doesn't need to include "qemu/timer.h", > "sysemu/dma.h", "hw/irq.h" (the types required are forward declared). > Include them in the source file which is the only one requiring the > function

[PATCH 07/26] hw/usb/hcd-dwc2: Restrict some headers to source

2020-07-04 Thread Philippe Mathieu-Daudé
The header "usb/hcd-dwc2.h" doesn't need to include "qemu/timer.h", "sysemu/dma.h", "hw/irq.h" (the types required are forward declared). Include them in the source file which is the only one requiring the function declarations. Signed-off-by: Philippe Mathieu-Daudé --- hw/usb/hcd-dwc2.h | 3