diff options
author | Andy Walls <awalls@radix.net> | 2008-08-31 23:40:41 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2008-10-12 07:36:58 -0400 |
commit | c641d09c60bfa36c7cf70444f24265090e51f5ce (patch) | |
tree | 2f3a9cc7d86aed3e0d5f1c7631a7354bb490fb29 /drivers/media/video/cx18/cx18-queue.c | |
parent | b1526421eac9a912b2cda7e147f1da2aa31be278 (diff) |
V4L/DVB (8914): cx18: Throttle mmio to/from the CX23418 so boards work in older systems
cx18: Throttle mmio to/from the CX23418 so boards work in older systems. The
CX23418 couldn't reliably handle mmio at the rate at which the cx18 driver was
attempting to access the chip. The PCI bridge arrangements and settings on
modern motherboards still allowed the CX23418 to work OK, but it didn't work
well on many older motherboards: mysterious I2C errors, firmware loading errors,
etc. This patch adds a throttle to *all* mmio access to the CX23418. It
defaults to a delay of 31 ns, but is adjustable by the mmio_ndelay module parm.
My HVR-1600 and Raptor PAL/SECAM card now function for analog capture on a
motherboard with an Intel 82810E Northbridge and 82801AA Southbridge.
Signed-off-by: Andy Walls <awalls@radix.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/cx18/cx18-queue.c')
-rw-r--r-- | drivers/media/video/cx18/cx18-queue.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/media/video/cx18/cx18-queue.c b/drivers/media/video/cx18/cx18-queue.c index 48976833e238..a33ba04a2686 100644 --- a/drivers/media/video/cx18/cx18-queue.c +++ b/drivers/media/video/cx18/cx18-queue.c | |||
@@ -170,7 +170,6 @@ int cx18_stream_alloc(struct cx18_stream *s) | |||
170 | } | 170 | } |
171 | buf->id = cx->buffer_id++; | 171 | buf->id = cx->buffer_id++; |
172 | INIT_LIST_HEAD(&buf->list); | 172 | INIT_LIST_HEAD(&buf->list); |
173 | /* FIXME - check for mmio */ | ||
174 | buf->dma_handle = pci_map_single(s->cx->dev, | 173 | buf->dma_handle = pci_map_single(s->cx->dev, |
175 | buf->buf, s->buf_size, s->dma); | 174 | buf->buf, s->buf_size, s->dma); |
176 | cx18_buf_sync_for_cpu(s, buf); | 175 | cx18_buf_sync_for_cpu(s, buf); |
@@ -194,7 +193,6 @@ void cx18_stream_free(struct cx18_stream *s) | |||
194 | 193 | ||
195 | /* empty q_free */ | 194 | /* empty q_free */ |
196 | while ((buf = cx18_dequeue(s, &s->q_free))) { | 195 | while ((buf = cx18_dequeue(s, &s->q_free))) { |
197 | /* FIXME - check for mmio */ | ||
198 | pci_unmap_single(s->cx->dev, buf->dma_handle, | 196 | pci_unmap_single(s->cx->dev, buf->dma_handle, |
199 | s->buf_size, s->dma); | 197 | s->buf_size, s->dma); |
200 | kfree(buf->buf); | 198 | kfree(buf->buf); |