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

Reply via email to