Revert "media: uvcvideo: Fix integer overflow calculating timestamp"
This reverts commit c5bc5ad44b
.
This commit is contained in:
parent
228acff879
commit
42755cc6a5
1 changed files with 5 additions and 5 deletions
|
@ -723,11 +723,11 @@ void uvc_video_clock_update(struct uvc_streaming *stream,
|
|||
unsigned long flags;
|
||||
u64 timestamp;
|
||||
u32 delta_stc;
|
||||
u32 y1;
|
||||
u32 y1, y2;
|
||||
u32 x1, x2;
|
||||
u32 mean;
|
||||
u32 sof;
|
||||
u64 y, y2;
|
||||
u64 y;
|
||||
|
||||
if (!uvc_hw_timestamps_param)
|
||||
return;
|
||||
|
@ -767,7 +767,7 @@ void uvc_video_clock_update(struct uvc_streaming *stream,
|
|||
sof = y;
|
||||
|
||||
uvc_trace(UVC_TRACE_CLOCK, "%s: PTS %u y %llu.%06llu SOF %u.%06llu "
|
||||
"(x1 %u x2 %u y1 %u y2 %llu SOF offset %u)\n",
|
||||
"(x1 %u x2 %u y1 %u y2 %u SOF offset %u)\n",
|
||||
stream->dev->name, buf->pts,
|
||||
y >> 16, div_u64((y & 0xffff) * 1000000, 65536),
|
||||
sof >> 16, div_u64(((u64)sof & 0xffff) * 1000000LLU, 65536),
|
||||
|
@ -782,7 +782,7 @@ void uvc_video_clock_update(struct uvc_streaming *stream,
|
|||
goto done;
|
||||
|
||||
y1 = NSEC_PER_SEC;
|
||||
y2 = ktime_to_ns(ktime_sub(last->host_time, first->host_time)) + y1;
|
||||
y2 = (u32)ktime_to_ns(ktime_sub(last->host_time, first->host_time)) + y1;
|
||||
|
||||
/* Interpolated and host SOF timestamps can wrap around at slightly
|
||||
* different times. Handle this by adding or removing 2048 to or from
|
||||
|
@ -802,7 +802,7 @@ void uvc_video_clock_update(struct uvc_streaming *stream,
|
|||
timestamp = ktime_to_ns(first->host_time) + y - y1;
|
||||
|
||||
uvc_trace(UVC_TRACE_CLOCK, "%s: SOF %u.%06llu y %llu ts %llu "
|
||||
"buf ts %llu (x1 %u/%u/%u x2 %u/%u/%u y1 %u y2 %llu)\n",
|
||||
"buf ts %llu (x1 %u/%u/%u x2 %u/%u/%u y1 %u y2 %u)\n",
|
||||
stream->dev->name,
|
||||
sof >> 16, div_u64(((u64)sof & 0xffff) * 1000000LLU, 65536),
|
||||
y, timestamp, vbuf->vb2_buf.timestamp,
|
||||
|
|
Loading…
Reference in a new issue