2012/7/3 Daniel Vetter <daniel.vet...@ffwll.ch>: > We already have this pattern at quite a few places, and moving part of > the modeset helper stuff into the driver will add more. > > v2: Don't clobber the crtc struct name with the macro parameter ... > > Signed-Off-by: Daniel Vetter <daniel.vet...@ffwll.ch>
Looks like you're missing some places: intel_lvds_mode_fixup() and intel_tv_mode_fixup(). http://paste.debian.net/177750/ (patch only compiled, not tested) (changes are trivial, so if pastebin expires it, you still know what to do) With that: Reviewed-by: Paulo Zanoni <paulo.r.zan...@intel.com> -- Paulo Zanoni _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx