All of lore.kernel.org
 help / color / mirror / Atom feed
* [patch] doc: fix an incorrect suggestion to pass NULL for PCI like buses
@ 2008-04-28 16:33 Matti Linnanvuori
  2008-04-28 17:03 ` Matthew Wilcox
  2008-04-28 17:09 ` Jesse Barnes
  0 siblings, 2 replies; 3+ messages in thread
From: Matti Linnanvuori @ 2008-04-28 16:33 UTC (permalink / raw)
  To: jbarnes; +Cc: linux-kernel, linux-pci, willy

From: Matti Linnanvuori <mattilinnanvuori@yahoo.com>

Fix an incorrect suggestion to pass NULL to pci_alloc_consistent
for PCI like buses where devices don't have struct pci_dev (like ISA, EISA).

Signed-off-by: Matti Linnanvuori <mattilinnanvuori@yahoo.com>

---

This patch is to be applied on top of
[patch v3] doc: DMA-mapping.txt has undeclared variables [Bug 10397]

--- linux-2.6/Documentation/DMA-mapping.txt    2008-04-28 19:09:36.124693500 +0300
+++ linux/Documentation/DMA-mapping.txt    2008-04-28 19:17:07.711024500 +0300
@@ -317,9 +317,9 @@ you should do:
 
     cpu_addr = pci_alloc_consistent(pdev, size, &dma_handle);
 
-where pdev is a struct pci_dev *. You should pass NULL for PCI like buses
-where devices don't have struct pci_dev (like ISA, EISA).  This may be
-called in interrupt context. 
+where pdev is a struct pci_dev *. This may be called in interrupt context.
+You should use dma_alloc_coherent (see DMA-API.txt) for buses
+where devices don't have struct pci_dev (like ISA, EISA).
 
 This argument is needed because the DMA translations may be bus
 specific (and often is private to the bus which the device is attached


      ____________________________________________________________________________________
Be a better friend, newshound, and 
know-it-all with Yahoo! Mobile.  Try it now.  http://mobile.yahoo.com/;_ylt=Ahu06i62sR8HDtDypao8Wcj9tAcJ

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch] doc: fix an incorrect suggestion to pass NULL for PCI like buses
  2008-04-28 16:33 [patch] doc: fix an incorrect suggestion to pass NULL for PCI like buses Matti Linnanvuori
@ 2008-04-28 17:03 ` Matthew Wilcox
  2008-04-28 17:09 ` Jesse Barnes
  1 sibling, 0 replies; 3+ messages in thread
From: Matthew Wilcox @ 2008-04-28 17:03 UTC (permalink / raw)
  To: Matti Linnanvuori; +Cc: jbarnes, linux-kernel, linux-pci, willy

On Mon, Apr 28, 2008 at 09:33:27AM -0700, Matti Linnanvuori wrote:
> From: Matti Linnanvuori <mattilinnanvuori@yahoo.com>
> 
> Fix an incorrect suggestion to pass NULL to pci_alloc_consistent
> for PCI like buses where devices don't have struct pci_dev (like ISA, EISA).
> 
> Signed-off-by: Matti Linnanvuori <mattilinnanvuori@yahoo.com>

Acked-by: Matthew Wilcox <willy@linux.intel.com>

-- 
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [patch] doc: fix an incorrect suggestion to pass NULL for PCI like buses
  2008-04-28 16:33 [patch] doc: fix an incorrect suggestion to pass NULL for PCI like buses Matti Linnanvuori
  2008-04-28 17:03 ` Matthew Wilcox
@ 2008-04-28 17:09 ` Jesse Barnes
  1 sibling, 0 replies; 3+ messages in thread
From: Jesse Barnes @ 2008-04-28 17:09 UTC (permalink / raw)
  To: Matti Linnanvuori; +Cc: linux-kernel, linux-pci, willy

On Monday, April 28, 2008 9:33 am Matti Linnanvuori wrote:
> From: Matti Linnanvuori <mattilinnanvuori@yahoo.com>
>
> Fix an incorrect suggestion to pass NULL to pci_alloc_consistent
> for PCI like buses where devices don't have struct pci_dev (like ISA,
> EISA).
>
> Signed-off-by: Matti Linnanvuori <mattilinnanvuori@yahoo.com>

This one was whitespace damaged too, but I fixed it up by hand and applied it.  
Before sending future patches, please fix up your mailer so that the diff is 
preserved...

Thanks,
Jesse

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-04-28 17:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-04-28 16:33 [patch] doc: fix an incorrect suggestion to pass NULL for PCI like buses Matti Linnanvuori
2008-04-28 17:03 ` Matthew Wilcox
2008-04-28 17:09 ` Jesse Barnes

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.