diff mbox series

extcon: max77843: Replace irqchip mask_invert with unmask_base

Message ID 20221112152446.41690-1-aidanmacdonald.0x0@gmail.com
State New
Headers
Series extcon: max77843: Replace irqchip mask_invert with unmask_base |

Commit Message

Aidan MacDonald Nov. 12, 2022, 3:24 p.m. UTC 
  Remove use of the deprecated mask_invert flag. Inverted mask
registers (where a '1' bit enables an IRQ) can be described more
directly as an unmask register.

Signed-off-by: Aidan MacDonald <aidanmacdonald.0x0@gmail.com>
---
 drivers/extcon/extcon-max77843.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Krzysztof Kozlowski Nov. 14, 2022, 8:03 a.m. UTC | #1 
On 12/11/2022 16:24, Aidan MacDonald wrote:
> Remove use of the deprecated mask_invert flag. Inverted mask
> registers (where a '1' bit enables an IRQ) can be described more
> directly as an unmask register.
> 
> Signed-off-by: Aidan MacDonald <aidanmacdonald.0x0@gmail.com>

This is a resend? Or v2?

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Aidan MacDonald Nov. 15, 2022, 2:37 p.m. UTC | #2 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> writes:

> On 12/11/2022 16:24, Aidan MacDonald wrote:
>> Remove use of the deprecated mask_invert flag. Inverted mask
>> registers (where a '1' bit enables an IRQ) can be described more
>> directly as an unmask register.
>>
>> Signed-off-by: Aidan MacDonald <aidanmacdonald.0x0@gmail.com>
>
> This is a resend? Or v2?
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> Best regards,
> Krzysztof

It's a resend. The last submission couldn't be applied at the time
because of dependency troubles.
Chanwoo Choi Nov. 20, 2022, 11:57 a.m. UTC | #3 
On 22. 11. 13. 00:24, Aidan MacDonald wrote:
> Remove use of the deprecated mask_invert flag. Inverted mask
> registers (where a '1' bit enables an IRQ) can be described more
> directly as an unmask register.
> 
> Signed-off-by: Aidan MacDonald <aidanmacdonald.0x0@gmail.com>
> ---
>  drivers/extcon/extcon-max77843.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/extcon/extcon-max77843.c b/drivers/extcon/extcon-max77843.c
> index 8e6e97ec65a8..1bc0426ce3f1 100644
> --- a/drivers/extcon/extcon-max77843.c
> +++ b/drivers/extcon/extcon-max77843.c
> @@ -189,8 +189,7 @@ static const struct regmap_irq max77843_muic_irq[] = {
>  static const struct regmap_irq_chip max77843_muic_irq_chip = {
>  	.name           = "max77843-muic",
>  	.status_base    = MAX77843_MUIC_REG_INT1,
> -	.mask_base      = MAX77843_MUIC_REG_INTMASK1,
> -	.mask_invert    = true,
> +	.unmask_base    = MAX77843_MUIC_REG_INTMASK1,
>  	.num_regs       = 3,
>  	.irqs           = max77843_muic_irq,
>  	.num_irqs       = ARRAY_SIZE(max77843_muic_irq),

Applied it. Thanks.
diff mbox series

Patch

diff --git a/drivers/extcon/extcon-max77843.c b/drivers/extcon/extcon-max77843.c
index 8e6e97ec65a8..1bc0426ce3f1 100644
--- a/drivers/extcon/extcon-max77843.c
+++ b/drivers/extcon/extcon-max77843.c
@@ -189,8 +189,7 @@  static const struct regmap_irq max77843_muic_irq[] = {
 static const struct regmap_irq_chip max77843_muic_irq_chip = {
 	.name           = "max77843-muic",
 	.status_base    = MAX77843_MUIC_REG_INT1,
-	.mask_base      = MAX77843_MUIC_REG_INTMASK1,
-	.mask_invert    = true,
+	.unmask_base    = MAX77843_MUIC_REG_INTMASK1,
 	.num_regs       = 3,
 	.irqs           = max77843_muic_irq,
 	.num_irqs       = ARRAY_SIZE(max77843_muic_irq),