mfd: axp20x: Fix axp288 PEK_DBR and PEK_DBF irqs being swapped
authorHans de Goede <hdegoede@redhat.com>
Sat, 7 Oct 2017 22:37:44 +0000 (22:37 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Nov 2017 09:06:29 +0000 (10:06 +0100)
[ Upstream commit 1af468ebe45591651ec3bafc2e9ddc6fdef70ae0 ]

The R in PEK_DBR stands for rising, so it should be mapped to
AXP288_IRQ_POKP where the last P stands for positive edge.

Likewise PEK_DBF should be mapped to the falling edge, aka the
_N_egative edge, so it should be mapped to AXP288_IRQ_POKN.

This fixes the inverted powerbutton status reporting by the
axp20x-pek driver.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Chen-Yu Tsai <wens@csie.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/mfd/axp20x.c

index 9842199e2e6c1101eea2c05ce12bdbfb5d7db570..89a2dd4d212ad54b955f703e91cc0e8875d743c5 100644 (file)
@@ -164,14 +164,14 @@ static struct resource axp22x_pek_resources[] = {
 static struct resource axp288_power_button_resources[] = {
        {
                .name   = "PEK_DBR",
-               .start  = AXP288_IRQ_POKN,
-               .end    = AXP288_IRQ_POKN,
+               .start  = AXP288_IRQ_POKP,
+               .end    = AXP288_IRQ_POKP,
                .flags  = IORESOURCE_IRQ,
        },
        {
                .name   = "PEK_DBF",
-               .start  = AXP288_IRQ_POKP,
-               .end    = AXP288_IRQ_POKP,
+               .start  = AXP288_IRQ_POKN,
+               .end    = AXP288_IRQ_POKN,
                .flags  = IORESOURCE_IRQ,
        },
 };