Re: [PATCH v2] net: wwan: qcom_bam_dmux: Fix missing pm_runtime_disable()

From: Sergey Ryazanov
Date: Tue Sep 24 2024 - 13:16:25 EST


On 23.09.2024 14:57, Jinjie Ruan wrote:
It's important to undo pm_runtime_use_autosuspend() with
pm_runtime_dont_use_autosuspend() at driver exit time.

But the pm_runtime_disable() and pm_runtime_dont_use_autosuspend()
is missing in the error path for bam_dmux_probe(). So add it.

Found by code review. Compile-tested only.

Fixes: 21a0ffd9b38c ("net: wwan: Add Qualcomm BAM-DMUX WWAN network driver")
Suggested-by: Stephan Gerhold <stephan.gerhold@xxxxxxxxxx>
Signed-off-by: Jinjie Ruan <ruanjinjie@xxxxxxxxxx>

Reviewed-by: Sergey Ryazanov <ryazanov.s.a@xxxxxxxxx>