iio: adc: ad7192: Organize chip info
authorAlisa-Dariana Roman <alisa.roman@analog.com>
Tue, 10 Oct 2023 12:49:24 +0000 (15:49 +0300)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Wed, 11 Oct 2023 14:54:40 +0000 (15:54 +0100)
Move all chip specific data into chip_info structure.

Signed-off-by: Alisa-Dariana Roman <alisa.roman@analog.com>
Link: https://lore.kernel.org/r/20231010124927.143343-2-alisadariana@gmail.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/adc/ad7192.c

index e0be394ccb346bbcab68cbb914843a6f9950d334..d72a190c51098802bdb099810c707546c6b09918 100644 (file)
@@ -171,6 +171,9 @@ enum {
 struct ad7192_chip_info {
        unsigned int                    chip_id;
        const char                      *name;
+       const struct iio_chan_spec      *channels;
+       u8                              num_channels;
+       const struct iio_info           *info;
 };
 
 struct ad7192_state {
@@ -964,39 +967,33 @@ static const struct ad7192_chip_info ad7192_chip_info_tbl[] = {
        [ID_AD7190] = {
                .chip_id = CHIPID_AD7190,
                .name = "ad7190",
+               .channels = ad7192_channels,
+               .num_channels = ARRAY_SIZE(ad7192_channels),
+               .info = &ad7192_info,
        },
        [ID_AD7192] = {
                .chip_id = CHIPID_AD7192,
                .name = "ad7192",
+               .channels = ad7192_channels,
+               .num_channels = ARRAY_SIZE(ad7192_channels),
+               .info = &ad7192_info,
        },
        [ID_AD7193] = {
                .chip_id = CHIPID_AD7193,
                .name = "ad7193",
+               .channels = ad7193_channels,
+               .num_channels = ARRAY_SIZE(ad7193_channels),
+               .info = &ad7192_info,
        },
        [ID_AD7195] = {
                .chip_id = CHIPID_AD7195,
                .name = "ad7195",
+               .channels = ad7192_channels,
+               .num_channels = ARRAY_SIZE(ad7192_channels),
+               .info = &ad7195_info,
        },
 };
 
-static int ad7192_channels_config(struct iio_dev *indio_dev)
-{
-       struct ad7192_state *st = iio_priv(indio_dev);
-
-       switch (st->chip_info->chip_id) {
-       case CHIPID_AD7193:
-               indio_dev->channels = ad7193_channels;
-               indio_dev->num_channels = ARRAY_SIZE(ad7193_channels);
-               break;
-       default:
-               indio_dev->channels = ad7192_channels;
-               indio_dev->num_channels = ARRAY_SIZE(ad7192_channels);
-               break;
-       }
-
-       return 0;
-}
-
 static void ad7192_reg_disable(void *reg)
 {
        regulator_disable(reg);
@@ -1051,15 +1048,9 @@ static int ad7192_probe(struct spi_device *spi)
                st->chip_info = (void *)spi_get_device_id(spi)->driver_data;
        indio_dev->name = st->chip_info->name;
        indio_dev->modes = INDIO_DIRECT_MODE;
-
-       ret = ad7192_channels_config(indio_dev);
-       if (ret < 0)
-               return ret;
-
-       if (st->chip_info->chip_id == CHIPID_AD7195)
-               indio_dev->info = &ad7195_info;
-       else
-               indio_dev->info = &ad7192_info;
+       indio_dev->channels = st->chip_info->channels;
+       indio_dev->num_channels = st->chip_info->num_channels;
+       indio_dev->info = st->chip_info->info;
 
        ret = ad_sd_init(&st->sd, indio_dev, spi, &ad7192_sigma_delta_info);
        if (ret)