On 2025/1/22 18:03, Uwe Kleine-König wrote:
The rational is usually that mul_u64_u64_div_u64(ddata->clk_rate_hz, state->period, NSEC_PER_SEC) cannot overflow. Your driver relies on that, too. (Which somewhat proves that a comment is indeed necessary 🙂
Oh, thanks for pointing that out, I almost forgot about that. Will check this out in the next version.
Regards, Chen