diff options
author | Hauke Mehrtens <hauke@openwrt.org> | 2013-02-13 16:03:27 +0000 |
---|---|---|
committer | Hauke Mehrtens <hauke@openwrt.org> | 2013-02-13 16:03:27 +0000 |
commit | 9f68dd6769936a51c24dbd7d5f838d1415005e4a (patch) | |
tree | 0019dc6d378323641c5a1cb4dcb8237ded29315e | |
parent | e2fabaf65cb0d1359bd70fd68904e6a10a678eb2 (diff) | |
download | upstream-9f68dd6769936a51c24dbd7d5f838d1415005e4a.tar.gz upstream-9f68dd6769936a51c24dbd7d5f838d1415005e4a.tar.bz2 upstream-9f68dd6769936a51c24dbd7d5f838d1415005e4a.zip |
switch: reverse if else in handle_vlan_port_read()
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@35581 3c298f89-4303-0410-b956-a3cf2f4a3e73
-rw-r--r-- | package/switch/src/switch-robo.c | 30 |
1 files changed, 16 insertions, 14 deletions
diff --git a/package/switch/src/switch-robo.c b/package/switch/src/switch-robo.c index 9c78d8281f..0ab2f69341 100644 --- a/package/switch/src/switch-robo.c +++ b/package/switch/src/switch-robo.c @@ -419,17 +419,17 @@ static int handle_vlan_port_read(void *driver, char *buf, int nr) val16 = (nr) /* vlan */ | (0 << 12) /* read */ | (1 << 13) /* enable */; - if (!robo.is_5365) { - u32 val32; - robo_write16(ROBO_VLAN_PAGE, ROBO_VLAN_TABLE_ACCESS, val16); + if (robo.is_5365) { + robo_write16(ROBO_VLAN_PAGE, ROBO_VLAN_TABLE_ACCESS_5365, val16); /* actual read */ - val32 = robo_read32(ROBO_VLAN_PAGE, ROBO_VLAN_READ); - if ((val32 & (1 << 20)) /* valid */) { + val16 = robo_read16(ROBO_VLAN_PAGE, ROBO_VLAN_READ); + if ((val16 & (1 << 14)) /* valid */) { for (j = 0; j < 6; j++) { - if (val32 & (1 << j)) { + if (val16 & (1 << j)) { len += sprintf(buf + len, "%d", j); - if (val32 & (1 << (j + 6))) { - if (j == 5) buf[len++] = 'u'; + if (val16 & (1 << (j + 7))) { + if (j == 5) + buf[len++] = 'u'; } else { buf[len++] = 't'; if (robo_read16(ROBO_VLAN_PAGE, ROBO_VLAN_PORT0_DEF_TAG + (j << 1)) == nr) @@ -441,15 +441,17 @@ static int handle_vlan_port_read(void *driver, char *buf, int nr) len += sprintf(buf + len, "\n"); } } else { - robo_write16(ROBO_VLAN_PAGE, ROBO_VLAN_TABLE_ACCESS_5365, val16); + u32 val32; + robo_write16(ROBO_VLAN_PAGE, ROBO_VLAN_TABLE_ACCESS, val16); /* actual read */ - val16 = robo_read16(ROBO_VLAN_PAGE, ROBO_VLAN_READ); - if ((val16 & (1 << 14)) /* valid */) { + val32 = robo_read32(ROBO_VLAN_PAGE, ROBO_VLAN_READ); + if ((val32 & (1 << 20)) /* valid */) { for (j = 0; j < 6; j++) { - if (val16 & (1 << j)) { + if (val32 & (1 << j)) { len += sprintf(buf + len, "%d", j); - if (val16 & (1 << (j + 7))) { - if (j == 5) buf[len++] = 'u'; + if (val32 & (1 << (j + 6))) { + if (j == 5) + buf[len++] = 'u'; } else { buf[len++] = 't'; if (robo_read16(ROBO_VLAN_PAGE, ROBO_VLAN_PORT0_DEF_TAG + (j << 1)) == nr) |