An excessive abs for unsigned value

[dmitrygx]

Why is abs needed for the uint32_t value here? Seems it's excessive, because it is always positive or zero , right?

[dmitrygx]

ipath_time.c:282:9: error: taking the absolute value of unsigned type 'unsigned int' has no effect [-Werror,-Wabsolute-value]
    if (abs(new_pico_per_cycle - old_pico_per_cycle) < 5)
        ^
ipath_time.c:282:9: note: remove the call to 'abs' since unsigned values cannot be negative
    if (abs(new_pico_per_cycle - old_pico_per_cycle) < 5)