iio: adc: ad4630: Use NANO to get to the correct sampling frequency

machschmitt · nunojsa · commit 0b3d96fbc075 · 2024-09-20T08:39:07.000+02:00
The ad4630 driver calculated the inverse of the PWM period in pico seconds to report the sampling frequency provided by the device. However, that lead to a sampling frequency 1000 higher than the actual one since the PWM period is actually in the nano seconds. Use NANO instead of PICO to correctly calculate the sampling frequency in Hz. Fixes: 8f4838f ("iio: adc: ad4630: Add support for ADAQ4224") Signed-off-by: Marcelo Schmitt <marcelo.schmitt@analog.com>
diff --git a/drivers/iio/adc/ad4630.c b/drivers/iio/adc/ad4630.c
@@ -261,7 +261,7 @@ static void ad4630_get_sampling_freq(const struct ad4630_state *st, int *freq)
 	struct pwm_state conversion_state;
 
 	pwm_get_state(st->conv_trigger, &conversion_state);
-	*freq = DIV_ROUND_CLOSEST_ULL(PICO, conversion_state.period);
+	*freq = DIV_ROUND_CLOSEST_ULL(NANO, conversion_state.period);
 }
 
 static int ad4630_get_chan_gain(struct iio_dev *indio_dev, int ch, int *val)

Original file line number	Diff line number	Diff line change
`@@ -261,7 +261,7 @@ static void ad4630_get_sampling_freq(const struct ad4630_state st, int freq)`
`261`	`261`	`struct pwm_state conversion_state;`
`262`	`262`
`263`	`263`	`pwm_get_state(st->conv_trigger, &conversion_state);`
`264`		`- *freq = DIV_ROUND_CLOSEST_ULL(PICO, conversion_state.period);`
	`264`	`+ *freq = DIV_ROUND_CLOSEST_ULL(NANO, conversion_state.period);`
`265`	`265`	`}`
`266`	`266`
`267`	`267`	`static int ad4630_get_chan_gain(struct iio_dev indio_dev, int ch, int val)`