[PATCH v2 0/4] extcon: Fix wakeup source leaks on device unbind

Krzysztof Kozlowski posted 4 patches 9 months, 1 week ago
drivers/extcon/extcon-adc-jack.c       | 1 +
drivers/extcon/extcon-axp288.c         | 2 +-
drivers/extcon/extcon-fsa9480.c        | 2 +-
drivers/extcon/extcon-qcom-spmi-misc.c | 2 +-
4 files changed, 4 insertions(+), 3 deletions(-)
[PATCH v2 0/4] extcon: Fix wakeup source leaks on device unbind
Posted by Krzysztof Kozlowski 9 months, 1 week ago
Changes in v2:
- Patch #1: 0->false, extend commit msg
- Add Rb/Acks
- Link to v1: https://lore.kernel.org/r/20250406-device-wakeup-leak-extcon-v1-0-8873eca57465@linaro.org

Device can be unbound, so driver must also release memory for the wakeup
source.  Use devm for driver already having devm interface and manually
disable wakeup for drivers still having remove() callback to keep
consistent ordering of cleanups.

Best regards,
Krzysztof

---
Krzysztof Kozlowski (4):
      extcon: adc-jack: Fix wakeup source leaks on device unbind
      extcon: axp288: Fix wakeup source leaks on device unbind
      extcon: fsa9480: Fix wakeup source leaks on device unbind
      extcon: qcom-spmi-misc: Fix wakeup source leaks on device unbind

 drivers/extcon/extcon-adc-jack.c       | 1 +
 drivers/extcon/extcon-axp288.c         | 2 +-
 drivers/extcon/extcon-fsa9480.c        | 2 +-
 drivers/extcon/extcon-qcom-spmi-misc.c | 2 +-
 4 files changed, 4 insertions(+), 3 deletions(-)
---
base-commit: 393d0c54cae31317deaa9043320c5fd9454deabc
change-id: 20250406-device-wakeup-leak-extcon-dc1d4429a2b4

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Re: [PATCH v2 0/4] extcon: Fix wakeup source leaks on device unbind
Posted by Chanwoo Choi 9 months ago
Applied them.

Thanks.

On Thu, May 1, 2025 at 11:34 PM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> Changes in v2:
> - Patch #1: 0->false, extend commit msg
> - Add Rb/Acks
> - Link to v1: https://lore.kernel.org/r/20250406-device-wakeup-leak-extcon-v1-0-8873eca57465@linaro.org
>
> Device can be unbound, so driver must also release memory for the wakeup
> source.  Use devm for driver already having devm interface and manually
> disable wakeup for drivers still having remove() callback to keep
> consistent ordering of cleanups.
>
> Best regards,
> Krzysztof
>
> ---
> Krzysztof Kozlowski (4):
>       extcon: adc-jack: Fix wakeup source leaks on device unbind
>       extcon: axp288: Fix wakeup source leaks on device unbind
>       extcon: fsa9480: Fix wakeup source leaks on device unbind
>       extcon: qcom-spmi-misc: Fix wakeup source leaks on device unbind
>
>  drivers/extcon/extcon-adc-jack.c       | 1 +
>  drivers/extcon/extcon-axp288.c         | 2 +-
>  drivers/extcon/extcon-fsa9480.c        | 2 +-
>  drivers/extcon/extcon-qcom-spmi-misc.c | 2 +-
>  4 files changed, 4 insertions(+), 3 deletions(-)
> ---
> base-commit: 393d0c54cae31317deaa9043320c5fd9454deabc
> change-id: 20250406-device-wakeup-leak-extcon-dc1d4429a2b4
>
> Best regards,
> --
> Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
>


-- 
Best Regards,
Chanwoo Choi
Samsung Electronics