diff options
author | Jeremy Fertic <jeremyfertic@gmail.com> | 2018-12-22 21:57:43 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-05-16 19:45:02 +0200 |
commit | 90b8ab9ee10fc7e364097dbae4164926ea1de7db (patch) | |
tree | b1596be4d3b4c9900a1aba6cd98a1ddcd8232b7c /security/selinux/hooks.c | |
parent | 7c4744e2c75cdec381ee3759ad53f85f90fdfaa0 (diff) |
staging: iio: adt7316: fix the dac write calculation
commit 78accaea117c1ae878774974fab91ac4a0b0e2b0 upstream.
The lsb calculation is not masking the correct bits from the user input.
Subtract 1 from (1 << offset) to correctly set up the mask to be applied
to user input.
The lsb register stores its value starting at the bit 7 position.
adt7316_store_DAC() currently assumes the value is at the other end of the
register. Shift the lsb value before storing it in a new variable lsb_reg,
and write this variable to the lsb register.
Fixes: 35f6b6b86ede ("staging: iio: new ADT7316/7/8 and ADT7516/7/9 driver")
Signed-off-by: Jeremy Fertic <jeremyfertic@gmail.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'security/selinux/hooks.c')
0 files changed, 0 insertions, 0 deletions