r8169: make NUM_RX_DESC a signed int
authorHeiner Kallweit <hkallweit1@gmail.com>
Sun, 6 Dec 2020 00:00:54 +0000 (01:00 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sun, 6 Dec 2020 00:29:21 +0000 (16:29 -0800)
After recent changes there's no need any longer to define NUM_RX_DESC
as an unsigned value.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/r8169_main.c

index 3ea27a6..46d8510 100644 (file)
@@ -68,7 +68,7 @@
 #define R8169_REGS_SIZE                256
 #define R8169_RX_BUF_SIZE      (SZ_16K - 1)
 #define NUM_TX_DESC    256     /* Number of Tx descriptor registers */
-#define NUM_RX_DESC    256U    /* Number of Rx descriptor registers */
+#define NUM_RX_DESC    256     /* Number of Rx descriptor registers */
 #define R8169_TX_RING_BYTES    (NUM_TX_DESC * sizeof(struct TxDesc))
 #define R8169_RX_RING_BYTES    (NUM_RX_DESC * sizeof(struct RxDesc))
 
@@ -3844,7 +3844,7 @@ static struct page *rtl8169_alloc_rx_data(struct rtl8169_private *tp,
 
 static void rtl8169_rx_clear(struct rtl8169_private *tp)
 {
-       unsigned int i;
+       int i;
 
        for (i = 0; i < NUM_RX_DESC && tp->Rx_databuff[i]; i++) {
                dma_unmap_page(tp_to_dev(tp),
@@ -3859,7 +3859,7 @@ static void rtl8169_rx_clear(struct rtl8169_private *tp)
 
 static int rtl8169_rx_fill(struct rtl8169_private *tp)
 {
-       unsigned int i;
+       int i;
 
        for (i = 0; i < NUM_RX_DESC; i++) {
                struct page *data;