microsoft
/
WSL2-Linux-Kernel
зеркало из https://github.com/microsoft/WSL2-Linux-Kernel.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

iio: chemical: bme680: cleanup bme680_read_calib formatting 

Use the full 80 char width to reduce the number of lines taken
by function calls.

Remove blank lines where it aids clarity by bringing together related
code blocks (such as read hi and low bytes and then combine them into
one value).

Signed-off-by: David Frey <dpfrey@gmail.com>
Reviewed-by: Himanshu Jha <himanshujha199640@gmail.com>
Tested-by: Himanshu Jha <himanshujha199640@gmail.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
This commit is contained in:
David Frey 2018-08-17 12:03:14 -07:00 коммит произвёл Jonathan Cameron
Родитель 8a0ae7d83a
Коммит 6784ba340a
1 изменённых файлов: 12 добавлений и 24 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

32
drivers/iio/chemical/bme680_core.c
Просмотреть файл

@ -102,16 +102,14 @@ static int bme680_read_calib(struct bme680_data *data,
__le16 buf; __le16 buf;
/* Temperature related coefficients */ /* Temperature related coefficients */
ret = regmap_bulk_read(data->regmap, BME680_T1_LSB_REG, ret = regmap_bulk_read(data->regmap, BME680_T1_LSB_REG, (u8 *) &buf, 2);
(u8 *) &buf, 2);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_T1_LSB_REG\n"); dev_err(dev, "failed to read BME680_T1_LSB_REG\n");
return ret; return ret;
} }
calib->par_t1 = le16_to_cpu(buf); calib->par_t1 = le16_to_cpu(buf);
ret = regmap_bulk_read(data->regmap, BME680_T2_LSB_REG, ret = regmap_bulk_read(data->regmap, BME680_T2_LSB_REG, (u8 *) &buf, 2);
(u8 *) &buf, 2);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_T2_LSB_REG\n"); dev_err(dev, "failed to read BME680_T2_LSB_REG\n");
return ret; return ret;
@ -126,16 +124,14 @@ static int bme680_read_calib(struct bme680_data *data,
calib->par_t3 = tmp; calib->par_t3 = tmp;
/* Pressure related coefficients */ /* Pressure related coefficients */
ret = regmap_bulk_read(data->regmap, BME680_P1_LSB_REG, ret = regmap_bulk_read(data->regmap, BME680_P1_LSB_REG, (u8 *) &buf, 2);
(u8 *) &buf, 2);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_P1_LSB_REG\n"); dev_err(dev, "failed to read BME680_P1_LSB_REG\n");
return ret; return ret;
} }
calib->par_p1 = le16_to_cpu(buf); calib->par_p1 = le16_to_cpu(buf);
ret = regmap_bulk_read(data->regmap, BME680_P2_LSB_REG, ret = regmap_bulk_read(data->regmap, BME680_P2_LSB_REG, (u8 *) &buf, 2);
(u8 *) &buf, 2);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_P2_LSB_REG\n"); dev_err(dev, "failed to read BME680_P2_LSB_REG\n");
return ret; return ret;
@ -149,16 +145,14 @@ static int bme680_read_calib(struct bme680_data *data,
} }
calib->par_p3 = tmp; calib->par_p3 = tmp;
ret = regmap_bulk_read(data->regmap, BME680_P4_LSB_REG, ret = regmap_bulk_read(data->regmap, BME680_P4_LSB_REG, (u8 *) &buf, 2);
(u8 *) &buf, 2);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_P4_LSB_REG\n"); dev_err(dev, "failed to read BME680_P4_LSB_REG\n");
return ret; return ret;
} }
calib->par_p4 = le16_to_cpu(buf); calib->par_p4 = le16_to_cpu(buf);
ret = regmap_bulk_read(data->regmap, BME680_P5_LSB_REG, ret = regmap_bulk_read(data->regmap, BME680_P5_LSB_REG, (u8 *) &buf, 2);
(u8 *) &buf, 2);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_P5_LSB_REG\n"); dev_err(dev, "failed to read BME680_P5_LSB_REG\n");
return ret; return ret;
@ -179,16 +173,14 @@ static int bme680_read_calib(struct bme680_data *data,
} }
calib->par_p7 = tmp; calib->par_p7 = tmp;
ret = regmap_bulk_read(data->regmap, BME680_P8_LSB_REG, ret = regmap_bulk_read(data->regmap, BME680_P8_LSB_REG, (u8 *) &buf, 2);
(u8 *) &buf, 2);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_P8_LSB_REG\n"); dev_err(dev, "failed to read BME680_P8_LSB_REG\n");
return ret; return ret;
} }
calib->par_p8 = le16_to_cpu(buf); calib->par_p8 = le16_to_cpu(buf);
ret = regmap_bulk_read(data->regmap, BME680_P9_LSB_REG, ret = regmap_bulk_read(data->regmap, BME680_P9_LSB_REG, (u8 *) &buf, 2);
(u8 *) &buf, 2);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_P9_LSB_REG\n"); dev_err(dev, "failed to read BME680_P9_LSB_REG\n");
return ret; return ret;
@ -208,13 +200,11 @@ static int bme680_read_calib(struct bme680_data *data,
dev_err(dev, "failed to read BME680_H1_MSB_REG\n"); dev_err(dev, "failed to read BME680_H1_MSB_REG\n");
return ret; return ret;
} }
ret = regmap_read(data->regmap, BME680_H1_LSB_REG, &tmp_lsb); ret = regmap_read(data->regmap, BME680_H1_LSB_REG, &tmp_lsb);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_H1_LSB_REG\n"); dev_err(dev, "failed to read BME680_H1_LSB_REG\n");
return ret; return ret;
} }
calib->par_h1 = (tmp_msb << BME680_HUM_REG_SHIFT_VAL) | calib->par_h1 = (tmp_msb << BME680_HUM_REG_SHIFT_VAL) |
(tmp_lsb & BME680_BIT_H1_DATA_MSK); (tmp_lsb & BME680_BIT_H1_DATA_MSK);
@ -223,13 +213,11 @@ static int bme680_read_calib(struct bme680_data *data,
dev_err(dev, "failed to read BME680_H2_MSB_REG\n"); dev_err(dev, "failed to read BME680_H2_MSB_REG\n");
return ret; return ret;
} }
ret = regmap_read(data->regmap, BME680_H2_LSB_REG, &tmp_lsb); ret = regmap_read(data->regmap, BME680_H2_LSB_REG, &tmp_lsb);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_H2_LSB_REG\n"); dev_err(dev, "failed to read BME680_H2_LSB_REG\n");
return ret; return ret;
} }
calib->par_h2 = (tmp_msb << BME680_HUM_REG_SHIFT_VAL) | calib->par_h2 = (tmp_msb << BME680_HUM_REG_SHIFT_VAL) |
(tmp_lsb >> BME680_HUM_REG_SHIFT_VAL); (tmp_lsb >> BME680_HUM_REG_SHIFT_VAL);
@ -276,8 +264,8 @@ static int bme680_read_calib(struct bme680_data *data,
} }
calib->par_gh1 = tmp; calib->par_gh1 = tmp;
ret = regmap_bulk_read(data->regmap, BME680_GH2_LSB_REG, ret = regmap_bulk_read(data->regmap, BME680_GH2_LSB_REG, (u8 *) &buf,
(u8 *) &buf, 2); 2);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "failed to read BME680_GH2_LSB_REG\n"); dev_err(dev, "failed to read BME680_GH2_LSB_REG\n");
return ret; return ret;