From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 67EE7C369A2 for ; Tue, 8 Apr 2025 15:38:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=UbnR/Yx6+S/ucqw+KvxItEEpP/udNhYI5Mpn+75jpdI=; b=RQFk3VesMoDm2ZlWd2VNg+g0y5 WmC1xIcrT2Rh+NJTmFI9QUgpXgLJFQquWmN+6HoQiB3/RVnIrZ3pBPvOW4pgnfIWMgTI2TE7i8eIZ h2UWIWhdx2ptXWTvxeEq3hiA1tq4dY8Vq6J0hYiNMEkwRilgoNIXimbu6A2aucYtOFpLpGMd08a3i +4ls3Jj+gKl1b9SeB6YgHhgnPcR8KVBWJqsG3POGT+8j4NkoapOgCN0MNjslDIcuVedyfeEm9Kiz1 9jteoEd6jxbHrbmHDFPNddXqi59FoGOdBeNeaIANUonV77VQIIyUxTaygLvH3akLmgiGqBRKeI8Kw QB9urBTw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1u2B1g-00000004enQ-0ENk; Tue, 08 Apr 2025 15:38:12 +0000 Received: from out-173.mta0.migadu.com ([91.218.175.173]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1u2AxN-00000004dmN-1ELT for linux-arm-kernel@lists.infradead.org; Tue, 08 Apr 2025 15:33:47 +0000 Message-ID: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1744126422; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=UbnR/Yx6+S/ucqw+KvxItEEpP/udNhYI5Mpn+75jpdI=; b=TsxJD6OXGsMrUeMKz2rzK5y1cGKbuJ9CkV9KabVEfH0zmf3hFLua2xYw+Xt8QMM4LpiEfI pXY8+6qa/6lsECB5Hzz/E4VRhkaYn2GiRnOeeuaHHsUuqGAdeNqfnXWaZLDwYFclTjUb99 fp2rFxjAxB3rERLDz29xdme4H5tlPMg= Date: Tue, 8 Apr 2025 11:33:26 -0400 MIME-Version: 1.0 Subject: Re: [net-next PATCH v2 11/14] net: axienet: Convert to use PCS subsystem To: "Gupta, Suraj" , "netdev@vger.kernel.org" , Andrew Lunn , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Russell King Cc: Heiner Kallweit , "upstream@airoha.com" , Kory Maincent , Christian Marangi , "linux-kernel@vger.kernel.org" , "Simek, Michal" , "Pandey, Radhey Shyam" , Robert Hancock , "linux-arm-kernel@lists.infradead.org" References: <20250407231746.2316518-1-sean.anderson@linux.dev> <20250407232058.2317056-1-sean.anderson@linux.dev> Content-Language: en-US X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Sean Anderson In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Migadu-Flow: FLOW_OUT X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250408_083345_486705_772705F0 X-CRM114-Status: GOOD ( 18.41 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 4/8/25 08:19, Gupta, Suraj wrote: > [AMD Official Use Only - AMD Internal Distribution Only] > >> -----Original Message----- >> From: Sean Anderson >> Sent: Tuesday, April 8, 2025 4:51 AM >> To: netdev@vger.kernel.org; Andrew Lunn ; David S . >> Miller ; Eric Dumazet ; Jakub >> Kicinski ; Paolo Abeni ; Russell King >> >> Cc: Heiner Kallweit ; upstream@airoha.com; Kory >> Maincent ; Christian Marangi >> ; linux-kernel@vger.kernel.org; Simek, Michal >> ; Pandey, Radhey Shyam >> ; Robert Hancock >> ; linux-arm-kernel@lists.infradead.org; Sean >> Anderson >> Subject: [net-next PATCH v2 11/14] net: axienet: Convert to use PCS subsystem >> >> Caution: This message originated from an External Source. Use proper caution >> when opening attachments, clicking links, or responding. >> >> >> Convert the AXI Ethernet driver to use the PCS subsystem, including the new Xilinx >> PCA/PMA driver. Unfortunately, we must use a helper to work with bare MDIO >> nodes without a compatible. >> > > AXI ethernet changes looks fine to me, except one minor nit mentioned below. Using DT changesets for backward compatibility is impressive :) > I'll try reviewing pcs/pma patch also and test it with our setups. >> Signed-off-by: Sean Anderson > > Reviewed-by: Suraj Gupta >> --- >> >> (no changes since v1) >> >> drivers/net/ethernet/xilinx/Kconfig | 1 + >> drivers/net/ethernet/xilinx/xilinx_axienet.h | 4 +- >> .../net/ethernet/xilinx/xilinx_axienet_main.c | 104 ++++-------------- >> drivers/net/pcs/Kconfig | 1 - >> 4 files changed, 22 insertions(+), 88 deletions(-) >> >> diff --git a/drivers/net/ethernet/xilinx/Kconfig b/drivers/net/ethernet/xilinx/Kconfig >> index 7502214cc7d5..2eab64cf1646 100644 >> --- a/drivers/net/ethernet/xilinx/Kconfig >> +++ b/drivers/net/ethernet/xilinx/Kconfig >> @@ -27,6 +27,7 @@ config XILINX_AXI_EMAC >> tristate "Xilinx 10/100/1000 AXI Ethernet support" >> depends on HAS_IOMEM >> depends on XILINX_DMA >> + select OF_DYNAMIC if PCS_XILINX >> select PHYLINK >> select DIMLIB >> help >> diff --git a/drivers/net/ethernet/xilinx/xilinx_axienet.h >> b/drivers/net/ethernet/xilinx/xilinx_axienet.h >> index 5ff742103beb..f46e862245eb 100644 >> --- a/drivers/net/ethernet/xilinx/xilinx_axienet.h >> +++ b/drivers/net/ethernet/xilinx/xilinx_axienet.h >> @@ -473,7 +473,6 @@ struct skbuf_dma_descriptor { >> * @dev: Pointer to device structure >> * @phylink: Pointer to phylink instance >> * @phylink_config: phylink configuration settings >> - * @pcs_phy: Reference to PCS/PMA PHY if used >> * @pcs: phylink pcs structure for PCS PHY >> * @switch_x_sgmii: Whether switchable 1000BaseX/SGMII mode is enabled in >> the core >> * @axi_clk: AXI4-Lite bus clock >> @@ -553,8 +552,7 @@ struct axienet_local { >> struct phylink *phylink; >> struct phylink_config phylink_config; >> >> - struct mdio_device *pcs_phy; >> - struct phylink_pcs pcs; >> + struct phylink_pcs *pcs; >> >> bool switch_x_sgmii; >> >> diff --git a/drivers/net/ethernet/xilinx/xilinx_axienet_main.c >> b/drivers/net/ethernet/xilinx/xilinx_axienet_main.c >> index 054abf283ab3..07487c4b2141 100644 >> --- a/drivers/net/ethernet/xilinx/xilinx_axienet_main.c >> +++ b/drivers/net/ethernet/xilinx/xilinx_axienet_main.c >> @@ -35,6 +35,8 @@ >> #include >> #include >> #include >> +#include >> +#include >> #include >> #include >> #include >> @@ -2519,63 +2521,6 @@ static const struct ethtool_ops axienet_ethtool_ops = { >> .get_rmon_stats = axienet_ethtool_get_rmon_stats, }; >> >> -static struct axienet_local *pcs_to_axienet_local(struct phylink_pcs *pcs) -{ >> - return container_of(pcs, struct axienet_local, pcs); >> -} >> - >> -static void axienet_pcs_get_state(struct phylink_pcs *pcs, >> - unsigned int neg_mode, >> - struct phylink_link_state *state) >> -{ >> - struct mdio_device *pcs_phy = pcs_to_axienet_local(pcs)->pcs_phy; >> - >> - phylink_mii_c22_pcs_get_state(pcs_phy, neg_mode, state); >> -} >> - >> -static void axienet_pcs_an_restart(struct phylink_pcs *pcs) -{ >> - struct mdio_device *pcs_phy = pcs_to_axienet_local(pcs)->pcs_phy; >> - >> - phylink_mii_c22_pcs_an_restart(pcs_phy); >> -} >> - >> -static int axienet_pcs_config(struct phylink_pcs *pcs, unsigned int neg_mode, >> - phy_interface_t interface, >> - const unsigned long *advertising, >> - bool permit_pause_to_mac) >> -{ >> - struct mdio_device *pcs_phy = pcs_to_axienet_local(pcs)->pcs_phy; >> - struct net_device *ndev = pcs_to_axienet_local(pcs)->ndev; >> - struct axienet_local *lp = netdev_priv(ndev); >> - int ret; >> - >> - if (lp->switch_x_sgmii) { >> - ret = mdiodev_write(pcs_phy, XLNX_MII_STD_SELECT_REG, >> - interface == PHY_INTERFACE_MODE_SGMII ? >> - XLNX_MII_STD_SELECT_SGMII : 0); >> - if (ret < 0) { >> - netdev_warn(ndev, >> - "Failed to switch PHY interface: %d\n", >> - ret); >> - return ret; >> - } >> - } >> - >> - ret = phylink_mii_c22_pcs_config(pcs_phy, interface, advertising, >> - neg_mode); >> - if (ret < 0) >> - netdev_warn(ndev, "Failed to configure PCS: %d\n", ret); >> - >> - return ret; >> -} >> - >> -static const struct phylink_pcs_ops axienet_pcs_ops = { >> - .pcs_get_state = axienet_pcs_get_state, >> - .pcs_config = axienet_pcs_config, >> - .pcs_an_restart = axienet_pcs_an_restart, >> -}; >> - >> static struct phylink_pcs *axienet_mac_select_pcs(struct phylink_config *config, >> phy_interface_t interface) { @@ -2583,8 +2528,8 >> @@ static struct phylink_pcs *axienet_mac_select_pcs(struct phylink_config >> *config, >> struct axienet_local *lp = netdev_priv(ndev); >> >> if (interface == PHY_INTERFACE_MODE_1000BASEX || >> - interface == PHY_INTERFACE_MODE_SGMII) >> - return &lp->pcs; >> + interface == PHY_INTERFACE_MODE_SGMII) > > nit: unchanged check. Just fixing up the spacing while I was "in the area" --Sean