]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
iio: adc: adc128s052: Rename channel structs
authorMatti Vaittinen <mazziesaccount@gmail.com>
Mon, 18 Aug 2025 08:12:52 +0000 (11:12 +0300)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Mon, 18 Aug 2025 19:12:13 +0000 (20:12 +0100)
The adc128s052 can be used to drive a few other ADCs but the TI's ADCs
it was originally intended for. The TI's IC variants model numbers don't
(trivially) explain the channel configuration (for a reader working with
other than TI's ICs).

Rename the channel configuration structures to explicitly explain they
are used for simple ADCs, having 2, 4 or 8 channels.

Signed-off-by: Matti Vaittinen <mazziesaccount@gmail.com>
Reviewed-by: David Lechner <dlechner@baylibre.com>
Link: https://patch.msgid.link/e737f2b416e25d8e4e734e2765b0e21a3f0ae0bb.1755504346.git.mazziesaccount@gmail.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/adc/ti-adc128s052.c

index 81153253529e77717bf6e0114032b1748c6f1bcc..9b8ecaca01edff4a1bd98971c2f6af37f54cc580 100644 (file)
@@ -99,52 +99,52 @@ static int adc128_read_raw(struct iio_dev *indio_dev,
                .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE) \
        }
 
-static const struct iio_chan_spec adc128s052_channels[] = {
+static const struct iio_chan_spec simple_2chan_adc_channels[] = {
        ADC128_VOLTAGE_CHANNEL(0),
        ADC128_VOLTAGE_CHANNEL(1),
-       ADC128_VOLTAGE_CHANNEL(2),
-       ADC128_VOLTAGE_CHANNEL(3),
-       ADC128_VOLTAGE_CHANNEL(4),
-       ADC128_VOLTAGE_CHANNEL(5),
-       ADC128_VOLTAGE_CHANNEL(6),
-       ADC128_VOLTAGE_CHANNEL(7),
 };
 
-static const struct iio_chan_spec adc122s021_channels[] = {
+static const struct iio_chan_spec simple_4chan_adc_channels[] = {
        ADC128_VOLTAGE_CHANNEL(0),
        ADC128_VOLTAGE_CHANNEL(1),
+       ADC128_VOLTAGE_CHANNEL(2),
+       ADC128_VOLTAGE_CHANNEL(3),
 };
 
-static const struct iio_chan_spec adc124s021_channels[] = {
+static const struct iio_chan_spec simple_8chan_adc_channels[] = {
        ADC128_VOLTAGE_CHANNEL(0),
        ADC128_VOLTAGE_CHANNEL(1),
        ADC128_VOLTAGE_CHANNEL(2),
        ADC128_VOLTAGE_CHANNEL(3),
+       ADC128_VOLTAGE_CHANNEL(4),
+       ADC128_VOLTAGE_CHANNEL(5),
+       ADC128_VOLTAGE_CHANNEL(6),
+       ADC128_VOLTAGE_CHANNEL(7),
 };
 
 static const char * const bd79104_regulators[] = { "iovdd" };
 
 static const struct adc128_configuration adc122s_config = {
-       .channels = adc122s021_channels,
-       .num_channels = ARRAY_SIZE(adc122s021_channels),
+       .channels = simple_2chan_adc_channels,
+       .num_channels = ARRAY_SIZE(simple_2chan_adc_channels),
        .refname = "vref",
 };
 
 static const struct adc128_configuration adc124s_config = {
-       .channels = adc124s021_channels,
-       .num_channels = ARRAY_SIZE(adc124s021_channels),
+       .channels = simple_4chan_adc_channels,
+       .num_channels = ARRAY_SIZE(simple_4chan_adc_channels),
        .refname = "vref",
 };
 
 static const struct adc128_configuration adc128s_config = {
-       .channels = adc128s052_channels,
-       .num_channels = ARRAY_SIZE(adc128s052_channels),
+       .channels = simple_8chan_adc_channels,
+       .num_channels = ARRAY_SIZE(simple_8chan_adc_channels),
        .refname = "vref",
 };
 
 static const struct adc128_configuration bd79104_config = {
-       .channels = adc128s052_channels,
-       .num_channels = ARRAY_SIZE(adc128s052_channels),
+       .channels = simple_8chan_adc_channels,
+       .num_channels = ARRAY_SIZE(simple_8chan_adc_channels),
        .refname = "vdd",
        .other_regulators = &bd79104_regulators,
        .num_other_regulators = 1,