@samitouri / QOSamiQemu / commits / 561710e5d2

hexagon: Remove unnecessary 'monitor/monitor.h' header

The Monitor type is used in these 2 files, as a pointer. Since the type is forward-declared in "qemu/typedefs.h", which all source files include via "qemu/osdep.h", we do not need to include it. Do however include "exec/hwaddr.h" and "exec/mmu-access-type.h" which declare the types used by hex_tlb_find_match prototype: extern bool hex_tlb_find_match(CPUHexagonState *env, uint32_t VA, MMUAccessType access_type, hwaddr *PA, int *prot, ^^^^^^^^^^^^^ ^^^^^^ uint64_t *size, int32_t *excp, int mmu_idx); Signed-off-by: Philippe Mathieu-Daudé <philmd@oss.qualcomm.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Message-Id: <20260812211708.92824-2-philmd@oss.qualcomm.com>

Philippe Mathieu-Daudé committed Aug 12, 2026 at 10:58 UTC 561710e5d2356ed10456e36224a8f4ce9592afa6

2 files changed +3 -2

include/hw/hexagon/hexagon_tlb.h

+1 -1

@@ -12,7 +12,7 @@

12 #include "qom/object.h"

13 #include "exec/hwaddr.h"

14 #include "exec/mmu-access-type.h"

15 -#include "monitor/monitor.h"

15 +

16 #define TYPE_HEXAGON_TLB "hexagon-tlb"

17 OBJECT_DECLARE_SIMPLE_TYPE(HexagonTLBState, HEXAGON_TLB)

target/hexagon/hex_mmu.h

+2 -1

@@ -7,8 +7,9 @@

7 #ifndef HEXAGON_MMU_H

8 #define HEXAGON_MMU_H

10 +#include "exec/hwaddr.h"

11 +#include "exec/mmu-access-type.h"

12 #include "cpu.h"

11 -#include "monitor/monitor.h"

14 extern void hex_tlbw(CPUHexagonState *env, uint32_t index, uint64_t value);

15 extern uint32_t hex_tlb_lookup(CPUHexagonState *env, uint32_t ssr, uint32_t VA);