diff options
| author | Chandra Mohan Sundar <chandramohan.explore@gmail.com> | 2025-08-16 16:37:55 +0300 |
|---|---|---|
| committer | Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2025-08-17 17:31:54 +0300 |
| commit | 26d48cc203e1c4d79b0bb6c7e2cdcaea5f548971 (patch) | |
| tree | 66561fcfa22400f184379f1993e6b65193d61ca6 | |
| parent | 35e3d0cd8e89d811b915593cbd8d9891d21d4a1a (diff) | |
| download | linux-26d48cc203e1c4d79b0bb6c7e2cdcaea5f548971.tar.xz | |
iio: adc: ad7768-1: Remove logically dead code
The clamp macro returns a value within the specified range.
In ad7768_set_freq(), the value returned from clamp() is checked against
zero, which is not possible since the value would always be between
50 and 1024000.
Removed the 'if' check.
This issue was reported by static coverity analyser as logically
dead code.
Signed-off-by: Chandra Mohan Sundar <chandramohan.explore@gmail.com>
Link: https://patch.msgid.link/20250816133757.98624-1-chandramohan.explore@gmail.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
| -rw-r--r-- | drivers/iio/adc/ad7768-1.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/iio/adc/ad7768-1.c b/drivers/iio/adc/ad7768-1.c index ca8fa91796ca..a27591bbf7df 100644 --- a/drivers/iio/adc/ad7768-1.c +++ b/drivers/iio/adc/ad7768-1.c @@ -687,8 +687,6 @@ static int ad7768_set_freq(struct ad7768_state *st, int ret; freq = clamp(freq, 50, 1024000); - if (freq == 0) - return -EINVAL; mclk_div = DIV_ROUND_CLOSEST(st->mclk_freq, freq * st->oversampling_ratio); /* Find the closest match for the desired sampling frequency */ |