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...