Commit eb7b7a7d authored by Chen-Yu Tsai's avatar Chen-Yu Tsai Committed by Stephen Boyd
Browse files

clk: mediatek: mux: Implement error handling in register API 



The mux clk type registration function does not stop or return errors
if any clk failed to be registered, nor does it implement an error
handling path. This may result in a partially working device if any
step failed.

Make the register function return proper error codes, and bail out if
errors occur. Proper cleanup, i.e. unregister any clks that were
successfully registered, is done in the new error path.

Signed-off-by: default avatarChen-Yu Tsai <wenst@chromium.org>
Reviewed-by: default avatarMiles Chen <miles.chen@mediatek.com>
Reviewed-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Link: https://lore.kernel.org/r/20220208124034.414635-25-wenst@chromium.org


Reviewed-by: default avatarChun-Jie Chen <chun-jie.chen@mediatek.com>
Signed-off-by: default avatarStephen Boyd <sboyd@kernel.org>
parent 203ce39e

drivers/clk/mediatek/clk-mux.c

+14 −1
Original line number Diff line number Diff line
@@ -215,13 +215,26 @@ int mtk_clk_register_muxes(const struct mtk_mux *muxes, 


		if (IS_ERR(clk)) {

			pr_err("Failed to register clk %s: %pe\n", mux->name, clk);

			continue;

			goto err;

		}



		clk_data->clks[mux->id] = clk;

	}



	return 0;



err:

	while (--i >= 0) {

		const struct mtk_mux *mux = &muxes[i];



		if (IS_ERR_OR_NULL(clk_data->clks[mux->id]))

			continue;



		mtk_clk_unregister_mux(clk_data->clks[mux->id]);

		clk_data->clks[mux->id] = ERR_PTR(-ENOENT);

	}



	return PTR_ERR(clk);

}

EXPORT_SYMBOL_GPL(mtk_clk_register_muxes);