Re: [PATCH v3 5/6] iio: imu: adis16607: Add external clock sync support

[Jonathan Cameron]

On Thu, 15 Oct 2026 12:33:02 +0300
Radu Sabau via B4 Relay <devnull+radu.sabau.analog.com@xxxxxxxxxx> wrote:

> From: Radu Sabau <radu.sabau@xxxxxxxxxx>
> 
> Add support for an optional external clock (clocks property),
> selecting between SYNC_DIRECT and SYNC_SCALED modes depending on the
> supplied frequency, with the internal 8 kHz clock used as the default
> when no clock is provided.
> 
> Signed-off-by: Radu Sabau <radu.sabau@xxxxxxxxxx>

Just a very quick look as I'm out of time for today.

> +
> +	/* Default dec=0 after reset: period = 1 / clk_freq */

Why does the comment make sense now but didn't before?

>  	st->sample_period_ns = DIV_ROUND_CLOSEST_ULL((u64)NSEC_PER_SEC * 1000ULL,
> -						     st->clk_freq);
> +						      st->clk_freq);
Tidy this up.

> +
>  	return 0;
>  }
>  
>