the variable sender is dereferenced in line 95, so it is no reason to check null again in line 99. Signed-off-by: Cong Ding <dinggnu@xxxxxxxxx> --- drivers/gpu/drm/gma500/mdfld_dsi_output.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/drivers/gpu/drm/gma500/mdfld_dsi_output.c b/drivers/gpu/drm/gma500/mdfld_dsi_output.c index 2d4ab48..67e439a 100644 --- a/drivers/gpu/drm/gma500/mdfld_dsi_output.c +++ b/drivers/gpu/drm/gma500/mdfld_dsi_output.c @@ -96,11 +96,6 @@ void mdfld_dsi_brightness_init(struct mdfld_dsi_config *dsi_config, int pipe) struct drm_psb_private *dev_priv = dev->dev_private; u32 gen_ctrl_val; - if (!sender) { - DRM_ERROR("No sender found\n"); - return; - } - /* Set default display backlight value to 85% (0xd8)*/ mdfld_dsi_send_mcs_short(sender, write_display_brightness, 0xd8, 1, true); -- 1.7.10.4 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel