Commit b2920095 authored by ye xingchen's avatar ye xingchen Committed by Rob Clark
Browse files

drm/msm/dsi: Remove the unneeded result variable 



Return the value msm_dsi_phy_enable() directly instead of storing it in
another redundant variable.

Reported-by: default avatarZeal Robot <zealci@zte.com.cn>
Signed-off-by: default avatarye xingchen <ye.xingchen@zte.com.cn>
Reviewed-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Patchwork: https://patchwork.freedesktop.org/patch/499621/
Link: https://lore.kernel.org/r/20220826072821.253150-1-ye.xingchen@zte.com.cn


Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: default avatarRob Clark <robdclark@chromium.org>
parent ac229a22

drivers/gpu/drm/msm/dsi/dsi_manager.c

+1 −4
Original line number Diff line number Diff line
@@ -141,14 +141,11 @@ static int enable_phy(struct msm_dsi *msm_dsi, 
		      struct msm_dsi_phy_shared_timings *shared_timings)

{

	struct msm_dsi_phy_clk_request clk_req;

	int ret;

	bool is_bonded_dsi = IS_BONDED_DSI();



	msm_dsi_host_get_phy_clk_req(msm_dsi->host, &clk_req, is_bonded_dsi);



	ret = msm_dsi_phy_enable(msm_dsi->phy, &clk_req, shared_timings);



	return ret;

	return msm_dsi_phy_enable(msm_dsi->phy, &clk_req, shared_timings);

}



static int