iio: accel: adxl345: simplify tap suppress bit

Simplify setting the tap suppress bit by applying regmap_assign_bit(). As a
result, the defines can be reorganized for better clarity.

Signed-off-by: Lothar Rubusch <l.rubusch@gmail.com>
Link: https://patch.msgid.link/20250727210014.27766-2-l.rubusch@gmail.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

diff --git a/drivers/iio/accel/adxl345_core.c b/drivers/iio/accel/adxl345_core.c
index b7dfd0007aa068daefc8cd6ec627655303c6d198..e293fc91797233682c393dc89bfba8f942b86a48 100644 (file)
--- a/drivers/iio/accel/adxl345_core.c
+++ b/drivers/iio/accel/adxl345_core.c
@@ -34,12 +34,10 @@
 #define ADXL345_INT2                   1
 
 #define ADXL345_REG_TAP_AXIS_MSK       GENMASK(2, 0)
-#define ADXL345_REG_TAP_SUPPRESS_MSK   BIT(3)
-#define ADXL345_REG_TAP_SUPPRESS       BIT(3)
-
 #define ADXL345_TAP_Z_EN               BIT(0)
 #define ADXL345_TAP_Y_EN               BIT(1)
 #define ADXL345_TAP_X_EN               BIT(2)
+#define ADXL345_REG_TAP_SUPPRESS       BIT(3)
 
 /* single/double tap */
 enum adxl345_tap_type {
@@ -368,9 +366,8 @@ static int adxl345_set_doubletap_en(struct adxl345_state *st, bool en)
         * Generally suppress detection of spikes during the latency period as
         * double taps here, this is fully optional for double tap detection
         */
-       ret = regmap_update_bits(st->regmap, ADXL345_REG_TAP_AXIS,
-                                ADXL345_REG_TAP_SUPPRESS_MSK,
-                                en ? ADXL345_REG_TAP_SUPPRESS : 0x00);
+       ret = regmap_assign_bits(st->regmap, ADXL345_REG_TAP_AXIS,
+                                ADXL345_REG_TAP_SUPPRESS, en);
        if (ret)
                return ret;