Messages in this thread Patch in this message | | | Date | Tue, 30 Mar 2004 19:58:04 -0800 | From | "Randy.Dunlap" <> | Subject | Re: [PATCH] add PCI_DMA_{64,32}BIT constants |
| |
On Tue, 30 Mar 2004 17:26:15 -0800 Greg KH <greg@kroah.com> wrote:
| On Mon, Mar 29, 2004 at 10:36:04PM -0800, Randy.Dunlap wrote: | > On Tue, 23 Mar 2004 00:23:05 -0500 Jeff Garzik <jgarzik@pobox.com> wrote: | > | > | | > | Been meaning to do this for ages... | > | | > | Another one for the janitors. | > | > >>>Nice, I've pulled this to my pci tree and will forward it on to Linus in | > >>>the next round of pci patches after 2.6.5 is out. | > >> | > >>Yeah well... in the intervening time, somebody on IRC commented | > >> | > >>"so what is so PCI-specific about those constants?" | > >> | > >>They probably ought to be DMA_{32,64}BIT_MASK or somesuch. | > > | > > | > > Heh, ok, care to make up another patch for this? :) | > | > | > Here's an updated patch, applies to 2.6.5-rc2-bk9. | > I left the DMA_xxBIT_MASK defines in linux/pci.h, although | > they aren't necessarily PCI-specific. Would we prefer to | > have them in linux/dma-mapping.h ? | | Ok, this looks good. I've backed out Jeff's patch and applied this one | instead to my tree. If you want to move the defines, please send me a | patch relative to this one.
Yes, let's move the defines like Jeff suggested. Additional (relative) patch is below.
Thanks, -- ~Randy
// linux-2.6.5-rc3 // move DMA_nnBIT_MASK to linux/dma-mapping.h;
diffstat:= include/linux/dma-mapping.h | 3 +++ include/linux/pci.h | 3 --- 2 files changed, 3 insertions(+), 3 deletions(-)
diff -Naurp ./include/linux/pci.h~dma_bits ./include/linux/pci.h --- ./include/linux/pci.h~dma_bits 2004-03-30 17:50:00.000000000 -0800 +++ ./include/linux/pci.h 2004-03-30 17:53:49.000000000 -0800 @@ -362,9 +362,6 @@ enum pci_mmap_state { #define PCI_DMA_FROMDEVICE 2 #define PCI_DMA_NONE 3 -#define DMA_64BIT_MASK 0xffffffffffffffffULL -#define DMA_32BIT_MASK 0x00000000ffffffffULL - #define DEVICE_COUNT_COMPATIBLE 4 #define DEVICE_COUNT_RESOURCE 12 diff -Naurp ./include/linux/dma-mapping.h~dma_bits ./include/linux/dma-mapping.h --- ./include/linux/dma-mapping.h~dma_bits 2004-03-30 17:41:41.000000000 -0800 +++ ./include/linux/dma-mapping.h 2004-03-30 17:53:54.000000000 -0800 @@ -10,6 +10,9 @@ enum dma_data_direction { DMA_NONE = 3, }; +#define DMA_64BIT_MASK 0xffffffffffffffffULL +#define DMA_32BIT_MASK 0x00000000ffffffffULL + #include <asm/dma-mapping.h> /* Backwards compat, remove in 2.7.x */ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |