On Tue, Aug 06, 2013 at 10:24:10PM +0300, ville.syrjala@xxxxxxxxxxxxxxx wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Check plane->fb in intel_disable_plane() to determine if the plane > is already disabled. > > If the plane has an fb, then it must also have a crtc, so we can drop > the plane->crtc check and just call intel_enable_primary() directly. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Leave it in as a if (WARN(plane->crtc)) return -EINVAL; I don't think this is performance critical and it serves as a good reminder. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
