On Tue, Dec 16, 2014 at 05:53:31PM +0100, Thierry Reding wrote: > -void drm_wait_one_vblank(struct drm_device *dev, int crtc) > +void drm_wait_one_vblank(struct drm_device *dev, unsigned int pipe) > { > + struct drm_vblank_crtc *vblank = &dev->vblank[pipe]; > int ret; > u32 last; > > - ret = drm_vblank_get(dev, crtc); > - if (WARN(ret, "vblank not available on crtc %i, ret=%i\n", crtc, ret)) > + if (WARN_ON(pipe >= dev->num_crtcs)) This addition should be in the previous patch. I didn't spot anything else but it's a bit a tedious patch. But makes sense. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html