From 3814202702cc6ab529c1a0977ea5615c87ce48a8 Mon Sep 17 00:00:00 2001 From: mb Date: Wed, 27 Feb 2008 22:35:41 +0000 Subject: Fix the roboswitch code for the WRT350N git-svn-id: svn://svn.openwrt.org/openwrt/trunk@10531 3c298f89-4303-0410-b956-a3cf2f4a3e73 --- package/switch/src/switch-core.c | 10 +- package/switch/src/switch-core.h | 12 +- package/switch/src/switch-robo.c | 244 ++++++++++++++++++++++++++++----------- 3 files changed, 186 insertions(+), 80 deletions(-) (limited to 'package/switch/src') diff --git a/package/switch/src/switch-core.c b/package/switch/src/switch-core.c index 5eeb803aa..e0aa541f4 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 5292469c0..5a64efb75 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 009781dc0..5bcd85bd1 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 + * Copyright (C) 2008 Michael Buesch * Based on 'robocfg' by Oleg I. Vdovikin * * This program is free software; you can redistribute it and/or @@ -28,15 +29,18 @@ #include #include #include +#include #include #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); } -- cgit v1.2.3