Thread (12 messages) flat view 12 messages, 4 authors, 2012-07-16
STALE5168d REVIEWED: 1 (0M)

1 review trailer.

[PATCH] ARM: dma: mxs-dma: Export missing symbols from mxs-dma.c

From: Dong Aisheng <hidden>
Date: 2012-07-06 11:10:21

On Fri, Jul 06, 2012 at 07:09:52PM +0800, Attila Kinali wrote:
quoted hunk ↗ jump to hunk
mxs-dma.c provides two functions mxs_dma_is_apbh and mxs_dma_is_apbx
which are used at least in mxs-mmc.c. Building mxs-mmc as module
fails due to those two symbols not being exported.

Signed-off-by: Attila Kinali <redacted>
---
 drivers/dma/mxs-dma.c |    3 +++
 1 file changed, 3 insertions(+)
diff --git a/drivers/dma/mxs-dma.c b/drivers/dma/mxs-dma.c
index c96ab15..0437f49 100644
--- a/drivers/dma/mxs-dma.c
+++ b/drivers/dma/mxs-dma.c
@@ -201,6 +201,7 @@ int mxs_dma_is_apbh(struct dma_chan *chan)
 
 	return dma_is_apbh(mxs_dma);
 }
+EXPORT_SYMBOL(mxs_dma_is_apbh);
quoted hunk ↗ jump to hunk
 
 int mxs_dma_is_apbx(struct dma_chan *chan)
 {
@@ -209,6 +210,7 @@ int mxs_dma_is_apbx(struct dma_chan *chan)
 
 	return !dma_is_apbh(mxs_dma);
 }
+EXPORT_SYMBOL(mxs_dma_is_apbx);
 
Shawn,

Do we need this be EXPORT_SYMBOL_GPL?
quoted hunk ↗ jump to hunk
 static void mxs_dma_reset_chan(struct mxs_dma_chan *mxs_chan)
 {
@@ -779,3 +781,4 @@ static int __init mxs_dma_module_init(void)
 	return platform_driver_probe(&mxs_dma_driver, mxs_dma_probe);
 }
 subsys_initcall(mxs_dma_module_init);
+
This line change is unneeded.

else i'm ok with the patch:
Acked-by: Dong Aisheng <redacted>

Regards
Dong Aisheng
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help