Thread (6 messages) 6 messages, 3 authors, 2018-12-05

Re: [PATCH] dmaengine: remove DBGFS_FUNC_DECL()

From: Vinod Koul <vkoul@kernel.org>
Date: 2018-12-05 15:59:22
Also in: dmaengine, linux-arm-msm, lkml

On 05-12-18, 23:49, Frank Lee wrote:
On Wed, Dec 5, 2018 at 4:27 PM Vinod Koul [off-list ref] wrote:
quoted
On 30-11-18, 10:42, Yangtao Li wrote:
quoted
We already have the DEFINE_SHOW_ATTRIBUTE,There is no need to define
such a macro,so remove DBGFS_FUNC_DECL.Also use macro to simplify some
code.
Spaces after full stop and comma please
quoted
Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/dma/amba-pl08x.c     | 14 ++------------
 drivers/dma/mic_x100_dma.c   | 22 +++-------------------
 drivers/dma/pxa_dma.c        | 24 ++++++++++++------------
 drivers/dma/qcom/hidma_dbg.c | 33 ++++++---------------------------
Do you mind splitting it per file.
Hi Vinod:

Why not apply it once ?The change is not too big.
It is not about big or small. A patch should do one thing and touch a
single module.

It also helps people when they look at changes and backports.

-- 
~Vinod

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help