On Tue, Aug 06, 2013 at 10:24:10PM +0300, [email protected] wrote: > From: Ville Syrjälä <[email protected]> > > 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ä <[email protected]>
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 [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
