[xorg@ dropped from Cc.] Hi,
On Wed, May 13, 2009 at 09:44:50AM -0700, Defnet, Benjamin R wrote: > Fix xf86CrtcSetModeTransform() to properly process required rotation and > transformation data, even when using kernel mode setting. Previously, with > kernel mode setting enabled, this function just called set_mode_major() and > returned causing transformations to never be applied. > > This is my first time submitting a patch so please let me know if I'm not > doing this right. No, because this forces all rendering to be transformed, even if you can do rotation in hardware (cf. OMAP) and don't need any transformations applied. Why not just factor out the transformation into a helper function (or helper functions?). Also, as Maarten pointed out, set_mode_major isn't exclusive to KMS. Cheers, Daniel
signature.asc
Description: Digital signature
_______________________________________________ xorg-devel mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-devel
