Input: tps65219-pwrbutton - use regmap_set_bits()

regmap_set_bits() is equivalent to regmap_update_bits() if mask == val.
The probe function uses regmap_clear_bits() to enable irqs, so
symmetrically make use of regmap_set_bits() to disable them. There is no
semantic difference.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Reviewed-by: Markus Schneider-Pargmann <msp@baylibre.com>
Link: https://lore.kernel.org/r/20230605161458.117361-2-u.kleine-koenig@pengutronix.de
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>

diff --git a/drivers/input/misc/tps65219-pwrbutton.c b/drivers/input/misc/tps65219-pwrbutton.c
index b2d9e5d..eeb9f23 100644 (file)
--- a/drivers/input/misc/tps65219-pwrbutton.c
+++ b/drivers/input/misc/tps65219-pwrbutton.c
@@ -123,9 +123,8 @@ static void tps65219_pb_remove(struct platform_device *pdev)
        int ret;
 
        /* Disable interrupt for the pushbutton */
-       ret = regmap_update_bits(tps->regmap, TPS65219_REG_MASK_CONFIG,
-                                TPS65219_REG_MASK_INT_FOR_PB_MASK,
-                                TPS65219_REG_MASK_INT_FOR_PB_MASK);
+       ret = regmap_set_bits(tps->regmap, TPS65219_REG_MASK_CONFIG,
+                             TPS65219_REG_MASK_INT_FOR_PB_MASK);
        if (ret)
                dev_warn(&pdev->dev, "Failed to disable irq (%pe)\n", ERR_PTR(ret));
 }