Different sensors may have different enable and disable functions, so
introduce enable and disable in gpadc_data to support soc specific
function.

Signed-off-by: Yangtao Li <tiny.win...@gmail.com>
---
 drivers/iio/adc/sun4i-gpadc-iio.c | 37 ++++++++++++++++++++++++++-----
 1 file changed, 31 insertions(+), 6 deletions(-)

diff --git a/drivers/iio/adc/sun4i-gpadc-iio.c 
b/drivers/iio/adc/sun4i-gpadc-iio.c
index b41ec0d5964d..de6b8556a549 100644
--- a/drivers/iio/adc/sun4i-gpadc-iio.c
+++ b/drivers/iio/adc/sun4i-gpadc-iio.c
@@ -49,6 +49,8 @@ static unsigned int sun6i_gpadc_chan_select(unsigned int chan)
        return SUN6I_GPADC_CTRL1_ADC_CHAN_SELECT(chan);
 }
 
+struct sun4i_gpadc_iio;
+
 struct gpadc_data {
        int             temp_offset;
        int             temp_scale;
@@ -56,10 +58,15 @@ struct gpadc_data {
        unsigned int    tp_adc_select;
        unsigned int    (*adc_chan_select)(unsigned int chan);
        unsigned int    adc_chan_mask;
+       int             (*gpadc_enable)(struct sun4i_gpadc_iio *info);
+       int             (*gpadc_disable)(struct sun4i_gpadc_iio *info);
        unsigned int    sensor_count;
        unsigned int    temp_data_base;
 };
 
+static int sun4i_gpadc_disable(struct sun4i_gpadc_iio *info);
+static int sun4i_gpadc_enable(struct sun4i_gpadc_iio *info);
+
 static const struct gpadc_data sun4i_gpadc_data = {
        .temp_offset = -1932,
        .temp_scale = 133,
@@ -67,6 +74,8 @@ static const struct gpadc_data sun4i_gpadc_data = {
        .tp_adc_select = SUN4I_GPADC_CTRL1_TP_ADC_SELECT,
        .adc_chan_select = &sun4i_gpadc_chan_select,
        .adc_chan_mask = SUN4I_GPADC_CTRL1_ADC_CHAN_MASK,
+       .gpadc_enable = sun4i_gpadc_enable,
+       .gpadc_disable = sun4i_gpadc_disable,
        .sensor_count = 1,
        .temp_data_base = SUN4I_GPADC_TEMP_DATA,
 };
@@ -78,6 +87,8 @@ static const struct gpadc_data sun5i_gpadc_data = {
        .tp_adc_select = SUN4I_GPADC_CTRL1_TP_ADC_SELECT,
        .adc_chan_select = &sun4i_gpadc_chan_select,
        .adc_chan_mask = SUN4I_GPADC_CTRL1_ADC_CHAN_MASK,
+       .gpadc_enable = sun4i_gpadc_enable,
+       .gpadc_disable = sun4i_gpadc_disable,
        .sensor_count = 1,
        .temp_data_base = SUN4I_GPADC_TEMP_DATA,
 };
@@ -89,6 +100,8 @@ static const struct gpadc_data sun6i_gpadc_data = {
        .tp_adc_select = SUN6I_GPADC_CTRL1_TP_ADC_SELECT,
        .adc_chan_select = &sun6i_gpadc_chan_select,
        .adc_chan_mask = SUN6I_GPADC_CTRL1_ADC_CHAN_MASK,
+       .gpadc_enable = sun4i_gpadc_enable,
+       .gpadc_disable = sun4i_gpadc_disable,
        .sensor_count = 1,
        .temp_data_base = SUN4I_GPADC_TEMP_DATA,
 };
@@ -97,6 +110,8 @@ static const struct gpadc_data sun8i_a33_gpadc_data = {
        .temp_offset = -1662,
        .temp_scale = 162,
        .tp_mode_en = SUN8I_GPADC_CTRL1_CHOP_TEMP_EN,
+       .gpadc_enable = sun4i_gpadc_enable,
+       .gpadc_disable = sun4i_gpadc_disable,
        .sensor_count = 1,
        .temp_data_base = SUN4I_GPADC_TEMP_DATA,
 };
@@ -402,10 +417,8 @@ static irqreturn_t sun4i_gpadc_fifo_data_irq_handler(int 
irq, void *dev_id)
        return IRQ_HANDLED;
 }
 
-static int sun4i_gpadc_runtime_suspend(struct device *dev)
+static int sun4i_gpadc_disable(struct sun4i_gpadc_iio *info)
 {
-       struct sun4i_gpadc_iio *info = iio_priv(dev_get_drvdata(dev));
-
        /* Disable the ADC on IP */
        regmap_write(info->regmap, SUN4I_GPADC_CTRL1, 0);
        /* Disable temperature sensor on IP */
@@ -414,10 +427,8 @@ static int sun4i_gpadc_runtime_suspend(struct device *dev)
        return 0;
 }
 
-static int sun4i_gpadc_runtime_resume(struct device *dev)
+static int sun4i_gpadc_enable(struct sun4i_gpadc_iio *info)
 {
-       struct sun4i_gpadc_iio *info = iio_priv(dev_get_drvdata(dev));
-
        /* clkin = 6MHz */
        regmap_write(info->regmap, SUN4I_GPADC_CTRL0,
                     SUN4I_GPADC_CTRL0_ADC_CLK_DIVIDER(2) |
@@ -435,6 +446,20 @@ static int sun4i_gpadc_runtime_resume(struct device *dev)
        return 0;
 }
 
+static int sun4i_gpadc_runtime_suspend(struct device *dev)
+{
+       struct sun4i_gpadc_iio *info = iio_priv(dev_get_drvdata(dev));
+
+       return info->data->gpadc_disable(info);
+}
+
+static int sun4i_gpadc_runtime_resume(struct device *dev)
+{
+       struct sun4i_gpadc_iio *info = iio_priv(dev_get_drvdata(dev));
+
+       return info->data->gpadc_enable(info);
+}
+
 static int sun4i_gpadc_get_temp(void *data, int *temp)
 {
        struct sun4i_sensor_tzd *tzd = data;
-- 
2.17.1

Reply via email to