RE: [PATCH net-next v10 2/2] net: ethernet: Add driver for Sunplus SP7021

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Paolo,


>           On Mon, 2022-05-02 at 21:23 +0800, Wells Lu wrote:
> > Add driver for Sunplus SP7021 SoC.
> >
> > Reviewed-by: Andrew Lunn <andrew@xxxxxxx>
> > Signed-off-by: Wells Lu <wellslutw@xxxxxxxxx>
> > ---
> > Changes in v10
> >   - Addressed comments of Mr. Stephen Hemminger.
> >     - Use define NAPI_POLL_WEIGHT.
> >     - Return amount of processed packets in spl2sw_tx_poll() and spl2sw_rx_poll().
> >   - Addressed comments of Mr. Francois Romien.
> >     - Added spin locks for "read-modify-write of int-mask register".
> >
> >  MAINTAINERS                                   |   1 +
> >  drivers/net/ethernet/Kconfig                  |   1 +
> >  drivers/net/ethernet/Makefile                 |   1 +
> >  drivers/net/ethernet/sunplus/Kconfig          |  35 ++
> >  drivers/net/ethernet/sunplus/Makefile         |   6 +
> >  drivers/net/ethernet/sunplus/spl2sw_define.h  | 270 ++++++++
> >  drivers/net/ethernet/sunplus/spl2sw_desc.c    | 228 +++++++
> >  drivers/net/ethernet/sunplus/spl2sw_desc.h    |  19 +
> >  drivers/net/ethernet/sunplus/spl2sw_driver.c  | 578 ++++++++++++++++++
> >  drivers/net/ethernet/sunplus/spl2sw_int.c     | 273 +++++++++
> >  drivers/net/ethernet/sunplus/spl2sw_int.h     |  13 +
> >  drivers/net/ethernet/sunplus/spl2sw_mac.c     | 274 +++++++++
> >  drivers/net/ethernet/sunplus/spl2sw_mac.h     |  18 +
> >  drivers/net/ethernet/sunplus/spl2sw_mdio.c    | 126 ++++
> >  drivers/net/ethernet/sunplus/spl2sw_mdio.h    |  12 +
> >  drivers/net/ethernet/sunplus/spl2sw_phy.c     |  92 +++
> >  drivers/net/ethernet/sunplus/spl2sw_phy.h     |  12 +
> >  .../net/ethernet/sunplus/spl2sw_register.h    |  86 +++
> >  18 files changed, 2045 insertions(+)
> >  create mode 100644 drivers/net/ethernet/sunplus/Kconfig
> >  create mode 100644 drivers/net/ethernet/sunplus/Makefile
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_define.h
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_desc.c
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_desc.h
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_driver.c
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_int.c
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_int.h
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_mac.c
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_mac.h
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_mdio.c
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_mdio.h
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_phy.c
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_phy.h
> >  create mode 100644 drivers/net/ethernet/sunplus/spl2sw_register.h
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS index 22a2f9699..11a062c42
> > 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -18890,6 +18890,7 @@ L:	netdev@xxxxxxxxxxxxxxx
> >  S:	Maintained
> >  W:	https://sunplus.atlassian.net/wiki/spaces/doc/overview
> >  F:	Documentation/devicetree/bindings/net/sunplus,sp7021-emac.yaml
> > +F:	drivers/net/ethernet/sunplus/
> >
> >  SUNPLUS OCOTP DRIVER
> >  M:	Vincent Shih <vincent.sunplus@xxxxxxxxx>
> > diff --git a/drivers/net/ethernet/Kconfig
> > b/drivers/net/ethernet/Kconfig index 827993022..955abbc54 100644
> > --- a/drivers/net/ethernet/Kconfig
> > +++ b/drivers/net/ethernet/Kconfig
> > @@ -179,6 +179,7 @@ source "drivers/net/ethernet/smsc/Kconfig"
> >  source "drivers/net/ethernet/socionext/Kconfig"
> >  source "drivers/net/ethernet/stmicro/Kconfig"
> >  source "drivers/net/ethernet/sun/Kconfig"
> > +source "drivers/net/ethernet/sunplus/Kconfig"
> >  source "drivers/net/ethernet/synopsys/Kconfig"
> >  source "drivers/net/ethernet/tehuti/Kconfig"
> >  source "drivers/net/ethernet/ti/Kconfig"
> > diff --git a/drivers/net/ethernet/Makefile
> > b/drivers/net/ethernet/Makefile index 8ef43e0c3..9eb011699 100644
> > --- a/drivers/net/ethernet/Makefile
> > +++ b/drivers/net/ethernet/Makefile
> > @@ -90,6 +90,7 @@ obj-$(CONFIG_NET_VENDOR_SMSC) += smsc/
> >  obj-$(CONFIG_NET_VENDOR_SOCIONEXT) += socionext/
> >  obj-$(CONFIG_NET_VENDOR_STMICRO) += stmicro/
> >  obj-$(CONFIG_NET_VENDOR_SUN) += sun/
> > +obj-$(CONFIG_NET_VENDOR_SUNPLUS) += sunplus/
> >  obj-$(CONFIG_NET_VENDOR_TEHUTI) += tehuti/
> >  obj-$(CONFIG_NET_VENDOR_TI) += ti/
> >  obj-$(CONFIG_NET_VENDOR_TOSHIBA) += toshiba/ diff --git
> > a/drivers/net/ethernet/sunplus/Kconfig
> > b/drivers/net/ethernet/sunplus/Kconfig
> > new file mode 100644
> > index 000000000..d0144a2ab
> > --- /dev/null
> > +++ b/drivers/net/ethernet/sunplus/Kconfig
> > @@ -0,0 +1,35 @@
> > +# SPDX-License-Identifier: GPL-2.0
> > +#
> > +# Sunplus network device configuration #
> > +
> > +config NET_VENDOR_SUNPLUS
> > +	bool "Sunplus devices"
> > +	default y
> > +	depends on ARCH_SUNPLUS || COMPILE_TEST
> > +	help
> > +	  If you have a network (Ethernet) card belonging to this
> > +	  class, say Y here.
> > +
> > +	  Note that the answer to this question doesn't directly
> > +	  affect the kernel: saying N will just cause the configurator
> > +	  to skip all the questions about Sunplus cards. If you say Y,
> > +	  you will be asked for your specific card in the following
> > +	  questions.
> > +
> > +if NET_VENDOR_SUNPLUS
> > +
> > +config SP7021_EMAC
> > +	tristate "Sunplus Dual 10M/100M Ethernet devices"
> > +	depends on SOC_SP7021 || COMPILE_TEST
> > +	select PHYLIB
> > +	select COMMON_CLK_SP7021
> > +	select RESET_SUNPLUS
> > +	select NVMEM_SUNPLUS_OCOTP
> > +	help
> > +	  If you have Sunplus dual 10M/100M Ethernet devices, say Y.
> > +	  The network device creates two net-device interfaces.
> > +	  To compile this driver as a module, choose M here. The
> > +	  module will be called sp7021_emac.
> > +
> > +endif # NET_VENDOR_SUNPLUS
> > diff --git a/drivers/net/ethernet/sunplus/Makefile
> > b/drivers/net/ethernet/sunplus/Makefile
> > new file mode 100644
> > index 000000000..ef7d7d0a7
> > --- /dev/null
> > +++ b/drivers/net/ethernet/sunplus/Makefile
> > @@ -0,0 +1,6 @@
> > +# SPDX-License-Identifier: GPL-2.0
> > +#
> > +# Makefile for the Sunplus network device drivers.
> > +#
> > +obj-$(CONFIG_SP7021_EMAC) += sp7021_emac.o sp7021_emac-objs :=
> > +spl2sw_driver.o spl2sw_int.o spl2sw_desc.o spl2sw_mac.o spl2sw_mdio.o
> > +spl2sw_phy.o
> > diff --git a/drivers/net/ethernet/sunplus/spl2sw_define.h
> > b/drivers/net/ethernet/sunplus/spl2sw_define.h
> > new file mode 100644
> > index 000000000..acc6c1228
> > --- /dev/null
> > +++ b/drivers/net/ethernet/sunplus/spl2sw_define.h
> > @@ -0,0 +1,270 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +/* Copyright Sunplus Technology Co., Ltd.
> > + *       All rights reserved.
> > + */
> > +
> > +#ifndef __SPL2SW_DEFINE_H__
> > +#define __SPL2SW_DEFINE_H__
> > +
> > +#define MAX_NETDEV_NUM			2	/* Maximum # of net-device */
> > +
> > +/* Interrupt status */
> > +#define MAC_INT_DAISY_MODE_CHG		BIT(31) /* Daisy Mode Change             */
> > +#define MAC_INT_IP_CHKSUM_ERR		BIT(23) /* IP Checksum Append Error      */
> > +#define MAC_INT_WDOG_TIMER1_EXP		BIT(22) /* Watchdog Timer1 Expired       */
> > +#define MAC_INT_WDOG_TIMER0_EXP		BIT(21) /* Watchdog Timer0 Expired       */
> > +#define MAC_INT_INTRUDER_ALERT		BIT(20) /* Atruder Alert                 */
> > +#define MAC_INT_PORT_ST_CHG		BIT(19) /* Port Status Change            */
> > +#define MAC_INT_BC_STORM		BIT(18) /* Broad Cast Storm              */
> > +#define MAC_INT_MUST_DROP_LAN		BIT(17) /* Global Queue Exhausted        */
> > +#define MAC_INT_GLOBAL_QUE_FULL		BIT(16) /* Global Queue Full             */
> > +#define MAC_INT_TX_SOC_PAUSE_ON		BIT(15) /* Soc Port TX Pause On          */
> > +#define MAC_INT_RX_SOC_QUE_FULL		BIT(14) /* Soc Port Out Queue Full       */
> > +#define MAC_INT_TX_LAN1_QUE_FULL	BIT(9)  /* Port 1 Out Queue Full         */
> > +#define MAC_INT_TX_LAN0_QUE_FULL	BIT(8)  /* Port 0 Out Queue Full         */
> > +#define MAC_INT_RX_L_DESCF		BIT(7)  /* Low Priority Descriptor Full  */
> > +#define MAC_INT_RX_H_DESCF		BIT(6)  /* High Priority Descriptor Full */
> > +#define MAC_INT_RX_DONE_L		BIT(5)  /* RX Low Priority Done          */
> > +#define MAC_INT_RX_DONE_H		BIT(4)  /* RX High Priority Done         */
> > +#define MAC_INT_TX_DONE_L		BIT(3)  /* TX Low Priority Done          */
> > +#define MAC_INT_TX_DONE_H		BIT(2)  /* TX High Priority Done         */
> > +#define MAC_INT_TX_DES_ERR		BIT(1)  /* TX Descriptor Error           */
> > +#define MAC_INT_RX_DES_ERR		BIT(0)  /* Rx Descriptor Error           */
> > +
> > +#define MAC_INT_RX			(MAC_INT_RX_DONE_H | MAC_INT_RX_DONE_L | \
> > +					MAC_INT_RX_DES_ERR)
> > +#define MAC_INT_TX			(MAC_INT_TX_DONE_L | MAC_INT_TX_DONE_H | \
> > +					MAC_INT_TX_DES_ERR)
> > +#define MAC_INT_MASK_DEF		(MAC_INT_DAISY_MODE_CHG | MAC_INT_IP_CHKSUM_ERR | \
> > +					MAC_INT_WDOG_TIMER1_EXP | MAC_INT_WDOG_TIMER0_EXP | \
> > +					MAC_INT_INTRUDER_ALERT | MAC_INT_PORT_ST_CHG | \
> > +					MAC_INT_BC_STORM | MAC_INT_MUST_DROP_LAN | \
> > +					MAC_INT_GLOBAL_QUE_FULL | MAC_INT_TX_SOC_PAUSE_ON | \
> > +					MAC_INT_RX_SOC_QUE_FULL | MAC_INT_TX_LAN1_QUE_FULL | \
> > +					MAC_INT_TX_LAN0_QUE_FULL | MAC_INT_RX_L_DESCF | \
> > +					MAC_INT_RX_H_DESCF)
> > +
> > +/* Address table search */
> > +#define MAC_ADDR_LOOKUP_IDLE		BIT(2)
> > +#define MAC_SEARCH_NEXT_ADDR		BIT(1)
> > +#define MAC_BEGIN_SEARCH_ADDR		BIT(0)
> > +
> > +/* Address table status */
> > +#define MAC_HASH_LOOKUP_ADDR		GENMASK(31, 22)
> > +#define MAC_R_PORT_MAP			GENMASK(13, 12)
> > +#define MAC_R_CPU_PORT			GENMASK(11, 10)
> > +#define MAC_R_VID			GENMASK(9, 7)
> > +#define MAC_R_AGE			GENMASK(6, 4)
> > +#define MAC_R_PROXY			BIT(3)
> > +#define MAC_R_MC_INGRESS		BIT(2)
> > +#define MAC_AT_TABLE_END		BIT(1)
> > +#define MAC_AT_DATA_READY		BIT(0)
> > +
> > +/* Wt mac ad0 */
> > +#define MAC_W_PORT_MAP			GENMASK(13, 12)
> > +#define MAC_W_LAN_PORT_1		BIT(13)
> > +#define MAC_W_LAN_PORT_0		BIT(12)
> > +#define MAC_W_CPU_PORT			GENMASK(11, 10)
> > +#define MAC_W_CPU_PORT_1		BIT(11)
> > +#define MAC_W_CPU_PORT_0		BIT(10)
> > +#define MAC_W_VID			GENMASK(9, 7)
> > +#define MAC_W_AGE			GENMASK(6, 4)
> > +#define MAC_W_PROXY			BIT(3)
> > +#define MAC_W_MC_INGRESS		BIT(2)
> > +#define MAC_W_MAC_DONE			BIT(1)
> > +#define MAC_W_MAC_CMD			BIT(0)
> > +
> > +/* W mac 15_0 bus */
> > +#define MAC_W_MAC_15_0			GENMASK(15, 0)
> > +
> > +/* W mac 47_16 bus */
> > +#define MAC_W_MAC_47_16			GENMASK(31, 0)
> > +
> > +/* PVID config 0 */
> > +#define MAC_P1_PVID			GENMASK(6, 4)
> > +#define MAC_P0_PVID			GENMASK(2, 0)
> > +
> > +/* VLAN member config 0 */
> > +#define MAC_VLAN_MEMSET_3		GENMASK(27, 24)
> > +#define MAC_VLAN_MEMSET_2		GENMASK(19, 16)
> > +#define MAC_VLAN_MEMSET_1		GENMASK(11, 8)
> > +#define MAC_VLAN_MEMSET_0		GENMASK(3, 0)
> > +
> > +/* VLAN member config 1 */
> > +#define MAC_VLAN_MEMSET_5		GENMASK(11, 8)
> > +#define MAC_VLAN_MEMSET_4		GENMASK(3, 0)
> > +
> > +/* Port ability */
> > +#define MAC_PORT_ABILITY_LINK_ST	GENMASK(25, 24)
> > +
> > +/* CPU control */
> > +#define MAC_EN_SOC1_AGING		BIT(15)
> > +#define MAC_EN_SOC0_AGING		BIT(14)
> > +#define MAC_DIS_LRN_SOC1		BIT(13)
> > +#define MAC_DIS_LRN_SOC0		BIT(12)
> > +#define MAC_EN_CRC_SOC1			BIT(9)
> > +#define MAC_EN_CRC_SOC0			BIT(8)
> > +#define MAC_DIS_SOC1_CPU		BIT(7)
> > +#define MAC_DIS_SOC0_CPU		BIT(6)
> > +#define MAC_DIS_BC2CPU_P1		BIT(5)
> > +#define MAC_DIS_BC2CPU_P0		BIT(4)
> > +#define MAC_DIS_MC2CPU			GENMASK(3, 2)
> > +#define MAC_DIS_MC2CPU_P1		BIT(3)
> > +#define MAC_DIS_MC2CPU_P0		BIT(2)
> > +#define MAC_DIS_UN2CPU			GENMASK(1, 0)
> > +
> > +/* Port control 0 */
> > +#define MAC_DIS_PORT			GENMASK(25, 24)
> > +#define MAC_DIS_PORT1			BIT(25)
> > +#define MAC_DIS_PORT0			BIT(24)
> > +#define MAC_DIS_RMC2CPU_P1		BIT(17)
> > +#define MAC_DIS_RMC2CPU_P0		BIT(16)
> > +#define MAC_EN_FLOW_CTL_P1		BIT(9)
> > +#define MAC_EN_FLOW_CTL_P0		BIT(8)
> > +#define MAC_EN_BACK_PRESS_P1		BIT(1)
> > +#define MAC_EN_BACK_PRESS_P0		BIT(0)
> > +
> > +/* Port control 1 */
> > +#define MAC_DIS_SA_LRN_P1		BIT(9)
> > +#define MAC_DIS_SA_LRN_P0		BIT(8)
> > +
> > +/* Port control 2 */
> > +#define MAC_EN_AGING_P1			BIT(9)
> > +#define MAC_EN_AGING_P0			BIT(8)
> > +
> > +/* Switch Global control */
> > +#define MAC_RMC_TB_FAULT_RULE		GENMASK(26, 25)
> > +#define MAC_LED_FLASH_TIME		GENMASK(24, 23)
> > +#define MAC_BC_STORM_PREV		GENMASK(5, 4)
> > +
> > +/* LED port 0 */
> > +#define MAC_LED_ACT_HI			BIT(28)
> > +
> > +/* PHY control register 0  */
> > +#define MAC_CPU_PHY_WT_DATA		GENMASK(31, 16)
> > +#define MAC_CPU_PHY_CMD			GENMASK(14, 13)
> > +#define MAC_CPU_PHY_REG_ADDR		GENMASK(12, 8)
> > +#define MAC_CPU_PHY_ADDR		GENMASK(4, 0)
> > +
> > +/* PHY control register 1 */
> > +#define MAC_CPU_PHY_RD_DATA		GENMASK(31, 16)
> > +#define MAC_PHY_RD_RDY			BIT(1)
> > +#define MAC_PHY_WT_DONE			BIT(0)
> > +
> > +/* MAC force mode */
> > +#define MAC_EXT_PHY1_ADDR		GENMASK(28, 24)
> > +#define MAC_EXT_PHY0_ADDR		GENMASK(20, 16)
> > +#define MAC_FORCE_RMII_LINK		GENMASK(9, 8)
> > +#define MAC_FORCE_RMII_EN_1		BIT(7)
> > +#define MAC_FORCE_RMII_EN_0		BIT(6)
> > +#define MAC_FORCE_RMII_FC		GENMASK(5, 4)
> > +#define MAC_FORCE_RMII_DPX		GENMASK(3, 2)
> > +#define MAC_FORCE_RMII_SPD		GENMASK(1, 0)
> > +
> > +/* CPU transmit trigger */
> > +#define MAC_TRIG_L_SOC0			BIT(1)
> > +#define MAC_TRIG_H_SOC0			BIT(0)
> > +
> > +/* Config descriptor queue */
> > +#define TX_DESC_NUM			16	/* # of descriptors in TX queue   */
> > +#define MAC_GUARD_DESC_NUM		2	/* # of descriptors of gap      0 */
> > +#define RX_QUEUE0_DESC_NUM		16	/* # of descriptors in RX queue 0 */
> > +#define RX_QUEUE1_DESC_NUM		16	/* # of descriptors in RX queue 1 */
> > +#define TX_DESC_QUEUE_NUM		1	/* # of TX queue                  */
> > +#define RX_DESC_QUEUE_NUM		2	/* # of RX queue                  */
> > +
> > +#define MAC_RX_LEN_MAX			2047	/* Size of RX buffer       */
> > +
> > +/* Tx descriptor */
> > +/* cmd1 */
> > +#define TXD_OWN				BIT(31)
> > +#define TXD_ERR_CODE			GENMASK(29, 26)
> > +#define TXD_SOP				BIT(25)		/* start of a packet */
> > +#define TXD_EOP				BIT(24)		/* end of a packet */
> > +#define TXD_VLAN			GENMASK(17, 12)
> > +#define TXD_PKT_LEN			GENMASK(10, 0)	/* packet length */
> > +/* cmd2 */
> > +#define TXD_EOR				BIT(31)		/* end of ring */
> > +#define TXD_BUF_LEN2			GENMASK(22, 12)
> > +#define TXD_BUF_LEN1			GENMASK(10, 0)
> > +
> > +/* Rx descriptor */
> > +/* cmd1 */
> > +#define RXD_OWN				BIT(31)
> > +#define RXD_ERR_CODE			GENMASK(29, 26)
> > +#define RXD_TCP_UDP_CHKSUM		BIT(23)
> > +#define RXD_PROXY			BIT(22)
> > +#define RXD_PROTOCOL			GENMASK(21, 20)
> > +#define RXD_VLAN_TAG			BIT(19)
> > +#define RXD_IP_CHKSUM			BIT(18)
> > +#define RXD_ROUTE_TYPE			GENMASK(17, 16)
> > +#define RXD_PKT_SP			GENMASK(14, 12)	/* packet source port */
> > +#define RXD_PKT_LEN			GENMASK(10, 0)	/* packet length */
> > +/* cmd2 */
> > +#define RXD_EOR				BIT(31)		/* end of ring */
> > +#define RXD_BUF_LEN2			GENMASK(22, 12)
> > +#define RXD_BUF_LEN1			GENMASK(10, 0)
> > +
> > +/* structure of descriptor */
> > +struct spl2sw_mac_desc {
> > +	u32 cmd1;
> > +	u32 cmd2;
> > +	u32 addr1;
> > +	u32 addr2;
> > +};
> > +
> > +struct spl2sw_skb_info {
> > +	struct sk_buff *skb;
> > +	u32 mapping;
> > +	u32 len;
> > +};
> > +
> > +struct spl2sw_common {
> > +	void __iomem *l2sw_reg_base;
> > +
> > +	struct platform_device *pdev;
> > +	struct reset_control *rstc;
> > +	struct clk *clk;
> > +
> > +	void *desc_base;
> > +	dma_addr_t desc_dma;
> > +	s32 desc_size;
> > +	struct spl2sw_mac_desc *rx_desc[RX_DESC_QUEUE_NUM];
> > +	struct spl2sw_skb_info *rx_skb_info[RX_DESC_QUEUE_NUM];
> > +	u32 rx_pos[RX_DESC_QUEUE_NUM];
> > +	u32 rx_desc_num[RX_DESC_QUEUE_NUM];
> > +	u32 rx_desc_buff_size;
> > +
> > +	struct spl2sw_mac_desc *tx_desc;
> > +	struct spl2sw_skb_info tx_temp_skb_info[TX_DESC_NUM];
> > +	u32 tx_done_pos;
> > +	u32 tx_pos;
> > +	u32 tx_desc_full;
> > +
> > +	struct net_device *ndev[MAX_NETDEV_NUM];
> > +	struct mii_bus *mii_bus;
> > +
> > +	struct napi_struct rx_napi;
> > +	struct napi_struct tx_napi;
> > +
> > +	spinlock_t tx_lock;		/* spinlock for accessing tx buffer */
> > +	spinlock_t mdio_lock;		/* spinlock for mdio commands */
> > +	spinlock_t int_mask_lock;	/* spinlock for accessing int mask reg. */
> > +
> > +	u8 enable;
> > +};
> > +
> > +struct spl2sw_mac {
> > +	struct net_device *ndev;
> > +	struct spl2sw_common *comm;
> > +
> > +	u8 mac_addr[ETH_ALEN];
> > +	phy_interface_t phy_mode;
> > +	struct device_node *phy_node;
> > +
> > +	u8 lan_port;
> > +	u8 to_vlan;
> > +	u8 vlan_id;
> > +};
> > +
> > +#endif
> > diff --git a/drivers/net/ethernet/sunplus/spl2sw_desc.c
> > b/drivers/net/ethernet/sunplus/spl2sw_desc.c
> > new file mode 100644
> > index 000000000..3f0d9f78b
> > --- /dev/null
> > +++ b/drivers/net/ethernet/sunplus/spl2sw_desc.c
> > @@ -0,0 +1,228 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/* Copyright Sunplus Technology Co., Ltd.
> > + *       All rights reserved.
> > + */
> > +
> > +#include <linux/platform_device.h>
> > +#include <linux/netdevice.h>
> > +#include <linux/of_mdio.h>
> > +
> > +#include "spl2sw_define.h"
> > +#include "spl2sw_desc.h"
> > +
> > +void spl2sw_rx_descs_flush(struct spl2sw_common *comm) {
> > +	struct spl2sw_skb_info *rx_skbinfo;
> > +	struct spl2sw_mac_desc *rx_desc;
> > +	u32 i, j;
> > +
> > +	for (i = 0; i < RX_DESC_QUEUE_NUM; i++) {
> > +		rx_desc = comm->rx_desc[i];
> > +		rx_skbinfo = comm->rx_skb_info[i];
> > +		for (j = 0; j < comm->rx_desc_num[i]; j++) {
> > +			rx_desc[j].addr1 = rx_skbinfo[j].mapping;
> > +			rx_desc[j].cmd2 = (j == comm->rx_desc_num[i] - 1) ?
> > +					  RXD_EOR | comm->rx_desc_buff_size :
> > +					  comm->rx_desc_buff_size;
> > +			wmb();	/* Set RXD_OWN after other fields are ready. */
> > +			rx_desc[j].cmd1 = RXD_OWN;
> > +		}
> > +	}
> > +}
> > +
> > +void spl2sw_tx_descs_clean(struct spl2sw_common *comm) {
> > +	u32 i;
> > +
> > +	if (!comm->tx_desc)
> > +		return;
> > +
> > +	for (i = 0; i < TX_DESC_NUM; i++) {
> > +		comm->tx_desc[i].cmd1 = 0;
> > +		wmb();	/* Clear TXD_OWN and then set other fields. */
> > +		comm->tx_desc[i].cmd2 = 0;
> > +		comm->tx_desc[i].addr1 = 0;
> > +		comm->tx_desc[i].addr2 = 0;
> > +
> > +		if (comm->tx_temp_skb_info[i].mapping) {
> > +			dma_unmap_single(&comm->pdev->dev, comm->tx_temp_skb_info[i].mapping,
> > +					 comm->tx_temp_skb_info[i].skb->len, DMA_TO_DEVICE);
> > +			comm->tx_temp_skb_info[i].mapping = 0;
> > +		}
> > +
> > +		if (comm->tx_temp_skb_info[i].skb) {
> > +			dev_kfree_skb_any(comm->tx_temp_skb_info[i].skb);
> > +			comm->tx_temp_skb_info[i].skb = NULL;
> > +		}
> > +	}
> > +}
> > +
> > +void spl2sw_rx_descs_clean(struct spl2sw_common *comm) {
> > +	struct spl2sw_skb_info *rx_skbinfo;
> > +	struct spl2sw_mac_desc *rx_desc;
> > +	u32 i, j;
> > +
> > +	for (i = 0; i < RX_DESC_QUEUE_NUM; i++) {
> > +		if (!comm->rx_skb_info[i])
> > +			continue;
> > +
> > +		rx_desc = comm->rx_desc[i];
> > +		rx_skbinfo = comm->rx_skb_info[i];
> > +		for (j = 0; j < comm->rx_desc_num[i]; j++) {
> > +			rx_desc[j].cmd1 = 0;
> > +			wmb();	/* Clear RXD_OWN and then set other fields. */
> > +			rx_desc[j].cmd2 = 0;
> > +			rx_desc[j].addr1 = 0;
> > +
> > +			if (rx_skbinfo[j].skb) {
> > +				dma_unmap_single(&comm->pdev->dev, rx_skbinfo[j].mapping,
> > +						 comm->rx_desc_buff_size, DMA_FROM_DEVICE);
> > +				dev_kfree_skb_any(rx_skbinfo[j].skb);
> > +				rx_skbinfo[j].skb = NULL;
> > +				rx_skbinfo[j].mapping = 0;
> > +			}
> > +		}
> > +
> > +		kfree(rx_skbinfo);
> > +		comm->rx_skb_info[i] = NULL;
> > +	}
> > +}
> > +
> > +void spl2sw_descs_clean(struct spl2sw_common *comm) {
> > +	spl2sw_rx_descs_clean(comm);
> > +	spl2sw_tx_descs_clean(comm);
> > +}
> > +
> > +void spl2sw_descs_free(struct spl2sw_common *comm) {
> > +	u32 i;
> > +
> > +	spl2sw_descs_clean(comm);
> > +	comm->tx_desc = NULL;
> > +	for (i = 0; i < RX_DESC_QUEUE_NUM; i++)
> > +		comm->rx_desc[i] = NULL;
> > +
> > +	/*  Free descriptor area  */
> > +	if (comm->desc_base) {
> > +		dma_free_coherent(&comm->pdev->dev, comm->desc_size, comm->desc_base,
> > +				  comm->desc_dma);
> > +		comm->desc_base = NULL;
> > +		comm->desc_dma = 0;
> > +		comm->desc_size = 0;
> > +	}
> > +}
> > +
> > +void spl2sw_tx_descs_init(struct spl2sw_common *comm) {
> > +	memset(comm->tx_desc, '\0', sizeof(struct spl2sw_mac_desc) *
> > +	       (TX_DESC_NUM + MAC_GUARD_DESC_NUM)); }
> > +
> > +int spl2sw_rx_descs_init(struct spl2sw_common *comm) {
> > +	struct spl2sw_skb_info *rx_skbinfo;
> > +	struct spl2sw_mac_desc *rx_desc;
> > +	struct sk_buff *skb;
> > +	u32 mapping;
> > +	u32 i, j;
> > +
> > +	for (i = 0; i < RX_DESC_QUEUE_NUM; i++) {
> > +		comm->rx_skb_info[i] = kcalloc(comm->rx_desc_num[i], sizeof(*rx_skbinfo),
> > +					       GFP_KERNEL | GFP_DMA);
> > +		if (!comm->rx_skb_info[i])
> > +			goto mem_alloc_fail;
> > +
> > +		rx_skbinfo = comm->rx_skb_info[i];
> > +		rx_desc = comm->rx_desc[i];
> > +		for (j = 0; j < comm->rx_desc_num[i]; j++) {
> > +			skb = netdev_alloc_skb(NULL, comm->rx_desc_buff_size);
> > +			if (!skb)
> > +				goto mem_alloc_fail;
> > +
> > +			rx_skbinfo[j].skb = skb;
> > +			mapping = dma_map_single(&comm->pdev->dev, skb->data,
> > +						 comm->rx_desc_buff_size,
> > +						 DMA_FROM_DEVICE);
> > +			if (dma_mapping_error(&comm->pdev->dev, mapping))
> > +				goto mem_alloc_fail;
> > +
> > +			rx_skbinfo[j].mapping = mapping;
> > +			rx_desc[j].addr1 = mapping;
> > +			rx_desc[j].addr2 = 0;
> > +			rx_desc[j].cmd2 = (j == comm->rx_desc_num[i] - 1) ?
> > +					  RXD_EOR | comm->rx_desc_buff_size :
> > +					  comm->rx_desc_buff_size;
> > +			wmb();	/* Set RXD_OWN after other fields are effective. */
> > +			rx_desc[j].cmd1 = RXD_OWN;
> > +		}
> > +	}
> > +
> > +	return 0;
> > +
> > +mem_alloc_fail:
> > +	spl2sw_rx_descs_clean(comm);
> > +	return -ENOMEM;
> > +}
> > +
> > +int spl2sw_descs_alloc(struct spl2sw_common *comm) {
> > +	s32 desc_size;
> > +	u32 i;
> > +
> > +	/* Alloc descriptor area  */
> > +	desc_size = (TX_DESC_NUM + MAC_GUARD_DESC_NUM) * sizeof(struct spl2sw_mac_desc);
> > +	for (i = 0; i < RX_DESC_QUEUE_NUM; i++)
> > +		desc_size += comm->rx_desc_num[i] * sizeof(struct spl2sw_mac_desc);
> > +
> > +	comm->desc_base = dma_alloc_coherent(&comm->pdev->dev, desc_size,
> &comm->desc_dma,
> > +					     GFP_KERNEL);
> > +	if (!comm->desc_base)
> > +		return -ENOMEM;
> > +
> > +	comm->desc_size = desc_size;
> > +
> > +	/* Setup Tx descriptor */
> > +	comm->tx_desc = comm->desc_base;
> > +
> > +	/* Setup Rx descriptor */
> > +	comm->rx_desc[0] = &comm->tx_desc[TX_DESC_NUM + MAC_GUARD_DESC_NUM];
> > +	for (i = 1; i < RX_DESC_QUEUE_NUM; i++)
> > +		comm->rx_desc[i] = comm->rx_desc[i - 1] + comm->rx_desc_num[i - 1];
> > +
> > +	return 0;
> > +}
> > +
> > +int spl2sw_descs_init(struct spl2sw_common *comm) {
> > +	u32 i, ret;
> > +
> > +	/* Initialize rx descriptor's data */
> > +	comm->rx_desc_num[0] = RX_QUEUE0_DESC_NUM;
> > +	comm->rx_desc_num[1] = RX_QUEUE1_DESC_NUM;
> > +
> > +	for (i = 0; i < RX_DESC_QUEUE_NUM; i++) {
> > +		comm->rx_desc[i] = NULL;
> > +		comm->rx_skb_info[i] = NULL;
> > +		comm->rx_pos[i] = 0;
> > +	}
> > +	comm->rx_desc_buff_size = MAC_RX_LEN_MAX;
> > +
> > +	/* Initialize tx descriptor's data */
> > +	comm->tx_done_pos = 0;
> > +	comm->tx_desc = NULL;
> > +	comm->tx_pos = 0;
> > +	comm->tx_desc_full = 0;
> > +	for (i = 0; i < TX_DESC_NUM; i++)
> > +		comm->tx_temp_skb_info[i].skb = NULL;
> > +
> > +	/* Allocate tx & rx descriptors. */
> > +	ret = spl2sw_descs_alloc(comm);
> > +	if (ret)
> > +		return ret;
> > +
> > +	spl2sw_tx_descs_init(comm);
> > +
> > +	return spl2sw_rx_descs_init(comm);
> > +}
> > diff --git a/drivers/net/ethernet/sunplus/spl2sw_desc.h
> > b/drivers/net/ethernet/sunplus/spl2sw_desc.h
> > new file mode 100644
> > index 000000000..f04e2d85c
> > --- /dev/null
> > +++ b/drivers/net/ethernet/sunplus/spl2sw_desc.h
> > @@ -0,0 +1,19 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +/* Copyright Sunplus Technology Co., Ltd.
> > + *       All rights reserved.
> > + */
> > +
> > +#ifndef __SPL2SW_DESC_H__
> > +#define __SPL2SW_DESC_H__
> > +
> > +void spl2sw_rx_descs_flush(struct spl2sw_common *comm); void
> > +spl2sw_tx_descs_clean(struct spl2sw_common *comm); void
> > +spl2sw_rx_descs_clean(struct spl2sw_common *comm); void
> > +spl2sw_descs_clean(struct spl2sw_common *comm); void
> > +spl2sw_descs_free(struct spl2sw_common *comm); void
> > +spl2sw_tx_descs_init(struct spl2sw_common *comm); int
> > +spl2sw_rx_descs_init(struct spl2sw_common *comm); int
> > +spl2sw_descs_alloc(struct spl2sw_common *comm); int
> > +spl2sw_descs_init(struct spl2sw_common *comm);
> > +
> > +#endif
> > diff --git a/drivers/net/ethernet/sunplus/spl2sw_driver.c
> > b/drivers/net/ethernet/sunplus/spl2sw_driver.c
> > new file mode 100644
> > index 000000000..8320fa833
> > --- /dev/null
> > +++ b/drivers/net/ethernet/sunplus/spl2sw_driver.c
> > @@ -0,0 +1,578 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/* Copyright Sunplus Technology Co., Ltd.
> > + *       All rights reserved.
> > + */
> > +
> > +#include <linux/platform_device.h>
> > +#include <linux/nvmem-consumer.h>
> > +#include <linux/etherdevice.h>
> > +#include <linux/netdevice.h>
> > +#include <linux/spinlock.h>
> > +#include <linux/of_net.h>
> > +#include <linux/reset.h>
> > +#include <linux/clk.h>
> > +#include <linux/of.h>
> > +
> > +#include "spl2sw_register.h"
> > +#include "spl2sw_define.h"
> > +#include "spl2sw_desc.h"
> > +#include "spl2sw_mdio.h"
> > +#include "spl2sw_phy.h"
> > +#include "spl2sw_int.h"
> > +#include "spl2sw_mac.h"
> > +
> > +/* net device operations */
> > +static int spl2sw_ethernet_open(struct net_device *ndev) {
> > +	struct spl2sw_mac *mac = netdev_priv(ndev);
> > +	struct spl2sw_common *comm = mac->comm;
> > +	u32 mask;
> > +
> > +	netdev_dbg(ndev, "Open port = %x\n", mac->lan_port);
> > +
> > +	comm->enable |= mac->lan_port;
> > +
> > +	spl2sw_mac_hw_start(comm);
> > +
> > +	/* Enable TX and RX interrupts */
> > +	mask = readl(comm->l2sw_reg_base + L2SW_SW_INT_MASK_0);
> > +	mask &= ~(MAC_INT_TX | MAC_INT_RX);
> > +	writel(mask, comm->l2sw_reg_base + L2SW_SW_INT_MASK_0);
> > +
> > +	phy_start(ndev->phydev);
> > +
> > +	netif_start_queue(ndev);
> > +
> > +	return 0;
> > +}
> > +
> > +static int spl2sw_ethernet_stop(struct net_device *ndev) {
> > +	struct spl2sw_mac *mac = netdev_priv(ndev);
> > +	struct spl2sw_common *comm = mac->comm;
> > +
> > +	netif_stop_queue(ndev);
> > +
> > +	comm->enable &= ~mac->lan_port;
> > +
> > +	phy_stop(ndev->phydev);
> > +
> > +	spl2sw_mac_hw_stop(comm);
> > +
> > +	return 0;
> > +}
> > +
> > +static int spl2sw_ethernet_start_xmit(struct sk_buff *skb, struct
> > +net_device *ndev) {
> > +	struct spl2sw_mac *mac = netdev_priv(ndev);
> > +	struct spl2sw_common *comm = mac->comm;
> > +	struct spl2sw_skb_info *skbinfo;
> > +	struct spl2sw_mac_desc *txdesc;
> > +	unsigned long flags;
> > +	u32 mapping;
> > +	u32 tx_pos;
> > +	u32 cmd1;
> > +	u32 cmd2;
> > +
> > +	if (unlikely(comm->tx_desc_full == 1)) {
> > +		/* No TX descriptors left. Wait for tx interrupt. */
> > +		netdev_dbg(ndev, "TX descriptor queue full when xmit!\n");
> > +		return NETDEV_TX_BUSY;
> > +	}
> > +
> > +	/* If skb size is shorter than ETH_ZLEN (60), pad it with 0. */
> > +	if (unlikely(skb->len < ETH_ZLEN)) {
> > +		if (skb_padto(skb, ETH_ZLEN))
> > +			return NETDEV_TX_OK;
> > +
> > +		skb_put(skb, ETH_ZLEN - skb->len);
> > +	}
> > +
> > +	mapping = dma_map_single(&comm->pdev->dev, skb->data,
> > +				 skb->len, DMA_TO_DEVICE);
> > +	if (dma_mapping_error(&comm->pdev->dev, mapping)) {
> > +		ndev->stats.tx_errors++;
> > +		dev_kfree_skb(skb);
> > +		return NETDEV_TX_OK;
> > +	}
> > +
> > +	spin_lock_irqsave(&comm->tx_lock, flags);
> > +
> > +	tx_pos = comm->tx_pos;
> > +	txdesc = &comm->tx_desc[tx_pos];
> > +	skbinfo = &comm->tx_temp_skb_info[tx_pos];
> > +	skbinfo->mapping = mapping;
> > +	skbinfo->len = skb->len;
> > +	skbinfo->skb = skb;
> > +
> > +	/* Set up a TX descriptor */
> > +	cmd1 = TXD_OWN | TXD_SOP | TXD_EOP | (mac->to_vlan << 12) |
> > +	       (skb->len & TXD_PKT_LEN);
> > +	cmd2 = skb->len & TXD_BUF_LEN1;
> > +
> > +	if (tx_pos == (TX_DESC_NUM - 1))
> > +		cmd2 |= TXD_EOR;
> > +
> > +	txdesc->addr1 = skbinfo->mapping;
> > +	txdesc->cmd2 = cmd2;
> > +	wmb();	/* Set TXD_OWN after other fields are effective. */
> > +	txdesc->cmd1 = cmd1;
> > +
> > +	/* Move tx_pos to next position */
> > +	tx_pos = ((tx_pos + 1) == TX_DESC_NUM) ? 0 : tx_pos + 1;
> > +
> > +	if (unlikely(tx_pos == comm->tx_done_pos)) {
> > +		netif_stop_queue(ndev);
> > +		comm->tx_desc_full = 1;
> > +	}
> > +	comm->tx_pos = tx_pos;
> > +	wmb();		/* make sure settings are effective. */
> > +
> > +	/* Trigger mac to transmit */
> > +	writel(MAC_TRIG_L_SOC0, comm->l2sw_reg_base + L2SW_CPU_TX_TRIG);
> > +
> > +	spin_unlock_irqrestore(&comm->tx_lock, flags);
> > +	return NETDEV_TX_OK;
> > +}
> > +
> > +static void spl2sw_ethernet_set_rx_mode(struct net_device *ndev) {
> > +	struct spl2sw_mac *mac = netdev_priv(ndev);
> > +
> > +	spl2sw_mac_rx_mode_set(mac);
> > +}
> > +
> > +static int spl2sw_ethernet_set_mac_address(struct net_device *ndev,
> > +void *addr) {
> > +	struct spl2sw_mac *mac = netdev_priv(ndev);
> > +	int err;
> > +
> > +	err = eth_mac_addr(ndev, addr);
> > +	if (err)
> > +		return err;
> > +
> > +	/* Delete the old MAC address */
> > +	netdev_dbg(ndev, "Old Ethernet (MAC) address = %pM\n", mac->mac_addr);
> > +	if (is_valid_ether_addr(mac->mac_addr)) {
> > +		err = spl2sw_mac_addr_del(mac);
> > +		if (err)
> > +			return err;
> > +	}
> > +
> > +	/* Set the MAC address */
> > +	ether_addr_copy(mac->mac_addr, ndev->dev_addr);
> > +	return spl2sw_mac_addr_add(mac);
> > +}
> > +
> > +static void spl2sw_ethernet_tx_timeout(struct net_device *ndev,
> > +unsigned int txqueue) {
> > +	struct spl2sw_mac *mac = netdev_priv(ndev);
> > +	struct spl2sw_common *comm = mac->comm;
> > +	unsigned long flags;
> > +	int i;
> > +
> > +	netdev_err(ndev, "TX timed out!\n");
> > +	ndev->stats.tx_errors++;
> > +
> > +	spin_lock_irqsave(&comm->tx_lock, flags);
> > +
> > +	for (i = 0; i < MAX_NETDEV_NUM; i++)
> > +		if (comm->ndev[i])
> > +			netif_stop_queue(comm->ndev[i]);
> > +
> > +	spl2sw_mac_soft_reset(comm);
> > +
> > +	/* Accept TX packets again. */
> > +	for (i = 0; i < MAX_NETDEV_NUM; i++)
> > +		if (comm->ndev[i]) {
> > +			netif_trans_update(comm->ndev[i]);
> > +			netif_wake_queue(comm->ndev[i]);
> > +		}
> > +
> > +	spin_unlock_irqrestore(&comm->tx_lock, flags); }
> > +
> > +static const struct net_device_ops netdev_ops = {
> > +	.ndo_open = spl2sw_ethernet_open,
> > +	.ndo_stop = spl2sw_ethernet_stop,
> > +	.ndo_start_xmit = spl2sw_ethernet_start_xmit,
> > +	.ndo_set_rx_mode = spl2sw_ethernet_set_rx_mode,
> > +	.ndo_set_mac_address = spl2sw_ethernet_set_mac_address,
> > +	.ndo_do_ioctl = phy_do_ioctl,
> > +	.ndo_tx_timeout = spl2sw_ethernet_tx_timeout, };
> > +
> > +static void spl2sw_check_mac_vendor_id_and_convert(u8 *mac_addr) {
> > +	u8 tmp;
> > +
> > +	/* Byte order of MAC address of some samples are reversed.
> > +	 * Check vendor id and convert byte order if it is wrong.
> > +	 * OUI of Sunplus: fc:4b:bc
> > +	 */
> > +	if (mac_addr[5] == 0xfc && mac_addr[4] == 0x4b && mac_addr[3] == 0xbc &&
> > +	    (mac_addr[0] != 0xfc || mac_addr[1] != 0x4b || mac_addr[2] != 0xbc)) {
> > +		/* Swap mac_addr[0] and mac_addr[5] */
> > +		tmp = mac_addr[0];
> > +		mac_addr[0] = mac_addr[5];
> > +		mac_addr[5] = tmp;
> > +
> > +		/* Swap mac_addr[1] and mac_addr[4] */
> > +		tmp = mac_addr[1];
> > +		mac_addr[1] = mac_addr[4];
> > +		mac_addr[4] = tmp;
> > +
> > +		/* Swap mac_addr[2] and mac_addr[3] */
> > +		tmp = mac_addr[2];
> > +		mac_addr[2] = mac_addr[3];
> > +		mac_addr[3] = tmp;
> > +	}
> > +}
> > +
> > +static int spl2sw_nvmem_get_mac_address(struct device *dev, struct device_node *np,
> > +					void *addrbuf)
> > +{
> > +	struct nvmem_cell *cell;
> > +	ssize_t len;
> > +	u8 *mac;
> > +
> > +	/* Get nvmem cell of mac-address from dts. */
> > +	cell = of_nvmem_cell_get(np, "mac-address");
> > +	if (IS_ERR(cell))
> > +		return PTR_ERR(cell);
> > +
> > +	/* Read mac address from nvmem cell. */
> > +	mac = nvmem_cell_read(cell, &len);
> > +	nvmem_cell_put(cell);
> > +	if (IS_ERR(mac))
> > +		return PTR_ERR(mac);
> > +
> > +	if (len != ETH_ALEN) {
> > +		kfree(mac);
> > +		dev_info(dev, "Invalid length of mac address in nvmem!\n");
> > +		return -EINVAL;
> > +	}
> > +
> > +	/* Byte order of some samples are reversed.
> > +	 * Convert byte order here.
> > +	 */
> > +	spl2sw_check_mac_vendor_id_and_convert(mac);
> > +
> > +	/* Check if mac address is valid */
> > +	if (!is_valid_ether_addr(mac)) {
> > +		kfree(mac);
> > +		dev_info(dev, "Invalid mac address in nvmem (%pM)!\n", mac);
> > +		return -EINVAL;
> > +	}
> > +
> > +	ether_addr_copy(addrbuf, mac);
> > +	kfree(mac);
> > +	return 0;
> > +}
> > +
> > +static u32 spl2sw_init_netdev(struct platform_device *pdev, u8 *mac_addr,
> > +			      struct net_device **r_ndev)
> > +{
> > +	struct net_device *ndev;
> > +	struct spl2sw_mac *mac;
> > +	int ret;
> > +
> > +	/* Allocate the devices, and also allocate spl2sw_mac,
> > +	 * we can get it by netdev_priv().
> > +	 */
> > +	ndev = devm_alloc_etherdev(&pdev->dev, sizeof(*mac));
> > +	if (!ndev) {
> > +		*r_ndev = NULL;
> > +		return -ENOMEM;
> > +	}
> > +	SET_NETDEV_DEV(ndev, &pdev->dev);
> > +	ndev->netdev_ops = &netdev_ops;
> > +	mac = netdev_priv(ndev);
> > +	mac->ndev = ndev;
> > +	ether_addr_copy(mac->mac_addr, mac_addr);
> > +
> > +	eth_hw_addr_set(ndev, mac_addr);
> > +	dev_info(&pdev->dev, "Ethernet (MAC) address = %pM\n", mac_addr);
> > +
> > +	ret = register_netdev(ndev);
> > +	if (ret) {
> > +		dev_err(&pdev->dev, "Failed to register net device \"%s\"!\n",
> > +			ndev->name);
> > +		free_netdev(ndev);
> > +		*r_ndev = NULL;
> > +		return ret;
> > +	}
> > +	netdev_dbg(ndev, "Registered net device \"%s\" successfully.\n",
> > +ndev->name);
> > +
> > +	*r_ndev = ndev;
> > +	return 0;
> > +}
> > +
> > +static struct device_node *spl2sw_get_eth_child_node(struct
> > +device_node *ether_np, int id) {
> > +	struct device_node *port_np;
> > +	int port_id;
> > +
> > +	for_each_child_of_node(ether_np, port_np) {
> > +		/* It is not a 'port' node, continue. */
> > +		if (strcmp(port_np->name, "port"))
> > +			continue;
> > +
> > +		if (of_property_read_u32(port_np, "reg", &port_id) < 0)
> > +			continue;
> > +
> > +		if (port_id == id)
> > +			return port_np;
> > +	}
> > +
> > +	/* Not found! */
> > +	return NULL;
> > +}
> > +
> > +static int spl2sw_probe(struct platform_device *pdev) {
> > +	struct device_node *eth_ports_np;
> > +	struct device_node *port_np;
> > +	struct spl2sw_common *comm;
> > +	struct device_node *phy_np;
> > +	phy_interface_t phy_mode;
> > +	struct net_device *ndev;
> > +	struct spl2sw_mac *mac;
> > +	u8 mac_addr[ETH_ALEN];
> > +	int irq, i, ret;
> > +
> > +	if (platform_get_drvdata(pdev))
> > +		return -ENODEV;
> > +
> > +	/* Allocate memory for 'spl2sw_common' area. */
> > +	comm = devm_kzalloc(&pdev->dev, sizeof(*comm), GFP_KERNEL);
> > +	if (!comm)
> > +		return -ENOMEM;
> > +
> > +	comm->pdev = pdev;
> > +	platform_set_drvdata(pdev, comm);
> > +
> > +	spin_lock_init(&comm->tx_lock);
> > +	spin_lock_init(&comm->mdio_lock);
> > +	spin_lock_init(&comm->int_mask_lock);
> > +
> > +	/* Get memory resource 0 from dts. */
> > +	comm->l2sw_reg_base = devm_platform_ioremap_resource(pdev, 0);
> > +	if (IS_ERR(comm->l2sw_reg_base))
> > +		return PTR_ERR(comm->l2sw_reg_base);
> > +
> > +	/* Get irq resource from dts. */
> > +	ret = platform_get_irq(pdev, 0);
> > +	if (ret < 0)
> > +		return ret;
> > +	irq = ret;
> > +
> > +	/* Get clock controller. */
> > +	comm->clk = devm_clk_get(&pdev->dev, NULL);
> > +	if (IS_ERR(comm->clk)) {
> > +		dev_err_probe(&pdev->dev, PTR_ERR(comm->clk),
> > +			      "Failed to retrieve clock controller!\n");
> > +		return PTR_ERR(comm->clk);
> > +	}
> > +
> > +	/* Get reset controller. */
> > +	comm->rstc = devm_reset_control_get_exclusive(&pdev->dev, NULL);
> > +	if (IS_ERR(comm->rstc)) {
> > +		dev_err_probe(&pdev->dev, PTR_ERR(comm->rstc),
> > +			      "Failed to retrieve reset controller!\n");
> > +		return PTR_ERR(comm->rstc);
> > +	}
> > +
> > +	/* Enable clock. */
> > +	ret = clk_prepare_enable(comm->clk);
> > +	if (ret)
> > +		return ret;
> > +	udelay(1);
> > +
> > +	/* Reset MAC */
> > +	reset_control_assert(comm->rstc);
> > +	udelay(1);
> > +	reset_control_deassert(comm->rstc);
> > +	usleep_range(1000, 2000);
> > +
> > +	/* Request irq. */
> > +	ret = devm_request_irq(&pdev->dev, irq, spl2sw_ethernet_interrupt, 0,
> > +			       dev_name(&pdev->dev), comm);
> > +	if (ret) {
> > +		dev_err(&pdev->dev, "Failed to request irq #%d!\n", irq);
> > +		goto out_clk_disable;
> > +	}
> > +
> > +	/* Initialize TX and RX descriptors. */
> > +	ret = spl2sw_descs_init(comm);
> > +	if (ret) {
> > +		dev_err(&pdev->dev, "Fail to initialize mac descriptors!\n");
> > +		spl2sw_descs_free(comm);
> > +		goto out_clk_disable;
> > +	}
> > +
> > +	/* Initialize MAC. */
> > +	spl2sw_mac_init(comm);
> > +
> > +	/* Initialize mdio bus */
> > +	ret = spl2sw_mdio_init(comm);
> > +	if (ret) {
> > +		dev_err(&pdev->dev, "Failed to initialize mdio bus!\n");
> > +		goto out_clk_disable;
> > +	}
> > +
> > +	/* Get child node ethernet-ports. */
> > +	eth_ports_np = of_get_child_by_name(pdev->dev.of_node, "ethernet-ports");
> > +	if (!eth_ports_np) {
> > +		dev_err(&pdev->dev, "No ethernet-ports child node found!\n");
> > +		ret = -ENODEV;
> > +		goto out_free_mdio;
> > +	}
> > +
> > +	for (i = 0; i < MAX_NETDEV_NUM; i++) {
> > +		/* Get port@i of node ethernet-ports. */
> > +		port_np = spl2sw_get_eth_child_node(eth_ports_np, i);
> > +		if (!port_np)
> > +			continue;
> > +
> > +		/* Get phy-mode. */
> > +		if (of_get_phy_mode(port_np, &phy_mode)) {
> > +			dev_err(&pdev->dev, "Failed to get phy-mode property of port@%d!\n",
> > +				i);
> > +			continue;
> > +		}
> > +
> > +		/* Get phy-handle. */
> > +		phy_np = of_parse_phandle(port_np, "phy-handle", 0);
> > +		if (!phy_np) {
> > +			dev_err(&pdev->dev, "Failed to get phy-handle property of port@%d!\n",
> > +				i);
> > +			continue;
> > +		}
> > +
> > +		/* Get mac-address from nvmem. */
> > +		ret = spl2sw_nvmem_get_mac_address(&pdev->dev, port_np, mac_addr);
> > +		if (ret == -EPROBE_DEFER) {
> > +			goto out_unregister_dev;
> > +		} else if (ret) {
> > +			dev_info(&pdev->dev, "Generate a random mac address!\n");
> > +			eth_random_addr(mac_addr);
> > +		}
> > +
> > +		/* Initialize the net device. */
> > +		ret = spl2sw_init_netdev(pdev, mac_addr, &ndev);
> > +		if (ret)
> > +			goto out_unregister_dev;
> > +
> > +		ndev->irq = irq;
> > +		comm->ndev[i] = ndev;
> > +		mac = netdev_priv(ndev);
> > +		mac->phy_node = phy_np;
> > +		mac->phy_mode = phy_mode;
> > +		mac->comm = comm;
> > +
> > +		mac->lan_port = 0x1 << i;	/* forward to port i */
> > +		mac->to_vlan = 0x1 << i;	/* vlan group: i     */
> > +		mac->vlan_id = i;		/* vlan group: i     */
> > +
> > +		/* Set MAC address */
> > +		ret = spl2sw_mac_addr_add(mac);
> > +		if (ret)
> > +			goto out_unregister_dev;
> > +
> > +		spl2sw_mac_rx_mode_set(mac);
> > +	}
> > +
> > +	/* Find first valid net device. */
> > +	for (i = 0; i < MAX_NETDEV_NUM; i++) {
> > +		if (comm->ndev[i])
> > +			break;
> > +	}
> > +	if (i >= MAX_NETDEV_NUM) {
> > +		dev_err(&pdev->dev, "No valid ethernet port!\n");
> > +		ret = -ENODEV;
> > +		goto out_free_mdio;
> > +	}
> > +
> > +	/* Save first valid net device */
> > +	ndev = comm->ndev[i];
> > +
> > +	ret = spl2sw_phy_connect(comm);
> > +	if (ret) {
> > +		netdev_err(ndev, "Failed to connect phy!\n");
> > +		goto out_unregister_dev;
> > +	}
> > +
> > +	/* Add and enable napi. */
> > +	netif_napi_add(ndev, &comm->rx_napi, spl2sw_rx_poll, NAPI_POLL_WEIGHT);
> > +	napi_enable(&comm->rx_napi);
> > +	netif_napi_add(ndev, &comm->tx_napi, spl2sw_tx_poll, NAPI_POLL_WEIGHT);
> > +	napi_enable(&comm->tx_napi);
> > +	return 0;
> > +
> > +out_unregister_dev:
> > +	for (i = 0; i < MAX_NETDEV_NUM; i++)
> > +		if (comm->ndev[i])
> > +			unregister_netdev(comm->ndev[i]);
> > +
> > +out_free_mdio:
> > +	spl2sw_mdio_remove(comm);
> > +
> > +out_clk_disable:
> > +	clk_disable_unprepare(comm->clk);
> > +	return ret;
> > +}
> > +
> > +static int spl2sw_remove(struct platform_device *pdev) {
> > +	struct spl2sw_common *comm;
> > +	int i;
> > +
> > +	comm = platform_get_drvdata(pdev);
> > +
> > +	spl2sw_phy_remove(comm);
> > +
> > +	/* Unregister and free net device. */
> > +	for (i = 0; i < MAX_NETDEV_NUM; i++)
> > +		if (comm->ndev[i])
> > +			unregister_netdev(comm->ndev[i]);
> > +
> > +	comm->enable = 0;
> > +	spl2sw_mac_hw_stop(comm);
> > +	spl2sw_descs_free(comm);
> > +
> > +	/* Disable and delete napi. */
> > +	napi_disable(&comm->rx_napi);
> > +	netif_napi_del(&comm->rx_napi);
> > +	napi_disable(&comm->tx_napi);
> > +	netif_napi_del(&comm->tx_napi);
> > +
> > +	spl2sw_mdio_remove(comm);
> > +
> > +	clk_disable_unprepare(comm->clk);
> > +
> > +	return 0;
> > +}
> > +
> > +static const struct of_device_id spl2sw_of_match[] = {
> > +	{.compatible = "sunplus,sp7021-emac"},
> > +	{ /* sentinel */ }
> > +};
> > +
> > +MODULE_DEVICE_TABLE(of, spl2sw_of_match);
> > +
> > +static struct platform_driver spl2sw_driver = {
> > +	.probe = spl2sw_probe,
> > +	.remove = spl2sw_remove,
> > +	.driver = {
> > +		.name = "sp7021_emac",
> > +		.owner = THIS_MODULE,
> > +		.of_match_table = spl2sw_of_match,
> > +	},
> > +};
> > +
> > +module_platform_driver(spl2sw_driver);
> > +
> > +MODULE_AUTHOR("Wells Lu <wellslutw@xxxxxxxxx>");
> > +MODULE_DESCRIPTION("Sunplus Dual 10M/100M Ethernet driver");
> > +MODULE_LICENSE("GPL");
> > diff --git a/drivers/net/ethernet/sunplus/spl2sw_int.c
> > b/drivers/net/ethernet/sunplus/spl2sw_int.c
> > new file mode 100644
> > index 000000000..e8eb30c44
> > --- /dev/null
> > +++ b/drivers/net/ethernet/sunplus/spl2sw_int.c
> > @@ -0,0 +1,273 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/* Copyright Sunplus Technology Co., Ltd.
> > + *       All rights reserved.
> > + */
> > +
> > +#include <linux/platform_device.h>
> > +#include <linux/etherdevice.h>
> > +#include <linux/netdevice.h>
> > +#include <linux/bitfield.h>
> > +#include <linux/spinlock.h>
> > +#include <linux/of_mdio.h>
> > +
> > +#include "spl2sw_register.h"
> > +#include "spl2sw_define.h"
> > +#include "spl2sw_int.h"
> > +
> > +int spl2sw_rx_poll(struct napi_struct *napi, int budget) {
> > +	struct spl2sw_common *comm = container_of(napi, struct spl2sw_common, rx_napi);
> > +	struct spl2sw_mac_desc *desc, *h_desc;
> > +	struct net_device_stats *stats;
> > +	struct sk_buff *skb, *new_skb;
> > +	struct spl2sw_skb_info *sinfo;
> > +	int budget_left = budget;
> > +	unsigned long flags;
> > +	u32 rx_pos, pkg_len;
> > +	u32 num, rx_count;
> > +	s32 queue;
> > +	u32 mask;
> > +	int port;
> > +	u32 cmd;
> > +
> > +	/* Process high-priority queue and then low-priority queue. */
> > +	for (queue = 0; queue < RX_DESC_QUEUE_NUM; queue++) {
> > +		rx_pos = comm->rx_pos[queue];
> > +		rx_count = comm->rx_desc_num[queue];
> > +
> > +		for (num = 0; num < rx_count && budget_left; num++) {
> > +			sinfo = comm->rx_skb_info[queue] + rx_pos;
> > +			desc = comm->rx_desc[queue] + rx_pos;
> > +			cmd = desc->cmd1;
> > +
> > +			if (cmd & RXD_OWN)
> > +				break;
> > +
> > +			port = FIELD_GET(RXD_PKT_SP, cmd);
> > +			if (port < MAX_NETDEV_NUM && comm->ndev[port])
> > +				stats = &comm->ndev[port]->stats;
> > +			else
> > +				goto spl2sw_rx_poll_rec_err;
> > +
> > +			pkg_len = FIELD_GET(RXD_PKT_LEN, cmd);
> > +			if (unlikely((cmd & RXD_ERR_CODE) || pkg_len < ETH_ZLEN + 4)) {
> > +				stats->rx_length_errors++;
> > +				stats->rx_dropped++;
> > +				goto spl2sw_rx_poll_rec_err;
> > +			}
> > +
> > +			dma_unmap_single(&comm->pdev->dev, sinfo->mapping,
> > +					 comm->rx_desc_buff_size, DMA_FROM_DEVICE);
> > +
> > +			skb = sinfo->skb;
> > +			skb_put(skb, pkg_len - 4); /* Minus FCS */
> > +			skb->ip_summed = CHECKSUM_NONE;
> > +			skb->protocol = eth_type_trans(skb, comm->ndev[port]);
> > +			netif_receive_skb(skb);
> > +
> > +			stats->rx_packets++;
> > +			stats->rx_bytes += skb->len;
> > +
> > +			/* Allocate a new skb for receiving. */
> > +			new_skb = netdev_alloc_skb(NULL, comm->rx_desc_buff_size);
> > +			if (unlikely(!new_skb)) {
> > +				desc->cmd2 = (rx_pos == comm->rx_desc_num[queue] - 1) ?
> > +					     RXD_EOR : 0;
> > +				sinfo->skb = NULL;
> > +				sinfo->mapping = 0;
> > +				desc->addr1 = 0;
> > +				goto spl2sw_rx_poll_alloc_err;
> > +			}
> > +
> > +			sinfo->mapping = dma_map_single(&comm->pdev->dev, new_skb->data,
> > +							comm->rx_desc_buff_size,
> > +							DMA_FROM_DEVICE);
> > +			if (dma_mapping_error(&comm->pdev->dev, sinfo->mapping)) {
> > +				dev_kfree_skb_irq(new_skb);
> > +				desc->cmd2 = (rx_pos == comm->rx_desc_num[queue] - 1) ?
> > +					     RXD_EOR : 0;
> > +				sinfo->skb = NULL;
> > +				sinfo->mapping = 0;
> > +				desc->addr1 = 0;
> > +				goto spl2sw_rx_poll_alloc_err;
> > +			}
> > +
> > +			sinfo->skb = new_skb;
> > +			desc->addr1 = sinfo->mapping;
> > +
> > +spl2sw_rx_poll_rec_err:
> > +			desc->cmd2 = (rx_pos == comm->rx_desc_num[queue] - 1) ?
> > +				     RXD_EOR | comm->rx_desc_buff_size :
> > +				     comm->rx_desc_buff_size;
> > +
> > +			wmb();	/* Set RXD_OWN after other fields are effective. */
> > +			desc->cmd1 = RXD_OWN;
> > +
> > +spl2sw_rx_poll_alloc_err:
> > +			/* Move rx_pos to next position */
> > +			rx_pos = ((rx_pos + 1) == comm->rx_desc_num[queue]) ? 0 : rx_pos +
> > +1;
> > +
> > +			budget_left--;
> > +
> > +			/* If there are packets in high-priority queue,
> > +			 * stop processing low-priority queue.
> > +			 */
> > +			if (queue == 1 && !(h_desc->cmd1 & RXD_OWN))
> > +				break;
> > +		}
> > +
> > +		comm->rx_pos[queue] = rx_pos;
> > +
> > +		/* Save pointer to last rx descriptor of high-priority queue. */
> > +		if (queue == 0)
> > +			h_desc = comm->rx_desc[queue] + rx_pos;
> > +	}
> > +
> > +	wmb();	/* make sure settings are effective. */
> 
> It's unclear to me why a memory barrier is needed here. Which writes should be visibile
> before the following? Also the spin lock/unlock pair below implies a full memory barrier.
> 
> > +	spin_lock_irqsave(&comm->int_mask_lock, flags);
> > +	mask = readl(comm->l2sw_reg_base + L2SW_SW_INT_MASK_0);
> > +	mask &= ~MAC_INT_RX;
> > +	writel(mask, comm->l2sw_reg_base + L2SW_SW_INT_MASK_0);
> > +	spin_unlock_irqrestore(&comm->int_mask_lock, flags);
> > +
> > +	napi_complete(napi);
> > +	return budget - budget_left;
> > +}
> > +
> > +int spl2sw_tx_poll(struct napi_struct *napi, int budget) {
> > +	struct spl2sw_common *comm = container_of(napi, struct spl2sw_common, tx_napi);
> > +	struct spl2sw_skb_info *skbinfo;
> > +	struct net_device_stats *stats;
> > +	int budget_left = budget;
> > +	unsigned long flags;
> > +	u32 tx_done_pos;
> > +	u32 mask;
> > +	u32 cmd;
> > +	int i;
> > +
> > +	spin_lock(&comm->tx_lock);
> > +
> > +	tx_done_pos = comm->tx_done_pos;
> > +	while (((tx_done_pos != comm->tx_pos) || (comm->tx_desc_full == 1)) && budget_left)
> {
> > +		cmd = comm->tx_desc[tx_done_pos].cmd1;
> > +		if (cmd & TXD_OWN)
> > +			break;
> > +
> > +		skbinfo = &comm->tx_temp_skb_info[tx_done_pos];
> > +		if (unlikely(!skbinfo->skb))
> > +			goto spl2sw_tx_poll_next;
> > +
> > +		i = ffs(FIELD_GET(TXD_VLAN, cmd)) - 1;
> > +		if (i < MAX_NETDEV_NUM && comm->ndev[i])
> > +			stats = &comm->ndev[i]->stats;
> > +		else
> > +			goto spl2sw_tx_poll_unmap;
> > +
> > +		if (unlikely(cmd & (TXD_ERR_CODE))) {
> > +			stats->tx_errors++;
> > +		} else {
> > +			stats->tx_packets++;
> > +			stats->tx_bytes += skbinfo->len;
> > +		}
> > +
> > +spl2sw_tx_poll_unmap:
> > +		dma_unmap_single(&comm->pdev->dev, skbinfo->mapping, skbinfo->len,
> > +				 DMA_TO_DEVICE);
> > +		skbinfo->mapping = 0;
> > +		dev_kfree_skb_irq(skbinfo->skb);
> > +		skbinfo->skb = NULL;
> > +
> > +spl2sw_tx_poll_next:
> > +		/* Move tx_done_pos to next position */
> > +		tx_done_pos = ((tx_done_pos + 1) == TX_DESC_NUM) ? 0 : tx_done_pos
> > ++ 1;
> > +
> > +		if (comm->tx_desc_full == 1)
> > +			comm->tx_desc_full = 0;
> > +
> > +		budget_left--;
> > +	}
> > +
> > +	comm->tx_done_pos = tx_done_pos;
> > +	if (!comm->tx_desc_full)
> > +		for (i = 0; i < MAX_NETDEV_NUM; i++)
> > +			if (comm->ndev[i])
> > +				if (netif_queue_stopped(comm->ndev[i]))
> > +					netif_wake_queue(comm->ndev[i]);
> > +
> > +	spin_unlock(&comm->tx_lock);
> > +
> > +	wmb();			/* make sure settings are effective. */
> 
> Same here.
> 
> Thanks,
> 
> Paolo

I'll remove the unnecessary memory barriers.

Thank you for your review.


Best regards,
Wells




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux