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 D0599C677F1 for ; Mon, 16 Jan 2023 16:45:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: Content-Transfer-Encoding:Content-Type:MIME-Version:Message-ID:In-Reply-To: Date:References:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Qcp8xlkY5pqGPztw1JirgSBLbAT2r9HBNw+DIb8btZc=; b=IfFh6814J/yBbG 4fFfiGO48R0yWeUzBMlaztLu1zwEvGQauL+sXbiS19oXcMUkK4FziTwwSzcXE7P/bafoHbD7o6jk2 kBHyYbmcjiwlx9BnenLdBl1URbIVSKU8fsKEgY3T7HRYEHymcEwg3KP0KWVQbhS0HBuWL+RvswwuG +y75nLxpnAR9p4pFK3wdNOT0aeGfRqf1/PcICyz4GbqQL+1WZlyV9mkyr0BV0f9o/LSJhPzJ8O7aC HEw2an0F9DTx6GUA31PdHNbdUHHIpYppQmptsS4n8n4gHZDHmHFrBd0rr7WDR6/3e/A6YUFmXz7f3 tsBAVzl1r8hGBGyREqtg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pHScC-00BRjQ-GN; Mon, 16 Jan 2023 16:45:44 +0000 Received: from dilbert.mork.no ([2a01:4f9:c010:a439::d]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pHSc7-00BRhn-Kt; Mon, 16 Jan 2023 16:45:42 +0000 Received: from canardo.dyn.mork.no ([IPv6:2a01:799:c9a:3200:0:0:0:1]) (authenticated bits=0) by dilbert.mork.no (8.15.2/8.15.2) with ESMTPSA id 30GGjIot2102675 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=OK); Mon, 16 Jan 2023 16:45:19 GMT Received: from miraculix.mork.no ([IPv6:2a01:799:c9a:3202:549f:9f7a:c9d8:875b]) (authenticated bits=0) by canardo.dyn.mork.no (8.15.2/8.15.2) with ESMTPSA id 30GGjCAw2045407 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=OK); Mon, 16 Jan 2023 17:45:12 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mork.no; s=b; t=1673887513; bh=Qcp8xlkY5pqGPztw1JirgSBLbAT2r9HBNw+DIb8btZc=; h=From:To:Cc:Subject:References:Date:Message-ID:From; b=ki3+2OVSuhhCJ6R/10JXtk8u3HQaA0qMl05kLjpiFig2pIgv2O/MxCozSVGWUldEi 9Z+jR6FLLv94+NTJWW6NVWy2pLHD9rP+qXhu9iH4XVQSyBWCaQtgVEhjHaHTHq1dCt uysrckr110k4OSbFm188Xvrq7k/sur/AXiFw1/fs= Received: (nullmailer pid 377826 invoked by uid 1000); Mon, 16 Jan 2023 16:45:12 -0000 From: =?utf-8?Q?Bj=C3=B8rn_Mork?= To: "Russell King (Oracle)" Subject: Re: [PATCH v2] net: mtk_sgmii: implement mtk_pcs_ops Organization: m References: <87o7qy39v5.fsf@miraculix.mork.no> <87h6wq35dn.fsf@miraculix.mork.no> <87bkmy33ph.fsf@miraculix.mork.no> <875yd630cu.fsf@miraculix.mork.no> Date: Mon, 16 Jan 2023 17:45:12 +0100 In-Reply-To: <875yd630cu.fsf@miraculix.mork.no> (=?utf-8?Q?=22Bj=C3=B8rn?= Mork"'s message of "Mon, 16 Jan 2023 17:33:53 +0100") Message-ID: <871qnu2ztz.fsf@miraculix.mork.no> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Virus-Scanned: clamav-milter 0.103.7 at canardo X-Virus-Status: Clean X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230116_084540_032170_F398745C X-CRM114-Status: UNSURE ( 9.70 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: netdev@vger.kernel.org, Sean Wang , linux-kernel@vger.kernel.org, "David S. Miller" , Jakub Kicinski , Eric Dumazet , Alexander Couzens , John Crispin , Matthias Brugger , linux-mediatek@lists.infradead.org, Frank Wunderlich , Paolo Abeni , Mark Lee , linux-arm-kernel@lists.infradead.org, Felix Fietkau Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Bj=C3=B8rn Mork writes: >> You have bmcr=3D0x1000, but the code sets two bits - SGMII_AN_RESTART and >> SGMII_AN_ENABLE which are bits 9 and 12, so bmcr should be 0x1200, not >> 0x1000. Any ideas why? > > No, not really Doh! Looked over it again, and this was my fault of course. Had an "bmcr =3D SGMII_AN_ENABLE;" =20=20 line overwriting the original value from a previous attempt without changing the if condition.. Thanks for spotting that. But this still doesn't work any better: [ 43.019395] mtk_soc_eth 15100000.ethernet wan: Link is Down [ 45.099898] mtk_sgmii_select_pcs: id=3D1 [ 45.103653] mtk_pcs_config: interface=3D4 [ 45.107473] offset:0 0x140 [ 45.107476] offset:4 0x4d544950 [ 45.110181] offset:8 0x20 [ 45.113305] forcing AN [ 45.118256] mtk_pcs_config: rgc3=3D0x0, advertise=3D0x1 (changed), link_= timer=3D1600000, sgm_mode=3D0x103, bmcr=3D0x1200, use_an=3D1 [ 45.129191] mtk_pcs_link_up: interface=3D4 [ 45.133100] offset:0 0x81140 [ 45.133102] offset:4 0x4d544950 [ 45.135967] offset:8 0x1 [ 45.139104] mtk_soc_eth 15100000.ethernet wan: Link is Up - 1Gbps/Full -= flow control rx/tx Bj=C3=B8rn 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 23AACC67871 for ; Mon, 16 Jan 2023 16:46:43 +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:MIME-Version:Message-ID:In-Reply-To: Date:References:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=8Yx35qc/oYyMoMMqYiA0H1FWqSqSLUJgpV4Mj84fNtg=; b=fscvnOotDfmIag aXXVKG4ynpyPEqpaYeJpYZq5z2WHeafxx/aR1EF4PYJnSOiHgfEwAByM0nlKnNy2RcuSPW25N7xEq tqUzPgaVg84zgHtWBX4JzvyalIcxAkkcaL14tD95DsQZTEYAdrTPROT2DxbdTqzyuxz2Y1DoSEgVz J3VUjOFQg/Y52xafXO22JISXKJEw0EXRxpdcRsXLmseWdUnFn+QGYCLAreiGcBVtABDf40zYr555Q 6aJuT//9hiwh70Zu3SuorwdUZoAlZFrXNKLgC7Vfc9X03jPKHSLItKh4AxD6QP1bdKBytUMu8nNoy iesLKPYzMRbYMO6xtUGA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pHScE-00BRjs-4E; Mon, 16 Jan 2023 16:45:46 +0000 Received: from dilbert.mork.no ([2a01:4f9:c010:a439::d]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pHSc7-00BRhn-Kt; Mon, 16 Jan 2023 16:45:42 +0000 Received: from canardo.dyn.mork.no ([IPv6:2a01:799:c9a:3200:0:0:0:1]) (authenticated bits=0) by dilbert.mork.no (8.15.2/8.15.2) with ESMTPSA id 30GGjIot2102675 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=OK); Mon, 16 Jan 2023 16:45:19 GMT Received: from miraculix.mork.no ([IPv6:2a01:799:c9a:3202:549f:9f7a:c9d8:875b]) (authenticated bits=0) by canardo.dyn.mork.no (8.15.2/8.15.2) with ESMTPSA id 30GGjCAw2045407 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=OK); Mon, 16 Jan 2023 17:45:12 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mork.no; s=b; t=1673887513; bh=Qcp8xlkY5pqGPztw1JirgSBLbAT2r9HBNw+DIb8btZc=; h=From:To:Cc:Subject:References:Date:Message-ID:From; b=ki3+2OVSuhhCJ6R/10JXtk8u3HQaA0qMl05kLjpiFig2pIgv2O/MxCozSVGWUldEi 9Z+jR6FLLv94+NTJWW6NVWy2pLHD9rP+qXhu9iH4XVQSyBWCaQtgVEhjHaHTHq1dCt uysrckr110k4OSbFm188Xvrq7k/sur/AXiFw1/fs= Received: (nullmailer pid 377826 invoked by uid 1000); Mon, 16 Jan 2023 16:45:12 -0000 From: =?utf-8?Q?Bj=C3=B8rn_Mork?= To: "Russell King (Oracle)" Cc: Frank Wunderlich , Frank Wunderlich , linux-mediatek@lists.infradead.org, Alexander Couzens , Felix Fietkau , John Crispin , Sean Wang , Mark Lee , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Matthias Brugger , netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] net: mtk_sgmii: implement mtk_pcs_ops Organization: m References: <87o7qy39v5.fsf@miraculix.mork.no> <87h6wq35dn.fsf@miraculix.mork.no> <87bkmy33ph.fsf@miraculix.mork.no> <875yd630cu.fsf@miraculix.mork.no> Date: Mon, 16 Jan 2023 17:45:12 +0100 In-Reply-To: <875yd630cu.fsf@miraculix.mork.no> (=?utf-8?Q?=22Bj=C3=B8rn?= Mork"'s message of "Mon, 16 Jan 2023 17:33:53 +0100") Message-ID: <871qnu2ztz.fsf@miraculix.mork.no> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) MIME-Version: 1.0 X-Virus-Scanned: clamav-milter 0.103.7 at canardo X-Virus-Status: Clean X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230116_084540_032170_F398745C X-CRM114-Status: UNSURE ( 9.70 ) X-CRM114-Notice: Please train this message. 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 QmrDuHJuIE1vcmsgPGJqb3JuQG1vcmsubm8+IHdyaXRlczoKCj4+IFlvdSBoYXZlIGJtY3I9MHgx MDAwLCBidXQgdGhlIGNvZGUgc2V0cyB0d28gYml0cyAtIFNHTUlJX0FOX1JFU1RBUlQgYW5kCj4+ IFNHTUlJX0FOX0VOQUJMRSB3aGljaCBhcmUgYml0cyA5IGFuZCAxMiwgc28gYm1jciBzaG91bGQg YmUgMHgxMjAwLCBub3QKPj4gMHgxMDAwLiBBbnkgaWRlYXMgd2h5Pwo+Cj4gTm8sIG5vdCByZWFs bHkKCkRvaCEgTG9va2VkIG92ZXIgaXQgYWdhaW4sIGFuZCB0aGlzIHdhcyBteSBmYXVsdCBvZiBj b3Vyc2UuICBIYWQgYW4KCiAgImJtY3IgPSBTR01JSV9BTl9FTkFCTEU7IgogIApsaW5lIG92ZXJ3 cml0aW5nIHRoZSBvcmlnaW5hbCB2YWx1ZSBmcm9tIGEgcHJldmlvdXMgYXR0ZW1wdCB3aXRob3V0 CmNoYW5naW5nIHRoZSBpZiBjb25kaXRpb24uLiBUaGFua3MgZm9yIHNwb3R0aW5nIHRoYXQuCgpC dXQgdGhpcyBzdGlsbCBkb2Vzbid0IHdvcmsgYW55IGJldHRlcjoKClsgICA0My4wMTkzOTVdIG10 a19zb2NfZXRoIDE1MTAwMDAwLmV0aGVybmV0IHdhbjogTGluayBpcyBEb3duClsgICA0NS4wOTk4 OThdIG10a19zZ21paV9zZWxlY3RfcGNzOiBpZD0xClsgICA0NS4xMDM2NTNdIG10a19wY3NfY29u ZmlnOiBpbnRlcmZhY2U9NApbICAgNDUuMTA3NDczXSBvZmZzZXQ6MCAweDE0MApbICAgNDUuMTA3 NDc2XSBvZmZzZXQ6NCAweDRkNTQ0OTUwClsgICA0NS4xMTAxODFdIG9mZnNldDo4IDB4MjAKWyAg IDQ1LjExMzMwNV0gZm9yY2luZyBBTgpbICAgNDUuMTE4MjU2XSBtdGtfcGNzX2NvbmZpZzogcmdj Mz0weDAsIGFkdmVydGlzZT0weDEgKGNoYW5nZWQpLCBsaW5rX3RpbWVyPTE2MDAwMDAsICBzZ21f bW9kZT0weDEwMywgYm1jcj0weDEyMDAsIHVzZV9hbj0xClsgICA0NS4xMjkxOTFdIG10a19wY3Nf bGlua191cDogaW50ZXJmYWNlPTQKWyAgIDQ1LjEzMzEwMF0gb2Zmc2V0OjAgMHg4MTE0MApbICAg NDUuMTMzMTAyXSBvZmZzZXQ6NCAweDRkNTQ0OTUwClsgICA0NS4xMzU5NjddIG9mZnNldDo4IDB4 MQpbICAgNDUuMTM5MTA0XSBtdGtfc29jX2V0aCAxNTEwMDAwMC5ldGhlcm5ldCB3YW46IExpbmsg aXMgVXAgLSAxR2Jwcy9GdWxsIC0gZmxvdyBjb250cm9sIHJ4L3R4CgoKCkJqw7hybgoKX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KbGludXgtYXJtLWtlcm5l bCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6 Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtYXJtLWtlcm5lbAo= 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 690B4C46467 for ; Mon, 16 Jan 2023 17:04:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234144AbjAPREE (ORCPT ); Mon, 16 Jan 2023 12:04:04 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54630 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234246AbjAPRDe (ORCPT ); Mon, 16 Jan 2023 12:03:34 -0500 Received: from dilbert.mork.no (dilbert.mork.no [IPv6:2a01:4f9:c010:a439::d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C069E2FCE3; Mon, 16 Jan 2023 08:45:54 -0800 (PST) Received: from canardo.dyn.mork.no ([IPv6:2a01:799:c9a:3200:0:0:0:1]) (authenticated bits=0) by dilbert.mork.no (8.15.2/8.15.2) with ESMTPSA id 30GGjIot2102675 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=OK); Mon, 16 Jan 2023 16:45:19 GMT Received: from miraculix.mork.no ([IPv6:2a01:799:c9a:3202:549f:9f7a:c9d8:875b]) (authenticated bits=0) by canardo.dyn.mork.no (8.15.2/8.15.2) with ESMTPSA id 30GGjCAw2045407 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=OK); Mon, 16 Jan 2023 17:45:12 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mork.no; s=b; t=1673887513; bh=Qcp8xlkY5pqGPztw1JirgSBLbAT2r9HBNw+DIb8btZc=; h=From:To:Cc:Subject:References:Date:Message-ID:From; b=ki3+2OVSuhhCJ6R/10JXtk8u3HQaA0qMl05kLjpiFig2pIgv2O/MxCozSVGWUldEi 9Z+jR6FLLv94+NTJWW6NVWy2pLHD9rP+qXhu9iH4XVQSyBWCaQtgVEhjHaHTHq1dCt uysrckr110k4OSbFm188Xvrq7k/sur/AXiFw1/fs= Received: (nullmailer pid 377826 invoked by uid 1000); Mon, 16 Jan 2023 16:45:12 -0000 From: =?utf-8?Q?Bj=C3=B8rn_Mork?= To: "Russell King (Oracle)" Cc: Frank Wunderlich , Frank Wunderlich , linux-mediatek@lists.infradead.org, Alexander Couzens , Felix Fietkau , John Crispin , Sean Wang , Mark Lee , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Matthias Brugger , netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] net: mtk_sgmii: implement mtk_pcs_ops Organization: m References: <87o7qy39v5.fsf@miraculix.mork.no> <87h6wq35dn.fsf@miraculix.mork.no> <87bkmy33ph.fsf@miraculix.mork.no> <875yd630cu.fsf@miraculix.mork.no> Date: Mon, 16 Jan 2023 17:45:12 +0100 In-Reply-To: <875yd630cu.fsf@miraculix.mork.no> (=?utf-8?Q?=22Bj=C3=B8rn?= Mork"'s message of "Mon, 16 Jan 2023 17:33:53 +0100") Message-ID: <871qnu2ztz.fsf@miraculix.mork.no> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Virus-Scanned: clamav-milter 0.103.7 at canardo X-Virus-Status: Clean Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Bj=C3=B8rn Mork writes: >> You have bmcr=3D0x1000, but the code sets two bits - SGMII_AN_RESTART and >> SGMII_AN_ENABLE which are bits 9 and 12, so bmcr should be 0x1200, not >> 0x1000. Any ideas why? > > No, not really Doh! Looked over it again, and this was my fault of course. Had an "bmcr =3D SGMII_AN_ENABLE;" =20=20 line overwriting the original value from a previous attempt without changing the if condition.. Thanks for spotting that. But this still doesn't work any better: [ 43.019395] mtk_soc_eth 15100000.ethernet wan: Link is Down [ 45.099898] mtk_sgmii_select_pcs: id=3D1 [ 45.103653] mtk_pcs_config: interface=3D4 [ 45.107473] offset:0 0x140 [ 45.107476] offset:4 0x4d544950 [ 45.110181] offset:8 0x20 [ 45.113305] forcing AN [ 45.118256] mtk_pcs_config: rgc3=3D0x0, advertise=3D0x1 (changed), link_= timer=3D1600000, sgm_mode=3D0x103, bmcr=3D0x1200, use_an=3D1 [ 45.129191] mtk_pcs_link_up: interface=3D4 [ 45.133100] offset:0 0x81140 [ 45.133102] offset:4 0x4d544950 [ 45.135967] offset:8 0x1 [ 45.139104] mtk_soc_eth 15100000.ethernet wan: Link is Up - 1Gbps/Full -= flow control rx/tx Bj=C3=B8rn