staging: et131x: reduce split lines in et131x_config_rx_dma_regs
authorZHAO Gang <gamerh2o@gmail.com>
Mon, 9 Dec 2013 11:38:24 +0000 (19:38 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 11 Dec 2013 07:12:12 +0000 (23:12 -0800)
Signed-off-by: ZHAO Gang <gamerh2o@gmail.com>
Acked-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/et131x/et131x.c

index 5dd21706209658d6a34436c0bac9db7eea9f3ea1..d16ac57a544d83e4e9ee111f0aa29333a8e7f7d1 100644 (file)
@@ -1781,6 +1781,7 @@ static void et131x_config_rx_dma_regs(struct et131x_adapter *adapter)
                u32 __iomem *min_des;
                u32 __iomem *base_hi;
                u32 __iomem *base_lo;
+               struct fbr_lookup *fbr = rx_local->fbr[id];
 
                if (id == 0) {
                        num_des = &rx_dma->fbr0_num_des;
@@ -1797,12 +1798,10 @@ static void et131x_config_rx_dma_regs(struct et131x_adapter *adapter)
                }
 
                /* Now's the best time to initialize FBR contents */
-               fbr_entry =
-                   (struct fbr_desc *) rx_local->fbr[id]->ring_virtaddr;
-               for (entry = 0;
-                    entry < rx_local->fbr[id]->num_entries; entry++) {
-                       fbr_entry->addr_hi = rx_local->fbr[id]->bus_high[entry];
-                       fbr_entry->addr_lo = rx_local->fbr[id]->bus_low[entry];
+               fbr_entry = (struct fbr_desc *) fbr->ring_virtaddr;
+               for (entry = 0; entry < fbr->num_entries; entry++) {
+                       fbr_entry->addr_hi = fbr->bus_high[entry];
+                       fbr_entry->addr_lo = fbr->bus_low[entry];
                        fbr_entry->word2 = entry;
                        fbr_entry++;
                }
@@ -1810,19 +1809,16 @@ static void et131x_config_rx_dma_regs(struct et131x_adapter *adapter)
                /* Set the address and parameters of Free buffer ring 1 and 0
                 * into the 1310's registers
                 */
-               writel(upper_32_bits(rx_local->fbr[id]->ring_physaddr),
-                      base_hi);
-               writel(lower_32_bits(rx_local->fbr[id]->ring_physaddr),
-                      base_lo);
-               writel(rx_local->fbr[id]->num_entries - 1, num_des);
+               writel(upper_32_bits(fbr->ring_physaddr), base_hi);
+               writel(lower_32_bits(fbr->ring_physaddr), base_lo);
+               writel(fbr->num_entries - 1, num_des);
                writel(ET_DMA10_WRAP, full_offset);
 
                /* This variable tracks the free buffer ring 1 full position,
                 * so it has to match the above.
                 */
-               rx_local->fbr[id]->local_full = ET_DMA10_WRAP;
-               writel(((rx_local->fbr[id]->num_entries *
-                                       LO_MARK_PERCENT_FOR_RX) / 100) - 1,
+               fbr->local_full = ET_DMA10_WRAP;
+               writel(((fbr->num_entries * LO_MARK_PERCENT_FOR_RX) / 100) - 1,
                       min_des);
        }