On Tue, May 12, 2009 at 12:41:55PM -0700, David Daney wrote: > CPU_CAVIUM_OCTEON is mips_r2 which is handled before the switch. This > label in the switch statement is dead code, so we remove it. > > Signed-off-by: David Daney <ddaney@xxxxxxxxxxxxxxxxxx> > --- > arch/mips/mm/tlbex.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/arch/mips/mm/tlbex.c b/arch/mips/mm/tlbex.c > index 4108674..4dc4f3e 100644 > --- a/arch/mips/mm/tlbex.c > +++ b/arch/mips/mm/tlbex.c > @@ -311,7 +311,6 @@ static void __cpuinit build_tlb_write_entry(u32 **p, struct uasm_label **l, > case CPU_BCM3302: > case CPU_BCM4710: > case CPU_LOONGSON2: > - case CPU_CAVIUM_OCTEON: > case CPU_R5500: > if (m4kc_tlbp_war()) > uasm_i_nop(p); Fewer lines of code is good... Reviewed by: David VomLehn <dvomlehn@xxxxxxxxx>