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