On Sun, Jan 21, 2024 at 10:22:50AM +0100, Christophe JAILLET wrote:
Seems like entire conditional here is not needed as it's implied in the max().Ok, understood.
And max() should take both signed or unsigned types, different types might have
a side effect.
Thanks for the review.
+ /*
+ * Whichever is greater - integration time period or
+ * sampling period.
+ */
+ delay = max(intg_time,
+ apds9306_repeat_rate_period[repeat_rate_idx]);