diff options
author | Damien Douxchamps <ddouxchamps@users.sf.net> | 2005-11-19 21:32:03 -0500 |
---|---|---|
committer | Jody McIntyre <scjody@modernduck.com> | 2005-11-19 21:32:03 -0500 |
commit | e4cda1654e5c0be4b68e29011e8dc04977286df9 (patch) | |
tree | bd368877ba69f4a9833a094de535ff3cb34b0762 /drivers/ieee1394/video1394.c | |
parent | e27d3014f301e6aee7b65b62ad1da2940e1fd8de (diff) |
Fix incorrect video1394 timestamps.
This patch fixes the incoherent timestamps generated by video1394 since
the single-buffer patch was applied in 2.6.11. Credits have also been
removed from the header and a "//" comment was changed to "/* */".
Signed-off-by: Damien Douxchamps <ddouxchamps@users.sf.net>
Signed-off-by: Jody McIntyre <scjody@modernduck.com>
Diffstat (limited to 'drivers/ieee1394/video1394.c')
-rw-r--r-- | drivers/ieee1394/video1394.c | 17 |
1 files changed, 2 insertions, 15 deletions
diff --git a/drivers/ieee1394/video1394.c b/drivers/ieee1394/video1394.c index 07050f0e4098..608479b2df14 100644 --- a/drivers/ieee1394/video1394.c +++ b/drivers/ieee1394/video1394.c | |||
@@ -19,12 +19,6 @@ | |||
19 | * | 19 | * |
20 | * NOTES: | 20 | * NOTES: |
21 | * | 21 | * |
22 | * jds -- add private data to file to keep track of iso contexts associated | ||
23 | * with each open -- so release won't kill all iso transfers. | ||
24 | * | ||
25 | * Damien Douxchamps: Fix failure when the number of DMA pages per frame is | ||
26 | * one. | ||
27 | * | ||
28 | * ioctl return codes: | 22 | * ioctl return codes: |
29 | * EFAULT is only for invalid address for the argp | 23 | * EFAULT is only for invalid address for the argp |
30 | * EINVAL for out of range values | 24 | * EINVAL for out of range values |
@@ -34,12 +28,6 @@ | |||
34 | * ENOTTY for unsupported ioctl request | 28 | * ENOTTY for unsupported ioctl request |
35 | * | 29 | * |
36 | */ | 30 | */ |
37 | |||
38 | /* Markus Tavenrath <speedygoo@speedygoo.de> : | ||
39 | - fixed checks for valid buffer-numbers in video1394_icotl | ||
40 | - changed the ways the dma prg's are used, now it's possible to use | ||
41 | even a single dma buffer | ||
42 | */ | ||
43 | #include <linux/config.h> | 31 | #include <linux/config.h> |
44 | #include <linux/kernel.h> | 32 | #include <linux/kernel.h> |
45 | #include <linux/list.h> | 33 | #include <linux/list.h> |
@@ -503,7 +491,7 @@ static void wakeup_dma_ir_ctx(unsigned long l) | |||
503 | if (d->ir_prg[i][d->nb_cmd-1].status & cpu_to_le32(0xFFFF0000)) { | 491 | if (d->ir_prg[i][d->nb_cmd-1].status & cpu_to_le32(0xFFFF0000)) { |
504 | reset_ir_status(d, i); | 492 | reset_ir_status(d, i); |
505 | d->buffer_status[d->buffer_prg_assignment[i]] = VIDEO1394_BUFFER_READY; | 493 | d->buffer_status[d->buffer_prg_assignment[i]] = VIDEO1394_BUFFER_READY; |
506 | do_gettimeofday(&d->buffer_time[i]); | 494 | do_gettimeofday(&d->buffer_time[d->buffer_prg_assignment[i]]); |
507 | } | 495 | } |
508 | } | 496 | } |
509 | 497 | ||
@@ -1010,7 +998,6 @@ static int __video1394_ioctl(struct file *file, | |||
1010 | 998 | ||
1011 | /* set time of buffer */ | 999 | /* set time of buffer */ |
1012 | v.filltime = d->buffer_time[v.buffer]; | 1000 | v.filltime = d->buffer_time[v.buffer]; |
1013 | // printk("Buffer %d time %d\n", v.buffer, (d->buffer_time[v.buffer]).tv_usec); | ||
1014 | 1001 | ||
1015 | /* | 1002 | /* |
1016 | * Look ahead to see how many more buffers have been received | 1003 | * Look ahead to see how many more buffers have been received |
@@ -1068,7 +1055,7 @@ static int __video1394_ioctl(struct file *file, | |||
1068 | 1055 | ||
1069 | spin_lock_irqsave(&d->lock,flags); | 1056 | spin_lock_irqsave(&d->lock,flags); |
1070 | 1057 | ||
1071 | // last_buffer is last_prg | 1058 | /* last_buffer is last_prg */ |
1072 | next_prg = (d->last_buffer + 1) % d->num_desc; | 1059 | next_prg = (d->last_buffer + 1) % d->num_desc; |
1073 | if (d->buffer_status[v.buffer]!=VIDEO1394_BUFFER_FREE) { | 1060 | if (d->buffer_status[v.buffer]!=VIDEO1394_BUFFER_FREE) { |
1074 | PRINT(KERN_ERR, ohci->host->id, | 1061 | PRINT(KERN_ERR, ohci->host->id, |