All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch
@ 2015-01-12 12:08 Codrin Ciubotariu
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 01/11] net/fm: Fix error when FMAN MAC has no PHY Codrin Ciubotariu
                   ` (10 more replies)
  0 siblings, 11 replies; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

This patches add support for VSC9953, a Vitesse L2 Switch IP which is integrated in the T1040/T1020 Freescale SoCs.

About Device:
=============
The Seville Gigabit Ethernet switch core contains eight 10/100/1000 Mbps Ethernet ports and two 10/100/1000/2500 Mbps ports. It provides a rich set of Ethernet switching features such as advanced TCAM-based VLAN and QoS processing as well as security processing using a TCAM.

Switch interfaces:
	-	8 Gigabit switch ports (ports 0 to 7) are external and
		are connected to external PHYs
	-	2 switch ports (ports 8 and 9) of 2.5 G are connected
		(fixed links) to FMan ports (FM1 at DTSEC1 and FM1 at DTSEC2)

Commands Overview:
=============
Commands supported
	- enable/disable a port
	- check a port's link speed, duplexity and status.

Commands syntax
	ethsw port <port_nr> enable|disable
		- enable/disable an l2 switch port
	ethsw port <port_nr> show
		- show an l2 switch port's configuration

	port_nr=0..9; use "all" for all ports

=> ethsw port all show
    Port   Status     Link    Speed   Duplex
       0  enabled     down       10     half
       1  enabled     down       10     half
       2  enabled     down       10     half
       3  enabled       up     1000     full
       4 disabled     down        -     half
       5 disabled     down        -     half
       6 disabled     down        -     half
       7 disabled     down        -     half
       8  enabled       up     2500     full
       9  enabled       up     2500     full
=>

Changes for v2:
	- split the bug fix from the implementation of
		SerDes protocols of L2 Switch ports patch;
	- added debug messages if MDIO reads or writes timeout;
	- added debug messages when reset of VSC9953 switch fails;
	- replaced Copyright and license to the generic one;
	- added patch descriptions;

Changes for v3:
	- replaced the arrays type from 'int' to 'u8' int the
		 fix for mapping of Freescale SerDes protocols;
	- removed "Change-id" lines from almost all the patches;


Codrin Ciubotariu (11):
  net/fm: Fix error when FMAN MAC has no PHY
  arch/powerpc: Fix mapping of Freescale SerDes protocols
  arch/powerpc: Add SGMII support for the L2 Switch ports
  net/fm: Enable FMAN ports if l2switch ports are connected over SGMII
  net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP
  arch/powerpc: Enable VSC9953 driver on T1040 and T1020
  board/T1040qds: Fix lane-to-slot mapping for SerDes protocol 0x89
  board/T1040qds: T1040 FMAN ports FM1 at DTSEC1 and FM1 at DTSEC2 have no
    PHYs
  board/T104xrdb: T1040 FMAN ports FM1 at DTSEC1 and FM1 at DTSEC2 have no
    PHYs
  board/T1040qds: Add VSC9953 support for T1040qds board
  board/T1040rdb: Add VSC9953 support for T1040rdb board

 arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c |  73 ++--
 arch/powerpc/cpu/mpc85xx/t1040_serdes.c        |   8 +-
 arch/powerpc/cpu/mpc8xxx/cpu.c                 |   5 +
 arch/powerpc/include/asm/config_mpc85xx.h      |   3 +
 arch/powerpc/include/asm/fsl_serdes.h          |   7 +
 board/freescale/t1040qds/eth.c                 |  93 +++++
 board/freescale/t104xrdb/eth.c                 |  59 ++-
 doc/README.t1040-l2switch                      |  49 +++
 drivers/net/Makefile                           |   1 +
 drivers/net/fm/eth.c                           |  30 +-
 drivers/net/fm/t1040.c                         |   3 +-
 drivers/net/vsc9953.c                          | 497 +++++++++++++++++++++++++
 include/configs/T1040QDS.h                     |   4 +
 include/configs/T104xRDB.h                     |   6 +
 include/vsc9953.h                              | 402 ++++++++++++++++++++
 15 files changed, 1191 insertions(+), 49 deletions(-)
 create mode 100644 doc/README.t1040-l2switch
 create mode 100644 drivers/net/vsc9953.c
 create mode 100644 include/vsc9953.h

-- 
1.7.11.7

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 01/11] net/fm: Fix error when FMAN MAC has no PHY
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-23  0:37   ` York Sun
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 02/11] arch/powerpc: Fix mapping of Freescale SerDes protocols Codrin Ciubotariu
                   ` (9 subsequent siblings)
  10 siblings, 1 reply; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

U-boot assumes that all FMAN ports have a PHY. Some SoCs (like T1040)
have fixed links. This means that the ports are connected MAC to MAc
and there is no Ethernet PHY attatched. This patch initializes a
FMAN MAC even if it doesn't have a PHY attached.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2: None

Changes for v3: None

 drivers/net/fm/eth.c | 30 ++++++++++++++++++------------
 1 file changed, 18 insertions(+), 12 deletions(-)

diff --git a/drivers/net/fm/eth.c b/drivers/net/fm/eth.c
index f1e39b9..1d1089d 100644
--- a/drivers/net/fm/eth.c
+++ b/drivers/net/fm/eth.c
@@ -410,10 +410,15 @@ static int fm_eth_open(struct eth_device *dev, bd_t *bd)
 	fmc_tx_port_graceful_stop_disable(fm_eth);
 
 #ifdef CONFIG_PHYLIB
-	ret = phy_startup(fm_eth->phydev);
-	if (ret) {
-		printf("%s: Could not initialize\n", fm_eth->phydev->dev->name);
-		return ret;
+	if (fm_eth->phydev) {
+		ret = phy_startup(fm_eth->phydev);
+		if (ret) {
+			printf("%s: Could not initialize\n",
+			       fm_eth->phydev->dev->name);
+			return ret;
+		}
+	} else {
+		return 0;
 	}
 #else
 	fm_eth->phydev->speed = SPEED_1000;
@@ -447,7 +452,8 @@ static void fm_eth_halt(struct eth_device *dev)
 	/* disable bmi Rx port */
 	bmi_rx_port_disable(fm_eth->rx_port);
 
-	phy_shutdown(fm_eth->phydev);
+	if (fm_eth->phydev)
+		phy_shutdown(fm_eth->phydev);
 }
 
 static int fm_eth_send(struct eth_device *dev, void *buf, int len)
@@ -625,11 +631,12 @@ static int init_phy(struct eth_device *dev)
 	if (fm_eth->bus) {
 		phydev = phy_connect(fm_eth->bus, fm_eth->phyaddr, dev,
 					fm_eth->enet_if);
-	}
-
-	if (!phydev) {
-		printf("Failed to connect\n");
-		return -1;
+		if (!phydev) {
+			printf("Failed to connect\n");
+			return -1;
+		}
+	} else {
+		return 0;
 	}
 
 	if (fm_eth->type == FM_ETH_1G_E) {
@@ -711,8 +718,7 @@ int fm_eth_initialize(struct ccsr_fman *reg, struct fm_eth_info *info)
 	if (!fm_eth_startup(fm_eth))
 		return 0;
 
-	if (init_phy(dev))
-		return 0;
+	init_phy(dev);
 
 	/* clear the ethernet address */
 	for (i = 0; i < 6; i++)
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 02/11] arch/powerpc: Fix mapping of Freescale SerDes protocols
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 01/11] net/fm: Fix error when FMAN MAC has no PHY Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-23  0:38   ` York Sun
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 03/11] arch/powerpc: Add SGMII support for the L2 Switch ports Codrin Ciubotariu
                   ` (8 subsequent siblings)
  10 siblings, 1 reply; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

The number of supported serdes protocols on Freescale SoCs
has increased over time. Until now, an u64 variable have been
initialized on boot with the configured protocols. However,
since this number has increased (enum srds_prtcl has more
than 64 values), 64 bits are no longer sufficient to hold track
of all the configured protocols.
This patch replaces the u64 map values with static arrays.
To keep track of the number of serdes protocols, the
SERDES_PRCTL_COUNT vale has been added at the end of
enum srds_prtcl. This value must always be the last one.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2: There is no v1 version of this patch;

Changes for v3:
	- replaced 'int' type of seredesX_prtcl_map[] with 'u8';
	- removed "Change-id" line from comment;

 arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c | 67 ++++++++++++++------------
 arch/powerpc/include/asm/fsl_serdes.h          |  1 +
 2 files changed, 38 insertions(+), 30 deletions(-)

diff --git a/arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c b/arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c
index 5cfae47..c7d9622 100644
--- a/arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c
+++ b/arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c
@@ -15,16 +15,16 @@
 #include "fsl_corenet2_serdes.h"
 
 #ifdef CONFIG_SYS_FSL_SRDS_1
-static u64 serdes1_prtcl_map;
+static u8 serdes1_prtcl_map[SERDES_PRCTL_COUNT];
 #endif
 #ifdef CONFIG_SYS_FSL_SRDS_2
-static u64 serdes2_prtcl_map;
+static u8 serdes2_prtcl_map[SERDES_PRCTL_COUNT];
 #endif
 #ifdef CONFIG_SYS_FSL_SRDS_3
-static u64 serdes3_prtcl_map;
+static u8 serdes3_prtcl_map[SERDES_PRCTL_COUNT];
 #endif
 #ifdef CONFIG_SYS_FSL_SRDS_4
-static u64 serdes4_prtcl_map;
+static u8 serdes4_prtcl_map[SERDES_PRCTL_COUNT];
 #endif
 
 #ifdef DEBUG
@@ -83,19 +83,19 @@ static const char *serdes_prtcl_str[] = {
 
 int is_serdes_configured(enum srds_prtcl device)
 {
-	u64 ret = 0;
+	int ret = 0;
 
 #ifdef CONFIG_SYS_FSL_SRDS_1
-	ret |= (1ULL << device) & serdes1_prtcl_map;
+	ret |= serdes1_prtcl_map[device];
 #endif
 #ifdef CONFIG_SYS_FSL_SRDS_2
-	ret |= (1ULL << device) & serdes2_prtcl_map;
+	ret |= serdes2_prtcl_map[device];
 #endif
 #ifdef CONFIG_SYS_FSL_SRDS_3
-	ret |= (1ULL << device) & serdes3_prtcl_map;
+	ret |= serdes3_prtcl_map[device];
 #endif
 #ifdef CONFIG_SYS_FSL_SRDS_4
-	ret |= (1ULL << device) & serdes4_prtcl_map;
+	ret |= serdes4_prtcl_map[device];
 #endif
 
 	return !!ret;
@@ -171,12 +171,14 @@ int serdes_get_first_lane(u32 sd, enum srds_prtcl device)
 #define BCAP_OVD_MASK		0x10000000
 #define BYP_CAL_MASK		0x02000000
 
-u64 serdes_init(u32 sd, u32 sd_addr, u32 sd_prctl_mask, u32 sd_prctl_shift)
+void serdes_init(u32 sd, u32 sd_addr, u32 sd_prctl_mask, u32 sd_prctl_shift,
+		u8 serdes_prtcl_map[SERDES_PRCTL_COUNT])
 {
 	ccsr_gur_t *gur = (void __iomem *)(CONFIG_SYS_MPC85xx_GUTS_ADDR);
-	u64 serdes_prtcl_map = 0;
 	u32 cfg;
 	int lane;
+
+	memset(serdes_prtcl_map, 0, sizeof(serdes_prtcl_map));
 #ifdef CONFIG_SYS_FSL_ERRATUM_A007186
 	struct ccsr_sfp_regs  __iomem *sfp_regs =
 			(struct ccsr_sfp_regs __iomem *)(CONFIG_SYS_SFP_ADDR);
@@ -312,38 +314,43 @@ u64 serdes_init(u32 sd, u32 sd_addr, u32 sd_prctl_mask, u32 sd_prctl_shift)
 
 	for (lane = 0; lane < SRDS_MAX_LANES; lane++) {
 		enum srds_prtcl lane_prtcl = serdes_get_prtcl(sd, cfg, lane);
-		serdes_prtcl_map |= (1ULL << lane_prtcl);
+		if (unlikely(lane_prtcl >= SERDES_PRCTL_COUNT))
+			debug("Unknown SerDes lane protocol %d\n", lane_prtcl);
+		else
+			serdes_prtcl_map[lane_prtcl] = 1;
 	}
-
-	return serdes_prtcl_map;
 }
 
 void fsl_serdes_init(void)
 {
 
 #ifdef CONFIG_SYS_FSL_SRDS_1
-	serdes1_prtcl_map = serdes_init(FSL_SRDS_1,
-		CONFIG_SYS_FSL_CORENET_SERDES_ADDR,
-		FSL_CORENET2_RCWSR4_SRDS1_PRTCL,
-		FSL_CORENET2_RCWSR4_SRDS1_PRTCL_SHIFT);
+	serdes_init(FSL_SRDS_1,
+		    CONFIG_SYS_FSL_CORENET_SERDES_ADDR,
+		    FSL_CORENET2_RCWSR4_SRDS1_PRTCL,
+		    FSL_CORENET2_RCWSR4_SRDS1_PRTCL_SHIFT,
+		    serdes1_prtcl_map);
 #endif
 #ifdef CONFIG_SYS_FSL_SRDS_2
-	serdes2_prtcl_map = serdes_init(FSL_SRDS_2,
-		CONFIG_SYS_FSL_CORENET_SERDES_ADDR + FSL_SRDS_2 * 0x1000,
-		FSL_CORENET2_RCWSR4_SRDS2_PRTCL,
-		FSL_CORENET2_RCWSR4_SRDS2_PRTCL_SHIFT);
+	serdes_init(FSL_SRDS_2,
+		    CONFIG_SYS_FSL_CORENET_SERDES_ADDR + FSL_SRDS_2 * 0x1000,
+		    FSL_CORENET2_RCWSR4_SRDS2_PRTCL,
+		    FSL_CORENET2_RCWSR4_SRDS2_PRTCL_SHIFT,
+		    serdes2_prtcl_map);
 #endif
 #ifdef CONFIG_SYS_FSL_SRDS_3
-	serdes3_prtcl_map = serdes_init(FSL_SRDS_3,
-		CONFIG_SYS_FSL_CORENET_SERDES_ADDR + FSL_SRDS_3 * 0x1000,
-		FSL_CORENET2_RCWSR4_SRDS3_PRTCL,
-		FSL_CORENET2_RCWSR4_SRDS3_PRTCL_SHIFT);
+	serdes_init(FSL_SRDS_3,
+		    CONFIG_SYS_FSL_CORENET_SERDES_ADDR + FSL_SRDS_3 * 0x1000,
+		    FSL_CORENET2_RCWSR4_SRDS3_PRTCL,
+		    FSL_CORENET2_RCWSR4_SRDS3_PRTCL_SHIFT,
+		    serdes3_prtcl_map);
 #endif
 #ifdef CONFIG_SYS_FSL_SRDS_4
-	serdes4_prtcl_map = serdes_init(FSL_SRDS_4,
-		CONFIG_SYS_FSL_CORENET_SERDES_ADDR + FSL_SRDS_4 * 0x1000,
-		FSL_CORENET2_RCWSR4_SRDS4_PRTCL,
-		FSL_CORENET2_RCWSR4_SRDS4_PRTCL_SHIFT);
+	serdes_init(FSL_SRDS_4,
+		    CONFIG_SYS_FSL_CORENET_SERDES_ADDR + FSL_SRDS_4 * 0x1000,
+		    FSL_CORENET2_RCWSR4_SRDS4_PRTCL,
+		    FSL_CORENET2_RCWSR4_SRDS4_PRTCL_SHIFT,
+		    serdes4_prtcl_map);
 #endif
 
 }
diff --git a/arch/powerpc/include/asm/fsl_serdes.h b/arch/powerpc/include/asm/fsl_serdes.h
index 8e0e190..2dd32c2 100644
--- a/arch/powerpc/include/asm/fsl_serdes.h
+++ b/arch/powerpc/include/asm/fsl_serdes.h
@@ -87,6 +87,7 @@ enum srds_prtcl {
 	SGMII_2500_FM2_DTSEC6,
 	SGMII_2500_FM2_DTSEC9,
 	SGMII_2500_FM2_DTSEC10,
+	SERDES_PRCTL_COUNT	/* Keep this item the last one */
 };
 
 enum srds {
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 03/11] arch/powerpc: Add SGMII support for the L2 Switch ports
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 01/11] net/fm: Fix error when FMAN MAC has no PHY Codrin Ciubotariu
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 02/11] arch/powerpc: Fix mapping of Freescale SerDes protocols Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-23  0:39   ` York Sun
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 04/11] net/fm: Enable FMAN ports if l2switch ports are connected over SGMII Codrin Ciubotariu
                   ` (7 subsequent siblings)
  10 siblings, 1 reply; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

Some Freescale SoCs like T1020 and T1040 have an integrated
L2 Switch. The L2 Switch ports may be connected to Ethernet PHYs
over SGMII and QSGMII.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2:
	- split a bug fix regarding Freescale SerDes protocols
	into a different patch;

Changes for v3: Removed "Change-id" line from comment;

 arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c | 6 ++++++
 arch/powerpc/cpu/mpc85xx/t1040_serdes.c        | 8 ++++----
 arch/powerpc/include/asm/fsl_serdes.h          | 6 ++++++
 3 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c b/arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c
index c7d9622..acb1353 100644
--- a/arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c
+++ b/arch/powerpc/cpu/mpc85xx/fsl_corenet2_serdes.c
@@ -78,6 +78,12 @@ static const char *serdes_prtcl_str[] = {
 	[INTERLAKEN] = "INTERLAKEN",
 	[QSGMII_SW1_A] = "QSGMII_SW1_A",
 	[QSGMII_SW1_B] = "QSGMII_SW1_B",
+	[SGMII_SW1_MAC1] = "SGMII_SW1_MAC1",
+	[SGMII_SW1_MAC2] = "SGMII_SW1_MAC2",
+	[SGMII_SW1_MAC3] = "SGMII_SW1_MAC3",
+	[SGMII_SW1_MAC4] = "SGMII_SW1_MAC4",
+	[SGMII_SW1_MAC5] = "SGMII_SW1_MAC5",
+	[SGMII_SW1_MAC6] = "SGMII_SW1_MAC6",
 };
 #endif
 
diff --git a/arch/powerpc/cpu/mpc85xx/t1040_serdes.c b/arch/powerpc/cpu/mpc85xx/t1040_serdes.c
index d86bb27..d5dccd5 100644
--- a/arch/powerpc/cpu/mpc85xx/t1040_serdes.c
+++ b/arch/powerpc/cpu/mpc85xx/t1040_serdes.c
@@ -33,10 +33,10 @@ static u8 serdes_cfg_tbl[][SRDS_MAX_LANES] = {
 		PCIE2, PCIE2, SGMII_FM1_DTSEC4, SGMII_FM1_DTSEC5},
 	[0x87] = {PCIE1, SGMII_FM1_DTSEC3, SGMII_FM1_DTSEC1, SGMII_FM1_DTSEC2,
 		PCIE2, PCIE3, PCIE4, SGMII_FM1_DTSEC5},
-	[0x89] = {PCIE1, QSGMII_SW1_A, QSGMII_SW1_A, QSGMII_SW1_A,
-		PCIE2, PCIE3, QSGMII_SW1_B, SATA1},
-	[0x8D] = {PCIE1, QSGMII_SW1_A, QSGMII_SW1_A, QSGMII_SW1_A,
-		PCIE2, QSGMII_SW1_B, QSGMII_SW1_B, QSGMII_SW1_B},
+	[0x89] = {PCIE1, SGMII_SW1_MAC3, SGMII_SW1_MAC1, SGMII_SW1_MAC2,
+		PCIE2, PCIE3, SGMII_SW1_MAC4, SATA1},
+	[0x8D] = {PCIE1, SGMII_SW1_MAC3, SGMII_SW1_MAC1, SGMII_SW1_MAC2,
+		PCIE2, SGMII_SW1_MAC6, SGMII_SW1_MAC4, SGMII_SW1_MAC5},
 	[0x8F] = {PCIE1, SGMII_FM1_DTSEC3, SGMII_FM1_DTSEC1, SGMII_FM1_DTSEC2,
 		AURORA, NONE, SGMII_FM1_DTSEC4, SGMII_FM1_DTSEC5},
 	[0xA5] = {PCIE1, SGMII_FM1_DTSEC3, SGMII_FM1_DTSEC1, SGMII_FM1_DTSEC2,
diff --git a/arch/powerpc/include/asm/fsl_serdes.h b/arch/powerpc/include/asm/fsl_serdes.h
index 2dd32c2..45e248e 100644
--- a/arch/powerpc/include/asm/fsl_serdes.h
+++ b/arch/powerpc/include/asm/fsl_serdes.h
@@ -87,6 +87,12 @@ enum srds_prtcl {
 	SGMII_2500_FM2_DTSEC6,
 	SGMII_2500_FM2_DTSEC9,
 	SGMII_2500_FM2_DTSEC10,
+	SGMII_SW1_MAC1,
+	SGMII_SW1_MAC2,
+	SGMII_SW1_MAC3,
+	SGMII_SW1_MAC4,
+	SGMII_SW1_MAC5,
+	SGMII_SW1_MAC6,
 	SERDES_PRCTL_COUNT	/* Keep this item the last one */
 };
 
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 04/11] net/fm: Enable FMAN ports if l2switch ports are connected over SGMII
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
                   ` (2 preceding siblings ...)
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 03/11] arch/powerpc: Add SGMII support for the L2 Switch ports Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-23  0:39   ` York Sun
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP Codrin Ciubotariu
                   ` (6 subsequent siblings)
  10 siblings, 1 reply; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

If SerDes is configured to connect L2 Switch ports from T1040
over SGMII or QSGMII, the two FMAN fixed ports (FM1 at DTSEC1 and FM2 at DTSEC2)
that are connected to two L2 swtch ports must be enabled. These
ports don't have PHYs and must be treated accordingly.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2: None

Changes for v3: Removed "Change-id" line from comment;

 drivers/net/fm/t1040.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/fm/t1040.c b/drivers/net/fm/t1040.c
index d2a097e..0458366 100644
--- a/drivers/net/fm/t1040.c
+++ b/drivers/net/fm/t1040.c
@@ -50,7 +50,8 @@ phy_interface_t fman_port_enet_if(enum fm_port port)
 	switch (port) {
 	case FM1_DTSEC1:
 	case FM1_DTSEC2:
-		if (is_serdes_configured(QSGMII_SW1_A + port - FM1_DTSEC1))
+		if (is_serdes_configured(QSGMII_SW1_A + port - FM1_DTSEC1) ||
+		    is_serdes_configured(SGMII_SW1_MAC1  + port - FM1_DTSEC1))
 			return PHY_INTERFACE_MODE_QSGMII;
 	case FM1_DTSEC3:
 	case FM1_DTSEC4:
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
                   ` (3 preceding siblings ...)
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 04/11] net/fm: Enable FMAN ports if l2switch ports are connected over SGMII Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-15 20:59   ` York Sun
                     ` (3 more replies)
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 06/11] arch/powerpc: Enable VSC9953 driver on T1040 and T1020 Codrin Ciubotariu
                   ` (5 subsequent siblings)
  10 siblings, 4 replies; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

This patch adds a driver for VSC9953 L2 Switch. This Vitesse IP
is integrated in Freescale T1040 and T1020 SoCs.
The L2 switch has 10 Ethernet ports: 2 internal fixed-links
(ports 8 and 9) at 2.5 Gbps and and 8 external ports at 1 Gbps.
The external ports may be connected to PHYs over QSGMII and SGMII.

Commands have also been added to enable/disable a port and to
check a port's link speed, duplexity and status. The commands are:

ethsw port <port_nr> enable|disable - enable/disable an l2 switch port
ethsw port <port_nr> show - show an l2 switch port's configuration

port_nr=0..9; use "all" for all ports

For more detailse please see doc/README.t1040-l2switch

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2:
	- added debug messages if MDIO reads or writes timeout;
	- added debug messages when reset of VSC9953 switch fails;
	- replaced Copyright and license to the generic one;

Changes for v3: Removed "Change-id" line from comment;

 doc/README.t1040-l2switch |  49 +++++
 drivers/net/Makefile      |   1 +
 drivers/net/vsc9953.c     | 497 ++++++++++++++++++++++++++++++++++++++++++++++
 include/vsc9953.h         | 402 +++++++++++++++++++++++++++++++++++++
 4 files changed, 949 insertions(+)
 create mode 100644 doc/README.t1040-l2switch
 create mode 100644 drivers/net/vsc9953.c
 create mode 100644 include/vsc9953.h

diff --git a/doc/README.t1040-l2switch b/doc/README.t1040-l2switch
new file mode 100644
index 0000000..6324f18
--- /dev/null
+++ b/doc/README.t1040-l2switch
@@ -0,0 +1,49 @@
+This file contains information for VSC9953, a Vitesse L2 Switch IP
+which is integrated in the T1040/T1020 Freescale SoCs.
+
+About Device:
+=============
+VSC9953 is an 8-port Gigabit Ethernet switch supports the following features:
+	-	8192 MAC addresses
+	-	Static Address provisioning
+	-	Dynamic learning of MAC addresses and aging
+	-	4096 VLANs
+	-	Independent and shared VLAN learning (IVL, SVL)
+	-	Policing with storm control and MC/BC protection
+	-	IPv4 and IPv6 multicast
+	-	Jumbo frames (9.6 KB)
+	-	Access Control List
+	-	VLAN editing, translation and remarking
+	-	RMON counters per port
+
+Switch interfaces:
+	-	8 Gigabit switch ports (ports 0 to 7) are external and are connected to external PHYs
+	-	2 switch ports (ports 8 and 9) of 2.5 G are connected (fixed links)
+		to FMan ports (FM1 at DTSEC1 and FM1 at DTSEC2)
+
+Commands Overview:
+=============
+Commands supported
+	- enable/disable a port
+	- check a port's link speed, duplexity and status.
+
+Commands syntax
+	ethsw port <port_nr> enable|disable		- enable/disable an l2 switch port
+	ethsw port <port_nr> show			- show an l2 switch port's configuration
+
+	port_nr=0..9; use "all" for all ports
+
+=> ethsw port all show
+    Port   Status     Link    Speed   Duplex
+       0  enabled     down       10     half
+       1  enabled     down       10     half
+       2  enabled     down       10     half
+       3  enabled       up     1000     full
+       4 disabled     down        -     half
+       5 disabled     down        -     half
+       6 disabled     down        -     half
+       7 disabled     down        -     half
+       8  enabled       up     2500     full
+       9  enabled       up     2500     full
+=>
+
diff --git a/drivers/net/Makefile b/drivers/net/Makefile
index fb0cf8c..46c4ac6 100644
--- a/drivers/net/Makefile
+++ b/drivers/net/Makefile
@@ -66,3 +66,4 @@ obj-$(CONFIG_XILINX_LL_TEMAC) += xilinx_ll_temac.o xilinx_ll_temac_mdio.o \
 		xilinx_ll_temac_fifo.o xilinx_ll_temac_sdma.o
 obj-$(CONFIG_ZYNQ_GEM) += zynq_gem.o
 obj-$(CONFIG_FSL_MC_ENET) += fsl_mc/
+obj-$(CONFIG_VSC9953) += vsc9953.o
diff --git a/drivers/net/vsc9953.c b/drivers/net/vsc9953.c
new file mode 100644
index 0000000..9fc3c18
--- /dev/null
+++ b/drivers/net/vsc9953.c
@@ -0,0 +1,497 @@
+/*
+ *  Copyright 2014 Freescale Semiconductor, Inc.
+ *
+ *  SPDX-License-Identifier:      GPL-2.0+
+ *
+ *  Driver for the Vitesse VSC9953 L2 Switch
+ */
+
+#include <asm/io.h>
+#include <asm/fsl_serdes.h>
+#include <fm_eth.h>
+#include <asm/fsl_memac.h>
+#include <vsc9953.h>
+
+static struct vsc9953_info vsc9953_l2sw = {
+		.port[0] = VSC9953_PORT_INFO_INITIALIZER(0),
+		.port[1] = VSC9953_PORT_INFO_INITIALIZER(1),
+		.port[2] = VSC9953_PORT_INFO_INITIALIZER(2),
+		.port[3] = VSC9953_PORT_INFO_INITIALIZER(3),
+		.port[4] = VSC9953_PORT_INFO_INITIALIZER(4),
+		.port[5] = VSC9953_PORT_INFO_INITIALIZER(5),
+		.port[6] = VSC9953_PORT_INFO_INITIALIZER(6),
+		.port[7] = VSC9953_PORT_INFO_INITIALIZER(7),
+		.port[8] = VSC9953_PORT_INFO_INITIALIZER(8),
+		.port[9] = VSC9953_PORT_INFO_INITIALIZER(9),
+};
+
+void vsc9953_port_info_set_mdio(int port, struct mii_dev *bus)
+{
+	if (!VSC9953_PORT_CHECK(port))
+		return;
+
+	vsc9953_l2sw.port[port].bus = bus;
+}
+
+void vsc9953_port_info_set_phy_address(int port, int address)
+{
+	if (!VSC9953_PORT_CHECK(port))
+		return;
+
+	vsc9953_l2sw.port[port].phyaddr = address;
+}
+
+void vsc9953_port_info_set_phy_int(int port, phy_interface_t phy_int)
+{
+	if (!VSC9953_PORT_CHECK(port))
+		return;
+
+	vsc9953_l2sw.port[port].enet_if = phy_int;
+}
+
+void vsc9953_port_enable(int port)
+{
+	if (!VSC9953_PORT_CHECK(port))
+		return;
+
+	vsc9953_l2sw.port[port].enabled = 1;
+}
+
+void vsc9953_port_disable(int port)
+{
+	if (!VSC9953_PORT_CHECK(port))
+		return;
+
+	vsc9953_l2sw.port[port].enabled = 0;
+}
+
+static void vsc9953_mdio_write(struct vsc9953_mii_mng *phyregs, int port_addr,
+		int regnum, int value)
+{
+	int			timeout = 50000;
+
+	out_le32(&phyregs->miimcmd, (0x1 << 31) | ((port_addr & 0x1f) << 25) |
+			((regnum & 0x1f) << 20) | ((value & 0xffff) << 4) |
+			(0x1 << 1));
+	asm("sync");
+
+	while ((in_le32(&phyregs->miimstatus) & 0x8) && --timeout)
+		udelay(1);
+
+	if (timeout == 0)
+		debug("Timeout waiting for MDIO write\n");
+}
+
+static int vsc9953_mdio_read(struct vsc9953_mii_mng *phyregs, int port_addr,
+		int regnum)
+{
+	int			value = 0xFFFF;
+	int			timeout = 50000;
+
+	while ((in_le32(&phyregs->miimstatus) & MIIMIND_OPR_PEND) && --timeout)
+		udelay(1);
+	if (timeout == 0) {
+		debug("Timeout waiting for MDIO operation to finish\n");
+		return value;
+	}
+
+	/* Put the address of the phy, and the register
+	 * number into MIICMD
+	 */
+	out_le32(&phyregs->miimcmd, (0x1 << 31) | ((port_addr & 0x1f) << 25) |
+			((regnum & 0x1f) << 20) | ((value & 0xffff) << 4) |
+			(0x2 << 1));
+
+	timeout = 50000;
+	/* Wait for the the indication that the read is done */
+	while ((in_le32(&phyregs->miimstatus) & 0x8) && --timeout)
+		udelay(1);
+	if (timeout == 0)
+		debug("Timeout waiting for MDIO read\n");
+
+	/* Grab the value read from the PHY */
+	value = in_le32(&phyregs->miimdata);
+
+	if ((value & 0x00030000) == 0)
+		return value & 0x0000ffff;
+
+	return value;
+}
+
+static int init_phy(struct eth_device *dev)
+{
+	struct vsc9953_port_info	*l2sw_port = dev->priv;
+	struct phy_device		*phydev = NULL;
+
+#ifdef CONFIG_PHYLIB
+	if (!l2sw_port->bus)
+		return 0;
+	phydev = phy_connect(l2sw_port->bus, l2sw_port->phyaddr, dev,
+			l2sw_port->enet_if);
+	if (!phydev) {
+		printf("Failed to connect\n");
+		return -1;
+	}
+
+	phydev->supported &= SUPPORTED_10baseT_Half |
+			SUPPORTED_10baseT_Full |
+			SUPPORTED_100baseT_Half |
+			SUPPORTED_100baseT_Full |
+			SUPPORTED_1000baseT_Full;
+	phydev->advertising = phydev->supported;
+
+	l2sw_port->phydev = phydev;
+
+	phy_config(phydev);
+#endif
+
+	return 0;
+}
+
+static int vsc9953_port_init(int port)
+{
+	struct eth_device		*dev;
+
+	/* Internal ports never have a PHY */
+	if (VSC9953_INTERNAL_PORT_CHECK(port))
+		return 0;
+
+	/* alloc eth device */
+	dev = (struct eth_device *)calloc(1, sizeof(struct eth_device));
+	if (!dev)
+		return 1;
+
+	sprintf(dev->name, "SW at PORT%d", port);
+	dev->priv = &vsc9953_l2sw.port[port];
+	dev->init = NULL;
+	dev->halt = NULL;
+	dev->send = NULL;
+	dev->recv = NULL;
+
+	if (init_phy(dev)) {
+		free(dev);
+		return 1;
+	}
+
+	return 0;
+}
+
+void vsc9953_init(bd_t *bis)
+{
+	u32				i, hdx_cfg = 0, phy_addr = 0;
+	int				timeout;
+	struct vsc9953_system_reg	*l2sys_reg;
+	struct vsc9953_qsys_reg		*l2qsys_reg;
+	struct vsc9953_dev_gmii		*l2dev_gmii_reg;
+	struct vsc9953_analyzer		*l2ana_reg;
+	struct vsc9953_devcpu_gcb	*l2dev_gcb;
+
+	l2dev_gmii_reg = (struct vsc9953_dev_gmii *)(VSC9953_OFFSET +
+			VSC9953_DEV_GMII_OFFSET);
+
+	l2ana_reg = (struct vsc9953_analyzer *)(VSC9953_OFFSET +
+			VSC9953_ANA_OFFSET);
+
+	l2sys_reg = (struct vsc9953_system_reg *)(VSC9953_OFFSET +
+			VSC9953_SYS_OFFSET);
+
+	l2qsys_reg = (struct vsc9953_qsys_reg *)(VSC9953_OFFSET +
+			VSC9953_QSYS_OFFSET);
+
+	l2dev_gcb = (struct vsc9953_devcpu_gcb *)(VSC9953_OFFSET +
+			VSC9953_DEVCPU_GCB);
+
+	out_le32(&l2dev_gcb->chip_regs.soft_rst,
+		 CONFIG_VSC9953_SOFT_SWC_RST_ENA);
+	timeout = 50000;
+	while ((in_le32(&l2dev_gcb->chip_regs.soft_rst) &
+			CONFIG_VSC9953_SOFT_SWC_RST_ENA) && --timeout)
+		udelay(1); /* busy wait for vsc9953 soft reset */
+	if (timeout == 0)
+		debug("Timeout waiting for VSC9953 to reset\n");
+
+	out_le32(&l2sys_reg->sys.reset_cfg, CONFIG_VSC9953_MEM_ENABLE |
+		 CONFIG_VSC9953_MEM_INIT);
+
+	timeout = 50000;
+	while ((in_le32(&l2sys_reg->sys.reset_cfg) &
+		CONFIG_VSC9953_MEM_INIT) && --timeout)
+		udelay(1); /* busy wait for vsc9953 memory init */
+	if (timeout == 0)
+		debug("Timeout waiting for VSC9953 memory to initialize\n");
+
+	out_le32(&l2sys_reg->sys.reset_cfg, (in_le32(&l2sys_reg->sys.reset_cfg)
+			| CONFIG_VSC9953_CORE_ENABLE));
+
+	/* VSC9953 Setting to be done once only */
+	out_le32(&l2qsys_reg->sys.ext_cpu_cfg, 0x00000b00);
+
+	for (i = 0; i < VSC9953_MAX_PORTS; i++) {
+		if (vsc9953_port_init(i))
+			printf("Failed to initialize l2switch port %d\n", i);
+
+		/* Enable VSC9953 GMII Ports Port ID 0 - 7 */
+		if (VSC9953_INTERNAL_PORT_CHECK(i)) {
+			out_le32(&l2ana_reg->pfc[i].pfc_cfg,
+				 CONFIG_VSC9953_PFC_FC_QSGMII);
+			out_le32(&l2sys_reg->pause_cfg.mac_fc_cfg[i],
+				 CONFIG_VSC9953_MAC_FC_CFG_QSGMII);
+		} else {
+			out_le32(&l2ana_reg->pfc[i].pfc_cfg,
+				 CONFIG_VSC9953_PFC_FC);
+			out_le32(&l2sys_reg->pause_cfg.mac_fc_cfg[i],
+				 CONFIG_VSC9953_MAC_FC_CFG);
+		}
+		out_le32(&l2dev_gmii_reg->port_mode.clock_cfg,
+			 CONFIG_VSC9953_CLOCK_CFG);
+		out_le32(&l2dev_gmii_reg->mac_cfg_status.mac_ena_cfg,
+			 CONFIG_VSC9953_MAC_ENA_CFG);
+		out_le32(&l2dev_gmii_reg->mac_cfg_status.mac_mode_cfg,
+			 CONFIG_VSC9953_MAC_MODE_CFG);
+		out_le32(&l2dev_gmii_reg->mac_cfg_status.mac_ifg_cfg,
+			 CONFIG_VSC9953_MAC_IFG_CFG);
+		/* mac_hdx_cfg varies with port id*/
+		hdx_cfg = (CONFIG_VSC9953_MAC_HDX_CFG | (i << 16));
+		out_le32(&l2dev_gmii_reg->mac_cfg_status.mac_hdx_cfg, hdx_cfg);
+		out_le32(&l2sys_reg->sys.front_port_mode[i],
+			 CONFIG_VSC9953_FRONT_PORT_MODE);
+		out_le32(&l2qsys_reg->sys.switch_port_mode[i],
+			 CONFIG_VSC9953_PORT_ENA);
+		out_le32(&l2dev_gmii_reg->mac_cfg_status.mac_maxlen_cfg,
+			 CONFIG_VSC9953_MAC_MAX_LEN);
+		out_le32(&l2sys_reg->pause_cfg.pause_cfg[i],
+			 CONFIG_VSC9953_PAUSE_CFG);
+		/* WAIT FOR 2 us*/
+		udelay(2);
+
+		l2dev_gmii_reg = (struct vsc9953_dev_gmii *)(
+				(char *)l2dev_gmii_reg
+				+ T1040_SWITCH_GMII_DEV_OFFSET);
+
+		/* Initialize Lynx PHY Wrappers */
+		phy_addr = 0;
+		if (vsc9953_l2sw.port[i].enet_if ==
+				PHY_INTERFACE_MODE_QSGMII)
+			phy_addr = (i + 0x4) & 0x1F;
+		else if (vsc9953_l2sw.port[i].enet_if ==
+				PHY_INTERFACE_MODE_SGMII)
+			phy_addr = (i + 1) & 0x1F;
+
+		if (phy_addr) {
+			/* SGMII IF mode + AN enable */
+			vsc9953_mdio_write(&l2dev_gcb->mii_mng[0], phy_addr,
+					   0x14, PHY_SGMII_IF_MODE_AN |
+					   PHY_SGMII_IF_MODE_SGMII);
+			/* Dev ability according to SGMII specification */
+			vsc9953_mdio_write(&l2dev_gcb->mii_mng[0], phy_addr,
+					   0x4, PHY_SGMII_DEV_ABILITY_SGMII);
+			/* Adjust link timer for SGMII
+			 * 1.6 ms in units of 8 ns = 2 * 10^5 = 0x30d40
+			 */
+			vsc9953_mdio_write(&l2dev_gcb->mii_mng[0], phy_addr,
+					   0x13, 0x0003);
+			vsc9953_mdio_write(&l2dev_gcb->mii_mng[0], phy_addr,
+					   0x12, 0x0d40);
+			/* Restart AN */
+			vsc9953_mdio_write(&l2dev_gcb->mii_mng[0], phy_addr,
+					   0x0, PHY_SGMII_CR_DEF_VAL |
+					   PHY_SGMII_CR_RESET_AN);
+
+			timeout = 50000;
+			while ((vsc9953_mdio_read(&l2dev_gcb->mii_mng[0],
+					phy_addr, 0x01) & 0x0020) && --timeout)
+				udelay(1); /* wait for AN to complete */
+			if (timeout == 0)
+				debug("Timeout waiting for AN to complete\n");
+		}
+	}
+
+	printf("VSC9953 L2 switch initialized\n");
+	return;
+}
+
+#ifdef CONFIG_VSC9953_CMD
+/* Enable/disable status of a VSC9953 port */
+static void vsc9953_port_status_set(int port_nr, u8 enabled)
+{
+	u32			val;
+	struct vsc9953_qsys_reg	*l2qsys_reg;
+
+	/* Administrative down */
+	if (vsc9953_l2sw.port[port_nr].enabled == 0)
+		return;
+
+	l2qsys_reg = (struct vsc9953_qsys_reg *)(VSC9953_OFFSET +
+			VSC9953_QSYS_OFFSET);
+
+	val = in_le32(&l2qsys_reg->sys.switch_port_mode[port_nr]);
+	if (enabled == 1)
+		val |= (1 << 13);
+	else
+		val &= ~(1 << 13);
+
+	out_le32(&l2qsys_reg->sys.switch_port_mode[port_nr], val);
+}
+
+/* Set all VSC9953 ports' status */
+static void vsc9953_port_all_status_set(u8 enabled)
+{
+	int		i;
+
+	for (i = 0; i < VSC9953_MAX_PORTS; i++)
+		vsc9953_port_status_set(i, enabled);
+}
+
+/* Start autonegotiation for a VSC9953 PHY */
+static void vsc9953_phy_autoneg(int port_nr)
+{
+	if (!vsc9953_l2sw.port[port_nr].phydev)
+		return;
+
+	if (vsc9953_l2sw.port[port_nr].phydev->drv->startup(
+			vsc9953_l2sw.port[port_nr].phydev))
+		printf("Failed to start PHY for port %d\n", port_nr);
+}
+
+/* Start autonegotiation for all VSC9953 PHYs */
+static void vsc9953_phy_all_autoneg(void)
+{
+	int		i;
+
+	for (i = 0; i < VSC9953_MAX_PORTS; i++)
+		vsc9953_phy_autoneg(i);
+}
+
+/* Print a VSC9953 port's configuration */
+static void vsc9953_port_config_show(int port)
+{
+	int			speed;
+	int			duplex;
+	int			link;
+	u8			enabled;
+	u32			val;
+	struct vsc9953_qsys_reg	*l2qsys_reg;
+
+	l2qsys_reg = (struct vsc9953_qsys_reg *)(VSC9953_OFFSET +
+			VSC9953_QSYS_OFFSET);
+
+	val = in_le32(&l2qsys_reg->sys.switch_port_mode[port]);
+	enabled = vsc9953_l2sw.port[port].enabled &
+			((val & 0x00002000) >> 13);
+
+	/* internal ports (8 and 9) are fixed */
+	if (VSC9953_INTERNAL_PORT_CHECK(port)) {
+		link = 1;
+		speed = SPEED_2500;
+		duplex = DUPLEX_FULL;
+	} else {
+		if (vsc9953_l2sw.port[port].phydev) {
+			link = vsc9953_l2sw.port[port].phydev->link;
+			speed = vsc9953_l2sw.port[port].phydev->speed;
+			duplex = vsc9953_l2sw.port[port].phydev->duplex;
+		} else {
+			link = -1;
+			speed = -1;
+			duplex = -1;
+		}
+	}
+
+	printf("%8d ", port);
+	printf("%8s ", enabled == 1 ? "enabled" : "disabled");
+	printf("%8s ", link == 1 ? "up" : "down");
+
+	switch (speed) {
+	case SPEED_10:
+		printf("%8d ", 10);
+		break;
+	case SPEED_100:
+		printf("%8d ", 100);
+		break;
+	case SPEED_1000:
+		printf("%8d ", 1000);
+		break;
+	case SPEED_2500:
+		printf("%8d ", 2500);
+		break;
+	case SPEED_10000:
+		printf("%8d ", 10000);
+		break;
+	default:
+		printf("%8s ", "-");
+	}
+
+	printf("%8s\n", duplex == DUPLEX_FULL ? "full" : "half");
+}
+
+/* Print VSC9953 ports' configuration */
+static void vsc9953_port_all_config_show(void)
+{
+	int		i;
+
+	for (i = 0; i < VSC9953_MAX_PORTS; i++)
+		vsc9953_port_config_show(i);
+}
+
+/* function to interpret commands starting with "ethsw " */
+static int do_ethsw(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+{
+	u8 enable;
+	u32 port;
+
+	if (argc < 4)
+		return -1;
+
+	if (strcmp(argv[1], "port"))
+		return -1;
+
+	if (!strcmp(argv[3], "show")) {
+		if (!strcmp(argv[2], "all")) {
+			vsc9953_phy_all_autoneg();
+			printf("%8s %8s %8s %8s %8s\n",
+			       "Port", "Status", "Link", "Speed",
+			       "Duplex");
+			vsc9953_port_all_config_show();
+			return 0;
+		} else {
+			port = simple_strtoul(argv[2], NULL, 10);
+			if (!VSC9953_PORT_CHECK(port))
+				return -1;
+			vsc9953_phy_autoneg(port);
+			printf("%8s %8s %8s %8s %8s\n",
+			       "Port", "Status", "Link", "Speed",
+			       "Duplex");
+			vsc9953_port_config_show(port);
+			return 0;
+		}
+	} else if (!strcmp(argv[3], "enable")) {
+		enable = 1;
+	} else if (!strcmp(argv[3], "disable")) {
+		enable = 0;
+	} else {
+		return -1;
+	}
+
+	if (!strcmp(argv[2], "all")) {
+		vsc9953_port_all_status_set(enable);
+		return 0;
+	} else {
+		port = simple_strtoul(argv[2], NULL, 10);
+		if (!VSC9953_PORT_CHECK(port))
+			return -1;
+		vsc9953_port_status_set(port, enable);
+		return 0;
+	}
+
+	return -1;
+}
+
+U_BOOT_CMD(ethsw, 5, 0, do_ethsw,
+	   "vsc9953 l2 switch commands",
+	   "port <port_nr> enable|disable\n"
+	   "    - enable/disable an l2 switch port\n"
+	   "      port_nr=0..9; use \"all\" for all ports\n"
+	   "ethsw port <port_nr> show\n"
+	   "    - show an l2 switch port's configuration\n"
+	   "      port_nr=0..9; use \"all\" for all ports\n"
+);
+#endif /* CONFIG_VSC9953_CMD */
diff --git a/include/vsc9953.h b/include/vsc9953.h
new file mode 100644
index 0000000..3d11b87
--- /dev/null
+++ b/include/vsc9953.h
@@ -0,0 +1,402 @@
+/*
+ *  vsc9953.h
+ *
+ *  Driver for the Vitesse VSC9953 L2 Switch
+ *
+ *  This software may be used and distributed according to the
+ *  terms of the GNU Public License, Version 2, incorporated
+ *  herein by reference.
+ *
+ * Copyright 2013  Freescale Semiconductor, Inc.
+ *
+ */
+
+#ifndef _VSC9953_H_
+#define _VSC9953_H_
+
+#include <config.h>
+#include <miiphy.h>
+#include <asm/types.h>
+#include <malloc.h>
+
+#define VSC9953_OFFSET			(CONFIG_SYS_CCSRBAR_DEFAULT + 0x800000)
+
+#define VSC9953_SYS_OFFSET		0x010000
+#define VSC9953_DEV_GMII_OFFSET		0x100000
+#define VSC9953_QSYS_OFFSET		0x200000
+#define VSC9953_ANA_OFFSET		0x280000
+#define VSC9953_DEVCPU_GCB		0x070000
+#define VSC9953_ES0			0x040000
+#define VSC9953_IS1			0x050000
+#define VSC9953_IS2			0x060000
+
+#define T1040_SWITCH_GMII_DEV_OFFSET	0x010000
+#define VSC9953_PHY_REGS_OFFST		0x0000AC
+
+#define CONFIG_VSC9953_SOFT_SWC_RST_ENA	0x00000001
+#define CONFIG_VSC9953_CORE_ENABLE	0x80
+#define CONFIG_VSC9953_MEM_ENABLE	0x40
+#define CONFIG_VSC9953_MEM_INIT		0x20
+
+#define CONFIG_VSC9953_PORT_ENA		0x00003a00
+#define CONFIG_VSC9953_MAC_ENA_CFG	0x00000011
+#define CONFIG_VSC9953_MAC_MODE_CFG	0x00000011
+#define CONFIG_VSC9953_MAC_IFG_CFG	0x00000515
+#define CONFIG_VSC9953_MAC_HDX_CFG	0x00001043
+#define CONFIG_VSC9953_CLOCK_CFG	0x00000001
+#define CONFIG_VSC9953_CLOCK_CFG_1000M	0x00000001
+#define CONFIG_VSC9953_PFC_FC		0x00000001
+#define CONFIG_VSC9953_PFC_FC_QSGMII	0x00000000
+#define CONFIG_VSC9953_MAC_FC_CFG	0x04700000
+#define CONFIG_VSC9953_MAC_FC_CFG_QSGMII	0x00700000
+#define CONFIG_VSC9953_PAUSE_CFG	0x001ffffe
+#define CONFIG_VSC9953_TOT_TAIL_DROP_LVL	0x000003ff
+#define CONFIG_VSC9953_FRONT_PORT_MODE	0x00000000
+#define CONFIG_VSC9953_MAC_MAX_LEN	0x000005ee
+
+#define	CONFIG_VSC9953_VCAP_MV_CFG	0x0000ffff
+#define	CONFIG_VSC9953_VCAP_UPDATE_CTRL	0x01000004
+#define VSC9953_MAX_PORTS		10
+#define VSC9953_PORT_CHECK(port)	\
+	(((port) < 0 || (port) >= VSC9953_MAX_PORTS) ? 0 : 1)
+#define VSC9953_INTERNAL_PORT_CHECK(port) ( \
+	( \
+		(port) < VSC9953_MAX_PORTS - 2 || (port) >= VSC9953_MAX_PORTS \
+	) ? 0 : 1 \
+)
+
+#define DEFAULT_VSC9953_MDIO_NAME	"VSC9953_MDIO0"
+
+#define MIIMIND_OPR_PEND		0x00000004
+
+struct vsc9953_mdio_info {
+	struct vsc9953_mii_mng	*regs;
+	char	*name;
+};
+
+/* VSC9953 ANA structure for T1040 U-boot*/
+
+struct	vsc9953_ana_port {
+	u32	vlan_cfg;
+	u32	drop_cfg;
+	u32	qos_cfg;
+	u32	vcap_cfg;
+	u32	vcap_s1_key_cfg[3];
+	u32	vcap_s2_cfg;
+	u32	qos_pcp_dei_map_cfg[16];
+	u32	cpu_fwd_cfg;
+	u32	cpu_fwd_bpdu_cfg;
+	u32	cpu_fwd_garp_cfg;
+	u32	cpu_fwd_ccm_cfg;
+	u32	port_cfg;
+	u32	pol_cfg;
+	u32	reserved[34];
+};
+
+struct vsc9953_ana_pol {
+	u32	pol_pir_cfg;
+	u32	pol_cir_cfg;
+	u32	pol_mode_cfg;
+	u32	pol_pir_state;
+	u32	pol_cir_state;
+	u32	reserved1[3];
+};
+
+struct vsc9953_ana_ana_tables {
+	u32	entry_lim[11];
+	u32	an_moved;
+	u32	mach_data;
+	u32	macl_data;
+	u32	mac_access;
+	u32	mact_indx;
+	u32	vlan_access;
+	u32	vlan_tidx;
+};
+
+struct vsc9953_ana_ana {
+	u32	adv_learn;
+	u32	vlan_mask;
+	u32	anag_efil;
+	u32	an_events;
+	u32	storm_limit_burst;
+	u32	storm_limit_cfg[4];
+	u32	isolated_prts;
+	u32	community_ports;
+	u32	auto_age;
+	u32	mac_options;
+	u32	learn_disc;
+	u32	agen_ctrl;
+	u32	mirror_ports;
+	u32	emirror_ports;
+	u32	flooding;
+	u32	flooding_ipmc;
+	u32	sflow_cfg[11];
+	u32	port_mode[12];
+};
+
+struct vsc9953_ana_pgid {
+	u32	port_grp_id[91];
+};
+
+struct	vsc9953_ana_pfc {
+	u32	pfc_cfg;
+	u32	reserved1[15];
+};
+
+struct vsc9953_ana_pol_misc {
+	u32	pol_flowc[10];
+	u32	reserved1[17];
+	u32	pol_hyst;
+};
+
+struct	vsc9953_ana_common {
+	u32	aggr_cfg;
+	u32	cpuq_cfg;
+	u32	cpuq_8021_cfg;
+	u32	dscp_cfg;
+	u32	dscp_rewr_cfg;
+	u32	vcap_rng_type_cfg;
+	u32	vcap_rng_val_cfg;
+	u32	discard_cfg;
+	u32	fid_cfg;
+};
+
+struct vsc9953_analyzer {
+	struct vsc9953_ana_port	port[11];
+	u32	reserved1[9536];
+	struct vsc9953_ana_pol	pol[164];
+	struct vsc9953_ana_ana_tables	ana_tables;
+	u32	reserved2[14];
+	struct vsc9953_ana_ana	ana;
+	u32	reserved3[22];
+	struct vsc9953_ana_pgid	port_id_tbl;
+	u32	reserved4[549];
+	struct vsc9953_ana_pfc	pfc[10];
+	struct vsc9953_ana_pol_misc	pol_misc;
+	u32	reserved5[196];
+	struct vsc9953_ana_common	common;
+};
+/* END VSC9953 ANA structure for T1040 U-boot*/
+
+/* VSC9953 DEV_GMII structure for T1040 U-boot*/
+
+struct	vsc9953_dev_gmii_port_mode {
+	u32	clock_cfg;
+	u32	port_misc;
+	u32	reserved1;
+	u32	eee_cfg;
+};
+
+struct	vsc9953_dev_gmii_mac_cfg_status {
+	u32	mac_ena_cfg;
+	u32	mac_mode_cfg;
+	u32	mac_maxlen_cfg;
+	u32	mac_tags_cfg;
+	u32	mac_adv_chk_cfg;
+	u32	mac_ifg_cfg;
+	u32	mac_hdx_cfg;
+	u32	mac_fc_mac_low_cfg;
+	u32	mac_fc_mac_high_cfg;
+	u32	mac_sticky;
+};
+
+struct vsc9953_dev_gmii {
+	struct vsc9953_dev_gmii_port_mode	port_mode;
+	struct vsc9953_dev_gmii_mac_cfg_status	mac_cfg_status;
+};
+
+/* END VSC9953 DEV_GMII structure for T1040 U-boot*/
+
+/* VSC9953 QSYS structure for T1040 U-boot*/
+
+struct	vsc9953_qsys_hsch {
+	u32	cir_cfg;
+	u32	reserved1;
+	u32	se_cfg;
+	u32	se_dwrr_cfg[8];
+	u32	cir_state;
+	u32	reserved2[20];
+};
+
+struct	vsc9953_qsys_sys {
+	u32	port_mode[12];
+	u32	switch_port_mode[11];
+	u32	stat_cnt_cfg;
+	u32	eee_cfg[10];
+	u32	eee_thrs;
+	u32	igr_no_sharing;
+	u32	egr_no_sharing;
+	u32	sw_status[11];
+	u32	ext_cpu_cfg;
+	u32	cpu_group_map;
+	u32	reserved1[23];
+};
+
+struct	vsc9953_qsys_qos_cfg {
+	u32	red_profile[16];
+	u32	res_qos_mode;
+};
+
+struct	vsc9953_qsys_drop_cfg {
+	u32	egr_drop_mode;
+};
+
+struct	vsc9953_qsys_mmgt {
+	u32	eq_cntrl;
+	u32	reserved1;
+};
+
+struct	vsc9953_qsys_hsch_misc {
+	u32	hsch_misc_cfg;
+	u32	reserved1[546];
+};
+
+struct	vsc9953_qsys_res_ctrl {
+	u32	res_cfg;
+	u32	res_stat;
+
+};
+
+struct	vsc9953_qsys_reg {
+	struct vsc9953_qsys_hsch	hsch[108];
+	struct vsc9953_qsys_sys	sys;
+	struct vsc9953_qsys_qos_cfg	qos_cfg;
+	struct vsc9953_qsys_drop_cfg	drop_cfg;
+	struct vsc9953_qsys_mmgt	mmgt;
+	struct vsc9953_qsys_hsch_misc	hsch_misc;
+	struct vsc9953_qsys_res_ctrl	res_ctrl[1024];
+};
+
+/* END VSC9953 QSYS structure for T1040 U-boot*/
+
+/* VSC9953 SYS structure for T1040 U-boot*/
+
+struct	vsc9953_sys_stat {
+	u32	rx_cntrs[64];
+	u32	tx_cntrs[64];
+	u32	drop_cntrs[64];
+	u32	reserved1[6];
+};
+
+struct	vsc9953_sys_sys {
+	u32	reset_cfg;
+	u32	reserved1;
+	u32	vlan_etype_cfg;
+	u32	port_mode[12];
+	u32	front_port_mode[10];
+	u32	frame_aging;
+	u32	stat_cfg;
+	u32	reserved2[50];
+};
+
+struct	vsc9953_sys_pause_cfg {
+	u32	pause_cfg[11];
+	u32	pause_tot_cfg;
+	u32	tail_drop_level[11];
+	u32	tot_tail_drop_lvl;
+	u32	mac_fc_cfg[10];
+};
+
+struct	vsc9953_sys_mmgt {
+	u16	free_cnt;
+};
+
+struct	vsc9953_system_reg {
+	struct vsc9953_sys_stat	stat;
+	struct vsc9953_sys_sys	sys;
+	struct vsc9953_sys_pause_cfg	pause_cfg;
+	struct vsc9953_sys_mmgt	mmgt;
+};
+
+/* END VSC9953 SYS structure for T1040 U-boot*/
+
+
+/* VSC9953 DEVCPU_GCB structure for T1040 U-boot*/
+
+struct	vsc9953_chip_regs {
+	u32	chipd_id;
+	u32	gpr;
+	u32	soft_rst;
+};
+
+struct	vsc9953_gpio {
+	u32	gpio_out_set[10];
+	u32	gpio_out_clr[10];
+	u32	gpio_out[10];
+	u32	gpio_in[10];
+};
+
+struct vsc9953_mii_mng {
+	u32	miimstatus;
+	u32	reserved1;
+	u32	miimcmd;
+	u32	miimdata;
+	u32	miimcfg;
+	u32	miimscan_0;
+	u32	miimscan_1;
+	u32	miiscan_lst_rslts;
+	u32	miiscan_lst_rslts_valid;
+};
+
+struct	vsc9953_mii_read_scan {
+	u32	mii_scan_results_sticky[2];
+};
+
+struct	vsc9953_devcpu_gcb {
+	struct vsc9953_chip_regs	chip_regs;
+	struct vsc9953_gpio		gpio;
+	struct vsc9953_mii_mng	mii_mng[2];
+	struct vsc9953_mii_read_scan	mii_read_scan;
+};
+
+/* END VSC9953 DEVCPU_GCB structure for T1040 U-boot*/
+
+/* VSC9953 IS* structure for T1040 U-boot*/
+
+struct	vsc9953_vcap_core_cfg	{
+	u32	vcap_update_ctrl;
+	u32	vcap_mv_cfg;
+};
+
+struct	vsc9953_vcap {
+struct	vsc9953_vcap_core_cfg	vcap_core_cfg;
+};
+
+/* END VSC9953 IS* structure for T1040 U-boot*/
+
+#define VSC9953_PORT_INFO_INITIALIZER(idx) \
+{									\
+	.enabled	= 0,						\
+	.phyaddr	= 0,						\
+	.index		= idx,						\
+	.phy_regs	= NULL,						\
+	.enet_if	= PHY_INTERFACE_MODE_NONE,			\
+	.bus		= NULL,						\
+	.phydev		= NULL,						\
+}
+
+/* Structure to describe a VSC9953 port */
+struct vsc9953_port_info {
+	u8	enabled;
+	u8	phyaddr;
+	int	index;
+	void	*phy_regs;
+	phy_interface_t	enet_if;
+	struct mii_dev	*bus;
+	struct phy_device	*phydev;
+};
+
+/* Structure to describe a VSC9953 switch */
+struct vsc9953_info {
+	struct	vsc9953_port_info	port[VSC9953_MAX_PORTS];
+};
+
+void vsc9953_init(bd_t *bis);
+
+void vsc9953_port_info_set_mdio(int port, struct mii_dev *bus);
+void vsc9953_port_info_set_phy_address(int port, int address);
+void vsc9953_port_enable(int port);
+void vsc9953_port_disable(int port);
+void vsc9953_port_info_set_phy_int(int port, phy_interface_t phy_int);
+
+#endif /* _VSC9953_H_ */
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 06/11] arch/powerpc: Enable VSC9953 driver on T1040 and T1020
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
                   ` (4 preceding siblings ...)
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-15 21:26   ` York Sun
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 07/11] board/T1040qds: Fix lane-to-slot mapping for SerDes protocol 0x89 Codrin Ciubotariu
                   ` (4 subsequent siblings)
  10 siblings, 1 reply; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

T1040 and T1020 are two Freescale SoCs with an integrated
VSC9953 Gigabit L2 Switch. This patch initializes this L2
switch on boards with T1040 and T1020.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2:
	- added patch description;

Changes for v3:
	- Removed "Change-id" line from comment;

 arch/powerpc/cpu/mpc8xxx/cpu.c            | 5 +++++
 arch/powerpc/include/asm/config_mpc85xx.h | 3 +++
 2 files changed, 8 insertions(+)

diff --git a/arch/powerpc/cpu/mpc8xxx/cpu.c b/arch/powerpc/cpu/mpc8xxx/cpu.c
index 2d28eb2..c92589f 100644
--- a/arch/powerpc/cpu/mpc8xxx/cpu.c
+++ b/arch/powerpc/cpu/mpc8xxx/cpu.c
@@ -15,6 +15,7 @@
 #include <netdev.h>
 #include <asm/cache.h>
 #include <asm/io.h>
+#include <vsc9953.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -271,5 +272,9 @@ int cpu_eth_init(bd_t *bis)
 #ifdef CONFIG_FMAN_ENET
 	fm_standard_init(bis);
 #endif
+
+#ifdef CONFIG_VSC9953
+	vsc9953_init(bis);
+#endif
 	return 0;
 }
diff --git a/arch/powerpc/include/asm/config_mpc85xx.h b/arch/powerpc/include/asm/config_mpc85xx.h
index 01b0905..79caac1 100644
--- a/arch/powerpc/include/asm/config_mpc85xx.h
+++ b/arch/powerpc/include/asm/config_mpc85xx.h
@@ -747,6 +747,9 @@ defined(CONFIG_PPC_T1020) || defined(CONFIG_PPC_T1022)
 #define CONFIG_SYS_FSL_QORIQ_CHASSIS2	/* Freescale Chassis generation 2 */
 #define CONFIG_SYS_FSL_CORES_PER_CLUSTER 1
 #define CONFIG_SYS_FSL_QMAN_V3		/* QMAN version 3 */
+#if defined(CONFIG_PPC_T1040) || defined(CONFIG_PPC_T1020)
+#define CONFIG_VSC9953			/* Vitesse L2 Switch */
+#endif
 #ifdef CONFIG_SYS_FSL_DDR4
 #define CONFIG_SYS_FSL_DDRC_GEN4
 #endif
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 07/11] board/T1040qds: Fix lane-to-slot mapping for SerDes protocol 0x89
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
                   ` (5 preceding siblings ...)
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 06/11] arch/powerpc: Enable VSC9953 driver on T1040 and T1020 Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-23  0:42   ` York Sun
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 08/11] board/T1040qds: T1040 FMAN ports FM1@DTSEC1 and FM1@DTSEC2 have no PHYs Codrin Ciubotariu
                   ` (3 subsequent siblings)
  10 siblings, 1 reply; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2: None

Changes for v3:
	- Removed "Change-id" line from comment;

 board/freescale/t1040qds/eth.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/board/freescale/t1040qds/eth.c b/board/freescale/t1040qds/eth.c
index 06d9086..c6fc146 100644
--- a/board/freescale/t1040qds/eth.c
+++ b/board/freescale/t1040qds/eth.c
@@ -216,6 +216,7 @@ static void initialize_lane_to_slot(void)
 		lane_to_slot[1] = 7;
 		lane_to_slot[2] = 7;
 		lane_to_slot[3] = 7;
+		lane_to_slot[6] = 7;
 		lane_to_slot[7] = 7;
 		break;
 	case 0x8d:
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 08/11] board/T1040qds: T1040 FMAN ports FM1@DTSEC1 and FM1@DTSEC2 have no PHYs
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
                   ` (6 preceding siblings ...)
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 07/11] board/T1040qds: Fix lane-to-slot mapping for SerDes protocol 0x89 Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-23  0:42   ` York Sun
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 09/11] board/T104xrdb: " Codrin Ciubotariu
                   ` (2 subsequent siblings)
  10 siblings, 1 reply; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

Freescale's T1040qds board may be configured to have up to
5 FMAN ports (FM1 at DTSEC1 to FM1 at DTSEC5). From these 5 ports,
2 of them may be fixed-links (FM1 at DTSEC1 annd FM1 at DTSEC2),
connected to other two ports from an intergrated
VSC9953 L2 Switch (switch ports 8 and 9). These fixed-link
ports have no PHYs attatched, so they don't have a
corresponding MDIO.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2:
	- added patch description;

Changes for v3:
	- Removed "Change-id" line from comment;

 board/freescale/t1040qds/eth.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/board/freescale/t1040qds/eth.c b/board/freescale/t1040qds/eth.c
index c6fc146..2f8e753 100644
--- a/board/freescale/t1040qds/eth.c
+++ b/board/freescale/t1040qds/eth.c
@@ -478,6 +478,7 @@ int board_eth_init(bd_t *bis)
 	for (i = FM1_DTSEC1; i < FM1_DTSEC1 + CONFIG_SYS_NUM_FM1_DTSEC; i++) {
 		switch (fm_info_get_enet_if(i)) {
 		case PHY_INTERFACE_MODE_QSGMII:
+			fm_info_set_mdio(i, NULL);
 			break;
 		case PHY_INTERFACE_MODE_SGMII:
 			t1040_handle_phy_interface_sgmii(i);
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 09/11] board/T104xrdb: T1040 FMAN ports FM1@DTSEC1 and FM1@DTSEC2 have no PHYs
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
                   ` (7 preceding siblings ...)
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 08/11] board/T1040qds: T1040 FMAN ports FM1@DTSEC1 and FM1@DTSEC2 have no PHYs Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-23  0:43   ` York Sun
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 10/11] board/T1040qds: Add VSC9953 support for T1040qds board Codrin Ciubotariu
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 11/11] board/T1040rdb: Add VSC9953 support for T1040rdb board Codrin Ciubotariu
  10 siblings, 1 reply; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

Freescale's T1040qds board may be configured to have up to
5 FMAN ports (FM1 at DTSEC1 to FM1 at DTSEC5). From these 5 ports,
2 of them may be fixed-links (FM1 at DTSEC1 annd FM1 at DTSEC2),
connected to other two ports from an intergrated
VSC9953 L2 Switch (switch ports 8 and 9). These fixed-link
ports have no PHYs attatched, so they don't have a
corresponding MDIO.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2:
	- Added patch description;

Changes for v3:
	- Removed "Change-id" line from comment;

 board/freescale/t104xrdb/eth.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/board/freescale/t104xrdb/eth.c b/board/freescale/t104xrdb/eth.c
index c8b6c67..f5c0ec8 100644
--- a/board/freescale/t104xrdb/eth.c
+++ b/board/freescale/t104xrdb/eth.c
@@ -72,8 +72,13 @@ int board_eth_init(bd_t *bis)
 			fm_info_set_phy_address(i, 0);
 			break;
 		}
-		fm_info_set_mdio(i,
-				 miiphy_get_dev_by_name(DEFAULT_FM_MDIO_NAME));
+		if (fm_info_get_enet_if(i) == PHY_INTERFACE_MODE_QSGMII ||
+		    fm_info_get_enet_if(i) == PHY_INTERFACE_MODE_NONE)
+			fm_info_set_mdio(i, NULL);
+		else
+			fm_info_set_mdio(i,
+					 miiphy_get_dev_by_name(
+							DEFAULT_FM_MDIO_NAME));
 	}
 
 	cpu_eth_init(bis);
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 10/11] board/T1040qds: Add VSC9953 support for T1040qds board
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
                   ` (8 preceding siblings ...)
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 09/11] board/T104xrdb: " Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 11/11] board/T1040rdb: Add VSC9953 support for T1040rdb board Codrin Ciubotariu
  10 siblings, 0 replies; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

This patch configures and initializes the L2 switch on T1040QDS board.
The L2 switch ports must be initialized according to the SerDes
protocols.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2: None

Changes for v3:
	- Removed "Change-id" line from comment;

 board/freescale/t1040qds/eth.c | 91 ++++++++++++++++++++++++++++++++++++++++++
 include/configs/T1040QDS.h     |  4 ++
 2 files changed, 95 insertions(+)

diff --git a/board/freescale/t1040qds/eth.c b/board/freescale/t1040qds/eth.c
index 2f8e753..8c82934 100644
--- a/board/freescale/t1040qds/eth.c
+++ b/board/freescale/t1040qds/eth.c
@@ -18,6 +18,7 @@
 #include <fsl_mdio.h>
 #include <malloc.h>
 #include <asm/fsl_dtsec.h>
+#include <vsc9953.h>
 
 #include "../common/fman.h"
 #include "../common/qixis.h"
@@ -439,6 +440,12 @@ int board_eth_init(bd_t *bis)
 #ifdef CONFIG_FMAN_ENET
 	struct memac_mdio_info memac_mdio_info;
 	unsigned int i;
+#ifdef CONFIG_VSC9953
+	int lane;
+	int phy_addr;
+	phy_interface_t phy_int;
+	struct mii_dev *bus;
+#endif
 
 	printf("Initializing Fman\n");
 	set_brdcfg9_for_gtx_clk();
@@ -493,6 +500,90 @@ int board_eth_init(bd_t *bis)
 		}
 	}
 
+#ifdef CONFIG_VSC9953
+	for (i = 0; i < VSC9953_MAX_PORTS; i++) {
+		lane = -1;
+		phy_addr = 0;
+		phy_int = PHY_INTERFACE_MODE_NONE;
+		switch (i) {
+		case 0:
+		case 1:
+		case 2:
+		case 3:
+			lane = serdes_get_first_lane(FSL_SRDS_1, QSGMII_SW1_A);
+			/* PHYs connected over QSGMII */
+			if (lane >= 0) {
+				phy_addr = CONFIG_SYS_FM1_QSGMII21_PHY_ADDR +
+						i;
+				phy_int = PHY_INTERFACE_MODE_QSGMII;
+				break;
+			}
+			lane = serdes_get_first_lane(FSL_SRDS_1,
+					SGMII_SW1_MAC1 + i);
+
+			if (lane < 0)
+				break;
+
+			/* PHYs connected over QSGMII */
+			if (i != 3 || lane_to_slot[lane] == 7)
+				phy_addr = CONFIG_SYS_FM1_DTSEC1_RISER_PHY_ADDR
+					+ i;
+			else
+				phy_addr = CONFIG_SYS_FM1_DTSEC1_RISER_PHY_ADDR;
+			phy_int = PHY_INTERFACE_MODE_SGMII;
+			break;
+		case 4:
+		case 5:
+		case 6:
+		case 7:
+			lane = serdes_get_first_lane(FSL_SRDS_1, QSGMII_SW1_B);
+			/* PHYs connected over QSGMII */
+			if (lane >= 0) {
+				phy_addr = CONFIG_SYS_FM1_QSGMII11_PHY_ADDR +
+						i - 4;
+				phy_int = PHY_INTERFACE_MODE_QSGMII;
+				break;
+			}
+			lane = serdes_get_first_lane(FSL_SRDS_1,
+					SGMII_SW1_MAC1 + i);
+			/* PHYs connected over SGMII */
+			if (lane >= 0) {
+				phy_addr = CONFIG_SYS_FM1_DTSEC1_RISER_PHY_ADDR
+						+ i - 3;
+				phy_int = PHY_INTERFACE_MODE_SGMII;
+			}
+			break;
+		case 8:
+			if (serdes_get_first_lane(FSL_SRDS_1,
+						  SGMII_FM1_DTSEC1) < 0)
+				/* FM1 at DTSEC1 is connected to SW1@PORT8 */
+				vsc9953_port_enable(i);
+			break;
+		case 9:
+			if (serdes_get_first_lane(FSL_SRDS_1,
+						  SGMII_FM1_DTSEC2) < 0) {
+				/* Enable L2 On MAC2 using SCFG */
+				struct ccsr_scfg *scfg = (struct ccsr_scfg *)
+						CONFIG_SYS_MPC85xx_SCFG;
+
+				out_be32(&scfg->esgmiiselcr,
+					 in_be32(&scfg->esgmiiselcr) |
+					 (0x80000000));
+				vsc9953_port_enable(i);
+			}
+			break;
+		}
+
+		if (lane >= 0) {
+			bus = mii_dev_for_muxval(lane_to_slot[lane]);
+			vsc9953_port_info_set_mdio(i, bus);
+			vsc9953_port_enable(i);
+		}
+		vsc9953_port_info_set_phy_address(i, phy_addr);
+		vsc9953_port_info_set_phy_int(i, phy_int);
+	}
+
+#endif
 	cpu_eth_init(bis);
 #endif
 
diff --git a/include/configs/T1040QDS.h b/include/configs/T1040QDS.h
index b70bdfe..5b7c3c4 100644
--- a/include/configs/T1040QDS.h
+++ b/include/configs/T1040QDS.h
@@ -689,6 +689,10 @@ unsigned long get_board_ddr_clk(void);
 #define CONFIG_PHY_GIGE		/* Include GbE speed/duplex detection */
 #endif
 
+#define CONFIG_VSC9953_CMD
+#define CONFIG_SYS_FM1_QSGMII11_PHY_ADDR	0x14
+#define CONFIG_SYS_FM1_QSGMII21_PHY_ADDR	0x18
+
 /*
  * Dynamic MTD Partition support with mtdparts
  */
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 11/11] board/T1040rdb: Add VSC9953 support for T1040rdb board
  2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
                   ` (9 preceding siblings ...)
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 10/11] board/T1040qds: Add VSC9953 support for T1040qds board Codrin Ciubotariu
@ 2015-01-12 12:08 ` Codrin Ciubotariu
  10 siblings, 0 replies; 24+ messages in thread
From: Codrin Ciubotariu @ 2015-01-12 12:08 UTC (permalink / raw)
  To: u-boot

This patch configures and initializes the L2 switch on T1040rdb board.
The external L2 switch ports may be connected to PHYs only over
QSGMII, for T1040rdb.

Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
---

Changes for v2: None

Changed for v3:
	- Removed "Change-id" line from comment;

 board/freescale/t104xrdb/eth.c | 50 ++++++++++++++++++++++++++++++++++++++++++
 include/configs/T104xRDB.h     |  6 +++++
 2 files changed, 56 insertions(+)

diff --git a/board/freescale/t104xrdb/eth.c b/board/freescale/t104xrdb/eth.c
index f5c0ec8..7581a4cd 100644
--- a/board/freescale/t104xrdb/eth.c
+++ b/board/freescale/t104xrdb/eth.c
@@ -6,11 +6,13 @@
 
 #include <common.h>
 #include <netdev.h>
+#include <asm/fsl_serdes.h>
 #include <asm/immap_85xx.h>
 #include <fm_eth.h>
 #include <fsl_mdio.h>
 #include <malloc.h>
 #include <asm/fsl_dtsec.h>
+#include <vsc9953.h>
 
 #include "../common/fman.h"
 
@@ -20,6 +22,11 @@ int board_eth_init(bd_t *bis)
 	struct memac_mdio_info memac_mdio_info;
 	unsigned int i;
 	int phy_addr = 0;
+#ifdef CONFIG_VSC9953
+	phy_interface_t phy_int;
+	struct mii_dev *bus;
+#endif
+
 	printf("Initializing Fman\n");
 
 	memac_mdio_info.regs =
@@ -81,6 +88,49 @@ int board_eth_init(bd_t *bis)
 							DEFAULT_FM_MDIO_NAME));
 	}
 
+#ifdef CONFIG_VSC9953
+	/* SerDes configured for QSGMII */
+	if (serdes_get_first_lane(FSL_SRDS_1, QSGMII_SW1_A) >= 0) {
+		for (i = 0; i < 4; i++) {
+			bus = miiphy_get_dev_by_name(DEFAULT_FM_MDIO_NAME);
+			phy_addr = CONFIG_SYS_FM1_QSGMII11_PHY_ADDR + i;
+			phy_int = PHY_INTERFACE_MODE_QSGMII;
+
+			vsc9953_port_info_set_mdio(i, bus);
+			vsc9953_port_info_set_phy_address(i, phy_addr);
+			vsc9953_port_info_set_phy_int(i, phy_int);
+			vsc9953_port_enable(i);
+		}
+	}
+	if (serdes_get_first_lane(FSL_SRDS_1, QSGMII_SW1_B) >= 0) {
+		for (i = 4; i < 8; i++) {
+			bus = miiphy_get_dev_by_name(DEFAULT_FM_MDIO_NAME);
+			phy_addr = CONFIG_SYS_FM1_QSGMII21_PHY_ADDR + i - 4;
+			phy_int = PHY_INTERFACE_MODE_QSGMII;
+
+			vsc9953_port_info_set_mdio(i, bus);
+			vsc9953_port_info_set_phy_address(i, phy_addr);
+			vsc9953_port_info_set_phy_int(i, phy_int);
+			vsc9953_port_enable(i);
+		}
+	}
+
+	/* Connect DTSEC1 to L2 switch if it doesn't have a PHY */
+	if (serdes_get_first_lane(FSL_SRDS_1, SGMII_FM1_DTSEC1) < 0)
+		vsc9953_port_enable(8);
+
+	/* Connect DTSEC2 to L2 switch if it doesn't have a PHY */
+	if (serdes_get_first_lane(FSL_SRDS_1, SGMII_FM1_DTSEC2) < 0) {
+		/* Enable L2 On MAC2 using SCFG */
+		struct ccsr_scfg *scfg = (struct ccsr_scfg *)
+				CONFIG_SYS_MPC85xx_SCFG;
+
+		out_be32(&scfg->esgmiiselcr, in_be32(&scfg->esgmiiselcr) |
+			 (0x80000000));
+		vsc9953_port_enable(9);
+	}
+#endif
+
 	cpu_eth_init(bis);
 #endif
 
diff --git a/include/configs/T104xRDB.h b/include/configs/T104xRDB.h
index 57cdf72..c77b5d5 100644
--- a/include/configs/T104xRDB.h
+++ b/include/configs/T104xRDB.h
@@ -726,6 +726,12 @@
 #define CONFIG_SYS_RGMII1_PHY_ADDR		0x01
 #define CONFIG_SYS_RGMII2_PHY_ADDR		0x02
 
+#ifdef CONFIG_T1040RDB
+#define CONFIG_VSC9953_CMD
+#define CONFIG_SYS_FM1_QSGMII11_PHY_ADDR	0x04
+#define CONFIG_SYS_FM1_QSGMII21_PHY_ADDR	0x08
+#endif
+
 #define CONFIG_MII		/* MII PHY management */
 #define CONFIG_ETHPRIME		"FM1 at DTSEC4"
 #define CONFIG_PHY_GIGE		/* Include GbE speed/duplex detection */
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP Codrin Ciubotariu
@ 2015-01-15 20:59   ` York Sun
  2015-01-15 21:02   ` York Sun
                     ` (2 subsequent siblings)
  3 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-15 20:59 UTC (permalink / raw)
  To: u-boot

On 01/12/2015 04:08 AM, Codrin Ciubotariu wrote:
> This patch adds a driver for VSC9953 L2 Switch. This Vitesse IP
> is integrated in Freescale T1040 and T1020 SoCs.
> The L2 switch has 10 Ethernet ports: 2 internal fixed-links
> (ports 8 and 9) at 2.5 Gbps and and 8 external ports at 1 Gbps.
> The external ports may be connected to PHYs over QSGMII and SGMII.
> 
> Commands have also been added to enable/disable a port and to
> check a port's link speed, duplexity and status. The commands are:
> 
> ethsw port <port_nr> enable|disable - enable/disable an l2 switch port
> ethsw port <port_nr> show - show an l2 switch port's configuration
> 
> port_nr=0..9; use "all" for all ports
> 
> For more detailse please see doc/README.t1040-l2switch
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 

Joe,

I am trying to bring in this patch. Please let me know if you have any comment.
http://patchwork.ozlabs.org/patch/427693/

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP Codrin Ciubotariu
  2015-01-15 20:59   ` York Sun
@ 2015-01-15 21:02   ` York Sun
  2015-01-15 21:42   ` Joe Hershberger
  2015-01-23  0:40   ` York Sun
  3 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-15 21:02 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 04:08 AM, Codrin Ciubotariu wrote:
> This patch adds a driver for VSC9953 L2 Switch. This Vitesse IP
> is integrated in Freescale T1040 and T1020 SoCs.
> The L2 switch has 10 Ethernet ports: 2 internal fixed-links
> (ports 8 and 9) at 2.5 Gbps and and 8 external ports at 1 Gbps.
> The external ports may be connected to PHYs over QSGMII and SGMII.
> 
> Commands have also been added to enable/disable a port and to
> check a port's link speed, duplexity and status. The commands are:
> 
> ethsw port <port_nr> enable|disable - enable/disable an l2 switch port
> ethsw port <port_nr> show - show an l2 switch port's configuration
> 
> port_nr=0..9; use "all" for all ports
> 
> For more detailse please see doc/README.t1040-l2switch
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2:
> 	- added debug messages if MDIO reads or writes timeout;
> 	- added debug messages when reset of VSC9953 switch fails;
> 	- replaced Copyright and license to the generic one;
> 
> Changes for v3: Removed "Change-id" line from comment;
> 
>  doc/README.t1040-l2switch |  49 +++++
>  drivers/net/Makefile      |   1 +
>  drivers/net/vsc9953.c     | 497 ++++++++++++++++++++++++++++++++++++++++++++++
>  include/vsc9953.h         | 402 +++++++++++++++++++++++++++++++++++++
>  4 files changed, 949 insertions(+)
>  create mode 100644 doc/README.t1040-l2switch
>  create mode 100644 drivers/net/vsc9953.c
>  create mode 100644 include/vsc9953.h
> 
> diff --git a/doc/README.t1040-l2switch b/doc/README.t1040-l2switch
> new file mode 100644
> index 0000000..6324f18
> --- /dev/null
> +++ b/doc/README.t1040-l2switch
> @@ -0,0 +1,49 @@
> +This file contains information for VSC9953, a Vitesse L2 Switch IP
> +which is integrated in the T1040/T1020 Freescale SoCs.
> +
> +About Device:
> +=============
> +VSC9953 is an 8-port Gigabit Ethernet switch supports the following features:
> +	-	8192 MAC addresses
> +	-	Static Address provisioning
> +	-	Dynamic learning of MAC addresses and aging
> +	-	4096 VLANs
> +	-	Independent and shared VLAN learning (IVL, SVL)
> +	-	Policing with storm control and MC/BC protection
> +	-	IPv4 and IPv6 multicast
> +	-	Jumbo frames (9.6 KB)
> +	-	Access Control List
> +	-	VLAN editing, translation and remarking
> +	-	RMON counters per port
> +
> +Switch interfaces:
> +	-	8 Gigabit switch ports (ports 0 to 7) are external and are connected to external PHYs
> +	-	2 switch ports (ports 8 and 9) of 2.5 G are connected (fixed links)
> +		to FMan ports (FM1 at DTSEC1 and FM1 at DTSEC2)
> +
> +Commands Overview:
> +=============
> +Commands supported
> +	- enable/disable a port
> +	- check a port's link speed, duplexity and status.
> +
> +Commands syntax
> +	ethsw port <port_nr> enable|disable		- enable/disable an l2 switch port
> +	ethsw port <port_nr> show			- show an l2 switch port's configuration
> +
> +	port_nr=0..9; use "all" for all ports
> +
> +=> ethsw port all show
> +    Port   Status     Link    Speed   Duplex
> +       0  enabled     down       10     half
> +       1  enabled     down       10     half
> +       2  enabled     down       10     half
> +       3  enabled       up     1000     full
> +       4 disabled     down        -     half
> +       5 disabled     down        -     half
> +       6 disabled     down        -     half
> +       7 disabled     down        -     half
> +       8  enabled       up     2500     full
> +       9  enabled       up     2500     full
> +=>
> +

Extra line at the end. No need to respin the patch. I can remove it when I apply
the patch.

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 06/11] arch/powerpc: Enable VSC9953 driver on T1040 and T1020
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 06/11] arch/powerpc: Enable VSC9953 driver on T1040 and T1020 Codrin Ciubotariu
@ 2015-01-15 21:26   ` York Sun
  0 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-15 21:26 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 04:08 AM, Codrin Ciubotariu wrote:
> T1040 and T1020 are two Freescale SoCs with an integrated
> VSC9953 Gigabit L2 Switch. This patch initializes this L2
> switch on boards with T1040 and T1020.
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2:
> 	- added patch description;
> 
> Changes for v3:
> 	- Removed "Change-id" line from comment;
> 
>  arch/powerpc/cpu/mpc8xxx/cpu.c            | 5 +++++
>  arch/powerpc/include/asm/config_mpc85xx.h | 3 +++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/arch/powerpc/cpu/mpc8xxx/cpu.c b/arch/powerpc/cpu/mpc8xxx/cpu.c
> index 2d28eb2..c92589f 100644
> --- a/arch/powerpc/cpu/mpc8xxx/cpu.c
> +++ b/arch/powerpc/cpu/mpc8xxx/cpu.c
> @@ -15,6 +15,7 @@
>  #include <netdev.h>
>  #include <asm/cache.h>
>  #include <asm/io.h>
> +#include <vsc9953.h>
>  
>  DECLARE_GLOBAL_DATA_PTR;
>  
> @@ -271,5 +272,9 @@ int cpu_eth_init(bd_t *bis)
>  #ifdef CONFIG_FMAN_ENET
>  	fm_standard_init(bis);
>  #endif
> +
> +#ifdef CONFIG_VSC9953
> +	vsc9953_init(bis);
> +#endif
>  	return 0;
>  }
> diff --git a/arch/powerpc/include/asm/config_mpc85xx.h b/arch/powerpc/include/asm/config_mpc85xx.h
> index 01b0905..79caac1 100644
> --- a/arch/powerpc/include/asm/config_mpc85xx.h
> +++ b/arch/powerpc/include/asm/config_mpc85xx.h
> @@ -747,6 +747,9 @@ defined(CONFIG_PPC_T1020) || defined(CONFIG_PPC_T1022)
>  #define CONFIG_SYS_FSL_QORIQ_CHASSIS2	/* Freescale Chassis generation 2 */
>  #define CONFIG_SYS_FSL_CORES_PER_CLUSTER 1
>  #define CONFIG_SYS_FSL_QMAN_V3		/* QMAN version 3 */
> +#if defined(CONFIG_PPC_T1040) || defined(CONFIG_PPC_T1020)
> +#define CONFIG_VSC9953			/* Vitesse L2 Switch */
> +#endif
>  #ifdef CONFIG_SYS_FSL_DDR4
>  #define CONFIG_SYS_FSL_DDRC_GEN4
>  #endif
> 

Codrin,

A question, do you want to enable the L2 switch regardless if the ports are
connected on the board? Would it be better to enable the feature in board header
file?

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP Codrin Ciubotariu
  2015-01-15 20:59   ` York Sun
  2015-01-15 21:02   ` York Sun
@ 2015-01-15 21:42   ` Joe Hershberger
  2015-01-23  0:40   ` York Sun
  3 siblings, 0 replies; 24+ messages in thread
From: Joe Hershberger @ 2015-01-15 21:42 UTC (permalink / raw)
  To: u-boot

On Mon, Jan 12, 2015 at 6:08 AM, Codrin Ciubotariu <
codrin.ciubotariu@freescale.com> wrote:
>
> This patch adds a driver for VSC9953 L2 Switch. This Vitesse IP
> is integrated in Freescale T1040 and T1020 SoCs.
> The L2 switch has 10 Ethernet ports: 2 internal fixed-links
> (ports 8 and 9) at 2.5 Gbps and and 8 external ports at 1 Gbps.
> The external ports may be connected to PHYs over QSGMII and SGMII.
>
> Commands have also been added to enable/disable a port and to
> check a port's link speed, duplexity and status. The commands are:
>
> ethsw port <port_nr> enable|disable - enable/disable an l2 switch port
> ethsw port <port_nr> show - show an l2 switch port's configuration
>
> port_nr=0..9; use "all" for all ports
>
> For more detailse please see doc/README.t1040-l2switch
>
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>

Acked-by: Joe Hershberger <joe.hershberger@ni.com>

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 01/11] net/fm: Fix error when FMAN MAC has no PHY
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 01/11] net/fm: Fix error when FMAN MAC has no PHY Codrin Ciubotariu
@ 2015-01-23  0:37   ` York Sun
  0 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-23  0:37 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 06:08 AM, Codrin Ciubotariu wrote:
> U-boot assumes that all FMAN ports have a PHY. Some SoCs (like T1040)
> have fixed links. This means that the ports are connected MAC to MAc
> and there is no Ethernet PHY attatched. This patch initializes a
> FMAN MAC even if it doesn't have a PHY attached.
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2: None
> 
> Changes for v3: None

Applied to u-boot-mpc85xx master branch, awaiting upstream.

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 02/11] arch/powerpc: Fix mapping of Freescale SerDes protocols
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 02/11] arch/powerpc: Fix mapping of Freescale SerDes protocols Codrin Ciubotariu
@ 2015-01-23  0:38   ` York Sun
  0 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-23  0:38 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 06:08 AM, Codrin Ciubotariu wrote:
> The number of supported serdes protocols on Freescale SoCs
> has increased over time. Until now, an u64 variable have been
> initialized on boot with the configured protocols. However,
> since this number has increased (enum srds_prtcl has more
> than 64 values), 64 bits are no longer sufficient to hold track
> of all the configured protocols.
> This patch replaces the u64 map values with static arrays.
> To keep track of the number of serdes protocols, the
> SERDES_PRCTL_COUNT vale has been added at the end of
> enum srds_prtcl. This value must always be the last one.
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2: There is no v1 version of this patch;
> 
> Changes for v3:
> 	- replaced 'int' type of seredesX_prtcl_map[] with 'u8';
> 	- removed "Change-id" line from comment;
> 

Applied to u-boot-mpc85xx master branch, awaiting upstream.

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 03/11] arch/powerpc: Add SGMII support for the L2 Switch ports
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 03/11] arch/powerpc: Add SGMII support for the L2 Switch ports Codrin Ciubotariu
@ 2015-01-23  0:39   ` York Sun
  0 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-23  0:39 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 06:08 AM, Codrin Ciubotariu wrote:
> Some Freescale SoCs like T1020 and T1040 have an integrated
> L2 Switch. The L2 Switch ports may be connected to Ethernet PHYs
> over SGMII and QSGMII.
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2:
> 	- split a bug fix regarding Freescale SerDes protocols
> 	into a different patch;
> 
> Changes for v3: Removed "Change-id" line from comment;
> 
>
Applied to u-boot-mpc85xx master branch, awaiting upstream.

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 04/11] net/fm: Enable FMAN ports if l2switch ports are connected over SGMII
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 04/11] net/fm: Enable FMAN ports if l2switch ports are connected over SGMII Codrin Ciubotariu
@ 2015-01-23  0:39   ` York Sun
  0 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-23  0:39 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 06:08 AM, Codrin Ciubotariu wrote:
> If SerDes is configured to connect L2 Switch ports from T1040
> over SGMII or QSGMII, the two FMAN fixed ports (FM1 at DTSEC1 and FM2 at DTSEC2)
> that are connected to two L2 swtch ports must be enabled. These
> ports don't have PHYs and must be treated accordingly.
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2: None
> 
> Changes for v3: Removed "Change-id" line from comment;
> 

Applied to u-boot-mpc85xx master branch, awaiting upstream.

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP Codrin Ciubotariu
                     ` (2 preceding siblings ...)
  2015-01-15 21:42   ` Joe Hershberger
@ 2015-01-23  0:40   ` York Sun
  3 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-23  0:40 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 06:08 AM, Codrin Ciubotariu wrote:
> This patch adds a driver for VSC9953 L2 Switch. This Vitesse IP
> is integrated in Freescale T1040 and T1020 SoCs.
> The L2 switch has 10 Ethernet ports: 2 internal fixed-links
> (ports 8 and 9) at 2.5 Gbps and and 8 external ports at 1 Gbps.
> The external ports may be connected to PHYs over QSGMII and SGMII.
> 
> Commands have also been added to enable/disable a port and to
> check a port's link speed, duplexity and status. The commands are:
> 
> ethsw port <port_nr> enable|disable - enable/disable an l2 switch port
> ethsw port <port_nr> show - show an l2 switch port's configuration
> 
> port_nr=0..9; use "all" for all ports
> 
> For more detailse please see doc/README.t1040-l2switch
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2:
> 	- added debug messages if MDIO reads or writes timeout;
> 	- added debug messages when reset of VSC9953 switch fails;
> 	- replaced Copyright and license to the generic one;
> 
> Changes for v3: Removed "Change-id" line from comment;
> 

Applied to u-boot-mpc85xx master branch, awaiting upstream.

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 07/11] board/T1040qds: Fix lane-to-slot mapping for SerDes protocol 0x89
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 07/11] board/T1040qds: Fix lane-to-slot mapping for SerDes protocol 0x89 Codrin Ciubotariu
@ 2015-01-23  0:42   ` York Sun
  0 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-23  0:42 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 06:08 AM, Codrin Ciubotariu wrote:
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2: None
> 
> Changes for v3:
> 	- Removed "Change-id" line from comment;
> 

Applied to u-boot-mpc85xx master branch, awaiting upstream.

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 08/11] board/T1040qds: T1040 FMAN ports FM1@DTSEC1 and FM1@DTSEC2 have no PHYs
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 08/11] board/T1040qds: T1040 FMAN ports FM1@DTSEC1 and FM1@DTSEC2 have no PHYs Codrin Ciubotariu
@ 2015-01-23  0:42   ` York Sun
  0 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-23  0:42 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 06:08 AM, Codrin Ciubotariu wrote:
> Freescale's T1040qds board may be configured to have up to
> 5 FMAN ports (FM1 at DTSEC1 to FM1 at DTSEC5). From these 5 ports,
> 2 of them may be fixed-links (FM1 at DTSEC1 annd FM1 at DTSEC2),
> connected to other two ports from an intergrated
> VSC9953 L2 Switch (switch ports 8 and 9). These fixed-link
> ports have no PHYs attatched, so they don't have a
> corresponding MDIO.
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2:
> 	- added patch description;
> 
> Changes for v3:
> 	- Removed "Change-id" line from comment;
> 

Applied to u-boot-mpc85xx master branch, awaiting upstream.

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot] [PATCH v3 09/11] board/T104xrdb: T1040 FMAN ports FM1@DTSEC1 and FM1@DTSEC2 have no PHYs
  2015-01-12 12:08 ` [U-Boot] [PATCH v3 09/11] board/T104xrdb: " Codrin Ciubotariu
@ 2015-01-23  0:43   ` York Sun
  0 siblings, 0 replies; 24+ messages in thread
From: York Sun @ 2015-01-23  0:43 UTC (permalink / raw)
  To: u-boot



On 01/12/2015 06:08 AM, Codrin Ciubotariu wrote:
> Freescale's T1040qds board may be configured to have up to
> 5 FMAN ports (FM1 at DTSEC1 to FM1 at DTSEC5). From these 5 ports,
> 2 of them may be fixed-links (FM1 at DTSEC1 annd FM1 at DTSEC2),
> connected to other two ports from an intergrated
> VSC9953 L2 Switch (switch ports 8 and 9). These fixed-link
> ports have no PHYs attatched, so they don't have a
> corresponding MDIO.
> 
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@freescale.com>
> ---
> 
> Changes for v2:
> 	- Added patch description;
> 
> Changes for v3:
> 	- Removed "Change-id" line from comment;
> 

Applied to u-boot-mpc85xx master branch, awaiting upstream.

York

^ permalink raw reply	[flat|nested] 24+ messages in thread

end of thread, other threads:[~2015-01-23  0:43 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-12 12:08 [U-Boot] [PATCH v3 00/11] Add driver for VSC9953 Ethernet Switch Codrin Ciubotariu
2015-01-12 12:08 ` [U-Boot] [PATCH v3 01/11] net/fm: Fix error when FMAN MAC has no PHY Codrin Ciubotariu
2015-01-23  0:37   ` York Sun
2015-01-12 12:08 ` [U-Boot] [PATCH v3 02/11] arch/powerpc: Fix mapping of Freescale SerDes protocols Codrin Ciubotariu
2015-01-23  0:38   ` York Sun
2015-01-12 12:08 ` [U-Boot] [PATCH v3 03/11] arch/powerpc: Add SGMII support for the L2 Switch ports Codrin Ciubotariu
2015-01-23  0:39   ` York Sun
2015-01-12 12:08 ` [U-Boot] [PATCH v3 04/11] net/fm: Enable FMAN ports if l2switch ports are connected over SGMII Codrin Ciubotariu
2015-01-23  0:39   ` York Sun
2015-01-12 12:08 ` [U-Boot] [PATCH v3 05/11] net/vsc9953: Add driver for Vitesse VSC9953 L2 Switch IP Codrin Ciubotariu
2015-01-15 20:59   ` York Sun
2015-01-15 21:02   ` York Sun
2015-01-15 21:42   ` Joe Hershberger
2015-01-23  0:40   ` York Sun
2015-01-12 12:08 ` [U-Boot] [PATCH v3 06/11] arch/powerpc: Enable VSC9953 driver on T1040 and T1020 Codrin Ciubotariu
2015-01-15 21:26   ` York Sun
2015-01-12 12:08 ` [U-Boot] [PATCH v3 07/11] board/T1040qds: Fix lane-to-slot mapping for SerDes protocol 0x89 Codrin Ciubotariu
2015-01-23  0:42   ` York Sun
2015-01-12 12:08 ` [U-Boot] [PATCH v3 08/11] board/T1040qds: T1040 FMAN ports FM1@DTSEC1 and FM1@DTSEC2 have no PHYs Codrin Ciubotariu
2015-01-23  0:42   ` York Sun
2015-01-12 12:08 ` [U-Boot] [PATCH v3 09/11] board/T104xrdb: " Codrin Ciubotariu
2015-01-23  0:43   ` York Sun
2015-01-12 12:08 ` [U-Boot] [PATCH v3 10/11] board/T1040qds: Add VSC9953 support for T1040qds board Codrin Ciubotariu
2015-01-12 12:08 ` [U-Boot] [PATCH v3 11/11] board/T1040rdb: Add VSC9953 support for T1040rdb board Codrin Ciubotariu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.