<p dir="ltr">Seems those inlines are leftovers from first versions of the driver. Anyway, looks fine by me. I think Viresh can give you an Ack.</p>
<div class="gmail_quote">9.5.2013 12.22 "Denis Efremov" <<a href="mailto:yefremov.denis@gmail.com">yefremov.denis@gmail.com</a>> kirjoitti:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
EXPORT_SYMBOL and inline directives are contradictory to each other.<br>
The patch fixes this inconsistency.<br>
<br>
Found by Linux Driver Verification project (<a href="http://linuxtesting.org" target="_blank">linuxtesting.org</a>).<br>
<br>
Signed-off-by: Denis Efremov <<a href="mailto:yefremov.denis@gmail.com">yefremov.denis@gmail.com</a>><br>
---<br>
 drivers/dma/dw_dmac.c | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/drivers/dma/dw_dmac.c b/drivers/dma/dw_dmac.c<br>
index 43a5329..a526bb9 100644<br>
--- a/drivers/dma/dw_dmac.c<br>
+++ b/drivers/dma/dw_dmac.c<br>
@@ -574,14 +574,14 @@ static void dwc_handle_error(struct dw_dma *dw, struct dw_dma_chan *dwc)<br>
<br>
 /* --------------------- Cyclic DMA API extensions -------------------- */<br>
<br>
-inline dma_addr_t dw_dma_get_src_addr(struct dma_chan *chan)<br>
+dma_addr_t dw_dma_get_src_addr(struct dma_chan *chan)<br>
 {<br>
        struct dw_dma_chan *dwc = to_dw_dma_chan(chan);<br>
        return channel_readl(dwc, SAR);<br>
 }<br>
 EXPORT_SYMBOL(dw_dma_get_src_addr);<br>
<br>
-inline dma_addr_t dw_dma_get_dst_addr(struct dma_chan *chan)<br>
+dma_addr_t dw_dma_get_dst_addr(struct dma_chan *chan)<br>
 {<br>
        struct dw_dma_chan *dwc = to_dw_dma_chan(chan);<br>
        return channel_readl(dwc, DAR);<br>
--<br>
1.8.1.4<br>
<br>
--<br>
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in<br>
the body of a message to <a href="mailto:majordomo@vger.kernel.org">majordomo@vger.kernel.org</a><br>
More majordomo info at  <a href="http://vger.kernel.org/majordomo-info.html" target="_blank">http://vger.kernel.org/majordomo-info.html</a><br>
Please read the FAQ at  <a href="http://www.tux.org/lkml/" target="_blank">http://www.tux.org/lkml/</a><br>
</blockquote></div>