On Mon, Jun 20 2005, [email protected] wrote:
> From: Tobias Klauser <[email protected]>
>
>
>
> Use the DMA_{64,32}BIT_MASK constants from dma-mapping.h when calling
> pci_set_dma_mask() or pci_set_consistent_dma_mask()
> These patches include dma-mapping.h explicitly because it caused errors
> on some architectures otherwise.
> See http://marc.theaimsgroup.com/?t=108001993000001&r=1&w=2 for details
>
> Signed-off-by: Tobias Klauser <[email protected]>
> Signed-off-by: Domen Puncer <[email protected]>
> ---
> umem.c | 5 +++--
> 1 files changed, 3 insertions(+), 2 deletions(-)
>
> Index: quilt/drivers/block/umem.c
> ===================================================================
> --- quilt.orig/drivers/block/umem.c
> +++ quilt/drivers/block/umem.c
> @@ -50,6 +50,7 @@
> #include <linux/timer.h>
> #include <linux/pci.h>
> #include <linux/slab.h>
> +#include <linux/dma-mapping.h>
>
> #include <linux/fcntl.h> /* O_ACCMODE */
> #include <linux/hdreg.h> /* HDIO_GETGEO */
> @@ -892,8 +893,8 @@ static int __devinit mm_pci_probe(struct
> printk(KERN_INFO "Micro Memory(tm) controller #%d found at %02x:%02x (PCI Mem Module (Battery Backup))\n",
> card->card_number, dev->bus->number, dev->devfn);
>
> - if (pci_set_dma_mask(dev, 0xffffffffffffffffLL) &&
> - !pci_set_dma_mask(dev, 0xffffffffLL)) {
> + if (pci_set_dma_mask(dev, DMA_64BIT_MASK) &&
> + !pci_set_dma_mask(dev, DMA_32BIT_MASK)) {
> printk(KERN_WARNING "MM%d: NO suitable DMA found\n",num_cards);
> return -ENOMEM;
> }
Not from your patch, but that code looks a little strange. We error if
setting a 64-bit mask fails _but_ the 32-bit one succeeds? That can't be
right.
--
Jens Axboe
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
