Commit 62ca1c3a authored by Dmitry Baryshkov's avatar Dmitry Baryshkov
Browse files

drm/bridge: analogic_dp: drop panel_lock 



The analogix_dp_prepare_panel() function is called from bridge's
atomic_pre_enable() and atomic_post_disable() callbacks, which can not
happen simultaneously. Drop the useless mutex.

Tested-by: default avatarDamon Ding <damon.ding@rock-chips.com>
Reviewed-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Link: https://lore.kernel.org/r/20250401-panel-return-void-v1-4-93e1be33dc8d@oss.qualcomm.com


Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
parent 4fc72e1f

drivers/gpu/drm/bridge/analogix/analogix_dp_core.c

+0 −5
Original line number Diff line number Diff line
@@ -974,14 +974,11 @@ static int analogix_dp_prepare_panel(struct analogix_dp_device *dp, 
	if (!dp->plat_data->panel)

		return 0;



	mutex_lock(&dp->panel_lock);



	if (prepare)

		ret = drm_panel_prepare(dp->plat_data->panel);

	else

		ret = drm_panel_unprepare(dp->plat_data->panel);



	mutex_unlock(&dp->panel_lock);

	return ret;

}
@@ -1547,8 +1544,6 @@ analogix_dp_probe(struct device *dev, struct analogix_dp_plat_data *plat_data) 
	dp->dev = &pdev->dev;

	dp->dpms_mode = DRM_MODE_DPMS_OFF;



	mutex_init(&dp->panel_lock);



	/*

	 * platform dp driver need containor_of the plat_data to get

	 * the driver private data, so we need to store the point of

drivers/gpu/drm/bridge/analogix/analogix_dp_core.h

+0 −2
Original line number Diff line number Diff line
@@ -169,8 +169,6 @@ struct analogix_dp_device { 
	bool			fast_train_enable;

	bool			psr_supported;



	struct mutex		panel_lock;



	struct analogix_dp_plat_data *plat_data;

};