staging: most: hdm-dim2: relocate variable declarations
authorAndrey Shvetsov <andrey.shvetsov@k2l.de>
Thu, 15 Sep 2016 14:19:05 +0000 (16:19 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 15 Sep 2016 17:23:29 +0000 (19:23 +0200)
This patch eliminates nested variable declarations by putting them at
the beginning of the function in order to flatten the code.

Signed-off-by: Andrey Shvetsov <andrey.shvetsov@k2l.de>
Signed-off-by: Christian Gromm <christian.gromm@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/most/hdm-dim2/dim2_hal.c

index 20b69705cfdb215b17d5e9476a9be6bcb22d883b..0c58ab86281f8017182e5825a36f61a4dff3ada5 100644 (file)
@@ -2,7 +2,7 @@
  * dim2_hal.c - DIM2 HAL implementation
  * (MediaLB, Device Interface Macro IP, OS62420)
  *
- * Copyright (C) 2015, Microchip Technology Germany II GmbH & Co. KG
+ * Copyright (C) 2015-2016, Microchip Technology Germany II GmbH & Co. KG
  *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ -518,20 +518,17 @@ static inline bool service_channel(u8 ch_addr, u8 idx)
 {
        u8 const shift = idx * 16;
        u32 const adt1 = dim2_read_ctr(ADT + ch_addr, 1);
+       u32 mask[4] = { 0, 0, 0, 0 };
+       u32 adt_w[4] = { 0, 0, 0, 0 };
 
        if (((adt1 >> (ADT1_DNE_BIT + shift)) & 1) == 0)
                return false;
 
-       {
-               u32 mask[4] = { 0, 0, 0, 0 };
-               u32 adt_w[4] = { 0, 0, 0, 0 };
-
-               mask[1] =
-                       bit_mask(ADT1_DNE_BIT + shift) |
-                       bit_mask(ADT1_ERR_BIT + shift) |
-                       bit_mask(ADT1_RDY_BIT + shift);
-               dim2_write_ctr_mask(ADT + ch_addr, mask, adt_w);
-       }
+       mask[1] =
+               bit_mask(ADT1_DNE_BIT + shift) |
+               bit_mask(ADT1_ERR_BIT + shift) |
+               bit_mask(ADT1_RDY_BIT + shift);
+       dim2_write_ctr_mask(ADT + ch_addr, mask, adt_w);
 
        /* clear channel status bit */
        dimcb_io_write(&g.dim2->ACSR0, bit_mask(ch_addr));