On Sat, Aug 09, 2014 at 11:00:57PM +0100, Damien Lespiau wrote: > Running smatch adds quite a few checks to what sparse is doing. This is > one of them. > > Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_display.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index 4b89447..a6af1b2 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -8646,7 +8646,6 @@ void intel_release_load_detect_pipe(struct drm_connector *connector, > } > > goto unlock; > - return; I have a locking fix that removes the goto and keeps the return :) I'll send it out now. > } > > /* Switch crtc and encoder back off if necessary */ > -- > 1.8.3.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
