On Fri, 2007-11-16 at 16:54 +0100, Segher Boessenkool wrote: > > +#ifdef CONFIG_44x > > +static void dump_tlb_44x(void); > > +#endif > > No need to #ifdef this... > > > +#ifdef CONFIG_44x > > +static void dump_tlb_44x(void) > > +{ > > ...or this. I don't want to require binutils to understand the 44x TLB ops Ben.