On 16.12.2015 23:31, Thierry Reding wrote: > > - if (crtc < 0 || crtc >= rdev->num_crtc) { > - DRM_ERROR("Invalid crtc %d\n", crtc); > + if (pipe < 0 || pipe >= rdev->num_crtc) { > + DRM_ERROR("Invalid crtc %u\n", pipe); Does it still make sense to test for < 0? Doesn't that generate a compiler warning? Also, I don't really get the point of the crtc => pipe rename here... -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel