Re: [PATCH V3 4/7] iio: adc: refactor some functions for support more PMiCs

From: Jonathan Cameron
Date: Sun Apr 10 2022 - 12:13:07 EST


On Thu, 7 Apr 2022 16:21:45 +0800
Cixi Geng <gengcixi@xxxxxxxxx> wrote:

> From: Cixi Geng <cixi.geng1@xxxxxxxxxx>
>
> refactor the common adc_nvmem_cell_calib_data,adc_to_volt and call
> these in the origin sc27xx_adc_scale_calibration,sc27xx_adc_to_volt
>
> Signed-off-by: Cixi Geng <cixi.geng1@xxxxxxxxxx>
One trivial comment inline.

> ---
> drivers/iio/adc/sc27xx_adc.c | 57 ++++++++++++++++++++++++++----------
> 1 file changed, 41 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/iio/adc/sc27xx_adc.c b/drivers/iio/adc/sc27xx_adc.c
> index 28bd70c27420..60c0a6aa3f45 100644
> --- a/drivers/iio/adc/sc27xx_adc.c
> +++ b/drivers/iio/adc/sc27xx_adc.c
> @@ -136,16 +136,41 @@ static int sc27xx_adc_get_calib_data(u32 calib_data, int calib_adc)
> return ((calib_data & 0xff) + calib_adc - 128) * 4;
> }
>
> +/* get the adc nvmem cell calibration data */
> +static int adc_nvmem_cell_calib_data(struct sc27xx_adc_data *data, const char *cell_name)
> +{
> + struct nvmem_cell *cell;
> + void *buf;
> + u32 origin_calib_data = 0;
> + size_t len;
> +
> + if (!data)
> + return -EINVAL;
> +
> + cell = nvmem_cell_get(data->dev, cell_name);
> + if (IS_ERR(cell))
> + return PTR_ERR(cell);
> +
> + buf = nvmem_cell_read(cell, &len);
> + if (IS_ERR(buf)) {
> + nvmem_cell_put(cell);
> + return PTR_ERR(buf);
> + }
> +
> + memcpy(&origin_calib_data, buf, min(len, sizeof(u32)));
> +
> + kfree(buf);
> + nvmem_cell_put(cell);
> + return origin_calib_data;
> +}
> +
> static int sc27xx_adc_scale_calibration(struct sc27xx_adc_data *data,
> bool big_scale)
> {
> const struct sc27xx_adc_linear_graph *calib_graph;
> struct sc27xx_adc_linear_graph *graph;
> - struct nvmem_cell *cell;
> const char *cell_name;
> u32 calib_data = 0;
> - void *buf;
> - size_t len;
>
> if (big_scale) {
> calib_graph = data->var_data->bscale_cal;
> @@ -157,24 +182,13 @@ static int sc27xx_adc_scale_calibration(struct sc27xx_adc_data *data,
> cell_name = "small_scale_calib";
> }
>
> - cell = nvmem_cell_get(data->dev, cell_name);
> - if (IS_ERR(cell))
> - return PTR_ERR(cell);
> -
> - buf = nvmem_cell_read(cell, &len);
> - nvmem_cell_put(cell);
> -
> - if (IS_ERR(buf))
> - return PTR_ERR(buf);
> -
> - memcpy(&calib_data, buf, min(len, sizeof(u32)));
> + calib_data = adc_nvmem_cell_calib_data(data, cell_name);
>
> /* Only need to calibrate the adc values in the linear graph. */
> graph->adc0 = sc27xx_adc_get_calib_data(calib_data, calib_graph->adc0);
> graph->adc1 = sc27xx_adc_get_calib_data(calib_data >> 8,
> calib_graph->adc1);
>
> - kfree(buf);
> return 0;
> }
>
> @@ -285,6 +299,7 @@ static int sc27xx_adc_read(struct sc27xx_adc_data *data, int channel,
> disable_adc:
> regmap_update_bits(data->regmap, data->base + SC27XX_ADC_CTL,
> SC27XX_ADC_EN, 0);
> +

Unrelated change that shouldn't be in this patch.

> unlock_adc:
> hwspin_unlock_raw(data->hwlock);
>
> @@ -305,7 +320,7 @@ static void sc27xx_adc_volt_ratio(struct sc27xx_adc_data *data,
> *div_denominator = ratio & SC27XX_RATIO_DENOMINATOR_MASK;
> }
>
> -static int sc27xx_adc_to_volt(struct sc27xx_adc_linear_graph *graph,
> +static int adc_to_volt(struct sc27xx_adc_linear_graph *graph,
> int raw_adc)
> {
> int tmp;
> @@ -314,6 +329,16 @@ static int sc27xx_adc_to_volt(struct sc27xx_adc_linear_graph *graph,
> tmp /= (graph->adc0 - graph->adc1);
> tmp += graph->volt1;
>
> + return tmp;
> +}
> +
> +static int sc27xx_adc_to_volt(struct sc27xx_adc_linear_graph *graph,
> + int raw_adc)
> +{
> + int tmp;
> +
> + tmp = adc_to_volt(graph, raw_adc);
> +
> return tmp < 0 ? 0 : tmp;
> }
>