Skip to contentSkip to conversation

dma: imx-sdma: fix indentation

PATCH Applied Asaf Vertz 2 participants 4 messages Dec 10, 2014 – Dec 15, 2014

Subsystems (2)
lore
Share & tools
lore

Asaf Vertz

author
#0 lore raw

PATCH 1/1 dma: imx-sdma: fix indentation

Fixed a coding style error, switch and case should be at the same indent

---
 drivers/dma/imx-sdma.c |   18 +++++++++---------
 1 files changed, 9 insertions(+), 9 deletions(-)
drivers/dma/imx-sdma.c +9 -9
index 88afc48..8729877 100644
@@ -1301,15 +1301,15 @@ static void sdma_load_firmware(const struct firmware *fw, void *context)
 	if (header->ram_code_start + header->ram_code_size > fw->size)
 		goto err_firmware;
 	switch (header->version_major) {
-		case 1:
-			sdma->script_number = SDMA_SCRIPT_ADDRS_ARRAY_SIZE_V1;
-			break;
-		case 2:
-			sdma->script_number = SDMA_SCRIPT_ADDRS_ARRAY_SIZE_V2;
-			break;
-		default:
-			dev_err(sdma->dev, "unknown firmware version\n");
-			goto err_firmware;
+	case 1:
+		sdma->script_number = SDMA_SCRIPT_ADDRS_ARRAY_SIZE_V1;
+		break;
+	case 2:
+		sdma->script_number = SDMA_SCRIPT_ADDRS_ARRAY_SIZE_V2;
+		break;
+	default:
+		dev_err(sdma->dev, "unknown firmware version\n");
+		goto err_firmware;
 	}
 
 	addr = (void *)header + header->script_addrs_start;
Signature · 1 line
1.7.0.4
Applied
#1 lore raw
Quoted 1 line · Asaf Vertz

Fixed a coding style error, switch and case should be at the same indent

Please use the right subsystem name for the patches

Applied, now

Thanks

Signature · 1 line
~Vinod
#3 lore raw
Quoted 9 lines · Asaf Vertz
Quoted 3 lines · Vinod Koul
Quoted 1 line · Asaf Vertz

Fixed a coding style error, switch and case should be at the same indent

Please use the right subsystem name for the patches

what subsystem name I should have used?
I used the one from the last patches in the log

http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/log/drivers/dma/imx-sdma.c

$ git llog drivers/dma/pl330.c | grep "dma: pl" |wc -l
22
$ git log drivers/dma/pl330.c | grep "dmaengine: pl" |wc -l
32

and these are last commits

$ git log --oneline drivers/dma/pl330.c
046209f61b5f dmaengine: pl330: update author info
cee42392f5f0 dmaengine: pl330: Correct device assignment.
6e4a2a83f958 dmaengine: pl330: Fix NULL pointer dereference on driver unbind
0f5ebabdd03b dmaengine: pl330: Fix NULL pointer dereference on probe failure
c3cb38f43cb9 dmaengine: pl330: Remove unused 'regs' variable in pl330_submit_req()
937cb2f2498d dmaengine: pl330: Remove non-NULL check for pl330_submit_req parameters
31c1e5a1350a dmaengine: Remove the context argument to the prep_dma_cyclic operation
b1e51d771fbc dmaengine: pl330: Check if the DMA descriptor is NULL
c26939e5204c dmaengine: pl330: Remove pl330_chan_ctrl()
8ed30a14265f dmaengine: pl330: Simplify marking a request as unused
9dc5a315fe51 dmaengine: pl330: Embed pl330_req directly into dma_pl330_desc
f6f2421c0a1c dmaengine: pl330: Merge dma_pl330_dmac and pl330_dmac structs
fbbcd9be96a0 dmaengine: pl330: Simplify is_manager()
Signature · 1 line
~Vinod