Fix the roboswitch code for the WRT350N

SVN-Revision: 10531

3 files changed, 186 insertions, 80 deletions

diff --git a/package/switch/src/switch-core.c b/package/switch/src/switch-core.c
index 5eeb803..e0aa541 100644
--- a/package/switch/src/switch-core.c
+++ b/package/switch/src/switch-core.c
@@ -139,18 +139,18 @@ static ssize_t switch_proc_write(struct file *file, const char *buf, size_t coun
 
 static int handle_driver_name(void *driver, char *buf, int nr)
 {
-	char *name = ((switch_driver *) driver)->name;
+	const char *name = ((switch_driver *) driver)->name;
 	return sprintf(buf, "%s\n", name);
 }
 
 static int handle_driver_version(void *driver, char *buf, int nr)
 {
-	char *version = ((switch_driver *) driver)->version;
+	const char *version = ((switch_driver *) driver)->version;
 	strcpy(buf, version);
 	return sprintf(buf, "%s\n", version);
 }
 
-static void add_handler(switch_driver *driver, switch_config *handler, struct proc_dir_entry *parent, int nr)
+static void add_handler(switch_driver *driver, const switch_config *handler, struct proc_dir_entry *parent, int nr)
 {
 	switch_priv *priv = (switch_priv *) driver->data;
 	struct proc_dir_entry *p;
@@ -175,7 +175,7 @@ static void add_handler(switch_driver *driver, switch_config *handler, struct pr
 	}
 }
 
-static inline void add_handlers(switch_driver *driver, switch_config *handlers, struct proc_dir_entry *parent, int nr)
+static inline void add_handlers(switch_driver *driver, const switch_config *handlers, struct proc_dir_entry *parent, int nr)
 {
 	int i;
 	
@@ -408,7 +408,7 @@ int switch_register_driver(switch_driver *driver)
 	memcpy(new, driver, sizeof(switch_driver));
 	new->name = strdup(driver->name);
 	new->interface = strdup(driver->interface);
-	
+
 	if ((ret = do_register(new)) < 0) {
 		kfree(new->name);
 		kfree(new);
diff --git a/package/switch/src/switch-core.h b/package/switch/src/switch-core.h
index 5292469..5a64efb 100644
--- a/package/switch/src/switch-core.h
+++ b/package/switch/src/switch-core.h
@@ -20,19 +20,19 @@
 typedef int (*switch_handler)(void *driver, char *buf, int nr);
 
 typedef struct {
-	char *name;
+	const char *name;
 	switch_handler read, write;
 } switch_config;
 
 typedef struct {
 	struct list_head list;
-	char *name;
-	char *version;
-	char *interface;
+	const char *name;
+	const char *version;
+	const char *interface;
 	int cpuport;
 	int ports;
 	int vlans;
-	switch_config *driver_handlers, *port_handlers, *vlan_handlers;
+	const switch_config *driver_handlers, *port_handlers, *vlan_handlers;
 	void *data;
 	void *priv;
 } switch_driver;
@@ -48,7 +48,7 @@ extern switch_vlan_config *switch_parse_vlan(switch_driver *driver, char *buf);
 extern int switch_parse_media(char *buf);
 extern int switch_print_media(char *buf, int media);
 
-static inline char *strdup(char *str)
+static inline char *strdup(const char *str)
 {
 	char *new = kmalloc(strlen(str) + 1, GFP_KERNEL);
 	strcpy(new, str);
diff --git a/package/switch/src/switch-robo.c b/package/switch/src/switch-robo.c
index 009781d..5bcd85b 100644
--- a/package/switch/src/switch-robo.c
+++ b/package/switch/src/switch-robo.c
@@ -2,6 +2,7 @@
  * Broadcom BCM5325E/536x switch configuration module
  *
  * Copyright (C) 2005 Felix Fietkau <nbd@nbd.name>
+ * Copyright (C) 2008 Michael Buesch <mb@bu3sch.de>
  * Based on 'robocfg' by Oleg I. Vdovikin
  *
  * This program is free software; you can redistribute it and/or
@@ -28,15 +29,18 @@
 #include <linux/sockios.h>
 #include <linux/ethtool.h>
 #include <linux/mii.h>
+#include <linux/delay.h>
 #include <asm/uaccess.h>
 
 #include "switch-core.h"
 #include "etc53xx.h"
 
 #define DRIVER_NAME		"bcm53xx"
-#define DRIVER_VERSION	"0.01"
+#define DRIVER_VERSION		"0.02"
+#define PFX			"roboswitch: "
 
-#define ROBO_PHY_ADDR	0x1E	/* robo switch phy address */
+#define ROBO_PHY_ADDR		0x1E	/* robo switch phy address */
+#define ROBO_PHY_ADDR_TG3	0x01	/* Tigon3 PHY address */
 
 /* MII registers */
 #define REG_MII_PAGE	0x10	/* MII Page register */
@@ -47,16 +51,33 @@
 #define REG_MII_ADDR_WRITE	1
 #define REG_MII_ADDR_READ	2
 
+/* Robo device ID register (in ROBO_MGMT_PAGE) */
+#define ROBO_DEVICE_ID		0x30
+#define  ROBO_DEVICE_ID_5325	0x25 /* Faked */
+#define  ROBO_DEVICE_ID_5395	0x95
+#define  ROBO_DEVICE_ID_5397	0x97
+#define  ROBO_DEVICE_ID_5398	0x98
+
 /* Private et.o ioctls */
 #define SIOCGETCPHYRD           (SIOCDEVPRIVATE + 9)
 #define SIOCSETCPHYWR           (SIOCDEVPRIVATE + 10)
 
-static char *device;
-static int use_et = 0;
-static int is_5350 = 0;
-static struct ifreq ifr;
-static struct net_device *dev;
-static unsigned char port[6] = { 0, 1, 2, 3, 4, 8 };
+
+/* Data structure for a Roboswitch device. */
+struct robo_switch {
+	char *device;			/* The device name string (ethX) */
+	u16 devid;			/* ROBO_DEVICE_ID_53xx */
+	bool use_et;
+	bool is_5350;
+	u8 phy_addr;			/* PHY address of the device */
+	struct ifreq ifr;
+	struct net_device *dev;
+	unsigned char port[6];
+};
+
+/* Currently we can only have one device in the system. */
+static struct robo_switch robo;
+
 
 static int do_ioctl(int cmd, void *buf)
 {
@@ -64,10 +85,10 @@ static int do_ioctl(int cmd, void *buf)
 	int ret;
 
 	if (buf != NULL)
-		ifr.ifr_data = (caddr_t) buf;
+		robo.ifr.ifr_data = (caddr_t) buf;
 
 	set_fs(KERNEL_DS);
-	ret = dev->do_ioctl(dev, &ifr, cmd);
+	ret = robo.dev->do_ioctl(robo.dev, &robo.ifr, cmd);
 	set_fs(old_fs);
 
 	return ret;
@@ -75,11 +96,11 @@ static int do_ioctl(int cmd, void *buf)
 
 static u16 mdio_read(__u16 phy_id, __u8 reg)
 {
-	if (use_et) {
+	if (robo.use_et) {
 		int args[2] = { reg };
-		
-		if (phy_id != ROBO_PHY_ADDR) {
-			printk(
+
+		if (phy_id != robo.phy_addr) {
+			printk(KERN_ERR PFX
 				"Access to real 'phy' registers unavaliable.\n"
 				"Upgrade kernel driver.\n");
 
@@ -88,18 +109,20 @@ static u16 mdio_read(__u16 phy_id, __u8 reg)
 
 
 		if (do_ioctl(SIOCGETCPHYRD, &args) < 0) {
-			printk("[%s:%d] SIOCGETCPHYRD failed!\n", __FILE__, __LINE__);
+			printk(KERN_ERR PFX
+			       "[%s:%d] SIOCGETCPHYRD failed!\n", __FILE__, __LINE__);
 			return 0xffff;
 		}
 	
 		return args[1];
 	} else {
-		struct mii_ioctl_data *mii = (struct mii_ioctl_data *) &ifr.ifr_data;
+		struct mii_ioctl_data *mii = (struct mii_ioctl_data *) &robo.ifr.ifr_data;
 		mii->phy_id = phy_id;
 		mii->reg_num = reg;
 
 		if (do_ioctl(SIOCGMIIREG, NULL) < 0) {
-			printk("[%s:%d] SIOCGMIIREG failed!\n", __FILE__, __LINE__);
+			printk(KERN_ERR PFX
+			       "[%s:%d] SIOCGMIIREG failed!\n", __FILE__, __LINE__);
 
 			return 0xffff;
 		}
@@ -110,11 +133,11 @@ static u16 mdio_read(__u16 phy_id, __u8 reg)
 
 static void mdio_write(__u16 phy_id, __u8 reg, __u16 val)
 {
-	if (use_et) {
+	if (robo.use_et) {
 		int args[2] = { reg, val };
 
-		if (phy_id != ROBO_PHY_ADDR) {
-			printk(
+		if (phy_id != robo.phy_addr) {
+			printk(KERN_ERR PFX
 				"Access to real 'phy' registers unavaliable.\n"
 				"Upgrade kernel driver.\n");
 
@@ -122,18 +145,20 @@ static void mdio_write(__u16 phy_id, __u8 reg, __u16 val)
 		}
 		
 		if (do_ioctl(SIOCSETCPHYWR, args) < 0) {
-			printk("[%s:%d] SIOCGETCPHYWR failed!\n", __FILE__, __LINE__);
+			printk(KERN_ERR PFX
+			       "[%s:%d] SIOCGETCPHYWR failed!\n", __FILE__, __LINE__);
 			return;
 		}
 	} else {
-		struct mii_ioctl_data *mii = (struct mii_ioctl_data *)&ifr.ifr_data;
+		struct mii_ioctl_data *mii = (struct mii_ioctl_data *)&robo.ifr.ifr_data;
 
 		mii->phy_id = phy_id;
 		mii->reg_num = reg;
 		mii->val_in = val;
 
 		if (do_ioctl(SIOCSMIIREG, NULL) < 0) {
-			printk("[%s:%d] SIOCSMIIREG failed!\n", __FILE__, __LINE__);
+			printk(KERN_ERR PFX
+			       "[%s:%d] SIOCSMIIREG failed!\n", __FILE__, __LINE__);
 			return;
 		}
 	}
@@ -144,20 +169,20 @@ static int robo_reg(__u8 page, __u8 reg, __u8 op)
 	int i = 3;
 	
 	/* set page number */
-	mdio_write(ROBO_PHY_ADDR, REG_MII_PAGE, 
+	mdio_write(robo.phy_addr, REG_MII_PAGE, 
 		(page << 8) | REG_MII_PAGE_ENABLE);
 	
 	/* set register address */
-	mdio_write(ROBO_PHY_ADDR, REG_MII_ADDR, 
+	mdio_write(robo.phy_addr, REG_MII_ADDR, 
 		(reg << 8) | op);
 
 	/* check if operation completed */
 	while (i--) {
-		if ((mdio_read(ROBO_PHY_ADDR, REG_MII_ADDR) & 3) == 0)
+		if ((mdio_read(robo.phy_addr, REG_MII_ADDR) & 3) == 0)
 			return 0;
 	}
 
-	printk("[%s:%d] timeout in robo_reg!\n", __FILE__, __LINE__);
+	printk(KERN_ERR PFX "[%s:%d] timeout in robo_reg!\n", __FILE__, __LINE__);
 	
 	return 0;
 }
@@ -170,7 +195,7 @@ static void robo_read(__u8 page, __u8 reg, __u16 *val, int count)
 	robo_reg(page, reg, REG_MII_ADDR_READ);
 	
 	for (i = 0; i < count; i++)
-		val[i] = mdio_read(ROBO_PHY_ADDR, REG_MII_DATA0 + i);
+		val[i] = mdio_read(robo.phy_addr, REG_MII_DATA0 + i);
 }
 */
 
@@ -178,21 +203,21 @@ static __u16 robo_read16(__u8 page, __u8 reg)
 {
 	robo_reg(page, reg, REG_MII_ADDR_READ);
 	
-	return mdio_read(ROBO_PHY_ADDR, REG_MII_DATA0);
+	return mdio_read(robo.phy_addr, REG_MII_DATA0);
 }
 
 static __u32 robo_read32(__u8 page, __u8 reg)
 {
 	robo_reg(page, reg, REG_MII_ADDR_READ);
 	
-	return mdio_read(ROBO_PHY_ADDR, REG_MII_DATA0) +
-		(mdio_read(ROBO_PHY_ADDR, REG_MII_DATA0 + 1) << 16);
+	return mdio_read(robo.phy_addr, REG_MII_DATA0) +
+		(mdio_read(robo.phy_addr, REG_MII_DATA0 + 1) << 16);
 }
 
 static void robo_write16(__u8 page, __u8 reg, __u16 val16)
 {
 	/* write data */
-	mdio_write(ROBO_PHY_ADDR, REG_MII_DATA0, val16);
+	mdio_write(robo.phy_addr, REG_MII_DATA0, val16);
 
 	robo_reg(page, reg, REG_MII_ADDR_WRITE);
 }
@@ -200,8 +225,8 @@ static void robo_write16(__u8 page, __u8 reg, __u16 val16)
 static void robo_write32(__u8 page, __u8 reg, __u32 val32)
 {
 	/* write data */
-	mdio_write(ROBO_PHY_ADDR, REG_MII_DATA0, val32 & 65535);
-	mdio_write(ROBO_PHY_ADDR, REG_MII_DATA0 + 1, val32 >> 16);
+	mdio_write(robo.phy_addr, REG_MII_DATA0, val32 & 65535);
+	mdio_write(robo.phy_addr, REG_MII_DATA0 + 1, val32 >> 16);
 	
 	robo_reg(page, reg, REG_MII_ADDR_WRITE);
 }
@@ -217,42 +242,108 @@ static int robo_vlan5350(void)
 	return (robo_read16(ROBO_VLAN_PAGE, ROBO_VLAN_TABLE_ACCESS_5350) == val16);
 }
 
+static int robo_switch_enable(void)
+{
+	unsigned int i, last_port;
+	u16 val;
+
+	val = robo_read16(ROBO_CTRL_PAGE, ROBO_SWITCH_MODE);
+	if (!(val & (1 << 1))) {
+		/* Unmanaged mode */
+		val &= ~(1 << 0);
+		/* With forwarding */
+		val |= (1 << 1);
+		robo_write16(ROBO_CTRL_PAGE, ROBO_SWITCH_MODE, val);
+		val = robo_read16(ROBO_CTRL_PAGE, ROBO_SWITCH_MODE);
+		if (!(val & (1 << 1))) {
+			printk("Failed to enable switch\n");
+			return -EBUSY;
+		}
+
+		last_port = (robo.devid == ROBO_DEVICE_ID_5398) ?
+				ROBO_PORT6_CTRL : ROBO_PORT3_CTRL;
+		for (i = ROBO_PORT0_CTRL; i < last_port + 1; i++)
+			robo_write16(ROBO_CTRL_PAGE, i, 0);
+	}
+
+	/* WAN port LED */
+	robo_write16(ROBO_CTRL_PAGE, 0x16, 0x1F);
+
+	return 0;
+}
 
+static void robo_switch_reset(void)
+{
+	if ((robo.devid == ROBO_DEVICE_ID_5395) ||
+	    (robo.devid == ROBO_DEVICE_ID_5397) ||
+	    (robo.devid == ROBO_DEVICE_ID_5398)) {
+		/* Trigger a software reset. */
+		robo_write16(ROBO_CTRL_PAGE, 0x79, 0x83);
+		robo_write16(ROBO_CTRL_PAGE, 0x79, 0);
+	}
+}
 
 static int robo_probe(char *devname)
 {
 	__u32 phyid;
+	unsigned int i;
+	int err;
 
-	printk("Probing device %s: ", devname);
-	strcpy(ifr.ifr_name, devname);
+	printk(KERN_INFO PFX "Probing device %s: ", devname);
+	strcpy(robo.ifr.ifr_name, devname);
 
-	if ((dev = dev_get_by_name(devname)) == NULL) {
+	if ((robo.dev = dev_get_by_name(devname)) == NULL) {
 		printk("No such device\n");
 		return 1;
 	}
 
+	robo.device = devname;
+	for (i = 0; i < 5; i++)
+		robo.port[i] = i;
+	robo.port[5] = 8;
+
 	/* try access using MII ioctls - get phy address */
 	if (do_ioctl(SIOCGMIIPHY, NULL) < 0) {
-		use_et = 1;
+		robo.use_et = 1;
+		robo.phy_addr = ROBO_PHY_ADDR;
 	} else {
 		/* got phy address check for robo address */
-		struct mii_ioctl_data *mii = (struct mii_ioctl_data *) &ifr.ifr_data;
-		if (mii->phy_id != ROBO_PHY_ADDR) {
+		struct mii_ioctl_data *mii = (struct mii_ioctl_data *) &robo.ifr.ifr_data;
+		if ((mii->phy_id != ROBO_PHY_ADDR) &&
+		    (mii->phy_id != ROBO_PHY_ADDR_TG3)) {
 			printk("Invalid phy address (%d)\n", mii->phy_id);
 			return 1;
 		}
+		robo.use_et = 0;
+		/* The robo has a fixed PHY address that is different from the
+		 * Tigon3 PHY address. */
+		robo.phy_addr = ROBO_PHY_ADDR;
 	}
 
-	phyid = mdio_read(ROBO_PHY_ADDR, 0x2) | 
-		(mdio_read(ROBO_PHY_ADDR, 0x3) << 16);
+	phyid = mdio_read(robo.phy_addr, 0x2) | 
+		(mdio_read(robo.phy_addr, 0x3) << 16);
 
 	if (phyid == 0xffffffff || phyid == 0x55210022) {
 		printk("No Robo switch in managed mode found\n");
 		return 1;
 	}
-	
-	is_5350 = robo_vlan5350();
-	
+
+	/* Get the device ID */
+	for (i = 0; i < 10; i++) {
+		robo.devid = robo_read16(ROBO_MGMT_PAGE, ROBO_DEVICE_ID);
+		if (robo.devid)
+			break;
+		udelay(10);
+	}
+	if (!robo.devid)
+		robo.devid = ROBO_DEVICE_ID_5325; /* Fake it */
+	robo.is_5350 = robo_vlan5350();
+
+	robo_switch_reset();
+	err = robo_switch_enable();
+	if (err)
+		return err;
+
 	printk("found!\n");
 	return 0;
 }
@@ -266,7 +357,7 @@ static int handle_vlan_port_read(void *driver, char *buf, int nr)
 
 	val16 = (nr) /* vlan */ | (0 << 12) /* read */ | (1 << 13) /* enable */;
 	
-	if (is_5350) {
+	if (robo.is_5350) {
 		u32 val32;
 		robo_write16(ROBO_VLAN_PAGE, ROBO_VLAN_TABLE_ACCESS_5350, val16);
 		/* actual read */
@@ -332,7 +423,7 @@ static int handle_vlan_port_write(void *driver, char *buf, int nr)
 
 	/* write config now */
 	val16 = (nr) /* vlan */ | (1 << 12) /* write */ | (1 << 13) /* enable */;
-	if (is_5350) {
+	if (robo.is_5350) {
 		robo_write32(ROBO_VLAN_PAGE, ROBO_VLAN_WRITE_5350,
 			(1 << 20) /* valid */ | (c->untag << 6) | c->port);
 		robo_write16(ROBO_VLAN_PAGE, ROBO_VLAN_TABLE_ACCESS_5350, val16);
@@ -395,19 +486,21 @@ static int handle_reset(void *driver, char *buf, int nr)
 	set_switch(0);
 
 	/* reset vlans */
-	for (j = 0; j <= (is_5350 ? VLAN_ID_MAX5350 : VLAN_ID_MAX); j++) {
+	for (j = 0; j <= ((robo.is_5350) ? VLAN_ID_MAX5350 : VLAN_ID_MAX); j++) {
 		/* write config now */
 		val16 = (j) /* vlan */ | (1 << 12) /* write */ | (1 << 13) /* enable */;
-		if (is_5350)
+		if (robo.is_5350)
 			robo_write32(ROBO_VLAN_PAGE, ROBO_VLAN_WRITE_5350, 0);
 		else
 			robo_write16(ROBO_VLAN_PAGE, ROBO_VLAN_WRITE, 0);
-		robo_write16(ROBO_VLAN_PAGE, (is_5350 ? ROBO_VLAN_TABLE_ACCESS_5350 : ROBO_VLAN_TABLE_ACCESS), val16);
+		robo_write16(ROBO_VLAN_PAGE, robo.is_5350 ? ROBO_VLAN_TABLE_ACCESS_5350 :
+							    ROBO_VLAN_TABLE_ACCESS,
+			     val16);
 	}
 
 	/* reset ports to a known good state */
 	for (j = 0; j < d->ports; j++) {
-		robo_write16(ROBO_CTRL_PAGE, port[j], 0x0000);
+		robo_write16(ROBO_CTRL_PAGE, robo.port[j], 0x0000);
 		robo_write16(ROBO_VLAN_PAGE, ROBO_VLAN_PORT0_DEF_TAG + (j << 1), 0);
 	}
 
@@ -423,6 +516,7 @@ static int handle_reset(void *driver, char *buf, int nr)
 static int __init robo_init(void)
 {
 	int notfound = 1;
+	char *device;
 
 	device = strdup("ethX");
 	for (device[3] = '0'; (device[3] <= '3') && notfound; device[3]++) {
@@ -434,26 +528,38 @@ static int __init robo_init(void)
 		kfree(device);
 		return -ENODEV;
 	} else {
-		switch_config cfg[] = {
-			{"enable", handle_enable_read, handle_enable_write},
-			{"enable_vlan", handle_enable_vlan_read, handle_enable_vlan_write},
-			{"reset", NULL, handle_reset},
-			{NULL, NULL, NULL}
+		static const switch_config cfg[] = {
+			{
+				.name	= "enable",
+				.read	= handle_enable_read,
+				.write	= handle_enable_write
+			}, {
+				.name	= "enable_vlan",
+				.read	= handle_enable_vlan_read,
+				.write	= handle_enable_vlan_write
+			}, {
+				.name	= "reset",
+				.read	= NULL,
+				.write	= handle_reset
+			}, { NULL, },
 		};
-		switch_config vlan[] = {
-			{"ports", handle_vlan_port_read, handle_vlan_port_write},
-			{NULL, NULL, NULL}
+		static const switch_config vlan[] = {
+			{
+				.name	= "ports",
+				.read	= handle_vlan_port_read,
+				.write	= handle_vlan_port_write
+			}, { NULL, },
 		};
 		switch_driver driver = {
-			name: DRIVER_NAME,
-			version: DRIVER_VERSION,
-			interface: device,
-			cpuport: 5,
-			ports: 6,
-			vlans: 16,
-			driver_handlers: cfg,
-			port_handlers: NULL,
-			vlan_handlers: vlan,
+			.name			= DRIVER_NAME,
+			.version		= DRIVER_VERSION,
+			.interface		= device,
+			.cpuport		= 5,
+			.ports			= 6,
+			.vlans			= 16,
+			.driver_handlers	= cfg,
+			.port_handlers		= NULL,
+			.vlan_handlers		= vlan,
 		};
 
 		return switch_register_driver(&driver);
@@ -463,7 +569,7 @@ static int __init robo_init(void)
 static void __exit robo_exit(void)
 {
 	switch_unregister_driver(DRIVER_NAME);
-	kfree(device);
+	kfree(robo.device);
 }