Skip to content
  • Nishanth Menon's avatar
    extcon: palmas: Drop stray IRQF_EARLY_RESUME flag · f75587b8
    Nishanth Menon authored
    Palmas extcon IRQs are nested threaded and wired to the Palmas
    interrupt controller. So, this flag is not required for nested
    IRQs anymore, since commit 3c646f2c ("genirq: Don't suspend
    nested_thread irqs over system suspend") was merged. However, the
    fix in commit ae64e42c ("extcon: palmas: Drop IRQF_EARLY_RESUME
    flag") missed a stray flag causing the following crash on resume on
    BeagleBoard-X15 platform:
    
    [   53.670141] Unhandled fault: imprecise external abort (0x1406) at 0x00000000
    [..]
    [   53.670141] [<c04ae734>] (omap_set_gpio_triggering) from [<c04aeb94>] (omap_gpio_unmask_irq+0xc0/0xc4)
    [   53.670141] [<c04aeb94>] (omap_gpio_unmask_irq) from [<c01a0b88>] (irq_enable+0x30/0x44)
    [   53.670141] [<c01a0b88>] (irq_enable) from [<c019ebd8>] (__enable_irq+0x54/0x78)
    [   53.670141] [<c019ebd8>] (__enable_irq) from [<c01a4e60>] (resume_irqs+0xe8/0x100)
    [   53.670141] [<c01a4e60>] (resume_irqs) from [<c0514840>] (syscore_resume+0x94/0x298)
    [   53.670141] [<c0514840>] (syscore_resume) from [<c01981cc>] (suspend_devices_and_enter+0x790/0x9e4)
    [   53.670141] [<c01981cc>] (suspend_devices_and_enter) from [<c0198a60>] (pm_suspend+0x640/0x75c)
    [   53.670141] [<c0198a60>] (pm_suspend) from [<c0196bec>] (state_store+0x64/0xb8)
    [   53.670141] [<c0196bec>] (state_store) from [<c0307944>] (kernfs_fop_write+0xc0/0x1bc)
    [   53.670141] [<c0307944>] (kernfs_fop_write) from [<c028acb0>] (__vfs_write+0x1c/0xd8)
    [   53.670141] [<c028acb0>] (__vfs_write) from [<c028bba0>] (vfs_write+0x90/0x16c)
    [   53.670141] [<c028bba0>] (vfs_write) from [<c028c8c0>] (SyS_write+0x44/0x9c)
    [   53.670141] [<c028c8c0>] (SyS_write) from [<c0107840>] (ret_fast_syscall+0x0/0x1c)
    [..]
    
    Fixes: ae64e42c
    
     ("extcon: palmas: Drop IRQF_EARLY_RESUME flag")
    Cc: Grygorii Strashko <grygorii.strashko@ti.com>
    Cc: MyungJoo Ham <myungjoo.ham@samsung.com>
    Cc: Chanwoo Choi <cw00.choi@samsung.com>
    Cc: Tony Lindgren <tony@atomide.com>
    Cc: Lee Jones <lee.jones@linaro.org>
    Cc: Roger Quadros <rogerq@ti.com>
    
    Reviewed-by: default avatarGrygorii Strashko <grygorii.strashko@ti.com>
    Signed-off-by: default avatarNishanth Menon <nm@ti.com>
    Acked-by: default avatarRoger Quadros <rogerq@ti.com>
    Signed-off-by: default avatarChanwoo Choi <cw00.choi@samsung.com>
    f75587b8