Skip to content

Commit 92bac7d

Browse files
krzkchanwoochoi
authored andcommitted
extcon: adc-jack: Cleanup wakeup source only if it was enabled
Driver in the probe enables wakeup source conditionally, so the cleanup path should do the same - do not release the wakeup source memory if it was not allocated. Link: https://lore.kernel.org/lkml/[email protected]/ Reported-by: Christophe JAILLET <[email protected]> Closes: https://lore.kernel.org/r/[email protected]/ Fixes: 78b6a99 ("extcon: adc-jack: Fix wakeup source leaks on device unbind") Signed-off-by: Krzysztof Kozlowski <[email protected]> Signed-off-by: Chanwoo Choi <[email protected]>
1 parent 369259d commit 92bac7d

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

drivers/extcon/extcon-adc-jack.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,8 @@ static void adc_jack_remove(struct platform_device *pdev)
164164
{
165165
struct adc_jack_data *data = platform_get_drvdata(pdev);
166166

167-
device_init_wakeup(&pdev->dev, false);
167+
if (data->wakeup_source)
168+
device_init_wakeup(&pdev->dev, false);
168169
free_irq(data->irq, data);
169170
cancel_work_sync(&data->handler.work);
170171
}

0 commit comments

Comments
 (0)