From patchwork Sun May 30 22:59:35 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vladimir Oltean X-Patchwork-Id: 12288493 X-Patchwork-Delegate: kuba@kernel.org Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 00F83C47098 for ; Sun, 30 May 2021 23:00:07 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id C3A8E611EE for ; Sun, 30 May 2021 23:00:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230004AbhE3XBg (ORCPT ); Sun, 30 May 2021 19:01:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35576 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230002AbhE3XBd (ORCPT ); Sun, 30 May 2021 19:01:33 -0400 Received: from mail-ed1-x52b.google.com (mail-ed1-x52b.google.com [IPv6:2a00:1450:4864:20::52b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1CE72C06174A for ; Sun, 30 May 2021 15:59:53 -0700 (PDT) Received: by mail-ed1-x52b.google.com with SMTP id df21so11360603edb.3 for ; Sun, 30 May 2021 15:59:53 -0700 (PDT) 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 :mime-version:content-transfer-encoding; bh=sEYZwQwKRAcGo0HP4idQ4bX+Dp8jsmR4SWn1JE0MSe0=; b=e+fNNNGaPKAqBB36Xy2dyLsqbtJg5iZt4KYuBqe5ygXV5aFkVff2UzVLWdPn2MypdY 2wySJ2LzF6Ub3OuCexw06h9Mo/Ir698DEaK7KF/2PM05e7KzdoAy6WtXanKUPQuMDJ4S 9BrRXC07rxMvdGLhiN58vs4m3pjBt0HecQQrSR+fizg4mhCSqt9EUFt86+Kx3O4ts08t bkMk/pV8sijKy/EiRWUAkGXwmUg9r36CQUfoWpec2u0nKz0dwLp7lejmBEF/K7yD8ufk 9xhJ3igbAO37g8B3KI7Z+rMdhXWaxU99uKg1txL0nvMq+awIA07Nsy2i7toJ8CfCgqsN J9YA== 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=sEYZwQwKRAcGo0HP4idQ4bX+Dp8jsmR4SWn1JE0MSe0=; b=dJFdR0VvTl4nRy9dAu0Yp+HwBwVcxzd5Eky9IykR459c85U3N2okJFlCQa7Qc1OZzG 20f+jdBbvvJ8UE2pr3zZbyS85UcjfKn4tbTvZQPjK/VpubKLTDkNmt38DdlvAUaqgzQG TnaNpNHtcQqlBfV03GvwXtt+wemTXsWZYzHaW5l1DIi1Il+DZcb+EZocmRlbABW1lYOU PMVka+bFIoiBuDFP9k6R7GwwjEeegFo09nC9xGGEuksXKCUf5ic7ujyOHcFeotKwzBc7 qtoVmaGVjdSM89RFQNFAfnT3gsYn1WSHUOQ4hpUw70hUF2WIwykdVt4L+OS3YSKI7Ptu cJ6g== X-Gm-Message-State: AOAM533PmZL3trsiIxU1ZlzttaISj4cbd6+4j0wSncvGzK2KcRbjtoza NdH+97PAJhU4eSVlk4snjug= X-Google-Smtp-Source: ABdhPJyZlSGA7okcSOvcGdvac1DjTBx4HFB6k6jazGBxrlE09lVQ23h4L0ZjAq4JmLnl5eD8FnTE2w== X-Received: by 2002:a50:9fe5:: with SMTP id c92mr14760797edf.93.1622415591755; Sun, 30 May 2021 15:59:51 -0700 (PDT) Received: from localhost.localdomain ([188.26.52.84]) by smtp.gmail.com with ESMTPSA id c6sm5135120eje.9.2021.05.30.15.59.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 30 May 2021 15:59:51 -0700 (PDT) From: Vladimir Oltean To: Jakub Kicinski , "David S. Miller" , netdev@vger.kernel.org Cc: Florian Fainelli , Andrew Lunn , Vivien Didelot , Vladimir Oltean Subject: [PATCH v3 net-next 4/8] net: dsa: sja1105: cache the phy-mode port property Date: Mon, 31 May 2021 01:59:35 +0300 Message-Id: <20210530225939.772553-5-olteanv@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210530225939.772553-1-olteanv@gmail.com> References: <20210530225939.772553-1-olteanv@gmail.com> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org From: Vladimir Oltean So far we've succeeded in operating without keeping a copy of the phy-mode in the driver, since we already have the static config and we can look at the xMII Mode Parameters Table which already holds that information. But with the SJA1110, we cannot make the distinction between sgmii and 2500base-x, because to the hardware's static config, it's all SGMII. So add a phy_mode property per port inside struct sja1105_private. Signed-off-by: Vladimir Oltean Reviewed-by: Florian Fainelli --- Changes in v3: None. Changes in v2: None. drivers/net/dsa/sja1105/sja1105.h | 1 + drivers/net/dsa/sja1105/sja1105_main.c | 24 +++--------------------- 2 files changed, 4 insertions(+), 21 deletions(-) diff --git a/drivers/net/dsa/sja1105/sja1105.h b/drivers/net/dsa/sja1105/sja1105.h index 830ea5ca359f..d5c0217b1f65 100644 --- a/drivers/net/dsa/sja1105/sja1105.h +++ b/drivers/net/dsa/sja1105/sja1105.h @@ -210,6 +210,7 @@ struct sja1105_private { struct sja1105_static_config static_config; bool rgmii_rx_delay[SJA1105_MAX_NUM_PORTS]; bool rgmii_tx_delay[SJA1105_MAX_NUM_PORTS]; + phy_interface_t phy_mode[SJA1105_MAX_NUM_PORTS]; bool best_effort_vlan_filtering; unsigned long learn_ena; unsigned long ucast_egress_floods; diff --git a/drivers/net/dsa/sja1105/sja1105_main.c b/drivers/net/dsa/sja1105/sja1105_main.c index c8d93d810421..e4932243d0d3 100644 --- a/drivers/net/dsa/sja1105/sja1105_main.c +++ b/drivers/net/dsa/sja1105/sja1105_main.c @@ -871,6 +871,8 @@ static int sja1105_parse_ports_node(struct sja1105_private *priv, ports[index].role = XMII_MAC; else if (of_property_read_bool(child, "sja1105,role-phy")) ports[index].role = XMII_PHY; + + priv->phy_mode[index] = phy_mode; } return 0; @@ -1081,27 +1083,7 @@ static int sja1105_adjust_port_config(struct sja1105_private *priv, int port, static bool sja1105_phy_mode_mismatch(struct sja1105_private *priv, int port, phy_interface_t interface) { - struct sja1105_xmii_params_entry *mii; - sja1105_phy_interface_t phy_mode; - - mii = priv->static_config.tables[BLK_IDX_XMII_PARAMS].entries; - phy_mode = mii->xmii_mode[port]; - - switch (interface) { - case PHY_INTERFACE_MODE_MII: - return (phy_mode != XMII_MODE_MII); - case PHY_INTERFACE_MODE_RMII: - return (phy_mode != XMII_MODE_RMII); - case PHY_INTERFACE_MODE_RGMII: - case PHY_INTERFACE_MODE_RGMII_ID: - case PHY_INTERFACE_MODE_RGMII_RXID: - case PHY_INTERFACE_MODE_RGMII_TXID: - return (phy_mode != XMII_MODE_RGMII); - case PHY_INTERFACE_MODE_SGMII: - return (phy_mode != XMII_MODE_SGMII); - default: - return true; - } + return priv->phy_mode[port] != interface; } static void sja1105_mac_config(struct dsa_switch *ds, int port,