lkml.org 
[lkml]   [2008]   [Dec]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: drivers using the non-PCI dma_set_mask() on PCI devices
Date
Mikael Pettersson wrote:
> Several drivers do the following:
>
> struct pci_dev *pdev = ...;
> ...
> if (dma_set_mask(&pdev->dev, mask))
> ...
>
> But pdev->dev.dma_mask == &pdev->dma_mask, so this is essentially a
> roundabout way of saying
>
> if (pci_set_dma_mask(pdev, mask))
>
> except that it bypasses the PCI-specific operations pci_set_dma_mask()
> may do on that platform.
>
> Drivers doing this include drivers/scsi/aic7xxx/aic79xx_osm_pci.c,
> drivers/scsi/aic7xxxx/aic7xxx_osm_pci.c, drivers/scsi/qla2xxx/qla_os.c,
> drivers/net/wireless/rt2x00/rt2x00pci.c, and drivers/media/video/meye.c.
>
> Is it considered acceptable that drivers bypass the PCI DMA API on
> PCI devices like this, or are these drivers in error?
>
> I'm doing some work on an embedded platform (ARM IXP4xx) with some
> PCI DMA restrictions. To handle these the platform provides its own
> versions of pci_set_dma_mask() and pci_set_consistent_dma_mask(),
> but its dma_set_mask() currently does not do anything PCI-specific.
> The question is: should dma_set_mask() have PCI knowledge or not?

AFAIK pci_set_dma_mask is somewhat deprecated, and dma_set_mask should
be used instead. If the platform fails to do what's needed when
dma_set_mask is called on a PCI device then it would seem the platform
code is in error.



\
 
 \ /
  Last update: 2008-12-23 05:47    [W:0.079 / U:0.396 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site