]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
mfd: ezx-pcap: Avoid rescheduling after destroying workqueue
authorKrzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Thu, 5 Mar 2026 21:45:48 +0000 (22:45 +0100)
committerLee Jones <lee@kernel.org>
Wed, 25 Mar 2026 12:45:59 +0000 (12:45 +0000)
commit30eedf2446a9ada57a6bda22ec6b89533feb81ed
treea5e65f6651d7c0a7e3b15da4c14867b5975c053b
parentef5a54c542f5388df3f142a84de642d33790bd7b
mfd: ezx-pcap: Avoid rescheduling after destroying workqueue

Driver allocates workqueue and then registers additional interrupt
handler with devm interface.  This means that device removal will not
use a reversed order, but first destroy workqueue and then, via devm
release handlers, free the interrupt.

The interrupt handler registered with devm does not directly
use/schedule work items on the workqueue and the remove() function
correctly removes other IRQs handlers, however the code mixing devm and
non-devm interfaces is difficult to analyze and read.

Make the code flow much more obvious by using devm interface for
allocating the workqueue, so it will be freed with the rest of devm
resources.

Change is not equivalent in the workqueue itself: use non-legacy API
which does not set (__WQ_LEGACY | WQ_MEM_RECLAIM).  The workqueue is
used to update device registers, thus there is no point to run it for
memory reclaim.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://patch.msgid.link/20260305-workqueue-devm-v2-9-66a38741c652@oss.qualcomm.com
Signed-off-by: Lee Jones <lee@kernel.org>
drivers/mfd/ezx-pcap.c