patch "iio: dac: ad5686: fix ref bit initialization for single-channel parts" added to char-misc-linus
Subsystems (3)
This is a note to let you know that I've just added the patch titled
iio: dac: ad5686: fix ref bit initialization for single-channel parts
to my char-misc git tree which can be found at
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
in the char-misc-linus branch.
The patch will show up in the next release of the linux-next tree (usually sometime within the next 24 hours during the week.)
The patch will hopefully also be merged in Linus's tree for the next -rc kernel release.
If you have any questions about this process, please let me know.
From ecae2ae606d493cf11457946436335bd0e726663 Mon Sep 17 00:00:00 2001
From: Rodrigo Alencar <[email protected]>
Date: Fri, 1 May 2026 10:14:54 +0100
Subject: iio: dac: ad5686: fix ref bit initialization for single-channel parts
The reference bit position was ignored when writing the register at the probe() function (!!val was used). When such bit is 1, internal voltage reference is disabled so that an external one can be used. For multi-channel devices, bit 0 of the Internal Reference Setup command behaves the same way, so AD5686_REF_BIT_MSK is created. The issue exists since support for single-channel devices were first introduced.
--- drivers/iio/dac/ad5686.c | 6 +++--- drivers/iio/dac/ad5686.h | 1 + 2 files changed, 4 insertions(+), 3 deletions(-)
drivers/iio/dac/ad5686.c +3 -3
@@ -509,7 +509,7 @@ int ad5686_probe(struct device *dev,break ;case AD5686_REGMAP : cmd =AD5686_CMD_INTERNAL_REFER_SETUP ; - ref_bit_msk =0 ; + ref_bit_msk =AD5686_REF_BIT_MSK ;break ;case AD5693_REGMAP : cmd =AD5686_CMD_CONTROL_REG ;
@@ -520,9 +520,9 @@ int ad5686_probe(struct device *dev,return -EINVAL ; } - val = (has_external_vref | ref_bit_msk); + val = has_external_vref ? ref_bit_msk :0 ; - ret = st->write (st, cmd,0 , !!val); + ret = st->write (st, cmd,0 , val);if (ret)return ret;
drivers/iio/dac/ad5686.h +1 -0
@@ -46,6 +46,7 @@#define AD5310_REF_BIT_MSK BIT(8)#define AD5683_REF_BIT_MSK BIT(12) +#define AD5686_REF_BIT_MSK BIT(0)#define AD5693_REF_BIT_MSK BIT(12)/**
Signature · 1 line
2.54.0