RE: [PATCH v2] clocksource/fsl: Fix errata A-007728 for flextimer

2016-09-21 Thread Meng Yi
> > __iomem *base) > > > > static inline void ftm_irq_acknowledge(void __iomem *base) { > > - u32 val; > > + unsigned long timeout = jiffies + msecs_to_jiffies(100); > > Do you expect the jiffies to be updated when we are in the timer irq handler ? > Oops, my bad. Will correct that using

RE: [PATCH v2] clocksource/fsl: Fix errata A-007728 for flextimer

2016-09-21 Thread Meng Yi
> > __iomem *base) > > > > static inline void ftm_irq_acknowledge(void __iomem *base) { > > - u32 val; > > + unsigned long timeout = jiffies + msecs_to_jiffies(100); > > Do you expect the jiffies to be updated when we are in the timer irq handler ? > Oops, my bad. Will correct that using

Re: [PATCH v2] clocksource/fsl: Fix errata A-007728 for flextimer

2016-09-21 Thread Daniel Lezcano
On Sun, Sep 18, 2016 at 02:46:01PM +0800, Meng Yi wrote: > If the FTM counter reaches the FTM_MOD value between the reading of the > TOF bit and the writing of 0 to the TOF bit, the process of clearing the > TOF bit does not work as expected when FTMx_CONF[NUMTOF] != 0 and the > current TOF count

Re: [PATCH v2] clocksource/fsl: Fix errata A-007728 for flextimer

2016-09-21 Thread Daniel Lezcano
On Sun, Sep 18, 2016 at 02:46:01PM +0800, Meng Yi wrote: > If the FTM counter reaches the FTM_MOD value between the reading of the > TOF bit and the writing of 0 to the TOF bit, the process of clearing the > TOF bit does not work as expected when FTMx_CONF[NUMTOF] != 0 and the > current TOF count