Skip to content

Commit e512ed6

Browse files
jhovoldlumag
authored andcommitted
drm/msm/dsi: 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: Dmitry Baryshkov <[email protected]> Signed-off-by: Johan Hovold <[email protected]> Tested-by: Kuogee Hsieh <[email protected]> Reviewed-by: Kuogee Hsieh <[email protected]> Reviewed-by: Abhinav Kumar <[email protected]> Patchwork: https://patchwork.freedesktop.org/patch/502678/ Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Dmitry Baryshkov <[email protected]>
1 parent 9a5c158 commit e512ed6

File tree

1 file changed

+1
-6
lines changed
  • drivers/gpu/drm/msm/dsi

1 file changed

+1
-6
lines changed

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

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -211,14 +211,9 @@ void __exit msm_dsi_unregister(void)
211211
int msm_dsi_modeset_init(struct msm_dsi *msm_dsi, struct drm_device *dev,
212212
struct drm_encoder *encoder)
213213
{
214-
struct msm_drm_private *priv;
214+
struct msm_drm_private *priv = dev->dev_private;
215215
int ret;
216216

217-
if (WARN_ON(!encoder) || WARN_ON(!msm_dsi) || WARN_ON(!dev))
218-
return -EINVAL;
219-
220-
priv = dev->dev_private;
221-
222217
if (priv->num_bridges == ARRAY_SIZE(priv->bridges)) {
223218
DRM_DEV_ERROR(dev->dev, "too many bridges\n");
224219
return -ENOSPC;

0 commit comments

Comments
 (0)