On Thu, Apr 11, 2019 at 9:22 AM Marek Vasut <ma...@denx.de> wrote: > > On 4/11/19 8:25 AM, Simon Goldschmidt wrote: > > On Wed, Apr 10, 2019 at 2:01 PM Marek Vasut <ma...@denx.de> wrote: > >> > >> The generic timer count is an incrementing 64bit value and a timer driver > >> must return an incrementing 64bit value. The DW APB timer only provides a > >> 32bit timer counting down, thus the result must be inverted and converted > >> to a 64bit value. The current implementation is however missing the 64bit > >> up-conversion and this results in random timer roll-overs, which in turn > >> triggers random timeouts throughout the codebase. > >> > >> This patch adds the missing 64bit up-conversion to fix the issue. > >> > >> Signed-off-by: Marek Vasut <ma...@denx.de> > >> Cc: Chin Liang See <chin.liang....@intel.com> > >> Cc: Dinh Nguyen <dingu...@kernel.org> > >> Cc: Ley Foon Tan <ley.foon....@intel.com> > >> Cc: Simon Goldschmidt <simon.k.r.goldschm...@gmail.com> > >> Cc: Tien Fong Chee <tien.fong.c...@intel.com> > > > > Reviewed-by: Simon Goldschmidt <simon.k.r.goldschm...@gmail.com> > > Maybe it's time to convert Gen5 to this timer driver too :)
Yes, well, let's see when I find the time. Converting clock and pinctrl to devicetree is on my list as well... Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
