From: Gabor Juhos <juhosg@openwrt.org>
Date: Tue, 13 Mar 2012 17:33:12 +0000 (+0000)
Subject: generic: ar8216: merge multiple if statements in ar8216_hw_apply
X-Git-Tag: reboot~14558
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=e3af7c82f880aea86ade6de05904523c699ba60b;p=openwrt%2Fopenwrt.git

generic: ar8216: merge multiple if statements in ar8216_hw_apply

SVN-Revision: 30928
---

diff --git a/target/linux/generic/files/drivers/net/phy/ar8216.c b/target/linux/generic/files/drivers/net/phy/ar8216.c
index ffa561b36c..4b67cb6cbb 100644
--- a/target/linux/generic/files/drivers/net/phy/ar8216.c
+++ b/target/linux/generic/files/drivers/net/phy/ar8216.c
@@ -621,24 +621,18 @@ ar8216_hw_apply(struct switch_dev *dev)
 		int egress, ingress;
 		int pvid;
 
-		if (priv->vlan)
-			pvid = priv->vlan_id[priv->pvid[i]];
-		else
-			pvid = i;
-
 		if (priv->vlan) {
+			pvid = priv->vlan_id[priv->pvid[i]];
 			if (priv->vlan_tagged & (1 << i))
 				egress = AR8216_OUT_ADD_VLAN;
 			else
 				egress = AR8216_OUT_STRIP_VLAN;
+			ingress = AR8216_IN_SECURE;
 		} else {
+			pvid = i;
 			egress = AR8216_OUT_KEEP;
-		}
-
-		if (priv->vlan)
-			ingress = AR8216_IN_SECURE;
-		else
 			ingress = AR8216_IN_PORT_ONLY;
+		}
 
 		if (priv->chip == AR8236)
 			ar8236_setup_port(priv, i, egress, ingress, portmask[i],