[PATCH] staging: comedi: db_pcidas64: invert conditional

Tobin C. Harding me at tobin.cc
Mon Feb 27 06:36:08 UTC 2017


Checkpatch emits WARNING: Avoid multiple line dereference. It is
possible to reduce the level of indentation by inverting a conditional
and continuing loop if new conditional evaluates to true.

Invert conditional. Continue loop if new conditional evaluates to
true. Reduce subsequent code indentation level by 1. Move multi-line
dereference onto single line.

Signed-off-by: Tobin C. Harding <me at tobin.cc>
---
 drivers/staging/comedi/drivers/cb_pcidas64.c | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c b/drivers/staging/comedi/drivers/cb_pcidas64.c
index efbf277..b0ca5c8 100644
--- a/drivers/staging/comedi/drivers/cb_pcidas64.c
+++ b/drivers/staging/comedi/drivers/cb_pcidas64.c
@@ -1497,16 +1497,14 @@ static int alloc_and_init_dma_members(struct comedi_device *dev)
 			return -ENOMEM;
 	}
 	for (i = 0; i < AO_DMA_RING_COUNT; i++) {
-		if (ao_cmd_is_supported(board)) {
-			devpriv->ao_buffer[i] =
-				dma_alloc_coherent(&pcidev->dev,
-						   DMA_BUFFER_SIZE,
-						   &devpriv->
-						   ao_buffer_bus_addr[i],
-						   GFP_KERNEL);
-			if (!devpriv->ao_buffer[i])
-				return -ENOMEM;
-		}
+		if (!ao_cmd_is_supported(board))
+			continue;
+		devpriv->ao_buffer[i] =
+			dma_alloc_coherent(&pcidev->dev, DMA_BUFFER_SIZE,
+					&devpriv->ao_buffer_bus_addr[i],
+					GFP_KERNEL);
+		if (!devpriv->ao_buffer[i])
+			return -ENOMEM;
 	}
 	/* allocate dma descriptors */
 	devpriv->ai_dma_desc =
-- 
2.7.4



More information about the devel mailing list