On Mon May 27, 2024 at 9:13 AM AEST, BALATON Zoltan wrote: > Drop includes from header that is not needed by the header itself and > only include them from C files that really need it.
Acked-by: Nicholas Piggin <npig...@gmail.com> > > Signed-off-by: BALATON Zoltan <bala...@eik.bme.hu> > --- > target/ppc/mmu-book3s-v3.h | 3 --- > target/ppc/mmu-hash64.c | 1 + > target/ppc/mmu-radix64.c | 1 + > 3 files changed, 2 insertions(+), 3 deletions(-) > > diff --git a/target/ppc/mmu-book3s-v3.h b/target/ppc/mmu-book3s-v3.h > index 263ce55c1f..be66e26604 100644 > --- a/target/ppc/mmu-book3s-v3.h > +++ b/target/ppc/mmu-book3s-v3.h > @@ -20,9 +20,6 @@ > #ifndef PPC_MMU_BOOK3S_V3_H > #define PPC_MMU_BOOK3S_V3_H > > -#include "mmu-hash64.h" > -#include "mmu-books.h" > - > #ifndef CONFIG_USER_ONLY > > /* > diff --git a/target/ppc/mmu-hash64.c b/target/ppc/mmu-hash64.c > index 7bc0323f26..5e1983e334 100644 > --- a/target/ppc/mmu-hash64.c > +++ b/target/ppc/mmu-hash64.c > @@ -31,6 +31,7 @@ > #include "hw/hw.h" > #include "internal.h" > #include "mmu-book3s-v3.h" > +#include "mmu-books.h" > #include "helper_regs.h" > > #ifdef CONFIG_TCG > diff --git a/target/ppc/mmu-radix64.c b/target/ppc/mmu-radix64.c > index cf9619e847..be7a45f254 100644 > --- a/target/ppc/mmu-radix64.c > +++ b/target/ppc/mmu-radix64.c > @@ -28,6 +28,7 @@ > #include "internal.h" > #include "mmu-radix64.h" > #include "mmu-book3s-v3.h" > +#include "mmu-books.h" > > /* Radix Partition Table Entry Fields */ > #define PATE1_R_PRTB 0x0FFFFFFFFFFFF000