From patchwork Wed Jan 3 12:46:32 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Christian Marangi X-Patchwork-Id: 184734 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:6f82:b0:100:9c79:88ff with SMTP id tb2csp4995690dyb; Wed, 3 Jan 2024 04:47:47 -0800 (PST) X-Google-Smtp-Source: AGHT+IGCRa1WjZOM61qevSVY9TDW4ArtGt9MSLbCRChjiUAupOXiMI+pnU7dgUDT7lwiqrMjDpWn X-Received: by 2002:a05:6870:5686:b0:204:1fb4:2896 with SMTP id p6-20020a056870568600b002041fb42896mr15225969oao.116.1704286067332; Wed, 03 Jan 2024 04:47:47 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704286067; cv=none; d=google.com; s=arc-20160816; b=RWOCnR2MWiFVmlUDxUV1ilaJNdg9RksgJbwFVN/NBit3WeYE/11Rursn6ayuinsZIg aZiI3x73fomq6yVmaIxy6iCvdO+74YV6mS7+T9ysQd6rucCKapGEnwgBEpjnyN+hNAD9 8kQxccw7+QLoyZjycK9G23in1Nm3GLan0Wrv1rGWdyf0KdEHFwX9MAow2PdRq15ZxdhR Yd6nHyEwsuYG0xk3ZyKfDwseyUqf6dEc//GSrh6ZbE4myL89l68k9cXgLtRC2JacEOTr qc8Cblsd2ujXzpPHefi7qIUfEZeena4XuCZ3do2L4y9k5HnA8jjKMVyUgoCBzoI8v/4R H7Og== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :date:subject:cc:to:from:dkim-signature; bh=mndGvtpI2oeo6GQDcXQC8VQnYdadwT7o6axDFA5/9kY=; fh=utmrxdkgikAlLxVfQI1kldUG+rNjhrLN+3fAlOYj01s=; b=BcBoSLh9Qjz6RnT0JoUXys4PGsF7bsbzhpW+DcsN6bHZ9wALfuj3XNVu0wgdXOBA8w Fpd+/69OygiSXd+mN0ejNjma5+n4eJZd5zyy+pcjU974nG5/xIP8ErYNMPcW4ku6K1v1 rL7H7XLeACfLDnfMiA6YEjM0HQCqQQFfbMBnuAqaG7FbPyOHwJHzk4f8j2VlM0ShAls0 7DXgq073q9EwWwXwM1fKVd76C8w8tfnyJu7Yfp8h6PawU8S5meO2QkokkB0nDGhRvygi sC0FTLCFU/yH/HbB1QSRVkLYw4Sfr5OVvlDPozL73XvTjaB/hiico3lp5998+PKf0SZu uWlA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=QekCfjTI; spf=pass (google.com: domain of linux-kernel+bounces-15532-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-15532-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id c13-20020a05620a134d00b00781c56106a6si5813544qkl.645.2024.01.03.04.47.47 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 03 Jan 2024 04:47:47 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-15532-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=QekCfjTI; spf=pass (google.com: domain of linux-kernel+bounces-15532-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-15532-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 2028A1C2337A for ; Wed, 3 Jan 2024 12:47:47 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 367EB19BB9; Wed, 3 Jan 2024 12:46:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="QekCfjTI" X-Original-To: linux-kernel@vger.kernel.org Received: from mail-wm1-f41.google.com (mail-wm1-f41.google.com [209.85.128.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 399A619442; Wed, 3 Jan 2024 12:46:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-wm1-f41.google.com with SMTP id 5b1f17b1804b1-40d894764e7so22499105e9.1; Wed, 03 Jan 2024 04:46:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1704286010; x=1704890810; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=mndGvtpI2oeo6GQDcXQC8VQnYdadwT7o6axDFA5/9kY=; b=QekCfjTIL8QQj5Qe+8uR6eAWOlRqBAcLned5f27TgXvSPU32Qo6lLGnCtOUK94vYJH d6ZlCDduT6KD+TxB6ZQNQSLxDxHy3vCYKvGdtwF6T6hY5l8qfO56QBluN6VHDhktGt6b /E0aEGL9lq0RvyJPZr6N2N1wX4x2Y8GYU99Ls/Bn+qNTCpj0SYS3BoUbSPEtJGMukjMV j97s83+cSQALXLTEi/bfvf0Fl+NAfGqYvm/Jk6F6AeKcY8QoT26lWQLF8DSVZXzOURsu wE28t5Q4amt8Uj2m6icv05dYBC7jX6mhDwp/jC3ydF1YPf9IM59mVtHMhR+0dUh/22mK oZOw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1704286010; x=1704890810; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=mndGvtpI2oeo6GQDcXQC8VQnYdadwT7o6axDFA5/9kY=; b=dAO+moEnP2fHxB0OJLWnGKqwJ2wJaQkuMJmUfxMp1IigxAUnc01wGDlOs9RWNhW4GX 5z726PqhtBIOyC0TP7cy7aQovD55RupynsnlCVBh5nlntEg+TnDAWqwxo/svRK0bLU2R sDYy0Hzb7SUUkcVYJRVcMihllht0qB0PZTccNPLUMFqqCWHLZm9ieLHjf5Y1e4t2QsZI UYueXs96o3AlzzRfKZ6OBeDbdE/MODQnSikgaflxpBBxQUlIN1NaE2/j4TiGb9f7YfTU KyKlxWgtv2ZrK2NtU8bsCkqLgkP6pil1+JEjVNpM/dVWn8FO1+Jce92JGkAJ1K0huLR8 v7qg== X-Gm-Message-State: AOJu0YzYsHHg6HU/wS2Q0rIxlTvT1fIqploikDZoCQEPidZDYC/Lw+Hu P5x6mouY33taXQn0hPZxX0g= X-Received: by 2002:a05:600c:1c11:b0:40d:88fd:e1 with SMTP id j17-20020a05600c1c1100b0040d88fd00e1mr1453451wms.24.1704286010370; Wed, 03 Jan 2024 04:46:50 -0800 (PST) Received: from localhost.localdomain (host-87-10-250-100.retail.telecomitalia.it. [87.10.250.100]) by smtp.googlemail.com with ESMTPSA id fc16-20020a05600c525000b0040d5c58c41dsm2219676wmb.24.2024.01.03.04.46.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 03 Jan 2024 04:46:49 -0800 (PST) From: Christian Marangi To: Andrew Lunn , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Christian Marangi Subject: [net-next PATCH v3 1/4] net: phy: at803x: generalize cdt fault length function Date: Wed, 3 Jan 2024 13:46:32 +0100 Message-ID: <20240103124637.3078-2-ansuelsmth@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240103124637.3078-1-ansuelsmth@gmail.com> References: <20240103124637.3078-1-ansuelsmth@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1787073467386363013 X-GMAIL-MSGID: 1787073467386363013 Generalize cable test fault length function since they all base on the same magic values (already reverse engineered to understand the meaning of it) to have consistenct values on every PHY. Signed-off-by: Christian Marangi Reviewed-by: Simon Horman --- drivers/net/phy/at803x.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/net/phy/at803x.c b/drivers/net/phy/at803x.c index aaf6c654aaed..acf483fa0887 100644 --- a/drivers/net/phy/at803x.c +++ b/drivers/net/phy/at803x.c @@ -1192,10 +1192,8 @@ static bool at803x_cdt_fault_length_valid(u16 status) return false; } -static int at803x_cdt_fault_length(u16 status) +static int at803x_cdt_fault_length(int dt) { - int dt; - /* According to the datasheet the distance to the fault is * DELTA_TIME * 0.824 meters. * @@ -1211,8 +1209,6 @@ static int at803x_cdt_fault_length(u16 status) * With a VF of 0.69 we get the factor 0.824 mentioned in the * datasheet. */ - dt = FIELD_GET(AT803X_CDT_STATUS_DELTA_TIME_MASK, status); - return (dt * 824) / 10; } @@ -1265,9 +1261,11 @@ static int at803x_cable_test_one_pair(struct phy_device *phydev, int pair) ethnl_cable_test_result(phydev, ethtool_pair[pair], at803x_cable_test_result_trans(val)); - if (at803x_cdt_fault_length_valid(val)) + if (at803x_cdt_fault_length_valid(val)) { + val = FIELD_GET(AT803X_CDT_STATUS_DELTA_TIME_MASK, val); ethnl_cable_test_fault_length(phydev, ethtool_pair[pair], at803x_cdt_fault_length(val)); + } return 1; } @@ -1994,7 +1992,8 @@ static int qca808x_cdt_fault_length(struct phy_device *phydev, int pair) if (val < 0) return val; - return (FIELD_GET(QCA808X_CDT_DIAG_LENGTH, val) * 824) / 10; + val = FIELD_GET(QCA808X_CDT_DIAG_LENGTH, val); + return at803x_cdt_fault_length(val); } static int qca808x_cable_test_start(struct phy_device *phydev)