On Sat,  3 Mar 2018 20:49:37 -0500
Brian Masney <masn...@onstation.org> wrote:

> The statP and calP variables triggered an 'Avoid CamelCase' warning
> from checkpatch.pl. This patch renames these variables to stat and cal
> to fix this warning.
> 
> Signed-off-by: Brian Masney <masn...@onstation.org>
Applied.
> ---
>  drivers/staging/iio/light/tsl2x7x.c | 26 +++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
> 
> diff --git a/drivers/staging/iio/light/tsl2x7x.c 
> b/drivers/staging/iio/light/tsl2x7x.c
> index cc209b64ed5a..380a6c96a69a 100644
> --- a/drivers/staging/iio/light/tsl2x7x.c
> +++ b/drivers/staging/iio/light/tsl2x7x.c
> @@ -773,7 +773,7 @@ static int tsl2x7x_invoke_change(struct iio_dev 
> *indio_dev)
>  }
>  
>  static void tsl2x7x_prox_calculate(int *data, int length,
> -                                struct tsl2x7x_prox_stat *statP)
> +                                struct tsl2x7x_prox_stat *stat)
>  {
>       int i;
>       int sample_sum;
> @@ -783,21 +783,21 @@ static void tsl2x7x_prox_calculate(int *data, int 
> length,
>               length = 1;
>  
>       sample_sum = 0;
> -     statP->min = INT_MAX;
> -     statP->max = INT_MIN;
> +     stat->min = INT_MAX;
> +     stat->max = INT_MIN;
>       for (i = 0; i < length; i++) {
>               sample_sum += data[i];
> -             statP->min = min(statP->min, data[i]);
> -             statP->max = max(statP->max, data[i]);
> +             stat->min = min(stat->min, data[i]);
> +             stat->max = max(stat->max, data[i]);
>       }
>  
> -     statP->mean = sample_sum / length;
> +     stat->mean = sample_sum / length;
>       sample_sum = 0;
>       for (i = 0; i < length; i++) {
> -             tmp = data[i] - statP->mean;
> +             tmp = data[i] - stat->mean;
>               sample_sum += tmp * tmp;
>       }
> -     statP->stddev = int_sqrt((long)sample_sum / length);
> +     stat->stddev = int_sqrt((long)sample_sum / length);
>  }
>  
>  /**
> @@ -812,7 +812,7 @@ static void tsl2x7x_prox_cal(struct iio_dev *indio_dev)
>       int prox_history[MAX_SAMPLES_CAL + 1];
>       int i;
>       struct tsl2x7x_prox_stat prox_stat_data[2];
> -     struct tsl2x7x_prox_stat *calP;
> +     struct tsl2x7x_prox_stat *cal;
>       struct tsl2X7X_chip *chip = iio_priv(indio_dev);
>       u8 tmp_irq_settings;
>       u8 current_state = chip->tsl2x7x_chip_status;
> @@ -844,13 +844,13 @@ static void tsl2x7x_prox_cal(struct iio_dev *indio_dev)
>       }
>  
>       tsl2x7x_chip_off(indio_dev);
> -     calP = &prox_stat_data[PROX_STAT_CAL];
> +     cal = &prox_stat_data[PROX_STAT_CAL];
>       tsl2x7x_prox_calculate(prox_history,
> -                            chip->settings.prox_max_samples_cal, calP);
> -     chip->settings.prox_thres_high = (calP->max << 1) - calP->mean;
> +                            chip->settings.prox_max_samples_cal, cal);
> +     chip->settings.prox_thres_high = (cal->max << 1) - cal->mean;
>  
>       dev_info(&chip->client->dev, " cal min=%d mean=%d max=%d\n",
> -              calP->min, calP->mean, calP->max);
> +              cal->min, cal->mean, cal->max);
>       dev_info(&chip->client->dev,
>                "%s proximity threshold set to %d\n",
>                chip->client->name, chip->settings.prox_thres_high);

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to