diff mbox series

[1/2] target/i386/hvf: Remove unused includes in 'hvf-i386.h'

Message ID 20230905123906.6880-2-philmd@linaro.org
State Superseded
Headers show
Series sysemu/accel: Simplify sysemu/hvf.h | expand

Commit Message

Philippe Mathieu-Daudé Sept. 5, 2023, 12:39 p.m. UTC 
The only non standard type -- CPUArchState -- is forward
declared in "qemu/typedefs.h", so no particular header is
required here.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 target/i386/hvf/hvf-i386.h | 6 ------
 1 file changed, 6 deletions(-)

Comments

Roman Bolshakov Sept. 10, 2023, 8:54 p.m. UTC | #1 
On Tue, Sep 05, 2023 at 02:39:05PM +0200, Philippe Mathieu-Daudé wrote:
> The only non standard type -- CPUArchState -- is forward
> declared in "qemu/typedefs.h", so no particular header is
> required here.
> 

Hi Philippe,

Reviewed-by: Roman Bolshakov <roman@roolebo.dev>
Tested-by: Roman Bolshakov <roman@roolebo.dev>

Regards,
Roman
diff mbox series

Patch

diff --git a/target/i386/hvf/hvf-i386.h b/target/i386/hvf/hvf-i386.h
index 95b47c1c2e..243bc111cc 100644
--- a/target/i386/hvf/hvf-i386.h
+++ b/target/i386/hvf/hvf-i386.h
@@ -16,12 +16,6 @@ 
 #ifndef HVF_I386_H
 #define HVF_I386_H
 
-#include "qemu/accel.h"
-#include "sysemu/hvf.h"
-#include "sysemu/hvf_int.h"
-#include "cpu.h"
-#include "x86.h"
-
 void hvf_handle_io(CPUArchState *, uint16_t, void *, int, int, int);
 
 /* Host specific functions */