On Mon, Sep 10, 2018 at 11:34:49PM +0300, Sergei Shtylyov wrote: > On 09/10/2018 11:22 PM, Sergei Shtylyov wrote: > > > The driver seems to abuse *unsigned long* not only for the (32-bit) > > register values but also for the 'sh_cmt_channel::total_cycles' which > > needs to always be 64-bit -- as a result, the clocksource's mask is > > needlessly clamped down to 32-bits on the 32-bit machines... > > I thought this bug was present from the start but no... > > > Reported-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> > > Fixes: 19bdc9d061bc ("clocksource: sh_cmt clocksource support") Reviewed-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>