On Thu, 5 Jul 2012 19:22:37 +0200, Olivier Galibert <galibert at pobox.com> wrote: > On Thu, Jul 05, 2012 at 10:10:10AM +0100, Chris Wilson wrote: > > + if (INTEL_INFO(dev)->gen >= 4 && (x|y) & ~4095) { > > + DRM_ERROR("CRTC offset too larget (%d, %d)\n", x, y); > > larget? Not even a humorous typo. Daniel has posted a couple of patches to actually workaround the limits which obsoletes the need for rejecting the mode_set_base(). -Chris -- Chris Wilson, Intel Open Source Technology Centre