http://bugs.freedesktop.org/show_bug.cgi?id=7771
[EMAIL PROTECTED] changed:
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|[EMAIL PROTECTED] |[email protected]
------- Comment #20 from [EMAIL PROTECTED] 2007-07-30 01:42 PST -------
Looks good, I'd like to push it but there are still some minor issues:
* The minimum actual sleep time probably depends on CONFIG_HZ and related
factors.
* The timeout value in RADEONWaitForVerticalSync() is missing a digit. Might be
worth using a define such as RADEON_VSYNC_TIMEOUT to avoid mistakes like this.
* In standard cases like this, I still think it would be better to use a global
struct timeval or one in RADEONInfoRec instead of adding a struct timeval
declaration for each caller.
Do you want to do another spin, or are you fine with me pushing with these
fixed?
--
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
_______________________________________________
xorg-driver-ati mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-driver-ati
