summaryrefslogtreecommitdiffstats
path: root/sound
diff options
context:
space:
mode:
authorLinus Walleij <linus.walleij@linaro.org>2016-11-14 11:35:21 +0100
committerJonathan Cameron <jic23@kernel.org>2016-11-19 13:25:36 +0100
commit9049531c91b4ed7948d7c8bf6e945f5da56fc501 (patch)
tree38541e2dcd6d34b012d3bec7a00c5ac4d2939a3f /sound
parentiio: adc: Add support for STM32 ADC (diff)
downloadlinux-9049531c91b4ed7948d7c8bf6e945f5da56fc501.tar.xz
linux-9049531c91b4ed7948d7c8bf6e945f5da56fc501.zip
iio: accel: st_accel: inline per-sensor data
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>
Diffstat (limited to 'sound')
0 files changed, 0 insertions, 0 deletions