linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dma: mediatek: addres minor style issues flagged by clang-format.
@ 2019-03-17  8:44 Armando Miraglia
  2019-03-17 18:02 ` Sean Wang
  0 siblings, 1 reply; 2+ messages in thread
From: Armando Miraglia @ 2019-03-17  8:44 UTC (permalink / raw)
  To: matthias.bgg, neil
  Cc: devel, jbi.octave, gustavo, gregkh, linux-kernel,
	sergio.paracuellos, Armando Miraglia, linux-mediatek,
	bhanusreemahesh, christian, linux-arm-kernel

Running clang-format on mtk-hsdma.c I noticed that few lines would not
need to be wrapped, since they fit 80 columns. This change changes such
lines to better fit the style-guide.

Signed-off-by: Armando Miraglia <armax@google.com>
---
 drivers/staging/mt7621-dma/mtk-hsdma.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/mt7621-dma/mtk-hsdma.c b/drivers/staging/mt7621-dma/mtk-hsdma.c
index 97571f1d697b..74c374f7bef4 100644
--- a/drivers/staging/mt7621-dma/mtk-hsdma.c
+++ b/drivers/staging/mt7621-dma/mtk-hsdma.c
@@ -264,8 +264,7 @@ static void mtk_hsdma_reset(struct mtk_hsdam_engine *hsdma,
 	/* init desc value */
 	for (i = 0; i < HSDMA_DESCS_NUM; i++) {
 		chan->tx_ring[i].addr0 = 0;
-		chan->tx_ring[i].flags = HSDMA_DESC_LS0 |
-			HSDMA_DESC_DONE;
+		chan->tx_ring[i].flags = HSDMA_DESC_LS0 | HSDMA_DESC_DONE;
 	}
 	for (i = 0; i < HSDMA_DESCS_NUM; i++) {
 		chan->rx_ring[i].addr0 = 0;
@@ -435,8 +434,7 @@ static irqreturn_t mtk_hsdma_irq(int irq, void *devid)
 	if (likely(status & HSDMA_INT_RX_Q0))
 		tasklet_schedule(&hsdma->task);
 	else
-		dev_dbg(hsdma->ddev.dev, "unhandle irq status %08x\n",
-			status);
+		dev_dbg(hsdma->ddev.dev, "unhandle irq status %08x\n", status);
 	/* clean intr bits */
 	mtk_hsdma_write(hsdma, HSDMA_REG_INT_STATUS, status);
 
-- 
2.21.0.360.g471c308f928-goog


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] dma: mediatek: addres minor style issues flagged by clang-format.
  2019-03-17  8:44 [PATCH] dma: mediatek: addres minor style issues flagged by clang-format Armando Miraglia
@ 2019-03-17 18:02 ` Sean Wang
  0 siblings, 0 replies; 2+ messages in thread
From: Sean Wang @ 2019-03-17 18:02 UTC (permalink / raw)
  To: Armando Miraglia
  Cc: devel, jbi.octave, sergio.paracuellos, gustavo,
	Greg Kroah-Hartman, linux-kernel, Neil Brown, Armando Miraglia,
	linux-mediatek, bhanusreemahesh, Matthias Brugger, christian,
	linux-arm-kernel

On Sun, Mar 17, 2019 at 1:45 AM Armando Miraglia <arma2ff0@gmail.com> wrote:
>
> Running clang-format on mtk-hsdma.c I noticed that few lines would not
> need to be wrapped, since they fit 80 columns. This change changes such
> lines to better fit the style-guide.
>
> Signed-off-by: Armando Miraglia <armax@google.com>
> ---
>  drivers/staging/mt7621-dma/mtk-hsdma.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/mt7621-dma/mtk-hsdma.c b/drivers/staging/mt7621-dma/mtk-hsdma.c
> index 97571f1d697b..74c374f7bef4 100644
> --- a/drivers/staging/mt7621-dma/mtk-hsdma.c
> +++ b/drivers/staging/mt7621-dma/mtk-hsdma.c
> @@ -264,8 +264,7 @@ static void mtk_hsdma_reset(struct mtk_hsdam_engine *hsdma,
>         /* init desc value */
>         for (i = 0; i < HSDMA_DESCS_NUM; i++) {
>                 chan->tx_ring[i].addr0 = 0;
> -               chan->tx_ring[i].flags = HSDMA_DESC_LS0 |
> -                       HSDMA_DESC_DONE;
> +               chan->tx_ring[i].flags = HSDMA_DESC_LS0 | HSDMA_DESC_DONE;
>         }
>         for (i = 0; i < HSDMA_DESCS_NUM; i++) {
>                 chan->rx_ring[i].addr0 = 0;
> @@ -435,8 +434,7 @@ static irqreturn_t mtk_hsdma_irq(int irq, void *devid)
>         if (likely(status & HSDMA_INT_RX_Q0))
>                 tasklet_schedule(&hsdma->task);
>         else
> -               dev_dbg(hsdma->ddev.dev, "unhandle irq status %08x\n",
> -                       status);
> +               dev_dbg(hsdma->ddev.dev, "unhandle irq status %08x\n", status);
>         /* clean intr bits */
>         mtk_hsdma_write(hsdma, HSDMA_REG_INT_STATUS, status);
>

Acked-by: Sean Wang <sean.wang@kernel.org>

> --
> 2.21.0.360.g471c308f928-goog
>
>
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-03-17 18:02 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-17  8:44 [PATCH] dma: mediatek: addres minor style issues flagged by clang-format Armando Miraglia
2019-03-17 18:02 ` Sean Wang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).