[PATCH 1/9] clocksource: time-armada-370-xp: Marvell Armada 370/XP SoC timer driver

[Thomas Petazzoni]

Hello Thomas,

Le Mon, 2 Jul 2012 14:29:53 +0200 (CEST),
Thomas Gleixner <tglx at linutronix.de> a écrit :

> > +{
> > +	unsigned long flags;
> > +	u32 u;
> > +
> > +	if (delta == 0)
> > +		return -ETIME;
> 
> If you have a proper min_delta_ns value set up, then delta will never
> be 0. SO that check is pointless.
> 
> > +	local_irq_save(flags);
> 
> That code is guaranteed to be called with interrupts disabled. No need
> for disabling them again.
> > +	/*
> > +	 * Setup clockevent timer (interrupt-driven).
> > +	 */
> > +	setup_irq(timer_irq, &armada_370_xp_timer_irq);
> > +	armada_370_xp_clkevt.mult = div_sc(timer_clk, NSEC_PER_SEC,
> > +					   armada_370_xp_clkevt.shift);
> > +	armada_370_xp_clkevt.max_delta_ns = clockevent_delta2ns(0xfffffffe,
> > +							&armada_370_xp_clkevt);
> > +	armada_370_xp_clkevt.min_delta_ns =
> > +	    clockevent_delta2ns(1, &armada_370_xp_clkevt);
> 
> clockevents_config_and_register() please.

Thanks a lot for all those comments. I have fixed those in our tree,
will be part of the next v7 posting. We'll just wait a bit more time
in case other comments show up before posting this v7.

Thanks again for the review,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com