Re: [PATCH] iio: adc: ti-ads1018: fix type overflow for data rate

From: Chunyang Chen

Date: Thu Mar 05 2026 - 07:52:42 EST


Hi Kurt, Andy,

Thanks for your review and suggestions!
I have sent out the v2 patch, which addresses the following:

On Thu, Mar 5, 2026, Kurt Borja wrote:
> data_rate_mode_to_hz[] is u32 so I prefer for this to be u32. This also
> kinda matches ads1018_calc_delay() which takes unsigned int.

Changed 'drate' type to u32 to match the array definition and maintain
consistency, as suggested by Kurt.

On Thu, Mar 5, 2026, Andy Shevchenko wrote:
> Needs Fixes tag.

Added the Fixes tag as requested by Andy.

Best regards,
Chunyang Chen