[PATCH 03/11] drmmode: add drmmode_get_current_ust function

Richard Wilbur richard.wilbur at gmail.com
Mon Apr 29 13:06:03 PDT 2013


On Mon, Apr 29, 2013 at 9:36 AM, Ilija Hadzic
<ihadzic at research.bell-labs.com> wrote:
> OK, I'll fix it, but I will give it a few more days before sending out v2 of
> the patches to give others opportunity to comment.

No problem.

> BTW, I now notice that
> the comment is actually wrong. The units are supposed to be microseconds.
> So, if it read like this would that address your concern:
>
> /*
>  * Retrieves present UST that is compatible with timestamps used for
>  * vblank events. Depending on the kernel version and DRM kernel module
>  * configuration, the vblank timestamp can either be real time or
>  * monotonic.
>  */

Sorry I wasn't more clear in my comment.  You are correct that the
units should be microseconds and that was what I was trying to convey.
 Other than that it specified the units as milliseconds I have no
quarrel with the original comment.

Sincerely,

Richard


More information about the xorg-driver-ati mailing list