forked from luck/tmp_suning_uos_patched
9049531c91
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. Cc: Lorenzo Bianconi <lorenzo.bianconi83@gmail.com> Signed-off-by: Linus Walleij <linus.walleij@linaro.org> Acked-by: Lorenzo Bianconi <lorenzo.bianconi@st.com> Signed-off-by: Jonathan Cameron <jic23@kernel.org> |
||
---|---|---|
.. | ||
bma180.c | ||
bma220_spi.c | ||
bmc150-accel-core.c | ||
bmc150-accel-i2c.c | ||
bmc150-accel-spi.c | ||
bmc150-accel.h | ||
da280.c | ||
da311.c | ||
dmard06.c | ||
dmard09.c | ||
dmard10.c | ||
hid-sensor-accel-3d.c | ||
Kconfig | ||
kxcjk-1013.c | ||
kxsd9-i2c.c | ||
kxsd9-spi.c | ||
kxsd9.c | ||
kxsd9.h | ||
Makefile | ||
mc3230.c | ||
mma7455_core.c | ||
mma7455_i2c.c | ||
mma7455_spi.c | ||
mma7455.h | ||
mma7660.c | ||
mma8452.c | ||
mma9551_core.c | ||
mma9551_core.h | ||
mma9551.c | ||
mma9553.c | ||
mxc4005.c | ||
mxc6255.c | ||
sca3000.c | ||
ssp_accel_sensor.c | ||
st_accel_buffer.c | ||
st_accel_core.c | ||
st_accel_i2c.c | ||
st_accel_spi.c | ||
st_accel.h | ||
stk8ba50.c | ||
stk8312.c |