]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
usb: musb: cppi_dma.c: use DIV_ROUND_UP macro in cppi_next_(r|t)x_segment()
authorIvan Safonov <insafonov@gmail.com>
Mon, 17 Apr 2017 04:21:15 +0000 (23:21 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 18 Apr 2017 14:48:26 +0000 (16:48 +0200)
DIV_ROUND_UP is bit useful than series of "/" and "%" operations.
Replace "/%" sequence with DIV_ROUND_UP macro.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Bin Liu <b-liu@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/musb/cppi_dma.c

index c4fabe952ca6a34dd3cb5c1f938e9ecc80c470ba..a13bd362504399a88aba8d8f688d66f0fefecb7f 100644 (file)
@@ -582,9 +582,10 @@ cppi_next_tx_segment(struct musb *musb, struct cppi_channel *tx)
                maxpacket = length;
                n_bds = 1;
        } else {
-               n_bds = length / maxpacket;
-               if (!length || (length % maxpacket))
-                       n_bds++;
+               if (length)
+                       n_bds = DIV_ROUND_UP(length, maxpacket);
+               else
+                       n_bds = 1;
                n_bds = min(n_bds, (unsigned) NUM_TXCHAN_BD);
                length = min(n_bds * maxpacket, length);
        }
@@ -790,9 +791,7 @@ cppi_next_rx_segment(struct musb *musb, struct cppi_channel *rx, int onepacket)
                        n_bds = 0xffff / maxpacket;
                        length = n_bds * maxpacket;
                } else {
-                       n_bds = length / maxpacket;
-                       if (length % maxpacket)
-                               n_bds++;
+                       n_bds = DIV_ROUND_UP(length, maxpacket);
                }
                if (n_bds == 1)
                        onepacket = 1;