Commit 2405f8cc authored by Angel Iglesias's avatar Angel Iglesias Committed by Jonathan Cameron

iio: pressure: bmp280: use FIELD_GET, FIELD_PREP and GENMASK

Cleaned and simplified register values construction and extraction
converting to use FIELD_PREP and FIELD_GET macros.

Replaced hardcoded bit masks with GENMASK macro.
Signed-off-by: default avatarAngel Iglesias <ang.iglesiasg@gmail.com>
Link: https://lore.kernel.org/r/3cbe56f29c2a46bc5dc23c5b72e1b43c9207f44d.1663025017.git.ang.iglesiasg@gmail.comSigned-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent 5f0c359d
...@@ -22,6 +22,8 @@ ...@@ -22,6 +22,8 @@
#define pr_fmt(fmt) "bmp280: " fmt #define pr_fmt(fmt) "bmp280: " fmt
#include <linux/bitops.h>
#include <linux/bitfield.h>
#include <linux/device.h> #include <linux/device.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/regmap.h> #include <linux/regmap.h>
...@@ -248,7 +250,7 @@ static int bmp280_read_calib(struct bmp280_data *data, ...@@ -248,7 +250,7 @@ static int bmp280_read_calib(struct bmp280_data *data,
dev_err(dev, "failed to read H5 comp value\n"); dev_err(dev, "failed to read H5 comp value\n");
return ret; return ret;
} }
calib->H5 = sign_extend32(((le16_to_cpu(l16) >> 4) & 0xfff), 11); calib->H5 = sign_extend32(FIELD_GET(BMP280_COMP_H5_MASK, le16_to_cpu(l16)), 11);
ret = regmap_read(data->regmap, BMP280_REG_COMP_H6, &tmp); ret = regmap_read(data->regmap, BMP280_REG_COMP_H6, &tmp);
if (ret < 0) { if (ret < 0) {
...@@ -352,7 +354,7 @@ static int bmp280_read_temp(struct bmp280_data *data, ...@@ -352,7 +354,7 @@ static int bmp280_read_temp(struct bmp280_data *data,
return ret; return ret;
} }
adc_temp = be32_to_cpu(tmp) >> 12; adc_temp = FIELD_GET(BMP280_MEAS_TRIM_MASK, be32_to_cpu(tmp));
if (adc_temp == BMP280_TEMP_SKIPPED) { if (adc_temp == BMP280_TEMP_SKIPPED) {
/* reading was skipped */ /* reading was skipped */
dev_err(data->dev, "reading temperature skipped\n"); dev_err(data->dev, "reading temperature skipped\n");
...@@ -391,7 +393,7 @@ static int bmp280_read_press(struct bmp280_data *data, ...@@ -391,7 +393,7 @@ static int bmp280_read_press(struct bmp280_data *data,
return ret; return ret;
} }
adc_press = be32_to_cpu(tmp) >> 12; adc_press = FIELD_GET(BMP280_MEAS_TRIM_MASK, be32_to_cpu(tmp));
if (adc_press == BMP280_PRESS_SKIPPED) { if (adc_press == BMP280_PRESS_SKIPPED) {
/* reading was skipped */ /* reading was skipped */
dev_err(data->dev, "reading pressure skipped\n"); dev_err(data->dev, "reading pressure skipped\n");
...@@ -617,8 +619,8 @@ static const struct iio_info bmp280_info = { ...@@ -617,8 +619,8 @@ static const struct iio_info bmp280_info = {
static int bmp280_chip_config(struct bmp280_data *data) static int bmp280_chip_config(struct bmp280_data *data)
{ {
u8 osrs = BMP280_OSRS_TEMP_X(data->oversampling_temp + 1) | u8 osrs = FIELD_PREP(BMP280_OSRS_TEMP_MASK, data->oversampling_temp + 1) |
BMP280_OSRS_PRESS_X(data->oversampling_press + 1); FIELD_PREP(BMP280_OSRS_PRESS_MASK, data->oversampling_press + 1);
int ret; int ret;
ret = regmap_write_bits(data->regmap, BMP280_REG_CTRL_MEAS, ret = regmap_write_bits(data->regmap, BMP280_REG_CTRL_MEAS,
...@@ -660,7 +662,7 @@ static const struct bmp280_chip_info bmp280_chip_info = { ...@@ -660,7 +662,7 @@ static const struct bmp280_chip_info bmp280_chip_info = {
static int bme280_chip_config(struct bmp280_data *data) static int bme280_chip_config(struct bmp280_data *data)
{ {
u8 osrs = BMP280_OSRS_HUMIDITIY_X(data->oversampling_humid + 1); u8 osrs = FIELD_PREP(BMP280_OSRS_HUMIDITY_MASK, data->oversampling_humid + 1);
int ret; int ret;
/* /*
...@@ -717,7 +719,7 @@ static int bmp180_measure(struct bmp280_data *data, u8 ctrl_meas) ...@@ -717,7 +719,7 @@ static int bmp180_measure(struct bmp280_data *data, u8 ctrl_meas)
if (!ret) if (!ret)
dev_err(data->dev, "timeout waiting for completion\n"); dev_err(data->dev, "timeout waiting for completion\n");
} else { } else {
if (ctrl_meas == BMP180_MEAS_TEMP) if (FIELD_GET(BMP180_MEAS_CTRL_MASK, ctrl_meas) == BMP180_MEAS_TEMP)
delay_us = 4500; delay_us = 4500;
else else
delay_us = delay_us =
...@@ -742,7 +744,9 @@ static int bmp180_read_adc_temp(struct bmp280_data *data, int *val) ...@@ -742,7 +744,9 @@ static int bmp180_read_adc_temp(struct bmp280_data *data, int *val)
__be16 tmp; __be16 tmp;
int ret; int ret;
ret = bmp180_measure(data, BMP180_MEAS_TEMP); ret = bmp180_measure(data,
FIELD_PREP(BMP180_MEAS_CTRL_MASK, BMP180_MEAS_TEMP) |
BMP180_MEAS_SCO);
if (ret) if (ret)
return ret; return ret;
...@@ -839,7 +843,10 @@ static int bmp180_read_adc_press(struct bmp280_data *data, int *val) ...@@ -839,7 +843,10 @@ static int bmp180_read_adc_press(struct bmp280_data *data, int *val)
__be32 tmp = 0; __be32 tmp = 0;
int ret; int ret;
ret = bmp180_measure(data, BMP180_MEAS_PRESS_X(oss)); ret = bmp180_measure(data,
FIELD_PREP(BMP180_MEAS_CTRL_MASK, BMP180_MEAS_PRESS) |
FIELD_PREP(BMP180_OSRS_PRESS_MASK, oss) |
BMP180_MEAS_SCO);
if (ret) if (ret)
return ret; return ret;
......
...@@ -13,6 +13,9 @@ ...@@ -13,6 +13,9 @@
#define BMP280_REG_PRESS_LSB 0xF8 #define BMP280_REG_PRESS_LSB 0xF8
#define BMP280_REG_PRESS_MSB 0xF7 #define BMP280_REG_PRESS_MSB 0xF7
/* Helper mask to truncate excess 12 bits on pressure and temp readings */
#define BMP280_MEAS_TRIM_MASK GENMASK(31, 12)
#define BMP280_REG_CONFIG 0xF5 #define BMP280_REG_CONFIG 0xF5
#define BMP280_REG_CTRL_MEAS 0xF4 #define BMP280_REG_CTRL_MEAS 0xF4
#define BMP280_REG_STATUS 0xF3 #define BMP280_REG_STATUS 0xF3
...@@ -32,44 +35,43 @@ ...@@ -32,44 +35,43 @@
#define BMP280_REG_COMP_PRESS_START 0x8E #define BMP280_REG_COMP_PRESS_START 0x8E
#define BMP280_COMP_PRESS_REG_COUNT 18 #define BMP280_COMP_PRESS_REG_COUNT 18
#define BMP280_FILTER_MASK (BIT(4) | BIT(3) | BIT(2)) #define BMP280_COMP_H5_MASK GENMASK(15, 4)
#define BMP280_FILTER_MASK GENMASK(4, 2)
#define BMP280_FILTER_OFF 0 #define BMP280_FILTER_OFF 0
#define BMP280_FILTER_2X BIT(2) #define BMP280_FILTER_2X 1
#define BMP280_FILTER_4X BIT(3) #define BMP280_FILTER_4X 2
#define BMP280_FILTER_8X (BIT(3) | BIT(2)) #define BMP280_FILTER_8X 3
#define BMP280_FILTER_16X BIT(4) #define BMP280_FILTER_16X 4
#define BMP280_OSRS_HUMIDITY_MASK (BIT(2) | BIT(1) | BIT(0)) #define BMP280_OSRS_HUMIDITY_MASK GENMASK(2, 0)
#define BMP280_OSRS_HUMIDITIY_X(osrs_h) ((osrs_h) << 0)
#define BMP280_OSRS_HUMIDITY_SKIP 0 #define BMP280_OSRS_HUMIDITY_SKIP 0
#define BMP280_OSRS_HUMIDITY_1X BMP280_OSRS_HUMIDITIY_X(1) #define BMP280_OSRS_HUMIDITY_1X 1
#define BMP280_OSRS_HUMIDITY_2X BMP280_OSRS_HUMIDITIY_X(2) #define BMP280_OSRS_HUMIDITY_2X 2
#define BMP280_OSRS_HUMIDITY_4X BMP280_OSRS_HUMIDITIY_X(3) #define BMP280_OSRS_HUMIDITY_4X 3
#define BMP280_OSRS_HUMIDITY_8X BMP280_OSRS_HUMIDITIY_X(4) #define BMP280_OSRS_HUMIDITY_8X 4
#define BMP280_OSRS_HUMIDITY_16X BMP280_OSRS_HUMIDITIY_X(5) #define BMP280_OSRS_HUMIDITY_16X 5
#define BMP280_OSRS_TEMP_MASK (BIT(7) | BIT(6) | BIT(5)) #define BMP280_OSRS_TEMP_MASK GENMASK(7, 5)
#define BMP280_OSRS_TEMP_SKIP 0 #define BMP280_OSRS_TEMP_SKIP 0
#define BMP280_OSRS_TEMP_X(osrs_t) ((osrs_t) << 5) #define BMP280_OSRS_TEMP_1X 1
#define BMP280_OSRS_TEMP_1X BMP280_OSRS_TEMP_X(1) #define BMP280_OSRS_TEMP_2X 2
#define BMP280_OSRS_TEMP_2X BMP280_OSRS_TEMP_X(2) #define BMP280_OSRS_TEMP_4X 3
#define BMP280_OSRS_TEMP_4X BMP280_OSRS_TEMP_X(3) #define BMP280_OSRS_TEMP_8X 4
#define BMP280_OSRS_TEMP_8X BMP280_OSRS_TEMP_X(4) #define BMP280_OSRS_TEMP_16X 5
#define BMP280_OSRS_TEMP_16X BMP280_OSRS_TEMP_X(5)
#define BMP280_OSRS_PRESS_MASK GENMASK(4, 2)
#define BMP280_OSRS_PRESS_MASK (BIT(4) | BIT(3) | BIT(2))
#define BMP280_OSRS_PRESS_SKIP 0 #define BMP280_OSRS_PRESS_SKIP 0
#define BMP280_OSRS_PRESS_X(osrs_p) ((osrs_p) << 2) #define BMP280_OSRS_PRESS_1X 1
#define BMP280_OSRS_PRESS_1X BMP280_OSRS_PRESS_X(1) #define BMP280_OSRS_PRESS_2X 2
#define BMP280_OSRS_PRESS_2X BMP280_OSRS_PRESS_X(2) #define BMP280_OSRS_PRESS_4X 3
#define BMP280_OSRS_PRESS_4X BMP280_OSRS_PRESS_X(3) #define BMP280_OSRS_PRESS_8X 4
#define BMP280_OSRS_PRESS_8X BMP280_OSRS_PRESS_X(4) #define BMP280_OSRS_PRESS_16X 5
#define BMP280_OSRS_PRESS_16X BMP280_OSRS_PRESS_X(5)
#define BMP280_MODE_MASK GENMASK(1, 0)
#define BMP280_MODE_MASK (BIT(1) | BIT(0))
#define BMP280_MODE_SLEEP 0 #define BMP280_MODE_SLEEP 0
#define BMP280_MODE_FORCED BIT(0) #define BMP280_MODE_FORCED 1
#define BMP280_MODE_NORMAL (BIT(1) | BIT(0)) #define BMP280_MODE_NORMAL 3
/* BMP180 specific registers */ /* BMP180 specific registers */
#define BMP180_REG_OUT_XLSB 0xF8 #define BMP180_REG_OUT_XLSB 0xF8
...@@ -79,13 +81,15 @@ ...@@ -79,13 +81,15 @@
#define BMP180_REG_CALIB_START 0xAA #define BMP180_REG_CALIB_START 0xAA
#define BMP180_REG_CALIB_COUNT 22 #define BMP180_REG_CALIB_COUNT 22
#define BMP180_MEAS_CTRL_MASK GENMASK(4, 0)
#define BMP180_MEAS_TEMP 0x0E
#define BMP180_MEAS_PRESS 0x14
#define BMP180_MEAS_SCO BIT(5) #define BMP180_MEAS_SCO BIT(5)
#define BMP180_MEAS_TEMP (0x0E | BMP180_MEAS_SCO) #define BMP180_OSRS_PRESS_MASK GENMASK(7, 6)
#define BMP180_MEAS_PRESS_X(oss) ((oss) << 6 | 0x14 | BMP180_MEAS_SCO) #define BMP180_MEAS_PRESS_1X 0
#define BMP180_MEAS_PRESS_1X BMP180_MEAS_PRESS_X(0) #define BMP180_MEAS_PRESS_2X 1
#define BMP180_MEAS_PRESS_2X BMP180_MEAS_PRESS_X(1) #define BMP180_MEAS_PRESS_4X 2
#define BMP180_MEAS_PRESS_4X BMP180_MEAS_PRESS_X(2) #define BMP180_MEAS_PRESS_8X 3
#define BMP180_MEAS_PRESS_8X BMP180_MEAS_PRESS_X(3)
/* BMP180 and BMP280 common registers */ /* BMP180 and BMP280 common registers */
#define BMP280_REG_CTRL_MEAS 0xF4 #define BMP280_REG_CTRL_MEAS 0xF4
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment