Commit b71bf1eb authored by Johan Hovold's avatar Johan Hovold Committed by Vinod Koul
Browse files

phy: qcom-qmp-combo: clean up DP clock callbacks 



Clean up the DP clock callbacks somewhat by dropping the redundant
"qcom" prefix and removing line breaks after type specifiers.

Reviewed-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: default avatarJohan Hovold <johan+linaro@kernel.org>
Link: https://lore.kernel.org/r/20221121085058.31213-11-johan+linaro@kernel.org


Signed-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent ce51f7a7

drivers/phy/qualcomm/phy-qcom-qmp-combo.c

+14 −19
Original line number Diff line number Diff line
@@ -2304,8 +2304,7 @@ static int phy_pipe_clk_register(struct qmp_combo *qmp, struct device_node *np) 
 *              for DP pixel clock

 *

 */

static int qcom_qmp_dp_pixel_clk_determine_rate(struct clk_hw *hw,

						struct clk_rate_request *req)

static int qmp_dp_pixel_clk_determine_rate(struct clk_hw *hw, struct clk_rate_request *req)

{

	switch (req->rate) {

	case 1620000000UL / 2:
@@ -2317,8 +2316,7 @@ static int qcom_qmp_dp_pixel_clk_determine_rate(struct clk_hw *hw, 
	}

}



static unsigned long

qcom_qmp_dp_pixel_clk_recalc_rate(struct clk_hw *hw, unsigned long parent_rate)

static unsigned long qmp_dp_pixel_clk_recalc_rate(struct clk_hw *hw, unsigned long parent_rate)

{

	const struct qmp_combo *qmp;

	const struct phy_configure_opts_dp *dp_opts;
@@ -2340,13 +2338,12 @@ qcom_qmp_dp_pixel_clk_recalc_rate(struct clk_hw *hw, unsigned long parent_rate) 
	}

}



static const struct clk_ops qcom_qmp_dp_pixel_clk_ops = {

	.determine_rate = qcom_qmp_dp_pixel_clk_determine_rate,

	.recalc_rate = qcom_qmp_dp_pixel_clk_recalc_rate,

static const struct clk_ops qmp_dp_pixel_clk_ops = {

	.determine_rate	= qmp_dp_pixel_clk_determine_rate,

	.recalc_rate	= qmp_dp_pixel_clk_recalc_rate,

};



static int qcom_qmp_dp_link_clk_determine_rate(struct clk_hw *hw,

					       struct clk_rate_request *req)

static int qmp_dp_link_clk_determine_rate(struct clk_hw *hw, struct clk_rate_request *req)

{

	switch (req->rate) {

	case 162000000:
@@ -2359,8 +2356,7 @@ static int qcom_qmp_dp_link_clk_determine_rate(struct clk_hw *hw, 
	}

}



static unsigned long

qcom_qmp_dp_link_clk_recalc_rate(struct clk_hw *hw, unsigned long parent_rate)

static unsigned long qmp_dp_link_clk_recalc_rate(struct clk_hw *hw, unsigned long parent_rate)

{

	const struct qmp_combo *qmp;

	const struct phy_configure_opts_dp *dp_opts;
@@ -2379,13 +2375,12 @@ qcom_qmp_dp_link_clk_recalc_rate(struct clk_hw *hw, unsigned long parent_rate) 
	}

}



static const struct clk_ops qcom_qmp_dp_link_clk_ops = {

	.determine_rate = qcom_qmp_dp_link_clk_determine_rate,

	.recalc_rate = qcom_qmp_dp_link_clk_recalc_rate,

static const struct clk_ops qmp_dp_link_clk_ops = {

	.determine_rate	= qmp_dp_link_clk_determine_rate,

	.recalc_rate	= qmp_dp_link_clk_recalc_rate,

};



static struct clk_hw *

qcom_qmp_dp_clks_hw_get(struct of_phandle_args *clkspec, void *data)

static struct clk_hw *qmp_dp_clks_hw_get(struct of_phandle_args *clkspec, void *data)

{

	struct qmp_combo *qmp = data;

	unsigned int idx = clkspec->args[0];
@@ -2408,7 +2403,7 @@ static int phy_dp_clks_register(struct qmp_combo *qmp, struct device_node *np) 
	int ret;



	snprintf(name, sizeof(name), "%s::link_clk", dev_name(qmp->dev));

	init.ops = &qcom_qmp_dp_link_clk_ops;

	init.ops = &qmp_dp_link_clk_ops;

	init.name = name;

	qmp->dp_link_hw.init = &init;

	ret = devm_clk_hw_register(qmp->dev, &qmp->dp_link_hw);
@@ -2416,7 +2411,7 @@ static int phy_dp_clks_register(struct qmp_combo *qmp, struct device_node *np) 
		return ret;



	snprintf(name, sizeof(name), "%s::vco_div_clk", dev_name(qmp->dev));

	init.ops = &qcom_qmp_dp_pixel_clk_ops;

	init.ops = &qmp_dp_pixel_clk_ops;

	init.name = name;

	qmp->dp_pixel_hw.init = &init;

	ret = devm_clk_hw_register(qmp->dev, &qmp->dp_pixel_hw);
@@ -2452,7 +2447,7 @@ static int qmp_combo_register_clocks(struct qmp_combo *qmp, struct device_node * 
	if (ret)

		return ret;



	ret = of_clk_add_hw_provider(dp_np, qcom_qmp_dp_clks_hw_get, qmp);

	ret = of_clk_add_hw_provider(dp_np, qmp_dp_clks_hw_get, qmp);

	if (ret)

		return ret;