diff options
| author | Gregor Boirie <gregor.boirie@parrot.com> | 2016-09-02 20:27:46 +0200 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2016-09-05 21:21:32 +0100 |
| commit | 171c0091837c81ed5c949fec6966bb5afff2d1cf (patch) | |
| tree | a1d5f5ac471ef9c9a224632479e00623fd4c4a7e /drivers/iio/dac | |
| parent | 5dba4b14bafe801083d01e1f400816df7e5a8f2e (diff) | |
| download | linux-171c0091837c81ed5c949fec6966bb5afff2d1cf.tar.gz | |
iio:core: fix IIO_VAL_FRACTIONAL sign handling
7985e7c100 ("iio: Introduce a new fractional value type") introduced a
new IIO_VAL_FRACTIONAL value type meant to represent rational type numbers
expressed by a numerator and denominator combination.
Formating of IIO_VAL_FRACTIONAL values relies upon do_div() usage. This
fails handling negative values properly since parameters are reevaluated
as unsigned values.
Fix this by using div_s64_rem() instead. Computed integer part will carry
properly signed value. Formatted fractional part will always be positive.
Fixes: 7985e7c100 ("iio: Introduce a new fractional value type")
Signed-off-by: Gregor Boirie <gregor.boirie@parrot.com>
Reviewed-by: Lars-Peter Clausen <lars@metafoo.de>
Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio/dac')
0 files changed, 0 insertions, 0 deletions