From patchwork Thu Jan 11 16:19:27 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: POPESCU Catalin X-Patchwork-Id: 13517603 X-Patchwork-Delegate: kuba@kernel.org Received: from EUR02-AM0-obe.outbound.protection.outlook.com (mail-am0eur02on2099.outbound.protection.outlook.com [40.107.247.99]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8A71252F82; Thu, 11 Jan 2024 16:20:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=leica-geosystems.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=leica-geosystems.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=leica-geosystems.com header.i=@leica-geosystems.com header.b="cr5yCxDF" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=L1YAZT3A052JTJig41kSVRkBzHqe2uhjhgexY3nGRXf4Xi9dlx1CXtReLqADV7JLy999qnutm8httwMW9ZsCgUb/Xkl0j0kh8lo7D9VaEqu/IxOF9mcZObHZjsRwO6fdUBWFYZBqGeTyTsb0GSz/YNMzmYlVzBkeKVjFl2w29x1eG5rnByoB/3c3j9X2mh6utiyXVkDwsfBi94zHzhHcZ8xLb69r8en2MkiBcD1t8i4JCehI5t7pkMVT6Im5e6wqlOmX7wPx/pS/fiPJ7op4v0c0dzc5Y29yLlhU0ricfobvGw2ex11mGwVg+5l3DQpEA1NYjJg1lBgO1QA/WMFc8A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=bZmJu/MFCJvPTh6O8Bx6egjMmivl7bmsl9NwLHxy4pw=; b=bQ7aaFb93ssofWA2i1xoUUuVMMlXLfp6HgsnlelRPj3cchfyk7a3ozv/ygk+L6SoYsbRRP1sX1Ovrf+CBgKBIU+mUPPS/BIGiPYT8+y0uHcET3S+ajxw3YauZDMVDpWQnpPaS5GUv+CgDC3Ko5pHE7mN5iz9vtjm7cPAlp1JvpkottqnS8DUnx2UGTGUoiy29/zUatUY6QEeF6i3XrCiuhvcmhlQh8mTi0JyM/u9w0pXBqJFD48cKAPGbydFNkr8WjKGpy0gbj5aI5PMpY8puSiscedVIxCMSzhZUuN0g9Q39/6YzqtWF0tinYAX6pGQZsyH8t7LDGVIWBSyeJfzCQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 193.8.40.94) smtp.rcpttodomain=vger.kernel.org smtp.mailfrom=leica-geosystems.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=leica-geosystems.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=leica-geosystems.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=bZmJu/MFCJvPTh6O8Bx6egjMmivl7bmsl9NwLHxy4pw=; b=cr5yCxDF7ElU5xOY8lZZ9e1WU9kUKHJgbNlj02svz658svznaHj9m9qje3OZ7wGQ1ibZ6lui+VTMmv2j8uOnxac4Q63OV0awV5CW88jwDJrjHUq0XMOhkbBEUJPJm+ooVWDmeB8rV0u+LBGISarvsd0XE5TzmZw/cXJfITv1/c0= Received: from DU2PR04CA0037.eurprd04.prod.outlook.com (2603:10a6:10:234::12) by AM8PR06MB7764.eurprd06.prod.outlook.com (2603:10a6:20b:363::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7181.19; Thu, 11 Jan 2024 16:20:28 +0000 Received: from DB5PEPF00014B98.eurprd02.prod.outlook.com (2603:10a6:10:234:cafe::fa) by DU2PR04CA0037.outlook.office365.com (2603:10a6:10:234::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7181.18 via Frontend Transport; Thu, 11 Jan 2024 16:20:28 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 193.8.40.94) smtp.mailfrom=leica-geosystems.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=leica-geosystems.com; Received-SPF: Pass (protection.outlook.com: domain of leica-geosystems.com designates 193.8.40.94 as permitted sender) receiver=protection.outlook.com; client-ip=193.8.40.94; helo=hexagon.com; pr=C Received: from hexagon.com (193.8.40.94) by DB5PEPF00014B98.mail.protection.outlook.com (10.167.8.165) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7181.14 via Frontend Transport; Thu, 11 Jan 2024 16:20:28 +0000 Received: from aherlnxbspsrv01.lgs-net.com ([10.60.34.116]) by hexagon.com with Microsoft SMTPSVC(10.0.17763.1697); Thu, 11 Jan 2024 17:20:28 +0100 From: Catalin Popescu To: davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, afd@ti.com, andrew@lunn.ch, hkallweit1@gmail.com, linux@armlinux.org.uk Cc: netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Catalin Popescu Subject: [PATCH 3/3] net: phy: dp83826: add support for voltage tuning of logical levels Date: Thu, 11 Jan 2024 17:19:27 +0100 Message-Id: <20240111161927.3689084-3-catalin.popescu@leica-geosystems.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240111161927.3689084-1-catalin.popescu@leica-geosystems.com> References: <20240111161927.3689084-1-catalin.popescu@leica-geosystems.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-OriginalArrivalTime: 11 Jan 2024 16:20:28.0261 (UTC) FILETIME=[170B8150:01DA44AA] X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DB5PEPF00014B98:EE_|AM8PR06MB7764:EE_ X-MS-Office365-Filtering-Correlation-Id: 11169f95-f652-422b-9711-08dc12c139c6 X-SET-LOWER-SCL-SCANNER: YES X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Iqa4UR3jpGpJYGpdJO2wxVayIAF0wveM6VeadIhcQihe9n/ZWyEEpsEwZHto24P7P60KUuZ0NIulB+kgUJd1tjwz7xBg/GXwxgQCmMRdaf9fl+qiMR9cEWSh1yhTA5xYX/dfGd/tG1sgMRY1o/KfqimUxQGdW7wHLkrFbUN15mX13/aCTalMqIcR+5qUpW9wAaefV5pjKbSW8jNV8GGH9mV0wBd1bOl/sZ1kJFp6qak5lDLODkd9pwju7/wTf3HX0nX3UWfj6nx6MSLotnZxd2+/nR27d+ZdD+OQyJ35ToRI+iWfSVIazmgxWZnSGm5fGLwBcQjjhqpe0SjvcvkEtZkH6wZEqDC74eynuQgSlCZpM+27+vzb36qhUG25pmqv5MdSM//seowlv3NHmndYbF9hVu3o49GRJ6YwAmRfccghhOAYdCBCZpbqR6keZpeMvCfbP15TncpPJfbu9gVuORSCdZJWZj2cavsfucfALj88f50arSuRmot7W8Rk6bXby7bb7OZhjCmOnpVOnVKogWQ1drQV6KeOSRr8tWc80oBtaBlyrkx/BHA5+zadD0JnzecLca++ISdT2uijozB0XyQRjP4ZhavahJWLOP9c3mxz5FDafuff+MhID/EpImUQMR9nvIvKv6EkJcsPErr+MT21oocqaoGV7xfR1awlWn8r2b8JkE1jz6Gv13J1nmUAJ9rfPBLvZNBQ5laCUaKidSyo+hqbN3VCQiWAaemeXjkHWSC7O3ObKdUX4L/elGYlzCor927PVTmkOl22sQSQ7Q== X-Forefront-Antispam-Report: CIP:193.8.40.94;CTRY:CH;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:hexagon.com;PTR:ahersrvdom50.leica-geosystems.com;CAT:NONE;SFS:(13230031)(4636009)(136003)(396003)(346002)(376002)(39860400002)(230922051799003)(1800799012)(64100799003)(186009)(82310400011)(451199024)(36840700001)(46966006)(40470700004)(66899024)(336012)(26005)(1076003)(2616005)(36756003)(107886003)(82740400003)(86362001)(81166007)(356005)(83380400001)(47076005)(41300700001)(36860700001)(478600001)(921011)(44832011)(8936002)(8676002)(40480700001)(40460700003)(450100002)(4326008)(5660300002)(70206006)(70586007)(316002)(2906002);DIR:OUT;SFP:1102; X-OriginatorOrg: leica-geosystems.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2024 16:20:28.5665 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 11169f95-f652-422b-9711-08dc12c139c6 X-MS-Exchange-CrossTenant-Id: 1b16ab3e-b8f6-4fe3-9f3e-2db7fe549f6a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=1b16ab3e-b8f6-4fe3-9f3e-2db7fe549f6a;Ip=[193.8.40.94];Helo=[hexagon.com] X-MS-Exchange-CrossTenant-AuthSource: DB5PEPF00014B98.eurprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM8PR06MB7764 X-Patchwork-Delegate: kuba@kernel.org DP83826 offers the possibility to tune the voltage of logical levels of the MLT-3 encoded data. This is especially interesting when the data path is lossy and we want to increase the voltage levels to compensate the loss. Signed-off-by: Catalin Popescu --- drivers/net/phy/dp83822.c | 127 ++++++++++++++++++++++++++++++++++++-- 1 file changed, 123 insertions(+), 4 deletions(-) diff --git a/drivers/net/phy/dp83822.c b/drivers/net/phy/dp83822.c index b7cb71817780..e26aaacc7ea5 100644 --- a/drivers/net/phy/dp83822.c +++ b/drivers/net/phy/dp83822.c @@ -12,6 +12,7 @@ #include #include #include +#include #define DP83822_PHY_ID 0x2000a240 #define DP83825S_PHY_ID 0x2000a140 @@ -34,6 +35,10 @@ #define MII_DP83822_GENCFG 0x465 #define MII_DP83822_SOR1 0x467 +/* DP83826 specific registers */ +#define MII_DP83826_VOD_CFG1 0x30b +#define MII_DP83826_VOD_CFG2 0x30c + /* GENCFG */ #define DP83822_SIG_DET_LOW BIT(0) @@ -110,6 +115,16 @@ #define DP83822_RX_ER_STR_MASK GENMASK(9, 8) #define DP83822_RX_ER_SHIFT 8 +/* DP83826: VOD_CFG1 & VOD_CFG2 */ +#define DP83826_VOD_CFG1_MINUS_MDIX_MASK GENMASK(13, 12) +#define DP83826_VOD_CFG1_MINUS_MDI_MASK GENMASK(11, 6) +#define DP83826_VOD_CFG2_MINUS_MDIX_MASK GENMASK(15, 12) +#define DP83826_VOD_CFG2_PLUS_MDIX_MASK GENMASK(11, 6) +#define DP83826_VOD_CFG2_PLUS_MDI_MASK GENMASK(5, 0) +#define DP83826_CFG_DAC_MINUS_MDIX_5_TO_4 GENMASK(5, 4) +#define DP83826_CFG_DAC_MINUS_MDIX_3_TO_0 GENMASK(3, 0) +#define DP83826_CFG_DAC_RAW_VALUES_MAX 17 + #define MII_DP83822_FIBER_ADVERTISE (ADVERTISED_TP | ADVERTISED_MII | \ ADVERTISED_FIBRE | \ ADVERTISED_Pause | ADVERTISED_Asym_Pause) @@ -118,6 +133,8 @@ struct dp83822_private { bool fx_signal_det_low; int fx_enabled; u16 fx_sd_enable; + u8 cfg_dac_minus; + u8 cfg_dac_plus; }; static int dp83822_set_wol(struct phy_device *phydev, @@ -233,7 +250,7 @@ static int dp83822_config_intr(struct phy_device *phydev) DP83822_ENERGY_DET_INT_EN | DP83822_LINK_QUAL_INT_EN); - /* Private data pointer is NULL on DP83825/26 */ + /* Private data pointer is NULL on DP83825 */ if (!dp83822 || !dp83822->fx_enabled) misr_status |= DP83822_ANEG_COMPLETE_INT_EN | DP83822_DUP_MODE_CHANGE_INT_EN | @@ -254,7 +271,7 @@ static int dp83822_config_intr(struct phy_device *phydev) DP83822_PAGE_RX_INT_EN | DP83822_EEE_ERROR_CHANGE_INT_EN); - /* Private data pointer is NULL on DP83825/26 */ + /* Private data pointer is NULL on DP83825 */ if (!dp83822 || !dp83822->fx_enabled) misr_status |= DP83822_ANEG_ERR_INT_EN | DP83822_WOL_PKT_INT_EN; @@ -474,6 +491,46 @@ static int dp83822_config_init(struct phy_device *phydev) return dp8382x_disable_wol(phydev); } +static int dp83826_config_init(struct phy_device *phydev) +{ + struct dp83822_private *dp83822 = phydev->priv; + u16 vod_cfg1_val = 0, vod_cfg1_msk = 0, vod_cfg2_val = 0, vod_cfg2_msk = 0; + int ret; + + if (dp83822->cfg_dac_plus) { + vod_cfg2_val = FIELD_PREP(DP83826_VOD_CFG2_PLUS_MDIX_MASK, dp83822->cfg_dac_plus) | + FIELD_PREP(DP83826_VOD_CFG2_PLUS_MDI_MASK, dp83822->cfg_dac_plus); + vod_cfg2_msk = DP83826_VOD_CFG2_PLUS_MDIX_MASK | DP83826_VOD_CFG2_PLUS_MDI_MASK; + } + + if (dp83822->cfg_dac_minus) { + vod_cfg2_val |= FIELD_PREP(DP83826_VOD_CFG2_MINUS_MDIX_MASK, + FIELD_GET(DP83826_CFG_DAC_MINUS_MDIX_3_TO_0, + dp83822->cfg_dac_minus)); + vod_cfg2_msk |= DP83826_VOD_CFG2_MINUS_MDIX_MASK; + vod_cfg1_val = FIELD_PREP(DP83826_VOD_CFG1_MINUS_MDI_MASK, dp83822->cfg_dac_minus) | + FIELD_PREP(DP83826_VOD_CFG1_MINUS_MDIX_MASK, + FIELD_GET(DP83826_CFG_DAC_MINUS_MDIX_5_TO_4, + dp83822->cfg_dac_minus)); + vod_cfg1_msk = DP83826_VOD_CFG1_MINUS_MDIX_MASK | DP83826_VOD_CFG1_MINUS_MDI_MASK; + } + + if (vod_cfg1_msk) { + ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83826_VOD_CFG1, + vod_cfg1_msk, vod_cfg1_val); + if (ret) + return ret; + } + if (vod_cfg2_msk) { + ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83826_VOD_CFG2, + vod_cfg2_msk, vod_cfg2_val); + if (ret) + return ret; + } + + return 0; +} + static int dp8382x_config_init(struct phy_device *phydev) { return dp8382x_disable_wol(phydev); @@ -509,11 +566,41 @@ static int dp83822_of_init(struct phy_device *phydev) return 0; } + +u8 dp83826_cfg_dac_minus_raw[DP83826_CFG_DAC_RAW_VALUES_MAX] = {0x38, 0x37, 0x36, 0x35, 0x34, 0x33, + 0x32, 0x31, 0x30, 0x2f, 0x2e, 0x2d, + 0x2c, 0x2b, 0x2a, 0x29, 0x28}; +u8 dp83826_cfg_dac_plus_raw[DP83826_CFG_DAC_RAW_VALUES_MAX] = {0x08, 0x09, 0x0a, 0x0b, 0x0c, 0x0d, + 0x0e, 0x0f, 0x10, 0x11, 0x12, 0x13, + 0x14, 0x15, 0x16, 0x17, 0x18}; + +static int dp83826_of_init(struct phy_device *phydev) +{ + struct dp83822_private *dp83822 = phydev->priv; + struct device *dev = &phydev->mdio.dev; + u32 val; + int ret; + + ret = device_property_read_u32(dev, "ti,cfg-dac-minus", &val); + if (!ret && val < DP83826_CFG_DAC_RAW_VALUES_MAX) + dp83822->cfg_dac_minus = dp83826_cfg_dac_minus_raw[val]; + + ret = device_property_read_u32(dev, "ti,cfg-dac-plus", &val); + if (!ret && val < DP83826_CFG_DAC_RAW_VALUES_MAX) + dp83822->cfg_dac_plus = dp83826_cfg_dac_plus_raw[val]; + + return 0; +} #else static int dp83822_of_init(struct phy_device *phydev) { return 0; } + +static int dp83826_of_init(struct phy_device *phydev) +{ + return 0; +} #endif /* CONFIG_OF_MDIO */ static int dp83822_read_straps(struct phy_device *phydev) @@ -567,6 +654,22 @@ static int dp83822_probe(struct phy_device *phydev) return 0; } +static int dp83826_probe(struct phy_device *phydev) +{ + struct dp83822_private *dp83822; + + dp83822 = devm_kzalloc(&phydev->mdio.dev, sizeof(*dp83822), + GFP_KERNEL); + if (!dp83822) + return -ENOMEM; + + phydev->priv = dp83822; + + dp83826_of_init(phydev); + + return 0; +} + static int dp83822_suspend(struct phy_device *phydev) { int value; @@ -610,6 +713,22 @@ static int dp83822_resume(struct phy_device *phydev) .resume = dp83822_resume, \ } +#define DP83826_PHY_DRIVER(_id, _name) \ + { \ + PHY_ID_MATCH_MODEL(_id), \ + .name = (_name), \ + /* PHY_BASIC_FEATURES */ \ + .probe = dp83826_probe, \ + .soft_reset = dp83822_phy_reset, \ + .config_init = dp83826_config_init, \ + .get_wol = dp83822_get_wol, \ + .set_wol = dp83822_set_wol, \ + .config_intr = dp83822_config_intr, \ + .handle_interrupt = dp83822_handle_interrupt, \ + .suspend = dp83822_suspend, \ + .resume = dp83822_resume, \ + } + #define DP8382X_PHY_DRIVER(_id, _name) \ { \ PHY_ID_MATCH_MODEL(_id), \ @@ -628,8 +747,8 @@ static int dp83822_resume(struct phy_device *phydev) static struct phy_driver dp83822_driver[] = { DP83822_PHY_DRIVER(DP83822_PHY_ID, "TI DP83822"), DP8382X_PHY_DRIVER(DP83825I_PHY_ID, "TI DP83825I"), - DP8382X_PHY_DRIVER(DP83826C_PHY_ID, "TI DP83826C"), - DP8382X_PHY_DRIVER(DP83826NC_PHY_ID, "TI DP83826NC"), + DP83826_PHY_DRIVER(DP83826C_PHY_ID, "TI DP83826C"), + DP83826_PHY_DRIVER(DP83826NC_PHY_ID, "TI DP83826NC"), DP8382X_PHY_DRIVER(DP83825S_PHY_ID, "TI DP83825S"), DP8382X_PHY_DRIVER(DP83825CM_PHY_ID, "TI DP83825M"), DP8382X_PHY_DRIVER(DP83825CS_PHY_ID, "TI DP83825CS"),