V4L/DVB (13716): [Mantis] Bug: incorrect byte swap
authorDavid Woodhouse <dwmw2@infradead.org>
Thu, 3 Dec 2009 08:47:11 +0000 (05:47 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 17 Jan 2010 13:55:30 +0000 (11:55 -0200)
You know, the worst endianness errors are not the cases where
people forget to byte-swap, but the cases where they either byte-swap
with the wrong size, or byte-swap when they shouldn't have done so at
all. Those ones defeat the casual reader of the code.

Signed-off-by: David Woodhouse <dwmw2@infradead.org>
Signed-off-by: Manu Abraham <manu@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/mantis/mantis_dma.c

index ea23a89..f685992 100644 (file)
@@ -189,7 +189,7 @@ void mantis_dma_start(struct mantis_pci *mantis)
        dprintk(verbose, MANTIS_DEBUG, 1, "Mantis Start DMA engine");
 
        mantis_risc_program(mantis);
-       mmwrite(cpu_to_le32(mantis->risc_dma), MANTIS_RISC_START);
+       mmwrite(mantis->risc_dma, MANTIS_RISC_START);
        mmwrite(mmread(MANTIS_GPIF_ADDR) | MANTIS_GPIF_RDWRN, MANTIS_GPIF_ADDR);
 
        mmwrite(0, MANTIS_DMA_CTL);