On Fri, 8 Jul 2011 10:23:55 +0300 Dan Carpenter <error27@xxxxxxxxx> wrote: > The list cursor is never NULL in a list_for_each_entry() loop. > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> > > diff --git a/drivers/staging/gma500/mdfld_intel_display.c > b/drivers/staging/gma500/mdfld_intel_display.c index 1447a5b..fa84990 > 100644 --- a/drivers/staging/gma500/mdfld_intel_display.c > +++ b/drivers/staging/gma500/mdfld_intel_display.c > @@ -1093,8 +1093,6 @@ static int mdfld_crtc_mode_set(struct drm_crtc > *crtc, memcpy(&psb_intel_crtc->saved_adjusted_mode, adjusted_mode, > sizeof(struct drm_display_mode)); > list_for_each_entry(connector, &mode_config->connector_list, > head) { > - if(!connector) > - continue; > > encoder = connector->encoder; All three added to the my next patch stack -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html