| Message ID | 20241218155202.71931-1-philmd@linaro.org |
|---|---|
| Headers | show |
| Series | include: Header cleanups around "cpu.h" | expand |
On 12/18/24 09:51, Philippe Mathieu-Daudé wrote: > Philippe Mathieu-Daudé (4): > target/ppc: Include missing headers in mmu-hash[32,64].h > tcg/tci: Include missing 'disas/dis-asm.h' header > exec/ram_addr: Include missing 'exec/hwaddr.h' and 'exec/cpu-common.h' > exec/cpu-all: Include 'cpu.h' earlier so MMU_USER_IDX is always > defined Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Though, frankly these are hard to review in isolation. r~
On 18/12/24 17:42, Richard Henderson wrote: > On 12/18/24 09:51, Philippe Mathieu-Daudé wrote: >> Philippe Mathieu-Daudé (4): >> target/ppc: Include missing headers in mmu-hash[32,64].h >> tcg/tci: Include missing 'disas/dis-asm.h' header >> exec/ram_addr: Include missing 'exec/hwaddr.h' and 'exec/cpu-common.h' >> exec/cpu-all: Include 'cpu.h' earlier so MMU_USER_IDX is always >> defined > > Reviewed-by: Richard Henderson <richard.henderson@linaro.org> > > Though, frankly these are hard to review in isolation. Sorry for that. I'll send a PR with what I've already queued. Next patches should be closer to the main tree. Queuing patches 2-4.
Few more cleanups related to "cpu.h" in "exec/cpu-all.h". (I plan to merge that myself) Philippe Mathieu-Daudé (4): target/ppc: Include missing headers in mmu-hash[32,64].h tcg/tci: Include missing 'disas/dis-asm.h' header exec/ram_addr: Include missing 'exec/hwaddr.h' and 'exec/cpu-common.h' exec/cpu-all: Include 'cpu.h' earlier so MMU_USER_IDX is always defined include/exec/cpu-all.h | 5 ++++- include/exec/ram_addr.h | 3 +++ target/ppc/mmu-hash32.h | 3 +++ target/ppc/mmu-hash64.h | 2 ++ tcg/tci.c | 1 + 5 files changed, 13 insertions(+), 1 deletion(-)