gpio: 104-dio-48e: Utilize mask_buf_def in handle_mask_sync() callback
authorWilliam Breathitt Gray <william.gray@linaro.org>
Wed, 8 Feb 2023 10:55:42 +0000 (05:55 -0500)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Mon, 6 Mar 2023 09:49:53 +0000 (10:49 +0100)
The mask_buf_def argument provides a mask of all the maskable lines.
Utilize mask_buf_def rather than hardcode an "all_masked" mask.

Signed-off-by: William Breathitt Gray <william.gray@linaro.org>
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
drivers/gpio/gpio-104-dio-48e.c

index a3846faf3780dcbe7337fe5db085c657eed89fe0..74e2721f2613d91a3b23b5019d75b17f2d60176f 100644 (file)
@@ -106,7 +106,6 @@ static int dio48e_handle_mask_sync(struct regmap *const map, const int index,
 {
        unsigned int *const irq_mask = irq_drv_data;
        const unsigned int prev_mask = *irq_mask;
-       const unsigned int all_masked = GENMASK(1, 0);
        int err;
        unsigned int val;
 
@@ -118,7 +117,7 @@ static int dio48e_handle_mask_sync(struct regmap *const map, const int index,
        *irq_mask = mask_buf;
 
        /* if all previously masked, enable interrupts when unmasking */
-       if (prev_mask == all_masked) {
+       if (prev_mask == mask_buf_def) {
                err = regmap_write(map, DIO48E_CLEAR_INTERRUPT, 0x00);
                if (err)
                        return err;
@@ -126,7 +125,7 @@ static int dio48e_handle_mask_sync(struct regmap *const map, const int index,
        }
 
        /* if all are currently masked, disable interrupts */
-       if (mask_buf == all_masked)
+       if (mask_buf == mask_buf_def)
                return regmap_read(map, DIO48E_DISABLE_INTERRUPT, &val);
 
        return 0;