From patchwork Fri Feb 15 20:16:49 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Florian Fainelli X-Patchwork-Id: 1043162 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="dzT0s3B3"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 441PgM6n43z9sDr for ; Sat, 16 Feb 2019 07:17:03 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2387612AbfBOURC (ORCPT ); Fri, 15 Feb 2019 15:17:02 -0500 Received: from mail-yb1-f196.google.com ([209.85.219.196]:44138 "EHLO mail-yb1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729019AbfBOURB (ORCPT ); Fri, 15 Feb 2019 15:17:01 -0500 Received: by mail-yb1-f196.google.com with SMTP id j85so934798ybg.11 for ; Fri, 15 Feb 2019 12:17:01 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=y2qm7A+/CCgrfTstiWJF722XPuBVch01JWCyDALceMo=; b=dzT0s3B3ybv9FeE5+8hg3GWSrZUa1ySJtaIo1DnNRbBTgb4YAdOeJH6j0Mven+e296 l7R0ME5BJ8H7Hp5/5tmPM3TuRKbWHj6IxFA+dV6Tp/iIFEcxhz61QV20O33CRTIMSwwi q5AryqbkHJ2ALf/ZLwMWtJ0XvSsRhzhwzk6cs8WpC1QToLfKSdAT+/UtIL35a88GZh/5 cPp69zHDRIY77O9A/3ZoFaTDyky4XEetst9VZcFBvzE982B3q8FVIszPd9E71nIXkYvE DeOG4tSFoAlKfrPBv/BdUIcBxRKpktOyN1sLnyuPZ47PVri9iTcHf0f6IP4z5pBrTEMb NRkA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=y2qm7A+/CCgrfTstiWJF722XPuBVch01JWCyDALceMo=; b=ElmST1TBjbzrdrUyRTF2L3AKxht4Pah9tuQQmNJRVc6w6M5T1W/XHwfMF/TBfZnXzc bZoikjbVW9lz7C9ScyH4dWkHe/eUOvoIlHUEGc1EoLtxiXhoaX76ENUxuSzqPQ92iA8G 3ZfEaEzerreXeoiktmTSUe4g4Z+i4Ws7bfHzkAJkA3+BTyP9f9HpCJ6t/aa2KuAEHyfh IcPw1/Rh+sNyXzdR4V5cqO0YIcD7eXpVLd6CIfQP9d8qyfc90zsVzA2W44T0Zd7smYaZ 61kMQV6TuQilpa5OhmvFKMvvlRVmxRhE/ox+lxkfBB0y701j9SYsi0hUenSQIA0XH3+E F05Q== X-Gm-Message-State: AHQUAubfODz2kB6IU3Sr63cfx3LtM35clCXZXXvOPNUMbxYFBZBzj1f8 upeN3jx9Zz4hphzQAzWxN4RGVDci X-Google-Smtp-Source: AHgI3IbeiX3PvOcnXxpkj9lquga7+31Ax2YeUQgIfu3nrmJd+EIybLbthFROIc36flFChfhuDm8yQQ== X-Received: by 2002:a25:c9c6:: with SMTP id z189mr8559481ybf.377.1550261819795; Fri, 15 Feb 2019 12:16:59 -0800 (PST) Received: from fainelli-desktop.igp.broadcom.net ([192.19.223.250]) by smtp.gmail.com with ESMTPSA id y67sm2889574ywf.89.2019.02.15.12.16.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 15 Feb 2019 12:16:59 -0800 (PST) From: Florian Fainelli To: netdev@vger.kernel.org Cc: davem@davemloft.net, andrew@lunn.ch, vivien.didelot@gmail.com, Florian Fainelli Subject: [PATCH net 1/5] net: dsa: b53: Fix default VLAN ID Date: Fri, 15 Feb 2019 12:16:49 -0800 Message-Id: <20190215201653.20988-2-f.fainelli@gmail.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20190215201653.20988-1-f.fainelli@gmail.com> References: <20190215201653.20988-1-f.fainelli@gmail.com> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org We were not consistent in how the default VID of a given port was defined, b53_br_leave() would make sure the VLAN ID would be either 0/1 depending on the switch generation, but b53_configure_vlan(), which is the default configuration would unconditionally set it to 1. The correct value is 1 for 5325/5365 series and 0 otherwise. To avoid repeating that mistake ever again, introduce a helper function: b53_default_pvid() to factor that out. Fixes: 967dd82ffc52 ("net: dsa: b53: Add support for Broadcom RoboSwitch") Signed-off-by: Florian Fainelli --- drivers/net/dsa/b53/b53_common.c | 29 ++++++++++++++++------------- 1 file changed, 16 insertions(+), 13 deletions(-) diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/dsa/b53/b53_common.c index 0e4bbdcc614f..964a9ec4652a 100644 --- a/drivers/net/dsa/b53/b53_common.c +++ b/drivers/net/dsa/b53/b53_common.c @@ -632,15 +632,25 @@ static void b53_enable_mib(struct b53_device *dev) b53_write8(dev, B53_MGMT_PAGE, B53_GLOBAL_CONFIG, gc); } +static u16 b53_default_pvid(struct b53_device *dev) +{ + if (is5325(dev) || is5365(dev)) + return 1; + else + return 0; +} + int b53_configure_vlan(struct dsa_switch *ds) { struct b53_device *dev = ds->priv; struct b53_vlan vl = { 0 }; - int i; + int i, def_vid; + + def_vid = b53_default_pvid(dev); /* clear all vlan entries */ if (is5325(dev) || is5365(dev)) { - for (i = 1; i < dev->num_vlans; i++) + for (i = def_vid; i < dev->num_vlans; i++) b53_set_vlan_entry(dev, i, &vl); } else { b53_do_vlan_op(dev, VTA_CMD_CLEAR); @@ -650,7 +660,7 @@ int b53_configure_vlan(struct dsa_switch *ds) b53_for_each_port(dev, i) b53_write16(dev, B53_VLAN_PAGE, - B53_VLAN_PORT_DEF_TAG(i), 1); + B53_VLAN_PORT_DEF_TAG(i), def_vid); if (!is5325(dev) && !is5365(dev)) b53_set_jumbo(dev, dev->enable_jumbo, false); @@ -1326,12 +1336,8 @@ int b53_vlan_del(struct dsa_switch *ds, int port, vl->members &= ~BIT(port); - if (pvid == vid) { - if (is5325(dev) || is5365(dev)) - pvid = 1; - else - pvid = 0; - } + if (pvid == vid) + pvid = b53_default_pvid(dev); if (untagged && !dsa_is_cpu_port(ds, port)) vl->untag &= ~(BIT(port)); @@ -1644,10 +1650,7 @@ void b53_br_leave(struct dsa_switch *ds, int port, struct net_device *br) b53_write16(dev, B53_PVLAN_PAGE, B53_PVLAN_PORT_MASK(port), pvlan); dev->ports[port].vlan_ctl_mask = pvlan; - if (is5325(dev) || is5365(dev)) - pvid = 1; - else - pvid = 0; + pvid = b53_default_pvid(dev); /* Make this port join all VLANs without VLAN entries */ if (is58xx(dev)) {