On Monday 21 July 2008, Marvin wrote: > diff --git a/arch/powerpc/mm/ppc_mmu_32.c b/arch/powerpc/mm/ppc_mmu_32.c > index c53145f..c3509c8 100644 > --- a/arch/powerpc/mm/ppc_mmu_32.c > +++ b/arch/powerpc/mm/ppc_mmu_32.c > @@ -74,7 +74,7 @@ unsigned long p_mapped_by_bats(phys_addr_t pa) > > unsigned long __init mmu_mapin_ram(void) > { > -#ifdef CONFIG_POWER4 > +#ifdef CONFIG_PPC64 > return 0; > #else > unsigned long tot, bl, done;
ppc_mmu_32 is 32-bit only, so you can kill this #ifdef. > diff --git a/include/asm-powerpc/cputable.h b/include/asm-powerpc/cputable.h > index 0ef9abc..97fdbaf 100644 > --- a/include/asm-powerpc/cputable.h > +++ b/include/asm-powerpc/cputable.h > @@ -257,7 +257,7 @@ extern void do_feature_fixups(unsigned long value, void > *fixup_start, > #endif > > #define CLASSIC_PPC (!defined(CONFIG_8xx) && !defined(CONFIG_4xx) && \ > - !defined(CONFIG_POWER4) && !defined(CONFIG_BOOKE)) > + !defined(CONFIG_PPC64) && !defined(CONFIG_BOOKE)) > This would be more logical in the opposite way: #define CLASSIC_PPC (defined(CONFIG_PPC32) && !defined(CONFIG_8xx) && !defined(CONFIG_4xx) && !defined(CONFIG_BOOKE)) Arnd <>< _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev