[PATCH v1 01/13] iio: chemical: bme680: Fix indentation and unnecessary spaces

vamoirid posted 13 patches 1 month, 2 weeks ago
[PATCH v1 01/13] iio: chemical: bme680: Fix indentation and unnecessary spaces
Posted by vamoirid 1 month, 2 weeks ago
From: Vasileios Amoiridis <vassilisamir@gmail.com>

Fix indentation issues, line breaking and unnecessary spaces
reported by checkpatch.pl.

Signed-off-by: Vasileios Amoiridis <vassilisamir@gmail.com>
---
 drivers/iio/chemical/bme680_core.c | 33 ++++++++++++------------------
 1 file changed, 13 insertions(+), 20 deletions(-)

diff --git a/drivers/iio/chemical/bme680_core.c b/drivers/iio/chemical/bme680_core.c
index 5d2e750ca2b9..95a667d56527 100644
--- a/drivers/iio/chemical/bme680_core.c
+++ b/drivers/iio/chemical/bme680_core.c
@@ -224,7 +224,7 @@ static int bme680_read_calib(struct bme680_data *data,
 	calib->res_heat_val = data->bme680_cal_buf_3[RES_HEAT_VAL];
 
 	calib->res_heat_range = FIELD_GET(BME680_RHRANGE_MASK,
-					data->bme680_cal_buf_3[RES_HEAT_RANGE]);
+					  data->bme680_cal_buf_3[RES_HEAT_RANGE]);
 
 	calib->range_sw_err = FIELD_GET(BME680_RSERROR_MASK,
 					data->bme680_cal_buf_3[RANGE_SW_ERR]);
@@ -294,8 +294,7 @@ static int bme680_get_t_fine(struct bme680_data *data, s32 *t_fine)
 	return 0;
 }
 
-static s16 bme680_compensate_temp(struct bme680_data *data,
-				  u32 adc_temp)
+static s16 bme680_compensate_temp(struct bme680_data *data, u32 adc_temp)
 {
 	return (bme680_calc_t_fine(data, adc_temp) * 5 + 128) / 256;
 }
@@ -445,12 +444,12 @@ static u32 bme680_compensate_gas(struct bme680_data *data, u16 gas_res_adc,
 				2136746228u, 2147483647u, 2126008810u,
 				2147483647u, 2147483647u};
 
-	var1 = ((1340 + (5 * (s64) calib->range_sw_err)) *
-			((s64) lookupTable[gas_range])) >> 16;
+	var1 = ((1340 + (5 * (s64)calib->range_sw_err)) *
+			((s64)lookupTable[gas_range])) >> 16;
 	var2 = ((gas_res_adc << 15) - 16777216) + var1;
 	var3 = ((125000 << (15 - gas_range)) * var1) >> 9;
 	var3 += (var2 >> 1);
-	calc_gas_res = div64_s64(var3, (s64) var2);
+	calc_gas_res = div64_s64(var3, (s64)var2);
 
 	return calc_gas_res;
 }
@@ -468,7 +467,7 @@ static u8 bme680_calc_heater_res(struct bme680_data *data, u16 temp)
 	if (temp > 400) /* Cap temperature */
 		temp = 400;
 
-	var1 = (((s32) BME680_AMB_TEMP * calib->par_gh3) / 1000) * 256;
+	var1 = (((s32)BME680_AMB_TEMP * calib->par_gh3) / 1000) * 256;
 	var2 = (calib->par_gh1 + 784) * (((((calib->par_gh2 + 154009) *
 						temp * 5) / 100)
 						+ 3276800) / 10);
@@ -571,9 +570,8 @@ static int bme680_chip_config(struct bme680_data *data)
 	int ret;
 	u8 osrs;
 
-	osrs = FIELD_PREP(
-		BME680_OSRS_HUMIDITY_MASK,
-		bme680_oversampling_to_reg(data->oversampling_humid));
+	osrs = FIELD_PREP(BME680_OSRS_HUMIDITY_MASK,
+			  bme680_oversampling_to_reg(data->oversampling_humid));
 	/*
 	 * Highly recommended to set oversampling of humidity before
 	 * temperature/pressure oversampling.
@@ -587,8 +585,7 @@ static int bme680_chip_config(struct bme680_data *data)
 
 	/* IIR filter settings */
 	ret = regmap_update_bits(data->regmap, BME680_REG_CONFIG,
-				 BME680_FILTER_MASK,
-				 BME680_FILTER_COEFF_VAL);
+				 BME680_FILTER_MASK, BME680_FILTER_COEFF_VAL);
 	if (ret < 0) {
 		dev_err(dev, "failed to write config register\n");
 		return ret;
@@ -664,8 +661,7 @@ static int bme680_read_temp(struct bme680_data *data, int *val)
 	return IIO_VAL_INT;
 }
 
-static int bme680_read_press(struct bme680_data *data,
-			     int *val, int *val2)
+static int bme680_read_press(struct bme680_data *data, int *val, int *val2)
 {
 	int ret;
 	u32 adc_press;
@@ -684,8 +680,7 @@ static int bme680_read_press(struct bme680_data *data,
 	return IIO_VAL_FRACTIONAL;
 }
 
-static int bme680_read_humid(struct bme680_data *data,
-			     int *val, int *val2)
+static int bme680_read_humid(struct bme680_data *data, int *val, int *val2)
 {
 	int ret;
 	u32 adc_humidity, comp_humidity;
@@ -706,8 +701,7 @@ static int bme680_read_humid(struct bme680_data *data,
 	return IIO_VAL_FRACTIONAL;
 }
 
-static int bme680_read_gas(struct bme680_data *data,
-			   int *val)
+static int bme680_read_gas(struct bme680_data *data, int *val)
 {
 	struct device *dev = regmap_get_device(data->regmap);
 	int ret;
@@ -889,8 +883,7 @@ int bme680_core_probe(struct device *dev, struct regmap *regmap,
 	data->heater_temp = 320; /* degree Celsius */
 	data->heater_dur = 150;  /* milliseconds */
 
-	ret = regmap_write(regmap, BME680_REG_SOFT_RESET,
-			   BME680_CMD_SOFTRESET);
+	ret = regmap_write(regmap, BME680_REG_SOFT_RESET, BME680_CMD_SOFTRESET);
 	if (ret < 0)
 		return dev_err_probe(dev, ret, "Failed to reset chip\n");
 
-- 
2.43.0
Re: [PATCH v1 01/13] iio: chemical: bme680: Fix indentation and unnecessary spaces
Posted by Andy Shevchenko 1 month, 2 weeks ago
On Thu, Oct 10, 2024 at 11:00:18PM +0200, vamoirid wrote:
> From: Vasileios Amoiridis <vassilisamir@gmail.com>
> 
> Fix indentation issues, line breaking and unnecessary spaces
> reported by checkpatch.pl.

Can we move this to be the last (or close to the end) in the series?

-- 
With Best Regards,
Andy Shevchenko
Re: [PATCH v1 01/13] iio: chemical: bme680: Fix indentation and unnecessary spaces
Posted by Vasileios Aoiridis 1 month, 2 weeks ago
On Fri, Oct 11, 2024 at 12:55:00PM +0300, Andy Shevchenko wrote:
> On Thu, Oct 10, 2024 at 11:00:18PM +0200, vamoirid wrote:
> > From: Vasileios Amoiridis <vassilisamir@gmail.com>
> > 
> > Fix indentation issues, line breaking and unnecessary spaces
> > reported by checkpatch.pl.
> 
> Can we move this to be the last (or close to the end) in the series?
> 
> -- 
> With Best Regards,
> Andy Shevchenko
> 
>

Hi Andy,

Yes, will do it.