[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20200121074227.GC2841@vkoul-mobl>
Date: Tue, 21 Jan 2020 13:12:27 +0530
From: Vinod Koul <vkoul@...nel.org>
To: Peter Ujfalusi <peter.ujfalusi@...com>
Cc: dmaengine@...r.kernel.org, linux-kernel@...r.kernel.org,
dan.j.williams@...el.com, grygorii.strashko@...com
Subject: Re: [PATCH] dmaengine: ti: k3-psil: Fix warnings with C=1 and W=1
On 21-01-20, 09:01, Peter Ujfalusi wrote:
> Fixes the following warnings:
> drivers/dma/ti/k3-psil-j721e.c:62:16: warning: symbol 'j721e_src_ep_map' was not declared. Should it be static?
> drivers/dma/ti/k3-psil-j721e.c:172:16: warning: symbol 'j721e_dst_ep_map' was not declared. Should it be static?
> drivers/dma/ti/k3-psil-j721e.c:216:20: warning: symbol 'j721e_ep_map' was not declared. Should it be static?
> CC drivers/dma/ti/k3-psil-j721e.o
> drivers/dma/ti/k3-psil-am654.c:52:16: warning: symbol 'am654_src_ep_map' was not declared. Should it be static?
> drivers/dma/ti/k3-psil-am654.c:127:16: warning: symbol 'am654_dst_ep_map' was not declared. Should it be static?
> drivers/dma/ti/k3-psil-am654.c:169:20: warning: symbol 'am654_ep_map' was not declared. Should it be static?
We dont use titles like this :(
A title should reflect the changes in the patch, I have updated this to
"dmaengine: ti: k3-psil: make symbols static"
and applied now
--
~Vinod
Powered by blists - more mailing lists