Hi Vinod, On 13 January 2018 at 01:08, Vinod Koul <vinod.koul@xxxxxxxxx> wrote: > Sparse warns that 'sprd_dma_prep_dma_memcpy' should be static so make it > static. > > drivers/dma/sprd-dma.c:713:32: warning: > symbol'sprd_dma_prep_dma_memcpy' was not declared. Should it be static? > > Signed-off-by: Vinod Koul <vinod.koul@xxxxxxxxx> Thanks for fixing my mistake. Reviewed-by: Baolin Wang <baolin.wang@xxxxxxxxxx> > --- > drivers/dma/sprd-dma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/dma/sprd-dma.c b/drivers/dma/sprd-dma.c > index b652071a2096..b106e8a60af6 100644 > --- a/drivers/dma/sprd-dma.c > +++ b/drivers/dma/sprd-dma.c > @@ -710,7 +710,7 @@ static int sprd_dma_config(struct dma_chan *chan, struct sprd_dma_desc *sdesc, > return 0; > } > > -struct dma_async_tx_descriptor * > +static struct dma_async_tx_descriptor * > sprd_dma_prep_dma_memcpy(struct dma_chan *chan, dma_addr_t dest, dma_addr_t src, > size_t len, unsigned long flags) > { > -- > 2.7.4 > > -- > To unsubscribe from this list: send the line "unsubscribe dmaengine" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Baolin.wang Best Regards -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html