From patchwork Fri Jun 5 18:10:02 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jonathan McDowell X-Patchwork-Id: 1304342 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=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org (client-ip=23.128.96.18; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=earth.li Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=earth.li header.i=@earth.li header.a=rsa-sha256 header.s=the header.b=LxfpePMG; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by ozlabs.org (Postfix) with ESMTP id 49drKK55Pjz9sSd for ; Sat, 6 Jun 2020 04:10:13 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727797AbgFESKK (ORCPT ); Fri, 5 Jun 2020 14:10:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57250 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726077AbgFESKK (ORCPT ); Fri, 5 Jun 2020 14:10:10 -0400 Received: from the.earth.li (the.earth.li [IPv6:2a00:1098:86:4d:c0ff:ee:15:900d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E10A4C08C5C2; Fri, 5 Jun 2020 11:10:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=earth.li; s=the; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject :To:From:Date:Sender:Reply-To:Cc:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=uX7WkPPY5DHynO/kH95BbeRu3QRcHZ3RT6LKeg6R+k0=; b=LxfpePMGvwk9Hyci3o22CrWB8f e7a+sb2Wmh8g0jqcKTPAywOHbaxyy0yxzEQwFJ5tS50zNMRIrz8wn6tID27Nwr23Hs8cZrFUBCpiK 2qv2iFaR1gTfIHmdg4YOY/RojDX8uvWUsCuoobdK/oZ0yXNspTJMvKM/qM0QQ0N8UBLpZoZTZKBVT 3eN3/rBYXEPvkepqKXVEnTvKTn2ztfISyMilYPUK/ZOj7EBzOXaWzRNOcsjjcfHkOX7JYVBAdY0Vc fPGdcFgmdjAcHsD59cxaqZt8fGpalQdiJpJMFaVWvOik+JC2qZbIgpSDq4tXxpK8tjM3DusiBHZyo 3SsKK73g==; Received: from [2001:4d48:ad59:1409:4::2] (helo=youmian.o362.us) by the.earth.li with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jhGn8-0003Cr-ST; Fri, 05 Jun 2020 19:10:06 +0100 Date: Fri, 5 Jun 2020 19:10:02 +0100 From: Jonathan McDowell To: Andrew Lunn , Vivien Didelot , Florian Fainelli , "David S. Miller" , Jakub Kicinski , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Rob Herring , devicetree@vger.kernel.org Subject: [PATCH 1/2] dt-bindings: net: dsa: qca8k: document SGMII properties Message-ID: References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org This patch documents the qca8k's SGMII related properties that allow configuration of the SGMII port. Signed-off-by: Jonathan McDowell --- Documentation/devicetree/bindings/net/dsa/qca8k.txt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Documentation/devicetree/bindings/net/dsa/qca8k.txt b/Documentation/devicetree/bindings/net/dsa/qca8k.txt index ccbc6d89325d..9e7d74a248ad 100644 --- a/Documentation/devicetree/bindings/net/dsa/qca8k.txt +++ b/Documentation/devicetree/bindings/net/dsa/qca8k.txt @@ -11,7 +11,11 @@ Required properties: Optional properties: +- disable-serdes-autoneg: Boolean, disables auto-negotiation on the SerDes - reset-gpios: GPIO to be used to reset the whole device +- sgmii-delay: Boolean, presence delays SGMII clock by 2ns +- sgmii-mode: String, operation mode of the SGMII interface. + Supported values are: "basex", "mac", "phy". Subnodes: From patchwork Fri Jun 5 18:10:58 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jonathan McDowell X-Patchwork-Id: 1304344 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=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org (client-ip=23.128.96.18; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=earth.li Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=earth.li header.i=@earth.li header.a=rsa-sha256 header.s=the header.b=XtyZiFT4; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by ozlabs.org (Postfix) with ESMTP id 49drLP0qH9z9sRW for ; Sat, 6 Jun 2020 04:11:09 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728076AbgFESLG (ORCPT ); Fri, 5 Jun 2020 14:11:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57398 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726351AbgFESLF (ORCPT ); Fri, 5 Jun 2020 14:11:05 -0400 Received: from the.earth.li (the.earth.li [IPv6:2a00:1098:86:4d:c0ff:ee:15:900d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 59A1EC08C5C2; Fri, 5 Jun 2020 11:11:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=earth.li; s=the; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject :To:From:Date:Sender:Reply-To:Cc:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=l6oTlisUnQD7Ck9w10S9cYUiWyHUo3TvSfoCMjyvw0U=; b=XtyZiFT4aYxWhsDOc9z5aE2fm/ 20xvQOugblpAftsyUfYokEtrRfdMvrNXyt4rkLUvoVe37C70EZ9r7A8oVzuBynL3hrq4lGpNNFpDz 1FtMpyN//isLC+EWXCJc5KPw0Mhlr7gyBoCLx95oJh0vVbLz6CBO4bSRGqveHQIYUad76dNuF5vxW MRDJ+hAkefepVvg9UUOYm3KrSUDCHpIlawiWHg0UmdYlIc5Q8CG1Xiwb/xIt0hSqWecgJrz0anuAQ s0bhNS29s+0lcIMIahZPraM3oQRxvAzQM0D0lqN/AwCVdjJLAGoC0nYJvNz7BSMIA6UvHVz5PAM7D l9/1zhQQ==; Received: from [2001:4d48:ad59:1409:4::2] (helo=youmian.o362.us) by the.earth.li with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jhGo2-0003GS-40; Fri, 05 Jun 2020 19:11:02 +0100 Date: Fri, 5 Jun 2020 19:10:58 +0100 From: Jonathan McDowell To: Andrew Lunn , Vivien Didelot , Florian Fainelli , "David S. Miller" , Jakub Kicinski , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 2/2] net: dsa: qca8k: introduce SGMII configuration options Message-ID: <8ddd76e484e1bedd12c87ea0810826b60e004a65.1591380105.git.noodles@earth.li> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The QCA8337(N) has an SGMII port which can operate in MAC, PHY or BASE-X mode depending on what it's connected to (e.g. CPU vs external PHY or SFP). At present the driver does no configuration of this port even if it is selected. Add support for making sure the SGMII is enabled if it's in use, and device tree support for configuring the connection details. Signed-off-by: Jonathan McDowell Reported-by: kernel test robot Reported-by: Dan Carpenter --- drivers/net/dsa/qca8k.c | 44 ++++++++++++++++++++++++++++++++++++++++- drivers/net/dsa/qca8k.h | 12 +++++++++++ 2 files changed, 55 insertions(+), 1 deletion(-) diff --git a/drivers/net/dsa/qca8k.c b/drivers/net/dsa/qca8k.c index 9f4205b4439b..5b7979aca9b9 100644 --- a/drivers/net/dsa/qca8k.c +++ b/drivers/net/dsa/qca8k.c @@ -418,6 +418,40 @@ qca8k_mib_init(struct qca8k_priv *priv) mutex_unlock(&priv->reg_mutex); } +static int +qca8k_setup_sgmii(struct qca8k_priv *priv) +{ + const char *mode; + u32 val; + + val = qca8k_read(priv, QCA8K_REG_SGMII_CTRL); + + val |= QCA8K_SGMII_EN_PLL | QCA8K_SGMII_EN_RX | + QCA8K_SGMII_EN_TX | QCA8K_SGMII_EN_SD; + + if (of_property_read_bool(priv->dev->of_node, "sgmii-delay")) + val |= QCA8K_SGMII_CLK125M_DELAY; + + if (of_property_read_string(priv->dev->of_node, "sgmii-mode", &mode)) { + val &= ~QCA8K_SGMII_MODE_CTRL_MASK; + + if (!strcasecmp(mode, "basex")) { + val |= QCA8K_SGMII_MODE_CTRL_BASEX; + } else if (!strcasecmp(mode, "mac")) { + val |= QCA8K_SGMII_MODE_CTRL_MAC; + } else if (!strcasecmp(mode, "phy")) { + val |= QCA8K_SGMII_MODE_CTRL_PHY; + } else { + pr_err("Unrecognised SGMII mode %s\n", mode); + return -EINVAL; + } + } + + qca8k_write(priv, QCA8K_REG_SGMII_CTRL, val); + + return 0; +} + static int qca8k_set_pad_ctrl(struct qca8k_priv *priv, int port, int mode) { @@ -458,7 +492,8 @@ qca8k_set_pad_ctrl(struct qca8k_priv *priv, int port, int mode) break; case PHY_INTERFACE_MODE_SGMII: qca8k_write(priv, reg, QCA8K_PORT_PAD_SGMII_EN); - break; + + return qca8k_setup_sgmii(priv); default: pr_err("xMII mode %d not supported\n", mode); return -EINVAL; @@ -661,6 +696,13 @@ qca8k_setup(struct dsa_switch *ds) if (ret) return ret; + if (of_property_read_bool(priv->dev->of_node, + "disable-serdes-autoneg")) { + mask = qca8k_read(priv, QCA8K_REG_PWS) | + QCA8K_PWS_SERDES_AEN_DIS; + qca8k_write(priv, QCA8K_REG_PWS, mask); + } + /* Initialize CPU port pad mode (xMII type, delays...) */ ret = of_get_phy_mode(dsa_to_port(ds, QCA8K_CPU_PORT)->dn, &phy_mode); if (ret) { diff --git a/drivers/net/dsa/qca8k.h b/drivers/net/dsa/qca8k.h index 42d6ea24eb14..cd97c212f3f8 100644 --- a/drivers/net/dsa/qca8k.h +++ b/drivers/net/dsa/qca8k.h @@ -36,6 +36,8 @@ #define QCA8K_MAX_DELAY 3 #define QCA8K_PORT_PAD_RGMII_RX_DELAY_EN BIT(24) #define QCA8K_PORT_PAD_SGMII_EN BIT(7) +#define QCA8K_REG_PWS 0x010 +#define QCA8K_PWS_SERDES_AEN_DIS BIT(7) #define QCA8K_REG_MODULE_EN 0x030 #define QCA8K_MODULE_EN_MIB BIT(0) #define QCA8K_REG_MIB 0x034 @@ -77,6 +79,16 @@ #define QCA8K_PORT_HDR_CTRL_ALL 2 #define QCA8K_PORT_HDR_CTRL_MGMT 1 #define QCA8K_PORT_HDR_CTRL_NONE 0 +#define QCA8K_REG_SGMII_CTRL 0x0e0 +#define QCA8K_SGMII_EN_PLL BIT(1) +#define QCA8K_SGMII_EN_RX BIT(2) +#define QCA8K_SGMII_EN_TX BIT(3) +#define QCA8K_SGMII_EN_SD BIT(4) +#define QCA8K_SGMII_CLK125M_DELAY BIT(7) +#define QCA8K_SGMII_MODE_CTRL_MASK (BIT(22) | BIT(23)) +#define QCA8K_SGMII_MODE_CTRL_BASEX 0 +#define QCA8K_SGMII_MODE_CTRL_PHY BIT(22) +#define QCA8K_SGMII_MODE_CTRL_MAC BIT(23) /* EEE control registers */ #define QCA8K_REG_EEE_CTRL 0x100