nrf_timer_ms_to_ticks wrong ASSERT

[mcisek]

Maybe I'm wrong but this seems to be wrong ASSERT in the function: 

__STATIC_INLINE uint32_t nrf_timer_ms_to_ticks(uint32_t time_ms,
                                               nrf_timer_frequency_t frequency)
{
    // The "frequency" parameter here is actually the prescaler value, and the
    // timer runs at the following frequency: f = 16000 kHz / 2^prescaler.
    uint32_t prescaler = (uint32_t)frequency;
    ASSERT(time_ms <= (UINT32_MAX / 16000UL));
    return ((time_ms * 16000UL) >> prescaler);
}

My understanding is that max time which can be set on timer is dependent from timer frequency and timer bit width. But this assert is not dependent from prescaler(frequency) at all. What happens is: when I try to setup timer for 5min (which is 300 000) this assert occurs - changing timer frequency and/or bit width is not helping (because it can't). This is from SDK 14.2. 

[mcisek]

What I've finally done was calculating ticks for one second using this function and then multiply result by number of seconds I want - that value can be next moved to timer function 'nrf_drv_timer_extended_compare'

[Stian Røed Hafskjold]

I agree that this is a bug. It should be dependent on prescaler and bit width. I would suggest that you just comment out the ASSERT. I will report this internally