mirror of https://gitee.com/openkylin/linux.git
91a86a3b89
We have #defines for all the individual sensor registers and value/mask pairs #defined at the top of the file and used at exactly one spot. This is usually good if the #defines give a meaning to the opaque magic numbers. However in this case, the semantic meaning is inherent in the name of the C99-addressable fields, and that means duplication of information, and only makes the code hard to maintain since you every time have to add a new #define AND update the site where it is to be used. Get rid of the #defines and just open code the values into the appropriate struct elements. Make sure to explicitly address the .hz and .value fields in the st_sensor_odr_avl struct so that the meaning of all values is clear. This patch is purely syntactic should have no semantic effect. Signed-off-by: Linus Walleij <linus.walleij@linaro.org> Signed-off-by: Jonathan Cameron <jic23@kernel.org> |
||
|---|---|---|
| .. | ||
| Kconfig | ||
| Makefile | ||
| abp060mg.c | ||
| bmp280-core.c | ||
| bmp280-i2c.c | ||
| bmp280-regmap.c | ||
| bmp280-spi.c | ||
| bmp280.h | ||
| hid-sensor-press.c | ||
| hp03.c | ||
| hp206c.c | ||
| mpl115.c | ||
| mpl115.h | ||
| mpl115_i2c.c | ||
| mpl115_spi.c | ||
| mpl3115.c | ||
| ms5611.h | ||
| ms5611_core.c | ||
| ms5611_i2c.c | ||
| ms5611_spi.c | ||
| ms5637.c | ||
| st_pressure.h | ||
| st_pressure_buffer.c | ||
| st_pressure_core.c | ||
| st_pressure_i2c.c | ||
| st_pressure_spi.c | ||
| t5403.c | ||
| zpa2326.c | ||
| zpa2326.h | ||
| zpa2326_i2c.c | ||
| zpa2326_spi.c | ||