On Tue, 18 Dec 2012, Stephen Boyd wrote: > It seems that this assignment is done twice in a row. Remove the > duplicate assignment. > > Signed-off-by: Stephen Boyd <sboyd@xxxxxxxxxxxxxx> > Cc: Steven Miao <realmz6@xxxxxxxxx> > Cc: trivial@xxxxxxxxxx > --- > > Noticed while grepping for broadcast users. > > arch/blackfin/kernel/time-ts.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/arch/blackfin/kernel/time-ts.c b/arch/blackfin/kernel/time-ts.c > index f608f02..cb0a484 100644 > --- a/arch/blackfin/kernel/time-ts.c > +++ b/arch/blackfin/kernel/time-ts.c > @@ -329,12 +329,6 @@ void bfin_coretmr_clockevent_init(void) > evt->broadcast = smp_timer_broadcast; > #endif > > - > -#ifdef CONFIG_SMP > - evt->broadcast = smp_timer_broadcast; > -#endif > - > - > evt->name = "bfin_core_timer"; > evt->rating = 350; > evt->irq = -1; > Looks good to me. Feel free to add Reviewed-by: Jesper Juhl <jj@xxxxxxxxxxxxx> if you like. -- Jesper Juhl <jj@xxxxxxxxxxxxx> http://www.chaosbits.net/ Don't top-post http://www.catb.org/jargon/html/T/top-post.html Plain text mails only, please. -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
