On 16/1/26 13:58, Peter Maydell wrote:
This commit was created with scripts/clean-includes:
  ./scripts/clean-includes '--git' 'all' '--all'

and manually edited to remove one change to hw/virtio/cbor-helpers.c.
All these changes are header files that include osdep.h or some
system header that osdep.h pulls in; they don't need to do this.

All .c should include qemu/osdep.h first.  The script performs three
related cleanups:

* Ensure .c files include qemu/osdep.h first.
* Including it in a .h is redundant, since the .c  already includes
   it.  Drop such inclusions.
* Likewise, including headers qemu/osdep.h includes is redundant.
   Drop these, too.

Signed-off-by: Peter Maydell <[email protected]>
---
  hw/scsi/lasi_ncr710.h          | 1 -
  hw/scsi/ncr53c710.h            | 1 -
  include/hw/core/loader.h       | 1 -
  include/hw/i386/tdvf.h         | 1 -
  include/hw/ppc/spapr_fadump.h  | 1 -
  include/hw/riscv/iommu.h       | 1 -
  include/system/accel-irq.h     | 1 -
  linux-user/alpha/target_proc.h | 1 -
  qga/commands-windows-ssh.h     | 1 -
  qga/vss-win32/vss-debug.h      | 1 -
  target/i386/kvm/vmsr_energy.h  | 2 --
  11 files changed, 12 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <[email protected]>

Reply via email to