Commit f94eff09 authored by Johan Hovold's avatar Johan Hovold Committed by Dmitry Baryshkov
Browse files

drm/msm/dp: drop modeset sanity checks 



Drop the overly defensive modeset sanity checks of function parameters
which have already been checked or used by the callers.

Reviewed-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: default avatarJohan Hovold <johan+linaro@kernel.org>
Tested-by: default avatarKuogee Hsieh <quic_khsieh@quicinc.com>
Reviewed-by: default avatarKuogee Hsieh <quic_khsieh@quicinc.com>
Reviewed-by: default avatarAbhinav Kumar <quic_abhinavk@quicinc.com>
Patchwork: https://patchwork.freedesktop.org/patch/502674/
Link: https://lore.kernel.org/r/20220913085320.8577-10-johan+linaro@kernel.org


Signed-off-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
parent af309c0c

drivers/gpu/drm/msm/dp/dp_display.c

+1 −11
Original line number Diff line number Diff line
@@ -1600,20 +1600,10 @@ static int dp_display_get_next_bridge(struct msm_dp *dp) 
int msm_dp_modeset_init(struct msm_dp *dp_display, struct drm_device *dev,

			struct drm_encoder *encoder)

{

	struct msm_drm_private *priv;

	struct msm_drm_private *priv = dev->dev_private;

	struct dp_display_private *dp_priv;

	int ret;



	if (WARN_ON(!encoder) || WARN_ON(!dp_display) || WARN_ON(!dev))

		return -EINVAL;



	priv = dev->dev_private;



	if (priv->num_bridges == ARRAY_SIZE(priv->bridges)) {

		DRM_DEV_ERROR(dev->dev, "too many bridges\n");

		return -ENOSPC;

	}



	dp_display->drm_dev = dev;



	dp_priv = container_of(dp_display, struct dp_display_private, dp_display);