[net-next,v5,02/16] net: phy: Remove the call to phy_mii_ioctl in phy_hwstamp_get/set
Message ID | 20231009155138.86458-3-kory.maincent@bootlin.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:a888:0:b0:403:3b70:6f57 with SMTP id x8csp1961286vqo; Mon, 9 Oct 2023 08:52:21 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEV6ddQWNjZSt38w/eN+Vn4iYmwXj8sDEa5DrYRcp0gYpg+kVJH6csab9SnJRao+UbwgaA+ X-Received: by 2002:a17:90a:b114:b0:25b:c454:a366 with SMTP id z20-20020a17090ab11400b0025bc454a366mr15416083pjq.5.1696866741516; Mon, 09 Oct 2023 08:52:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1696866741; cv=none; d=google.com; s=arc-20160816; b=WVQD+g7LRcqMLmiiZY3Czdt1Trf0apj+sO+j4SaZBoC9+TPRfVgjp27yUiMe3bY2lt IGW1qNpMrrSYgR6kSotlACP2EmkKqFF9UEPK2n280t84Gc8FUWt7w7x9UV1vjyy4eZ7x V+dFGV15jhWQJV2uDCTRWr+9aidPoDMaT5nyIxdRD6mpwPZlK8XugSZrC4Wi8PKK50zp dSAwZMYHYkX0I285XqDqEsTjQkrcfbtrfPW2tGs2X7bE8rYRG74HktcuwLQnOHkQIGSj B00Ly93QQZCiaXACCbqVq5SIZijEMqaaHFhTrorT8MeHh9KpZcG2ZSj5I6djhdJVVfFa sDtA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=d10ol8bgQXRCQ3CN2K11usb0w6nOmBYdHJPNNd1RZzA=; fh=G6KN1zqmJBGIhpPSq1AWsD4aWgSLDy1jBh7tZkBaukM=; b=Sd0MUxooL1ZJvcupJCuKR2pvsI+gHYdCP9anKSHxsGq6QkdPXekbzZM2ocBbtM/77o oxlAgVYU1tzx0i5e8mEvEhoUeNot/xDf0Mlyx/3svF46XQQSIU0QQx+Kxdid9Jd9xd0/ LFUOsr0TDnccZ7IXR8/AOlP9kV9Whn8uMJAtz9NgucbFg1KCdOP7oQPa/U+qiWi7iO6g QE/vMaKv8iAOtCpm0M3bUDob9+ZMvt7fXqjtLMR/Y2wbrU+rnOmBhd9GdoKHKiNRSGmq jWgf6lpyjIuTajeWnmeR30Trw2iSrlauSmz3nQeURoQuA/H8wEIqr2ZQ5mbwyqbIfEIe Y2dg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@bootlin.com header.s=gm1 header.b=EBVAcZGQ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=bootlin.com Received: from morse.vger.email (morse.vger.email. [23.128.96.31]) by mx.google.com with ESMTPS id bj18-20020a17090b089200b0026961fdd37bsi9837648pjb.97.2023.10.09.08.52.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 09 Oct 2023 08:52:21 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) client-ip=23.128.96.31; Authentication-Results: mx.google.com; dkim=pass header.i=@bootlin.com header.s=gm1 header.b=EBVAcZGQ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=bootlin.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by morse.vger.email (Postfix) with ESMTP id 25805801C1E4; Mon, 9 Oct 2023 08:52:16 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at morse.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1376524AbjJIPv7 (ORCPT <rfc822;ezelljr.billy@gmail.com> + 18 others); Mon, 9 Oct 2023 11:51:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36618 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1377083AbjJIPvy (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Mon, 9 Oct 2023 11:51:54 -0400 Received: from relay5-d.mail.gandi.net (relay5-d.mail.gandi.net [IPv6:2001:4b98:dc4:8::225]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DD9989F; Mon, 9 Oct 2023 08:51:52 -0700 (PDT) Received: by mail.gandi.net (Postfix) with ESMTPSA id 7EDE01C0013; Mon, 9 Oct 2023 15:51:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1696866711; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=d10ol8bgQXRCQ3CN2K11usb0w6nOmBYdHJPNNd1RZzA=; b=EBVAcZGQOgfR3e24L5o1/MyUk//goW/HWM+hPvBZNWpBxJrgTtvw8dqQf54U1i4BPYp4BA t9ciHAM6nos42a6p9r29lyAz1GqSzzLenWVcLPQRxqQDpTwzaO94vaz6BA82IFYjDUztzc sbyvOKm/7aRL+DbRCIu9dOaNMcy9Y9GZkZDDJN0xEMJh4OR/oSTLonFHf/Kr3pTotx6AvZ Mv4C6jiLy3zJM6tyCknjOEgCykzkmgO8hUto1tyYh/BR1gB2Q3AKYo3xkBjXSItyf61CWw gbjRPYf+drzJRNIlH6QUX7KcJC0Md1AVSB95rTbePn+rENLQUl1J00c1b8jvbA== From: =?utf-8?q?K=C3=B6ry_Maincent?= <kory.maincent@bootlin.com> To: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>, "David S . Miller" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>, Jonathan Corbet <corbet@lwn.net>, Jay Vosburgh <j.vosburgh@gmail.com>, Andy Gospodarek <andy@greyhouse.net>, Nicolas Ferre <nicolas.ferre@microchip.com>, Claudiu Beznea <claudiu.beznea@tuxon.dev>, Horatiu Vultur <horatiu.vultur@microchip.com>, UNGLinuxDriver@microchip.com, Florian Fainelli <florian.fainelli@broadcom.com>, Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>, Andrew Lunn <andrew@lunn.ch>, Heiner Kallweit <hkallweit1@gmail.com>, Russell King <linux@armlinux.org.uk>, Richard Cochran <richardcochran@gmail.com>, Radu Pirea <radu-nicolae.pirea@oss.nxp.com>, Willem de Bruijn <willemdebruijn.kernel@gmail.com>, Vladimir Oltean <vladimir.oltean@nxp.com>, Michael Walle <michael@walle.cc>, Jacob Keller <jacob.e.keller@intel.com>, Maxime Chevallier <maxime.chevallier@bootlin.com>, Kory Maincent <kory.maincent@bootlin.com> Subject: [PATCH net-next v5 02/16] net: phy: Remove the call to phy_mii_ioctl in phy_hwstamp_get/set Date: Mon, 9 Oct 2023 17:51:24 +0200 Message-Id: <20231009155138.86458-3-kory.maincent@bootlin.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20231009155138.86458-1-kory.maincent@bootlin.com> References: <20231009155138.86458-1-kory.maincent@bootlin.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-GND-Sasl: kory.maincent@bootlin.com X-Spam-Status: No, score=2.7 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, RCVD_IN_SBL_CSS,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on morse.vger.email Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (morse.vger.email [0.0.0.0]); Mon, 09 Oct 2023 08:52:16 -0700 (PDT) X-Spam-Level: ** X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1779293740597748925 X-GMAIL-MSGID: 1779293740597748925 |
Series |
net: Make timestamping selectable
|
|
Commit Message
Köry Maincent
Oct. 9, 2023, 3:51 p.m. UTC
From: Kory Maincent <kory.maincent@bootlin.com> __phy_hwtstamp_set function were calling the phy_mii_ioctl function which will then use the ifreq pointer to call the hwtstamp callback. Now that ifreq has been removed from the hwstamp callback parameters it seems more logical to not go through the phy_mii_ioctl function and pass directly kernel_hwtstamp_config parameter to the hwtstamp callback. Lets do the same for __phy_hwtstamp_get function and return directly EOPNOTSUPP as SIOCGHWTSTAMP is not supported for now for the PHYs. Signed-off-by: Kory Maincent <kory.maincent@bootlin.com> --- drivers/net/phy/phy.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
Comments
On 10/9/23 08:51, Köry Maincent wrote: > From: Kory Maincent <kory.maincent@bootlin.com> > > __phy_hwtstamp_set function were calling the phy_mii_ioctl function > which will then use the ifreq pointer to call the hwtstamp callback. > Now that ifreq has been removed from the hwstamp callback parameters > it seems more logical to not go through the phy_mii_ioctl function and pass > directly kernel_hwtstamp_config parameter to the hwtstamp callback. > > Lets do the same for __phy_hwtstamp_get function and return directly > EOPNOTSUPP as SIOCGHWTSTAMP is not supported for now for the PHYs. > > Signed-off-by: Kory Maincent <kory.maincent@bootlin.com> Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com>
diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c index d058316666ba..3376e58e2b88 100644 --- a/drivers/net/phy/phy.c +++ b/drivers/net/phy/phy.c @@ -486,7 +486,7 @@ int __phy_hwtstamp_get(struct phy_device *phydev, if (!phydev) return -ENODEV; - return phy_mii_ioctl(phydev, config->ifr, SIOCGHWTSTAMP); + return -EOPNOTSUPP; } /** @@ -503,7 +503,10 @@ int __phy_hwtstamp_set(struct phy_device *phydev, if (!phydev) return -ENODEV; - return phy_mii_ioctl(phydev, config->ifr, SIOCSHWTSTAMP); + if (phydev->mii_ts && phydev->mii_ts->hwtstamp) + return phydev->mii_ts->hwtstamp(phydev->mii_ts, config, extack); + + return -EOPNOTSUPP; } /**