2008-06-12 22:13:22

by Adrian Bunk

[permalink] [raw]
Subject: [2.6 patch] mn10300: move sg_dma_{address,len}() to asm/scatterlist.h

mn10300 was the only archotecture where sg_dma_{address,len}() were not
in asm/scatterlist.h, and it's not a big surprise that this caused a
compile error somewhere:

<-- snip -->

..
CC drivers/media/video/videobuf-dma-sg.o
/home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/media/video/videobuf-dma-sg.c: In function `videobuf_dma_map':
/home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/media/video/videobuf-dma-sg.c:238: error: implicit declaration of function 'sg_dma_address'
...
make[4]: *** [drivers/media/video/videobuf-dma-sg.o] Error 1

<-- snip -->

Reported-by: Adrian Bunk <[email protected]>
Signed-off-by: Adrian Bunk <[email protected]>

---

include/asm-mn10300/pci.h | 9 ---------
include/asm-mn10300/scatterlist.h | 9 +++++++++
2 files changed, 9 insertions(+), 9 deletions(-)

b300203c4ab4c6ef3650159edc67a6806171f2f6 diff --git a/include/asm-mn10300/pci.h b/include/asm-mn10300/pci.h
index 205192c..cd9cc5c 100644
--- a/include/asm-mn10300/pci.h
+++ b/include/asm-mn10300/pci.h
@@ -74,15 +74,6 @@ struct pci_dev;
/* This is always fine. */
#define pci_dac_dma_supported(pci_dev, mask) (0)

-/*
- * These macros should be used after a pci_map_sg call has been done
- * to get bus addresses of each of the SG entries and their lengths.
- * You should only work with the number of sg entries pci_map_sg
- * returns.
- */
-#define sg_dma_address(sg) ((sg)->dma_address)
-#define sg_dma_len(sg) ((sg)->length)
-
/* Return the index of the PCI controller for device. */
static inline int pci_controller_num(struct pci_dev *dev)
{
diff --git a/include/asm-mn10300/scatterlist.h b/include/asm-mn10300/scatterlist.h
index e29d91d..6753590 100644
--- a/include/asm-mn10300/scatterlist.h
+++ b/include/asm-mn10300/scatterlist.h
@@ -43,4 +43,13 @@ struct scatterlist {

#define ISA_DMA_THRESHOLD (0x00ffffff)

+/*
+ * These macros should be used after a pci_map_sg call has been done
+ * to get bus addresses of each of the SG entries and their lengths.
+ * You should only work with the number of sg entries pci_map_sg
+ * returns.
+ */
+#define sg_dma_address(sg) ((sg)->dma_address)
+#define sg_dma_len(sg) ((sg)->length)
+
#endif /* _ASM_SCATTERLIST_H */


2008-06-13 12:05:28

by David Howells

[permalink] [raw]
Subject: Re: [2.6 patch] mn10300: move sg_dma_{address,len}() to asm/scatterlist.h

Adrian Bunk <[email protected]> wrote:

> mn10300 was the only archotecture where sg_dma_{address,len}() were not
> in asm/scatterlist.h, and it's not a big surprise that this caused a
> compile error somewhere:
>
> <-- snip -->
>
> ..
> CC drivers/media/video/videobuf-dma-sg.o
> /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/media/video/videobuf-dma-sg.c: In function `videobuf_dma_map':
> /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/media/video/videobuf-dma-sg.c:238: error: implicit declaration of function 'sg_dma_address'
> ...
> make[4]: *** [drivers/media/video/videobuf-dma-sg.o] Error 1
>
> <-- snip -->
>
> Reported-by: Adrian Bunk <[email protected]>
> Signed-off-by: Adrian Bunk <[email protected]>

Acked-by: David Howells <[email protected]>