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 0C43AECAAA3 for ; Fri, 26 Aug 2022 16:48:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=oO5ZPW1KmJUimsJX2Tb/2N4soIBRZFtbjODX85CwoLg=; b=Y+/W3zffhbc2W/ UfX6qpvNiaRCTte/JccQlVF5+krWWO06v37TuTnBu+XXGGk5JSIx1hhNZr/cMBRw41NX12p0tmXSC xwKon6TGW1GdOSDixWJdDMkOBDd3bAz7Yv9cIuE5/iK7iNA3lY9HkxJL5jhTgLjPtnOlMZf9DxbWW BiBVHG3zfGsSz5kI1E718L3oLKq4R75DLi0csL46mabBDH7r7R06hNWo7e1LxYoYb6HGkrmZvK0+f 5u74tXDsv3QToXyRhvRmJ/7yAjDLaBbYxM1gsED6tS63UF6txhR6rAQIuYdCp609H+h9nQtvgjkO7 HH6HAfg2kXQcPNYZu5Bw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oRcUd-008Yc7-AT; Fri, 26 Aug 2022 16:47:39 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:32c8:5054:ff:fe00:142]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oRcUa-008WOy-2J for linux-arm-kernel@lists.infradead.org; Fri, 26 Aug 2022 16:47:37 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: 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=1+qrU9950QfzNnLy0WDyq6B9ElOQ/hOzcQZFWjhLUSY=; b=DlSwWLoexpRsAd5rM+5oPmPbtA LYXDunXdQNtLVpyDSrjeR3EwECgk27UzGIrYOmkQH404X3qICozaO049ARu9AKHmjefl2co9PkToN otQ1nc5OCMdQbYMyhP6R5CiMDfjHb0izpE8oB2BOZ3e0svblbCrwd/RprEDzUpRhZ1RZahgm5oJXF pyXpiPNsXBrDu6QnqB0cD28tCVcEPOsfU8xGdMeXfnJszqX8mcYQPG+T/f+oO+KOTYLIW7E02qmvt RCdh6lj/3xaEAeUc09mxuFu12yd58MhPzvdh2foQsY5f3BsncoEr9xt3Ed63qo3NHN3SGLskRLsW6 Ug7ATD/w==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:33948) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1oRcSK-0006Un-7r; Fri, 26 Aug 2022 17:45:18 +0100 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1oRcSH-0006Dj-J4; Fri, 26 Aug 2022 17:45:13 +0100 Date: Fri, 26 Aug 2022 17:45:13 +0100 From: "Russell King (Oracle)" To: Maxime Chevallier Cc: davem@davemloft.net, Rob Herring , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, thomas.petazzoni@bootlin.com, Andrew Lunn , Jakub Kicinski , Eric Dumazet , Paolo Abeni , Florian Fainelli , Heiner Kallweit , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org Subject: Re: [PATCH net-next 3/5] net: pcs: add new PCS driver for altera TSE PCS Message-ID: References: <20220826135451.526756-1-maxime.chevallier@bootlin.com> <20220826135451.526756-4-maxime.chevallier@bootlin.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220826135451.526756-4-maxime.chevallier@bootlin.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220826_094736_125846_5C0764E2 X-CRM114-Status: GOOD ( 18.32 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Aug 26, 2022 at 03:54:49PM +0200, Maxime Chevallier wrote: > + > +/* SGMII PCS register addresses > + */ > +#define SGMII_PCS_SCRATCH 0x10 > +#define SGMII_PCS_REV 0x11 > +#define SGMII_PCS_LINK_TIMER_0 0x12 > +#define SGMII_PCS_LINK_TIMER_REG(x) (0x12 + (x)) > +#define SGMII_PCS_LINK_TIMER_1 0x13 > +#define SGMII_PCS_IF_MODE 0x14 > +#define PCS_IF_MODE_SGMII_ENA BIT(0) > +#define PCS_IF_MODE_USE_SGMII_AN BIT(1) > +#define PCS_IF_MODE_SGMI_SPEED_MASK GENMASK(3, 2) > +#define PCS_IF_MODE_SGMI_SPEED_10 (0 << 2) > +#define PCS_IF_MODE_SGMI_SPEED_100 (1 << 2) > +#define PCS_IF_MODE_SGMI_SPEED_1000 (2 << 2) > +#define PCS_IF_MODE_SGMI_HALF_DUPLEX BIT(4) > +#define PCS_IF_MODE_SGMI_PHY_AN BIT(5) This looks very similar to pcs-lynx's register layout. I wonder if it's the same underlying hardware. > +static int alt_tse_pcs_config(struct phylink_pcs *pcs, unsigned int mode, > + phy_interface_t interface, > + const unsigned long *advertising, > + bool permit_pause_to_mac) > +{ > + struct altera_tse_pcs *tse_pcs = phylink_pcs_to_tse_pcs(pcs); > + u32 ctrl, if_mode; > + > + if (interface != PHY_INTERFACE_MODE_SGMII && > + interface != PHY_INTERFACE_MODE_1000BASEX) > + return 0; I would suggest doing this check in .pcs_validate() to catch anyone attaching the PCS with an unsupported interface mode. > +static void alt_tse_pcs_an_restart(struct phylink_pcs *pcs) > +{ > + struct altera_tse_pcs *tse_pcs = phylink_pcs_to_tse_pcs(pcs); > + u16 bmcr; > + > + bmcr = tse_pcs_read(tse_pcs, MII_BMCR); > + bmcr |= BMCR_ANRESTART; > + tse_pcs_write(tse_pcs, MII_BMCR, bmcr); > + > + tse_pcs_reset(tse_pcs); Any ideas why a reset is necessary after setting BMCR_ANRESTART? Normally, this is not required. > diff --git a/include/linux/pcs-altera-tse.h b/include/linux/pcs-altera-tse.h > new file mode 100644 > index 000000000000..9c85e7c8ef70 > --- /dev/null > +++ b/include/linux/pcs-altera-tse.h > @@ -0,0 +1,17 @@ > +/* SPDX-License-Identifier: GPL-2.0 */ > +/* > + * Copyright (C) 2022 Bootlin > + * > + * Maxime Chevallier > + */ > + > +#ifndef __LINUX_PCS_ALTERA_TSE_H > +#define __LINUX_PCS_ALTERA_TSE_H > + > +struct phylink; Don't you want "struct phylink_pcs;" here? -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last! _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel