Re: [PATCH] Remove inclusion of hw/hw.h from files that don't need it

[Cédric Le Goater]

On 7/1/24 3:26 PM, Thomas Huth wrote:

hw/hw.h only contains the prototype of hw_error() nowadays, so
files that don't use this function don't need to include this
header.

Signed-off-by: Thomas Huth <th...@redhat.com>
---
  include/hw/misc/xlnx-cfi-if.h | 1 -
  hw/misc/edu.c                 | 1 -
  hw/vfio/container.c           | 1 -
  3 files changed, 3 deletions(-)

diff --git a/include/hw/misc/xlnx-cfi-if.h b/include/hw/misc/xlnx-cfi-if.h
index f9bd12292d..5010401517 100644
--- a/include/hw/misc/xlnx-cfi-if.h
+++ b/include/hw/misc/xlnx-cfi-if.h
@@ -11,7 +11,6 @@
  #define XLNX_CFI_IF_H 1

#include "qemu/help-texts.h"

-#include "hw/hw.h"
  #include "qom/object.h"

#define TYPE_XLNX_CFI_IF "xlnx-cfi-if"

diff --git a/hw/misc/edu.c b/hw/misc/edu.c
index fa052c44db..504178b4a2 100644
--- a/hw/misc/edu.c
+++ b/hw/misc/edu.c
@@ -26,7 +26,6 @@
  #include "qemu/log.h"
  #include "qemu/units.h"
  #include "hw/pci/pci.h"
-#include "hw/hw.h"
  #include "hw/pci/msi.h"
  #include "qemu/timer.h"
  #include "qom/object.h"
diff --git a/hw/vfio/container.c b/hw/vfio/container.c
index 2e7ecdf10e..88ede913d6 100644
--- a/hw/vfio/container.c
+++ b/hw/vfio/container.c
@@ -26,7 +26,6 @@
  #include "exec/address-spaces.h"
  #include "exec/memory.h"
  #include "exec/ram_addr.h"
-#include "hw/hw.h"
  #include "qemu/error-report.h"
  #include "qemu/range.h"
  #include "sysemu/reset.h"

Reviewed-by: Cédric Le Goater <c...@redhat.com>

Thanks,

C.