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 X-Spam-Level: X-Spam-Status: No, score=-3.5 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8A04EC48BC2 for ; Wed, 23 Jun 2021 13:05:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 7031660FDC for ; Wed, 23 Jun 2021 13:05:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230461AbhFWNH6 (ORCPT ); Wed, 23 Jun 2021 09:07:58 -0400 Received: from esa.microchip.iphmx.com ([68.232.153.233]:42954 "EHLO esa.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230121AbhFWNHt (ORCPT ); Wed, 23 Jun 2021 09:07:49 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1624453532; x=1655989532; h=message-id:subject:from:to:cc:date:in-reply-to: references:mime-version:content-transfer-encoding; bh=6ngQ1L8CvZrw2+GK1uugRRh4hu9W22hWObLewNrxIaM=; b=QvqHL9lx/mKseEyBa6+G31vnMCkg5O6or01+PAPa2odHOaE7XSslv/SE NCi3/akhz+Pbtu7wZ627qUmrRcn5foLUGxp/Gg0vhVx3+1aLvKMUudWUT Rg5v3tsOFMaQd3o+BR42Iw9/iPL2oQ+DYLLK5ozemLFM330VGU3YXiPsC NFHsrp0b6He1T7pY1b0FY24aFx9YApJzuD76l5GgKDAqzn2OKhCJCH3AO oHaWe0Ye+gip2OzssQEl9kBQ2lti7nVDTrbOSqP1wr4+sm40yw+STPCLn npQaYPTngXk6zIKtmHRqb51Qzdlp+c1jAR/wCaqW2NbkGXR6u+wWkJzYn w==; IronPort-SDR: X4O/Y/IWGPQ2/LS/+RUEM3lDTTnBUBp1MK5HxwzY4+kfmteJJvxqfgNZcwo3kAQ5H3Da2LvhuH Zccelp4qDQ0v7eUaHF7Qbw8uppLC74TG5LTv4NZ4FOH5Y2EwvhmjnbAtzJExa5dtoK93oLhz2k vVgunE2YP1lbZkb/r8J1cr6uZ8UbVFaZpeBHlf0viFmbn06zJ5FgpfV3y+rqn5GDOI64Aw8v3f 2f/vQLhSA8fKnabwgXMpi0/thGg5VvBQ0xVhOlUAuMzZboDffsa27x0w1Xe5ffhk94pmSYtk+p zZQ= X-IronPort-AV: E=Sophos;i="5.83,293,1616482800"; d="scan'208";a="126349718" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa3.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 23 Jun 2021 06:05:31 -0700 Received: from chn-vm-ex04.mchp-main.com (10.10.85.152) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.2; Wed, 23 Jun 2021 06:05:31 -0700 Received: from [10.205.21.35] (10.10.115.15) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server id 15.1.2176.2 via Frontend Transport; Wed, 23 Jun 2021 06:05:26 -0700 Message-ID: <13fc1b7dac31464f8a635336bf41143d7d02b04a.camel@microchip.com> Subject: Re: [PATCH net-next v4 04/10] net: sparx5: add port module support From: Steen Hegelund To: "Russell King (Oracle)" CC: "David S. Miller" , Jakub Kicinski , Andrew Lunn , Microchip Linux Driver Support , Alexandre Belloni , Madalin Bucur , Mark Einon , Masahiro Yamada , Arnd Bergmann , Philipp Zabel , "Simon Horman" , , , , Bjarni Jonasson , Lars Povlsen Date: Wed, 23 Jun 2021 15:05:25 +0200 In-Reply-To: <20210621143334.GN22278@shell.armlinux.org.uk> References: <20210615085034.1262457-1-steen.hegelund@microchip.com> <20210615085034.1262457-5-steen.hegelund@microchip.com> <20210621143334.GN22278@shell.armlinux.org.uk> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.40.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Russell, Thanks for your comment. On Mon, 2021-06-21 at 15:33 +0100, Russell King (Oracle) wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > Hi, > > On Tue, Jun 15, 2021 at 10:50:28AM +0200, Steen Hegelund wrote: > > +static int sparx5_port_pcs_low_set(struct sparx5 *sparx5, > > +                                struct sparx5_port *port, > > +                                struct sparx5_port_config *conf) > > +{ > > +     bool sgmii = false, inband_aneg = false; > > +     int err; > > + > > +     if (!port->conf.has_sfp) { > > +             sgmii = true; /* Phy is connnected to the MAC */ > > +     } else { > > +             if (conf->portmode == PHY_INTERFACE_MODE_SGMII || > > +                 conf->portmode == PHY_INTERFACE_MODE_QSGMII) > > +                     inband_aneg = true; /* Cisco-SGMII in-band-aneg */ > > +             else if (conf->portmode == PHY_INTERFACE_MODE_1000BASEX && > > +                      conf->autoneg) > > +                     inband_aneg = true; /* Clause-37 in-band-aneg */ > > I have to wonder why the presence of inband aneg depends on whether > there's a SFP or not... We don't do that kind of thing in other > drivers, so what is different here? Hmm. I have changed the implementation to use phylink_autoneg_inband() instead of a preconfigured value. > > Thanks. > > -- > RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ > FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last! -- BR Steen -=-=-=-=-=-=-=-=-=-=-=-=-=-= steen.hegelund@microchip.com 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 X-Spam-Level: X-Spam-Status: No, score=-4.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2770DC4743C for ; Wed, 23 Jun 2021 13:07:27 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id DFD956108E for ; Wed, 23 Jun 2021 13:07:26 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org DFD956108E Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=microchip.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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:MIME-Version:References:In-Reply-To: Date:CC:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=A063izRf0o8lmPJ085lT0xCzGRPsC6urvJVc/Qudqjw=; b=3+sWakkVQ6egNU mEt88BUpRwiLMhLYPPbp31PvytIV6UmRfJTkdmLhtZHVmxlaRoJJvVMpBapCLQ2JVV3GQdDjXDBSR R6cSi4YdG1m0t0XRr+RLPTrwQRKtXgQX4+FY15ChLw+maArYK8WEA/y18GUURhuOoIKDGsvdsylSS m1NFoIOZrWX6xbE4qLgEd9AEQQPPMOKD1z4QmiqE3iyF0D06vDvc+zXQ48Sk27DzXx6X6AdnNBfCn JR9RO/latb2ihHrTWTmcYDcHlRjK46cXnCzCJqOs6uvvmSUsGmiTRIy4riYklwqDPkLB/Jg+3N28v jZ23vMXXnpueSRipKhBw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lw2ZU-00AirF-CS; Wed, 23 Jun 2021 13:05:36 +0000 Received: from esa.microchip.iphmx.com ([68.232.153.233]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1lw2ZQ-00Aiqb-EU for linux-arm-kernel@lists.infradead.org; Wed, 23 Jun 2021 13:05:34 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1624453532; x=1655989532; h=message-id:subject:from:to:cc:date:in-reply-to: references:mime-version:content-transfer-encoding; bh=6ngQ1L8CvZrw2+GK1uugRRh4hu9W22hWObLewNrxIaM=; b=QvqHL9lx/mKseEyBa6+G31vnMCkg5O6or01+PAPa2odHOaE7XSslv/SE NCi3/akhz+Pbtu7wZ627qUmrRcn5foLUGxp/Gg0vhVx3+1aLvKMUudWUT Rg5v3tsOFMaQd3o+BR42Iw9/iPL2oQ+DYLLK5ozemLFM330VGU3YXiPsC NFHsrp0b6He1T7pY1b0FY24aFx9YApJzuD76l5GgKDAqzn2OKhCJCH3AO oHaWe0Ye+gip2OzssQEl9kBQ2lti7nVDTrbOSqP1wr4+sm40yw+STPCLn npQaYPTngXk6zIKtmHRqb51Qzdlp+c1jAR/wCaqW2NbkGXR6u+wWkJzYn w==; IronPort-SDR: X4O/Y/IWGPQ2/LS/+RUEM3lDTTnBUBp1MK5HxwzY4+kfmteJJvxqfgNZcwo3kAQ5H3Da2LvhuH Zccelp4qDQ0v7eUaHF7Qbw8uppLC74TG5LTv4NZ4FOH5Y2EwvhmjnbAtzJExa5dtoK93oLhz2k vVgunE2YP1lbZkb/r8J1cr6uZ8UbVFaZpeBHlf0viFmbn06zJ5FgpfV3y+rqn5GDOI64Aw8v3f 2f/vQLhSA8fKnabwgXMpi0/thGg5VvBQ0xVhOlUAuMzZboDffsa27x0w1Xe5ffhk94pmSYtk+p zZQ= X-IronPort-AV: E=Sophos;i="5.83,293,1616482800"; d="scan'208";a="126349718" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa3.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 23 Jun 2021 06:05:31 -0700 Received: from chn-vm-ex04.mchp-main.com (10.10.85.152) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.2; Wed, 23 Jun 2021 06:05:31 -0700 Received: from [10.205.21.35] (10.10.115.15) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server id 15.1.2176.2 via Frontend Transport; Wed, 23 Jun 2021 06:05:26 -0700 Message-ID: <13fc1b7dac31464f8a635336bf41143d7d02b04a.camel@microchip.com> Subject: Re: [PATCH net-next v4 04/10] net: sparx5: add port module support From: Steen Hegelund To: "Russell King (Oracle)" CC: "David S. Miller" , Jakub Kicinski , Andrew Lunn , Microchip Linux Driver Support , Alexandre Belloni , Madalin Bucur , Mark Einon , Masahiro Yamada , Arnd Bergmann , Philipp Zabel , "Simon Horman" , , , , Bjarni Jonasson , Lars Povlsen Date: Wed, 23 Jun 2021 15:05:25 +0200 In-Reply-To: <20210621143334.GN22278@shell.armlinux.org.uk> References: <20210615085034.1262457-1-steen.hegelund@microchip.com> <20210615085034.1262457-5-steen.hegelund@microchip.com> <20210621143334.GN22278@shell.armlinux.org.uk> User-Agent: Evolution 3.40.2 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210623_060532_536973_9C2FEA2A X-CRM114-Status: GOOD ( 18.12 ) 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="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org SGkgUnVzc2VsbCwKClRoYW5rcyBmb3IgeW91ciBjb21tZW50LgoKCk9uIE1vbiwgMjAyMS0wNi0y MSBhdCAxNTozMyArMDEwMCwgUnVzc2VsbCBLaW5nIChPcmFjbGUpIHdyb3RlOgo+IEVYVEVSTkFM IEVNQUlMOiBEbyBub3QgY2xpY2sgbGlua3Mgb3Igb3BlbiBhdHRhY2htZW50cyB1bmxlc3MgeW91 IGtub3cgdGhlIGNvbnRlbnQgaXMgc2FmZQo+IAo+IEhpLAo+IAo+IE9uIFR1ZSwgSnVuIDE1LCAy MDIxIGF0IDEwOjUwOjI4QU0gKzAyMDAsIFN0ZWVuIEhlZ2VsdW5kIHdyb3RlOgo+ID4gK3N0YXRp YyBpbnQgc3Bhcng1X3BvcnRfcGNzX2xvd19zZXQoc3RydWN0IHNwYXJ4NSAqc3Bhcng1LAo+ID4g K8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoMKgIHN0cnVjdCBzcGFyeDVfcG9ydCAqcG9ydCwKPiA+ICvCoMKgwqDCoMKgwqDCoMKgwqDC oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBzdHJ1Y3Qgc3Bhcng1 X3BvcnRfY29uZmlnICpjb25mKQo+ID4gK3sKPiA+ICvCoMKgwqDCoCBib29sIHNnbWlpID0gZmFs c2UsIGluYmFuZF9hbmVnID0gZmFsc2U7Cj4gPiArwqDCoMKgwqAgaW50IGVycjsKPiA+ICsKPiA+ ICvCoMKgwqDCoCBpZiAoIXBvcnQtPmNvbmYuaGFzX3NmcCkgewo+ID4gK8KgwqDCoMKgwqDCoMKg wqDCoMKgwqDCoCBzZ21paSA9IHRydWU7IC8qIFBoeSBpcyBjb25ubmVjdGVkIHRvIHRoZSBNQUMg Ki8KPiA+ICvCoMKgwqDCoCB9IGVsc2Ugewo+ID4gK8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBp ZiAoY29uZi0+cG9ydG1vZGUgPT0gUEhZX0lOVEVSRkFDRV9NT0RFX1NHTUlJIHx8Cj4gPiArwqDC oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAgY29uZi0+cG9ydG1vZGUgPT0gUEhZX0lOVEVS RkFDRV9NT0RFX1FTR01JSSkKPiA+ICvCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoMKgIGluYmFuZF9hbmVnID0gdHJ1ZTsgLyogQ2lzY28tU0dNSUkgaW4tYmFuZC1hbmVnICov Cj4gPiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIGVsc2UgaWYgKGNvbmYtPnBvcnRtb2RlID09 IFBIWV9JTlRFUkZBQ0VfTU9ERV8xMDAwQkFTRVggJiYKPiA+ICvCoMKgwqDCoMKgwqDCoMKgwqDC oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqAgY29uZi0+YXV0b25lZykKPiA+ICvCoMKgwqDCoMKgwqDC oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIGluYmFuZF9hbmVnID0gdHJ1ZTsgLyogQ2xhdXNl LTM3IGluLWJhbmQtYW5lZyAqLwo+IAo+IEkgaGF2ZSB0byB3b25kZXIgd2h5IHRoZSBwcmVzZW5j ZSBvZiBpbmJhbmQgYW5lZyBkZXBlbmRzIG9uIHdoZXRoZXIKPiB0aGVyZSdzIGEgU0ZQIG9yIG5v dC4uLiBXZSBkb24ndCBkbyB0aGF0IGtpbmQgb2YgdGhpbmcgaW4gb3RoZXIKPiBkcml2ZXJzLCBz byB3aGF0IGlzIGRpZmZlcmVudCBoZXJlPwoKSG1tLgpJIGhhdmUgY2hhbmdlZCB0aGUgaW1wbGVt ZW50YXRpb24gdG8gdXNlIHBoeWxpbmtfYXV0b25lZ19pbmJhbmQoKSBpbnN0ZWFkIG9mIGEgcHJl Y29uZmlndXJlZCB2YWx1ZS4KCgo+IAo+IFRoYW5rcy4KPiAKPiAtLQo+IFJNSydzIFBhdGNoIHN5 c3RlbTogaHR0cHM6Ly93d3cuYXJtbGludXgub3JnLnVrL2RldmVsb3Blci9wYXRjaGVzLwo+IEZU VFAgaXMgaGVyZSEgNDBNYnBzIGRvd24gMTBNYnBzIHVwLiBEZWNlbnQgY29ubmVjdGl2aXR5IGF0 IGxhc3QhCgotLSAKQlIKU3RlZW4KCi09LT0tPS09LT0tPS09LT0tPS09LT0tPS09LT0Kc3RlZW4u aGVnZWx1bmRAbWljcm9jaGlwLmNvbQoKCgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX18KbGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJt LWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21h aWxtYW4vbGlzdGluZm8vbGludXgtYXJtLWtlcm5lbAo=