From: Marijn Suijten Date: Mon, 1 May 2023 23:17:36 +0000 (+0200) Subject: iio: adc: qcom-spmi-adc5: Remove unnecessary datasheet_name NULL check X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=ac0abf75e1a94425f1cd503e1a895e003e2f40d6;p=linux.git iio: adc: qcom-spmi-adc5: Remove unnecessary datasheet_name NULL check datasheet_name is statically filled by a macro for every channel, and is nor should ever be set to NULL. Signed-off-by: Marijn Suijten Reviewed-by: Dmitry Baryshkov Link: https://lore.kernel.org/r/20230502-iio-adc-propagate-fw-node-label-v3-4-6be5db6e6b5a@somainline.org Signed-off-by: Jonathan Cameron --- diff --git a/drivers/iio/adc/qcom-spmi-adc5.c b/drivers/iio/adc/qcom-spmi-adc5.c index fe2000209d97d..0a4fd3a461138 100644 --- a/drivers/iio/adc/qcom-spmi-adc5.c +++ b/drivers/iio/adc/qcom-spmi-adc5.c @@ -657,8 +657,7 @@ static int adc5_get_fw_channel_data(struct adc5_chip *adc, chan = chan & ADC_CHANNEL_MASK; } - if (chan > ADC5_PARALLEL_ISENSE_VBAT_IDATA || - !data->adc_chans[chan].datasheet_name) { + if (chan > ADC5_PARALLEL_ISENSE_VBAT_IDATA) { dev_err(dev, "%s invalid channel number %d\n", name, chan); return -EINVAL; }