net: rtl8139: Minor cleanup of set_rx_mode()

Rename the function to rtl8139_set_rx_mode(), use unsigned variables
where applicable and inline mc_filter[] values. No functional chanage.

Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Joe Hershberger <joe.hershberger@ni.com>
This commit is contained in:
Marek Vasut 2020-04-12 21:35:12 +02:00 committed by marex
parent 198e6b571b
commit 89f3facffb

View file

@ -357,19 +357,17 @@ static const unsigned int rtl8139_rx_config =
(RX_FIFO_THRESH << 13) | (RX_FIFO_THRESH << 13) |
(RX_DMA_BURST << 8); (RX_DMA_BURST << 8);
static void set_rx_mode(struct eth_device *dev) { static void rtl8139_set_rx_mode(struct eth_device *dev)
unsigned int mc_filter[2]; {
int rx_mode;
/* !IFF_PROMISC */ /* !IFF_PROMISC */
rx_mode = RTL_REG_RXCONFIG_ACCEPTBROADCAST | unsigned int rx_mode = RTL_REG_RXCONFIG_ACCEPTBROADCAST |
RTL_REG_RXCONFIG_ACCEPTMULTICAST | RTL_REG_RXCONFIG_ACCEPTMULTICAST |
RTL_REG_RXCONFIG_ACCEPTMYPHYS; RTL_REG_RXCONFIG_ACCEPTMYPHYS;
mc_filter[1] = mc_filter[0] = 0xffffffff;
outl(rtl8139_rx_config | rx_mode, ioaddr + RTL_REG_RXCONFIG); outl(rtl8139_rx_config | rx_mode, ioaddr + RTL_REG_RXCONFIG);
outl(mc_filter[0], ioaddr + RTL_REG_MAR0 + 0); outl(0xffffffff, ioaddr + RTL_REG_MAR0 + 0);
outl(mc_filter[1], ioaddr + RTL_REG_MAR0 + 4); outl(0xffffffff, ioaddr + RTL_REG_MAR0 + 4);
} }
static void rtl_reset(struct eth_device *dev) static void rtl_reset(struct eth_device *dev)
@ -425,8 +423,7 @@ static void rtl_reset(struct eth_device *dev)
/* Start the chip's Tx and Rx process. */ /* Start the chip's Tx and Rx process. */
outl(0, ioaddr + RTL_REG_RXMISSED); outl(0, ioaddr + RTL_REG_RXMISSED);
/* set_rx_mode */ rtl8139_set_rx_mode(dev);
set_rx_mode(dev);
/* Disable all known interrupts by setting the interrupt mask. */ /* Disable all known interrupts by setting the interrupt mask. */
outw(0, ioaddr + RTL_REG_INTRMASK); outw(0, ioaddr + RTL_REG_INTRMASK);