spmi: mtk-pmif: Reorder driver remove sequence
authorFei Shao <fshao@chromium.org>
Wed, 6 Dec 2023 23:17:28 +0000 (15:17 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 15 Dec 2023 16:27:04 +0000 (17:27 +0100)
This driver enables clocks and then adds SPMI controller in probing, so
we expect the reversed sequence in removal.
Fix the order in the remove callback.

Signed-off-by: Fei Shao <fshao@chromium.org>
Link: https://lore.kernel.org/r/20230824104101.4083400-4-fshao@chromium.org
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
Link: https://lore.kernel.org/r/20231206231733.4031901-6-sboyd@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/spmi/spmi-mtk-pmif.c

index 6ebc6901490a78032c78f632c97960b52cfde088..cc660da6a03702288996ef389f7b25e92217eb2d 100644 (file)
@@ -515,9 +515,9 @@ static void mtk_spmi_remove(struct platform_device *pdev)
        struct spmi_controller *ctrl = platform_get_drvdata(pdev);
        struct pmif *arb = spmi_controller_get_drvdata(ctrl);
 
+       spmi_controller_remove(ctrl);
        clk_bulk_disable_unprepare(arb->nclks, arb->clks);
        clk_bulk_put(arb->nclks, arb->clks);
-       spmi_controller_remove(ctrl);
 }
 
 static const struct of_device_id mtk_spmi_match_table[] = {