On 04/08/2015 at 12:31:19 +0200, Daniel Lezcano wrote : > On 07/30/2015 02:22 AM, Alexandre Belloni wrote: > >The current slow clock rate is hardcoded. Properly get the slow clock > >and use its rate. > > > >Signed-off-by: Alexandre Belloni <[email protected]> > >--- > > [ ... ] > > >+struct clk *sclk; > > Do you need this variable global ? AFAICS, its scope is in the init > function, no ? >
Indeed. > >+int timer_latch; > > static. > Yeah, this was changed in v2 (that I forgot to label as v2). -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

