[RFC][PATCH 3/3] clocksource: Add Freescale FlexTimer Module (FTM) timer support
Li.Xiubo at freescale.com
Li.Xiubo at freescale.com
Thu Apr 17 20:55:46 PDT 2014
> > + freq = clk_get_rate(ftm_clk);
> > +
> > + calc_closest_cound_cyc(freq);
> > +
> > + BUG_ON(ftm_clocksource_init(freq));
> > +
> > + BUG_ON(ftm_clockevent_init(freq, irq));
> > +}
> > +CLOCKSOURCE_OF_DECLARE(vf610, "fsl,vf610-ftm-timer", ftm_timer_init);
>
>
> I am not a big fan of those BUG_ON every line. Could you please replace
> it by dev_err().
>
While, for the FlexTimer driver, there hasn't any device creation and
registering, so I'll use pr_err() instead of dev_err()...
Thanks
BRs
Xiubo
More information about the linux-arm-kernel
mailing list