From patchwork Sun Jul 5 23:15:50 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 234837 Delivered-To: patch@linaro.org Received: by 2002:a54:3249:0:0:0:0:0 with SMTP id g9csp3789815ecs; Sun, 5 Jul 2020 16:16:19 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy03CNSqqTO+dhJiRpQ9uE6aKS2YYNds3BEuiQvWSPK3GtLCjRW22JPOXkIR9dAMZNrsfPs X-Received: by 2002:a05:6402:1250:: with SMTP id l16mr53712941edw.362.1593990979039; Sun, 05 Jul 2020 16:16:19 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1593990979; cv=none; d=google.com; s=arc-20160816; b=ayGsGLwMrN3NMb4Zu+NyD56L7xgKts2n5VRSPj702UlgAVt48f22GP1uaoAM0kwQO5 uDtHG07I5ynpeacDK51mk4F61X/mTYMmgRYUqEFEYHVUvE7m5ErRHXW3ach+qwmcKNrf 5ccmjRIBqU9Nav7VKVyrEkqWSqo5JtXo7YSL5HbzMRi9sFPXAu1eYWUE7is/3DfsNt8I AjYamIQ0FWOmWSVgW5zUl8JRQd6535ADx3Z34fERxJtDzWUOCJENenwpPP6ooCNTmFov SMj+k8qThCMQhgu0uU3RPHq0anlLnYvMxwCoqQbn6U4rqUm9EJH768IhmmIhNgHKB2ti sR5Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=bEnvRIle9ouYWtzoth+BrQzE2gQE9/8+m5/d/sfeHBA=; b=M84MsxCrD72aCMSum3aaEknAoF4VndfrW/XbF4ecpc7gNpkVQJdQ6ByBDpk+cOol9r MwJq0d5OM0HQ3Sxg/e07ov+D14912mky8ewHEquMAYxTqZKNsTTvSmZ19cjEj+lyQB57 qjNUv+2UZVNPSX86ez/Uy/0lXh8YdAXYcnazrBEC7lsZ2ZhZrHf0D8VjkjqJHdoWxeWL rQskLqx8xFqsc3ucEzaktG0mXhaAKzhQw2KdU0p9zKBdy4Go/7RSs9pp/WOU6oN6+tIZ 8NMWKPCCFQt35rr6kZf69SNdkjkbNi14ijQfOE/htxDXOjJARoL4QcMP9x+WQb0cLkUL mebg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=TzjSZww2; spf=pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=netdev-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id dn17si15678910ejc.556.2020.07.05.16.16.18; Sun, 05 Jul 2020 16:16:19 -0700 (PDT) Received-SPF: pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=TzjSZww2; spf=pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=netdev-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728238AbgGEXQJ (ORCPT + 9 others); Sun, 5 Jul 2020 19:16:09 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56930 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728220AbgGEXQG (ORCPT ); Sun, 5 Jul 2020 19:16:06 -0400 Received: from mail-lf1-x143.google.com (mail-lf1-x143.google.com [IPv6:2a00:1450:4864:20::143]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 45FD2C061794 for ; Sun, 5 Jul 2020 16:16:06 -0700 (PDT) Received: by mail-lf1-x143.google.com with SMTP id k15so21554333lfc.4 for ; Sun, 05 Jul 2020 16:16:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=bEnvRIle9ouYWtzoth+BrQzE2gQE9/8+m5/d/sfeHBA=; b=TzjSZww2QTqv2PaavxaJsKSzwDtKnRMVtuXVlJyqgWjSVW8lj5c7cXA3mv8CEEMMw8 4dSo/ymC41Sdhx98vBWE1PI/SuwHkC+zeU5dpfJ3Nc9Rpa4IQS35zzClCv5iNw/kkv76 Ji6lSzGhLSf1t+LLTEsgKBiZDbVLhklV7bdR5STby9/XQ1qYtApqGJnJ58NF58/KSRFN NYTtDs/Tah11mOxkewCAPgpxmAwvgcL6X+nC942zDyCnEdlW1BjeoZlh2m8x5yLZsKO1 RXUd/g2aUfQXL/KtUotM5bF9U4lUhebl/RV/54VZO7OXRKCaDhT/yDUB12xhZEc7nKM3 Pkhw== 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:mime-version:content-transfer-encoding; bh=bEnvRIle9ouYWtzoth+BrQzE2gQE9/8+m5/d/sfeHBA=; b=dglD+tcWqdFUK9+QcyCVbWVffK12TeXfGl+d6DZwJffkViuOiiwR2s6rCGoTZaSIj5 wVAeAcQtpr6tucoao5padhxolUlWVRI1qYrqJ9zUkjqq46BKm0Zz+I4gNfsPI0fM9Ifx cOaEr1aLVYvF90a18zAVqA4iWLL8xkbtq7s35zG/vtL6SD5i7oX+clgjvJOHtBY4D+PC CSe18COqHxY0R5lczr3O2Hs8fO1f2Fb68GIUYsxzCSzwlY9wAZZbcJ/f71fD01wiitPB S6Xy+PiqhF190299m8EYmbJLb+a+KyVbwCFivw4Ez/JV14wKLw0+DZa8DI0mvEFBLh+p vWTQ== X-Gm-Message-State: AOAM531NQ0SJrLLzRc7Vs5bMbXJ41Zg16lhIlAoP0cK53pAy79E4ZE4T Le0lInbEq1XYiTviSvu3FV4Hqw== X-Received: by 2002:ac2:5a5e:: with SMTP id r30mr28480711lfn.30.1593990964759; Sun, 05 Jul 2020 16:16:04 -0700 (PDT) Received: from localhost.bredbandsbolaget (c-92d7225c.014-348-6c756e10.bbcust.telenor.se. [92.34.215.146]) by smtp.gmail.com with ESMTPSA id f14sm8439410lfa.35.2020.07.05.16.16.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 05 Jul 2020 16:16:04 -0700 (PDT) From: Linus Walleij To: Andrew Lunn , Vivien Didelot , Florian Fainelli , netdev@vger.kernel.org, "David S . Miller" Cc: Linus Walleij , DENG Qingfang , Mauri Sandberg Subject: [net-next PATCH 5/5 v3] net: dsa: rtl8366: Use top VLANs for default Date: Mon, 6 Jul 2020 01:15:50 +0200 Message-Id: <20200705231550.77946-6-linus.walleij@linaro.org> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20200705231550.77946-1-linus.walleij@linaro.org> References: <20200705231550.77946-1-linus.walleij@linaro.org> MIME-Version: 1.0 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The RTL8366 DSA switches will not work unless we set up a default VLAN for each port. We are currently using e.g. VLAN 1..6 for a 5-port switch as default VLANs. This is not very helpful for users, move it to allocate the top VLANs for default instead, for example on RTL8366RB there are 16 VLANs so instead of using VLAN 1..6 as default use VLAN 10..15 so VLAN 1 thru VLAN 9 is available for users. Cc: DENG Qingfang Cc: Mauri Sandberg Reviewed-by: Andrew Lunn Signed-off-by: Linus Walleij --- ChangeLog v2->v3: - Collect Andrew's reviewed-by. ChangeLog v1->v2: - Rebase on v5.8-rc1. --- drivers/net/dsa/rtl8366.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.26.2 diff --git a/drivers/net/dsa/rtl8366.c b/drivers/net/dsa/rtl8366.c index a000d458d121..06adcf68ff8d 100644 --- a/drivers/net/dsa/rtl8366.c +++ b/drivers/net/dsa/rtl8366.c @@ -260,8 +260,8 @@ static int rtl8366_set_default_vlan_and_pvid(struct realtek_smi *smi, u16 vid; int ret; - /* This is the reserved default VLAN for this port */ - vid = port + 1; + /* Use the top VLANs for per-port default VLAN */ + vid = smi->num_vlan_mc - smi->num_ports + port; if (port == smi->cpu_port) /* For the CPU port, make all ports members of this