[{"id":1776207,"web_url":"http://patchwork.ozlabs.org/comment/1776207/","msgid":"<20170927101508.z3u32gff7kqgnmr2@flea>","list_archive_url":null,"date":"2017-09-27T10:15:08","subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","submitter":{"id":12916,"url":"http://patchwork.ozlabs.org/api/people/12916/","name":"Maxime Ripard","email":"maxime.ripard@free-electrons.com"},"content":"Hi,\n\nOn Wed, Sep 27, 2017 at 07:34:08AM +0000, Corentin Labbe wrote:\n> This patch add documentation about the MDIO switch used on sun8i-h3-emac\n> for integrated PHY.\n> \n> Signed-off-by: Corentin Labbe \n\nThis should be squashed with patch 1.\n\nMaxime","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"IRyarQh+\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2DH838B0z9tXs\n\tfor ;\n\tWed, 27 Sep 2017 20:15:48 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dx9NY-0004HC-U0; Wed, 27 Sep 2017 10:15:44 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dx9NU-0003i3-UY for linux-arm-kernel@lists.infradead.org;\n\tWed, 27 Sep 2017 10:15:42 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 096A42089D; Wed, 27 Sep 2017 12:15:18 +0200 (CEST)","from localhost (unknown [195.81.232.10])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id D9DED20898;\n\tWed, 27 Sep 2017 12:15:07 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type:Cc:\n\tList-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:\n\tIn-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To:\n\tContent-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner;\n\tbh=HUKAHzi2eA+QPyCzUH2H++nq6fw3tN0gleXOad7zq2s=;\n\tb=IRyarQh+qOmFruPyTB0Lx1hML\n\tCemHaaGNHWNF/PQPel70fDXRlaGnilcOYEMP96mB2URcV6qqvRDEh908glGHUm6Q1rG4tISdlIJud\n\tlM9IevL/3wVmtpU+sHySFbfpAoAPdZaFjKPUetvOYVRtSyp8sP5QM9rLqI70O3Vwt6PiuexUH3+aO\n\tm+qmsgyoTaQxvh2K/dNAc+mxrEZKtx6UMhVhPDd9MyKC1eW8J6N5dloaEOC1bqm2NV8H8v+9gQSi7\n\tb5dNBWcTWVuRA9gKxoyyX1EGeFFpMiu53zIIw7ihaVnS0MNBY1If0582IeDhfJ/Zr0astT2fE5FDa\n\tpFPCrJqpw==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT\n\tshortcircuit=ham autolearn=disabled version=3.4.0","Date":"Wed, 27 Sep 2017 12:15:08 +0200","From":"Maxime Ripard ","To":"Corentin Labbe ","Subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","Message-ID":"<20170927101508.z3u32gff7kqgnmr2@flea>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-6-clabbe.montjoie@gmail.com>","MIME-Version":"1.0","In-Reply-To":"<20170927073414.17361-6-clabbe.montjoie@gmail.com>","User-Agent":"NeoMutt/20170914 (1.9.0)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_031541_141478_7056F6AD ","X-CRM114-Status":"UNSURE ( 9.09 )","X-CRM114-Notice":"Please train this message.","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, andrew@lunn.ch, f.fainelli@gmail.com,\n\talexandre.torgue@st.com, devicetree@vger.kernel.org,\n\tcatalin.marinas@arm.com, \n\tlinux-sunxi@googlegroups.com, will.deacon@arm.com,\n\tlinux@armlinux.org.uk, \n\tlinux-kernel@vger.kernel.org, wens@csie.org, robh+dt@kernel.org,\n\tnetdev@vger.kernel.org, peppe.cavallaro@st.com, frowand.list@gmail.com,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"multipart/mixed;\n\tboundary=\"===============8568274981790570822==\"","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776213,"web_url":"http://patchwork.ozlabs.org/comment/1776213/","msgid":"<20170927101622.v7odmreccwh7ldg2@flea>","list_archive_url":null,"date":"2017-09-27T10:16:22","subject":"Re: [PATCH v6 06/11] ARM: dts: sunxi: h3/h5: represent the mdio\n\tswitch used by sun8i-h3-emac","submitter":{"id":12916,"url":"http://patchwork.ozlabs.org/api/people/12916/","name":"Maxime Ripard","email":"maxime.ripard@free-electrons.com"},"content":"On Wed, Sep 27, 2017 at 07:34:09AM +0000, Corentin Labbe wrote:\n> Since dwmac-sun8i could use either an integrated PHY or an external PHY\n> (which could be at same MDIO address), we need to represent this selection\n> by a MDIO switch.\n> \n> Signed-off-by: Corentin Labbe \n> ---\n> arch/arm/boot/dts/sunxi-h3-h5.dtsi | 31 +++++++++++++++++++++++++------\n> 1 file changed, 25 insertions(+), 6 deletions(-)\n> \n> diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi\n> index 3b7d953429a6..a8e9b8f378ba 100644\n> --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi\n> +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi\n> @@ -422,14 +422,33 @@\n> \t\t\t#size-cells = <0>;\n> \t\t\tstatus = \"disabled\";\n> \n> -\t\t\tmdio: mdio {\n> +\t\t\tmdio0: mdio {\n> \t\t\t\t#address-cells = <1>;\n> \t\t\t\t#size-cells = <0>;\n> -\t\t\t\tint_mii_phy: ethernet-phy@1 {\n> -\t\t\t\t\tcompatible = \"ethernet-phy-ieee802.3-c22\";\n> -\t\t\t\t\treg = <1>;\n> -\t\t\t\t\tclocks = <&ccu CLK_BUS_EPHY>;\n> -\t\t\t\t\tresets = <&ccu RST_BUS_EPHY>;\n> +\t\t\t\tcompatible = \"snps,dwmac-mdio\";\n> +\n> +\t\t\t\tmdio-mux {\n> +\t\t\t\t\tcompatible = \"mdio-mux\";\n> +\t\t\t\t\t#address-cells = <1>;\n> +\t\t\t\t\t#size-cells = <0>;\n\nNewline\n\n> +\t\t\t\t\t/* Only one MDIO is usable at the time */\n> +\t\t\t\t\tinternal_mdio: mdio@1 {\n> +\t\t\t\t\t\treg = <1>;\n> +\t\t\t\t\t\t#address-cells = <1>;\n> +\t\t\t\t\t\t#size-cells = <0>;\n\nNewline\n\n> +\t\t\t\t\t\tint_mii_phy: ethernet-phy@1 {\n> +\t\t\t\t\t\t\tcompatible = \"ethernet-phy-ieee802.3-c22\";\n> +\t\t\t\t\t\t\treg = <1>;\n> +\t\t\t\t\t\t\tclocks = <&ccu CLK_BUS_EPHY>;\n> +\t\t\t\t\t\t\tresets = <&ccu RST_BUS_EPHY>;\n> +\t\t\t\t\t\t\tphy-is-integrated;\n> +\t\t\t\t\t\t};\n> +\t\t\t\t\t};\n\nNewline\n\n> +\t\t\t\t\tmdio: mdio@2 {\n\nThis is quite confusing. Why not call the label external_mdio?\n\nThanks","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"J2y+txyl\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2DJg1B07z9tXC\n\tfor ;\n\tWed, 27 Sep 2017 20:17:07 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dx9Oo-0005fM-BI; Wed, 27 Sep 2017 10:17:02 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dx9Of-00057P-Pr for linux-arm-kernel@lists.infradead.org;\n\tWed, 27 Sep 2017 10:17:00 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 322C220801; Wed, 27 Sep 2017 12:16:32 +0200 (CEST)","from localhost (unknown [195.81.232.10])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id 0D0482068F;\n\tWed, 27 Sep 2017 12:16:22 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type:Cc:\n\tList-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:\n\tIn-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To:\n\tContent-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner;\n\tbh=Ca1dpLmoXl4fyn5Xn7SJbFm8SIhHNEMAGoiVzUA4gA0=;\n\tb=J2y+txylwxVLHDwzbFVqU8i1p\n\tO5Z9wcclXs7/vHpp0ToeEMm91geEKU1XsAAndbwpCzhgGwXqCcEZKr2v9gyvg8BkEKyEBTE/YT2Ml\n\t+RNAfvxG2QALL0UyQU69V2tpaAp4OgnRoYS4dWuVFzbXICHImOFfoUSlQ5YdYtWg0OqPEGDnCzl0E\n\tpkWfaBJ4nuduVj/65q+le9f2uyKPd2UQEJjgLWTSS+XIccjpTCu9Z42+8epUhKN032bf8z+ybokKH\n\t50QDecvZI2fHBXg6yIBipyl8MYjG7TnmQzL6M3KIdAfFEeE9ecZ2mjg6v9gT3ndcyefnrOWNjr28q\n\t0VZZZaTcw==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT,\n\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","Date":"Wed, 27 Sep 2017 12:16:22 +0200","From":"Maxime Ripard ","To":"Corentin Labbe ","Subject":"Re: [PATCH v6 06/11] ARM: dts: sunxi: h3/h5: represent the mdio\n\tswitch used by sun8i-h3-emac","Message-ID":"<20170927101622.v7odmreccwh7ldg2@flea>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-7-clabbe.montjoie@gmail.com>","MIME-Version":"1.0","In-Reply-To":"<20170927073414.17361-7-clabbe.montjoie@gmail.com>","User-Agent":"NeoMutt/20170914 (1.9.0)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_031654_149467_1AC33B9C ","X-CRM114-Status":"GOOD ( 14.19 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, andrew@lunn.ch, f.fainelli@gmail.com,\n\talexandre.torgue@st.com, devicetree@vger.kernel.org,\n\tcatalin.marinas@arm.com, \n\tlinux-sunxi@googlegroups.com, will.deacon@arm.com,\n\tlinux@armlinux.org.uk, \n\tlinux-kernel@vger.kernel.org, wens@csie.org, robh+dt@kernel.org,\n\tnetdev@vger.kernel.org, peppe.cavallaro@st.com, frowand.list@gmail.com,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"multipart/mixed;\n\tboundary=\"===============5445460869612052801==\"","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776369,"web_url":"http://patchwork.ozlabs.org/comment/1776369/","msgid":"<20170927134709.GA14370@Red>","list_archive_url":null,"date":"2017-09-27T13:47:09","subject":"Re: [PATCH v6 06/11] ARM: dts: sunxi: h3/h5: represent the mdio\n\tswitch used by sun8i-h3-emac","submitter":{"id":64152,"url":"http://patchwork.ozlabs.org/api/people/64152/","name":"Corentin Labbe","email":"clabbe.montjoie@gmail.com"},"content":"On Wed, Sep 27, 2017 at 12:16:22PM +0200, Maxime Ripard wrote:\n> On Wed, Sep 27, 2017 at 07:34:09AM +0000, Corentin Labbe wrote:\n> > Since dwmac-sun8i could use either an integrated PHY or an external PHY\n> > (which could be at same MDIO address), we need to represent this selection\n> > by a MDIO switch.\n> > \n> > Signed-off-by: Corentin Labbe \n> > ---\n> > arch/arm/boot/dts/sunxi-h3-h5.dtsi | 31 +++++++++++++++++++++++++------\n> > 1 file changed, 25 insertions(+), 6 deletions(-)\n> > \n> > diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi\n> > index 3b7d953429a6..a8e9b8f378ba 100644\n> > --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi\n> > +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi\n> > @@ -422,14 +422,33 @@\n> > \t\t\t#size-cells = <0>;\n> > \t\t\tstatus = \"disabled\";\n> > \n> > -\t\t\tmdio: mdio {\n> > +\t\t\tmdio0: mdio {\n> > \t\t\t\t#address-cells = <1>;\n> > \t\t\t\t#size-cells = <0>;\n> > -\t\t\t\tint_mii_phy: ethernet-phy@1 {\n> > -\t\t\t\t\tcompatible = \"ethernet-phy-ieee802.3-c22\";\n> > -\t\t\t\t\treg = <1>;\n> > -\t\t\t\t\tclocks = <&ccu CLK_BUS_EPHY>;\n> > -\t\t\t\t\tresets = <&ccu RST_BUS_EPHY>;\n> > +\t\t\t\tcompatible = \"snps,dwmac-mdio\";\n> > +\n> > +\t\t\t\tmdio-mux {\n> > +\t\t\t\t\tcompatible = \"mdio-mux\";\n> > +\t\t\t\t\t#address-cells = <1>;\n> > +\t\t\t\t\t#size-cells = <0>;\n> \n> Newline\n> \n> > +\t\t\t\t\t/* Only one MDIO is usable at the time */\n> > +\t\t\t\t\tinternal_mdio: mdio@1 {\n> > +\t\t\t\t\t\treg = <1>;\n> > +\t\t\t\t\t\t#address-cells = <1>;\n> > +\t\t\t\t\t\t#size-cells = <0>;\n> \n> Newline\n> \n> > +\t\t\t\t\t\tint_mii_phy: ethernet-phy@1 {\n> > +\t\t\t\t\t\t\tcompatible = \"ethernet-phy-ieee802.3-c22\";\n> > +\t\t\t\t\t\t\treg = <1>;\n> > +\t\t\t\t\t\t\tclocks = <&ccu CLK_BUS_EPHY>;\n> > +\t\t\t\t\t\t\tresets = <&ccu RST_BUS_EPHY>;\n> > +\t\t\t\t\t\t\tphy-is-integrated;\n> > +\t\t\t\t\t\t};\n> > +\t\t\t\t\t};\n> \n> Newline\n> \n> > +\t\t\t\t\tmdio: mdio@2 {\n> \n> This is quite confusing. Why not call the label external_mdio?\n> \n\nI will do it. (at origin I was not changing it for limiting changes on board with external PHY, but now all DT are reverted, it will be easy)\n\nRegards","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"bn4GN73L\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"VzNUWDo9\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2K0c0VVXz9tXw\n\tfor ;\n\tWed, 27 Sep 2017 23:48:32 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxChL-0007oz-6V; Wed, 27 Sep 2017 13:48:23 +0000","from mail-wr0-x243.google.com ([2a00:1450:400c:c0c::243])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxCgf-0006yV-5e for linux-arm-kernel@lists.infradead.org;\n\tWed, 27 Sep 2017 13:47:47 +0000","by mail-wr0-x243.google.com with SMTP id z1so2045081wre.1\n\tfor ;\n\tWed, 27 Sep 2017 06:47:18 -0700 (PDT)","from Red ([2a01:cb1d:16e:1300:2e56:dcff:fed2:c6d6])\n\tby smtp.googlemail.com with ESMTPSA id\n\tb55sm8099879wrb.11.2017.09.27.06.47.16\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 27 Sep 2017 06:47:16 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=cNs+65VZB5SCloB+Ab1niF4UYsymjV4N6VpUoh5b80w=;\n\tb=bn4GN73L79BWCX\n\tjvyyWz3NddOxYE70w48LrJtN6EthBeOGjBWOUAYahTAkKTxG7k8Ea+TR5vrBXDLLTYn3coDWwNx0J\n\tAE7ioLrr7WCRnhCvZdBBDDoYwNpjo80IaeZGA54rxJgeG5JDeIylF5mZxAM42d6kr1R3eD7Ctr0ew\n\teO6a7Y8BYoeBSW9E+UagE9ktL1XU9fCuqfXi7z2SNQ6nrHqjL1oK0iQXSR7O27YTur7b3HMaY8g9c\n\tttct31ixoWfebxmFaLeeJpaLKfU+x53VrCvqjWeDBgezml1zhOb4uS4AWhueIQ3VIsiqJsoa3T2mA\n\tarMgGVMJB7h+FoRV/BaA==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:in-reply-to:user-agent;\n\tbh=3DQ1rD4LElWexKKKueQSd/4ThPr00I+qZ3ENPlxGpKo=;\n\tb=VzNUWDo9PiTmoemJe7MVqz9/NuZckUXOaa32CltC5lGXvxm4IBBSQvgQiwwR9c0tjs\n\txmQmy+rT4Q1KJZJojt4qTeHsSt+JGWdj4fgoXxzYSiUQj99Nx/wtKGTXSMoC0p1xfRz6\n\tCPwOBl9k543tfq8xqwsXYXEa+QtLoZOP96IcbmcDmv9svDcFmr0buOGRPwRNh9a7/xmE\n\tQy6ENFy6d7uJjr14YPrq7kbfusfabY12JE4k2r96LLfduNcTvfaTrdu7E0OkJcqJoT7k\n\tMv2NFA3H24vIct6WM/d/v8PakE7pw9djbcpOp+v7lHcSYbTvIowqKbpSqiUm2rH8PiNc\n\tONLw=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=3DQ1rD4LElWexKKKueQSd/4ThPr00I+qZ3ENPlxGpKo=;\n\tb=Nlhg3OvI7p+SB90UznlNsExiDzT/xZepgCuGE6mgJRfzTkiSeTrqBxNJcXlpap8ByK\n\tFb9A2eJG6/p7dfVaevXMhu0c0t5KawZFPOPX3px55aW/0+J/JgTPlXhcjtHe8B9G+d3P\n\tA0bdG3DGGuU7QN1wEFsRFeTh7T2pqzLOBx3VxdKtUGDO/N6eGEpgk+7bmrRBJDCkeakr\n\tKDNvwEJEoEgFPXCvZ/cQN5ru0vyc5a1/Lfjnqt+B1JvYFSlquEXOgKzb+FNwS6On6W8t\n\tdq7VNxnO3zSUPX7pjhJZsAkxD3vKRO0R+cukG0AjRWk/6pC1oue3jKkhleRyiu9QGk9x\n\tWREw==","X-Gm-Message-State":"AHPjjUiQAkMOXZtzdPKwYDzmNQIv+amE2b5TnHfuHHI4eV45tsXxLNCZ\n\tL+bEJ41KKu1Jn+ks9Gl3ANc=","X-Google-Smtp-Source":"AOwi7QA+0AiYZ4sMuwU7xA943iSguT1uccmPh1YTfNjb/U7O3ewKUCtc0wcYd3VciZL2SUNz697BCQ==","X-Received":"by 10.223.155.157 with SMTP id d29mr1186535wrc.24.1506520037365; \n\tWed, 27 Sep 2017 06:47:17 -0700 (PDT)","Date":"Wed, 27 Sep 2017 15:47:09 +0200","From":"Corentin Labbe ","To":"Maxime Ripard ","Subject":"Re: [PATCH v6 06/11] ARM: dts: sunxi: h3/h5: represent the mdio\n\tswitch used by sun8i-h3-emac","Message-ID":"<20170927134709.GA14370@Red>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-7-clabbe.montjoie@gmail.com>\n\t<20170927101622.v7odmreccwh7ldg2@flea>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170927101622.v7odmreccwh7ldg2@flea>","User-Agent":"Mutt/1.7.2 (2016-11-26)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_064741_376729_4EADECB6 ","X-CRM114-Status":"GOOD ( 16.95 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (clabbe.montjoie[at]gmail.com)\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a00:1450:400c:c0c:0:0:0:243 listed in] [list.dnswl.org]\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, andrew@lunn.ch, f.fainelli@gmail.com,\n\talexandre.torgue@st.com, devicetree@vger.kernel.org,\n\tcatalin.marinas@arm.com, \n\tlinux-sunxi@googlegroups.com, will.deacon@arm.com,\n\tlinux@armlinux.org.uk, \n\tlinux-kernel@vger.kernel.org, wens@csie.org, robh+dt@kernel.org,\n\tnetdev@vger.kernel.org, peppe.cavallaro@st.com, frowand.list@gmail.com,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776381,"web_url":"http://patchwork.ozlabs.org/comment/1776381/","msgid":"<20170927140210.GE13516@lunn.ch>","list_archive_url":null,"date":"2017-09-27T14:02:10","subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","submitter":{"id":13608,"url":"http://patchwork.ozlabs.org/api/people/13608/","name":"Andrew Lunn","email":"andrew@lunn.ch"},"content":"Hi Corentin\n\n> +Required properties for the mdio-mux node:\n> + - compatible = \"mdio-mux\"\n\nThis is too generic. Please add a more specific compatible for this\nparticular mux. You can keep \"mdio-mux\", since that is what the MDIO\nsubsystem will look for.\n\n> +Required properties of the integrated phy node:\n> - clocks: a phandle to the reference clock for the EPHY\n> - resets: a phandle to the reset control for the EPHY\n> +- phy-is-integrated\n\nSo the last thing you said is that the mux is not the problem\nhere. Something else is locking up. Did you discover what?\n\nI really would like phy-is-integrated to go away.\n\n Andrew","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"IJx2PGHz\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2KKH4Jdtz9tXy\n\tfor ;\n\tThu, 28 Sep 2017 00:02:59 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxCvO-0006eM-UD; Wed, 27 Sep 2017 14:02:54 +0000","from vps0.lunn.ch ([185.16.172.187])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxCvH-0006ZC-TO for linux-arm-kernel@lists.infradead.org;\n\tWed, 27 Sep 2017 14:02:53 +0000","from andrew by vps0.lunn.ch with local (Exim 4.84_2)\n\t(envelope-from )\n\tid 1dxCug-0002aa-AQ; Wed, 27 Sep 2017 16:02:10 +0200"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=q5VGy3zmmhWmEoxjxi+BolTabl+rqdJMzlHv+ZrAME0=;\n\tb=IJx2PGHzRPIROW\n\tNQ7NXHoKe0osVZaWRoiara8tKbUDiZZ/ceO/goxsfMH5vD/t0LMDXNRMAWaEy0s93epE1wERcCqNn\n\t4JKLWOEFy3Qi3oiPlGPUR6gYcs04x7IYsTZoQNha7pWbIjRd6Zsn0qppzjHsXqIlPyDcaQze05j4i\n\tMpoRjCahfwDeD9qv9+dDCTg7AaDNse91AdW7G3D1I9XjDcINhqskaB4EYQvL01PUHfvbMmrCps93z\n\tzayp6sMLb6UL/8iyO0ZUBa3q9iP52+irjSMYHEPXiElwu2m0ucpdaVGll8TimE7RMmrTG8bTPpPiB\n\tOULOUtUlEdzyshWfemfw==;","Date":"Wed, 27 Sep 2017 16:02:10 +0200","From":"Andrew Lunn ","To":"Corentin Labbe ","Subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","Message-ID":"<20170927140210.GE13516@lunn.ch>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-6-clabbe.montjoie@gmail.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170927073414.17361-6-clabbe.montjoie@gmail.com>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_070248_119095_5A48D967 ","X-CRM114-Status":"UNSURE ( 7.54 )","X-CRM114-Notice":"Please train this message.","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, devicetree@vger.kernel.org, f.fainelli@gmail.com, \n\talexandre.torgue@st.com, catalin.marinas@arm.com,\n\tlinux-sunxi@googlegroups.com, \n\twill.deacon@arm.com, linux@armlinux.org.uk,\n\tlinux-kernel@vger.kernel.org, \n\twens@csie.org, robh+dt@kernel.org, netdev@vger.kernel.org,\n\tpeppe.cavallaro@st.com, maxime.ripard@free-electrons.com,\n\tfrowand.list@gmail.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776387,"web_url":"http://patchwork.ozlabs.org/comment/1776387/","msgid":"<20170927141213.GF13516@lunn.ch>","list_archive_url":null,"date":"2017-09-27T14:12:13","subject":"Re: [PATCH v6 11/11] of: mdio: Prevent of_mdiobus_register from\n\tscanning mdio-mux nodes","submitter":{"id":13608,"url":"http://patchwork.ozlabs.org/api/people/13608/","name":"Andrew Lunn","email":"andrew@lunn.ch"},"content":"On Wed, Sep 27, 2017 at 09:34:14AM +0200, Corentin Labbe wrote:\n> Each child node of an MDIO node is scanned as a PHY when calling\n> of_mdiobus_register() givint the following result:\n> [ 18.175379] mdio_bus stmmac-0: /soc/ethernet@1c30000/mdio/mdio-mux has invalid PHY address\n> [ 18.175408] mdio_bus stmmac-0: scan phy mdio-mux at address 0\n> [ 18.175450] mdio_bus stmmac-0: scan phy mdio-mux at address 1\n> [...]\n> [ 18.176420] mdio_bus stmmac-0: scan phy mdio-mux at address 30\n> [ 18.176452] mdio_bus stmmac-0: scan phy mdio-mux at address 31\n> \n> Since mdio-mux nodes are not PHY, this patch a way to to not scan\n> them.\n\nHi Corentin\n\nI still don't like this, but ...\n\n> \n> Signed-off-by: Corentin Labbe \n> ---\n> drivers/of/of_mdio.c | 10 ++++++++++\n> 1 file changed, 10 insertions(+)\n> \n> diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c\n> index d94dd8b77abd..d90ddb0d90f2 100644\n> --- a/drivers/of/of_mdio.c\n> +++ b/drivers/of/of_mdio.c\n> @@ -190,6 +190,10 @@ int of_mdiobus_register(struct mii_bus *mdio, struct device_node *np)\n> \tstruct device_node *child;\n> \tbool scanphys = false;\n> \tint addr, rc;\n> +\tstatic const struct of_device_id do_not_scan[] = {\n> +\t\t{ .compatible = \"mdio-mux\" },\n> +\t\t{}\n> +\t};\n\nPlease rename this to some less generic. What i don't want is other\ncompatible strings added here. We want to make the exception for\nmuxes, but nothing else. So something like compatible_muxes?\n\n Andrew","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"Ns9RvRzQ\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2KXh6nS2z9t3m\n\tfor ;\n\tThu, 28 Sep 2017 00:12:52 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxD4z-0005KR-Ic; Wed, 27 Sep 2017 14:12:49 +0000","from vps0.lunn.ch ([185.16.172.187])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxD4r-00056u-1M for linux-arm-kernel@lists.infradead.org;\n\tWed, 27 Sep 2017 14:12:47 +0000","from andrew by vps0.lunn.ch with local (Exim 4.84_2)\n\t(envelope-from )\n\tid 1dxD4P-0002h0-Ld; Wed, 27 Sep 2017 16:12:13 +0200"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=Evul4TZ9wf8YgfXLioRwPH7H04Mw3cZWl+sjQ/w5b78=;\n\tb=Ns9RvRzQanKJD7\n\tR2Cy+vBoYrRAAgNRATxADeSQTQVipDPkLl+z8phERProHA4QXJauvPqrzl0JZzZZzH9ljaOwPiQNX\n\txchIsO8xQ5NCv5ua7wOSfk3fY4hXpc6j3cVuYre6hcmWuKFRYnTWLih/2fmDSHOmLMsShh5AKam61\n\tK0u56Q6DPIyfWEf8tYT6w7Y8q1/HVt6++AW8T0Rh22OjrqWN5Ma6hbuql7fZjqOfW2u2y4o6Rsio4\n\tKoNOyAOg19Ihd0BvIjNHwcOS8hVTwD2ZmU7/eiwvFybtLACVBA+X99q9cVZMouFt47m5ht8FLzcLQ\n\tJt46SgR/F5YxIzreok7Q==;","Date":"Wed, 27 Sep 2017 16:12:13 +0200","From":"Andrew Lunn ","To":"Corentin Labbe ","Subject":"Re: [PATCH v6 11/11] of: mdio: Prevent of_mdiobus_register from\n\tscanning mdio-mux nodes","Message-ID":"<20170927141213.GF13516@lunn.ch>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-12-clabbe.montjoie@gmail.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170927073414.17361-12-clabbe.montjoie@gmail.com>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_071241_247553_18606F3B ","X-CRM114-Status":"GOOD ( 12.77 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, devicetree@vger.kernel.org, f.fainelli@gmail.com, \n\talexandre.torgue@st.com, catalin.marinas@arm.com,\n\tlinux-sunxi@googlegroups.com, \n\twill.deacon@arm.com, linux@armlinux.org.uk,\n\tlinux-kernel@vger.kernel.org, \n\twens@csie.org, robh+dt@kernel.org, netdev@vger.kernel.org,\n\tpeppe.cavallaro@st.com, maxime.ripard@free-electrons.com,\n\tfrowand.list@gmail.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776760,"web_url":"http://patchwork.ozlabs.org/comment/1776760/","msgid":"","list_archive_url":null,"date":"2017-09-28T04:50:46","subject":"Re: [PATCH v6 11/11] of: mdio: Prevent of_mdiobus_register from\n\tscanning mdio-mux nodes","submitter":{"id":2800,"url":"http://patchwork.ozlabs.org/api/people/2800/","name":"Florian Fainelli","email":"f.fainelli@gmail.com"},"content":"On 09/27/2017 07:12 AM, Andrew Lunn wrote:\n> On Wed, Sep 27, 2017 at 09:34:14AM +0200, Corentin Labbe wrote:\n>> Each child node of an MDIO node is scanned as a PHY when calling\n>> of_mdiobus_register() givint the following result:\n>> [ 18.175379] mdio_bus stmmac-0: /soc/ethernet@1c30000/mdio/mdio-mux has invalid PHY address\n>> [ 18.175408] mdio_bus stmmac-0: scan phy mdio-mux at address 0\n>> [ 18.175450] mdio_bus stmmac-0: scan phy mdio-mux at address 1\n>> [...]\n>> [ 18.176420] mdio_bus stmmac-0: scan phy mdio-mux at address 30\n>> [ 18.176452] mdio_bus stmmac-0: scan phy mdio-mux at address 31\n>>\n>> Since mdio-mux nodes are not PHY, this patch a way to to not scan\n>> them.\n> \n> Hi Corentin\n> \n> I still don't like this, but ...\n\nMe neither, even more so as I don't understand the reasoning behind\nputting the mux as a child node of the MDIO bus controller in the first\nplace.\n\nAlso, you need to re-order patches such that this patch comes before the\nDTS changes.\n\n> \n>>\n>> Signed-off-by: Corentin Labbe \n>> ---\n>> drivers/of/of_mdio.c | 10 ++++++++++\n>> 1 file changed, 10 insertions(+)\n>>\n>> diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c\n>> index d94dd8b77abd..d90ddb0d90f2 100644\n>> --- a/drivers/of/of_mdio.c\n>> +++ b/drivers/of/of_mdio.c\n>> @@ -190,6 +190,10 @@ int of_mdiobus_register(struct mii_bus *mdio, struct device_node *np)\n>> \tstruct device_node *child;\n>> \tbool scanphys = false;\n>> \tint addr, rc;\n>> +\tstatic const struct of_device_id do_not_scan[] = {\n>> +\t\t{ .compatible = \"mdio-mux\" },\n>> +\t\t{}\n>> +\t};\n> \n> Please rename this to some less generic. What i don't want is other\n> compatible strings added here. We want to make the exception for\n> muxes, but nothing else. So something like compatible_muxes?\n> \n> Andrew\n>","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"cCOnC6lR\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"mXVolsHO\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2j2J4BjMz9sNc\n\tfor ;\n\tThu, 28 Sep 2017 14:51:20 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxQn6-0008Nd-US; Thu, 28 Sep 2017 04:51:16 +0000","from mail-oi0-x241.google.com ([2607:f8b0:4003:c06::241])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxQn3-0008Lq-Nv for linux-arm-kernel@lists.infradead.org;\n\tThu, 28 Sep 2017 04:51:15 +0000","by mail-oi0-x241.google.com with SMTP id h9so409262oia.4\n\tfor ;\n\tWed, 27 Sep 2017 21:50:51 -0700 (PDT)","from ?IPv6:2001:470:d:73f:356d:fcbf:65d5:3e98?\n\t([2001:470:d:73f:356d:fcbf:65d5:3e98])\n\tby smtp.gmail.com with ESMTPSA id\n\th124sm317723oib.34.2017.09.27.21.50.47\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 27 Sep 2017 21:50:49 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=Otk352cVqRTs8bQuS6TpPfEzzAVfd7a5jZXknkuxH7k=;\n\tb=cCOnC6lRfDvpPA\n\tCFwWTy7V/tf8CpMxSPDWDMpDDIDJYefqKSJ/ABabSg43pdB4KKfOHnUWIPD7hGRI2KZbYJaofaafo\n\tbfgBa4CAdrCFHjH0CEE0CzXSeeIahAFIbcFPYdisWKOO7PYz29I2oOiDiioT5rmmtZSeGj8RG0JVl\n\tphuSOiwpYs1RHXw33lVi+xYvqvCMS5nyM8rkH1yjaCRnCDjmXsgWBEKgabeNTjZGUvhEHuYF8+1pL\n\tZFK+bh3AMTvfdriv3+wpJDeNyzd9B3+gvSn4YTOjEiW9x1O3dpewI7kDXjcpyJLSsu8IAiVCieacH\n\tw398ZOeB79AJwrtoV1rA==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=subject:to:cc:references:from:message-id:date:user-agent\n\t:mime-version:in-reply-to:content-language:content-transfer-encoding; \n\tbh=/s1OQsH341N0B2sWiC2DAJ8B997KJqeklBs5s/S7lm8=;\n\tb=mXVolsHOGURnS2Q53Wzbkw0N5R/38UJdyKpNpdQ4zqst6EKtZdv6TlnVWfIaOIjrRQ\n\tSWU1MatMrqIlFDxBnD4N7P2sl+ly6fPwhKMmF40WOfBVaPxEnO1lSdAySRVdXvBAaSCC\n\tgbUMnwFhCPXz33JL2EZ5sQ3lpv2IHmLbaz/cdnsaIaYdo/CijDIfMPwA7fbIHNJGD4dA\n\tAF1DS3xnDStcTDJRO5Rl+x2j3A9UPoVgF4bdr4jo8ME9Ge9I7vTYQI4YhpVzh5s1WkRR\n\tTh//1fYQ2UjmMhaPJQtYhvPrmUFylx9CHawj1SXnP6aC4LqdSN58FknTuXeUWfm9DEtU\n\tBVxQ=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:subject:to:cc:references:from:message-id:date\n\t:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=/s1OQsH341N0B2sWiC2DAJ8B997KJqeklBs5s/S7lm8=;\n\tb=Gfxr5OrN5EBy7hzYS+UlpiXMleLS9gwh6acV70fo6iUR/yT/gaeAnXydWOmQa+2wKL\n\tPmajnbA1EpvMsnX8V26Nw3H/jOJoLQK8Y7G1YVJgCqdO4AamNMp0zXafJnIr4X1I1wNM\n\tW2AJaCgXIYhQARwyZoL6W7l0YZloRwNtyzh2cR9B3Y+On+R7uxeoVbh6W3Y4PeNNSkga\n\tApgfNnJ+SkL/2sySrBJVKLAwhD0yzduC92/NVXS+gsuDKxmV5hVz9bmyx0TZu8Lu7uNK\n\tXLhSZg5FJAtQwv49clqnwOX+fr2s+bd5Q62sLk/BEnmVe6El5CNGXVGDk9cUuEH3aa0X\n\tBGOw==","X-Gm-Message-State":"AMCzsaU5sUKvFL/tSs0MVAJHbR0OyRv0Cx6ottag97t++cJqcw7zonk2\n\tI03ZaCBQbcPhabLh4XlyqsE=","X-Google-Smtp-Source":"AOwi7QByAUyb+AM1qJkqJJDzHAmOSvB6cnmXay7DcRbRjOm+BuiEwhuVu1W/slPhlDrzRPSa3Knk5g==","X-Received":"by 10.157.29.239 with SMTP id w44mr295451otw.479.1506574250481; \n\tWed, 27 Sep 2017 21:50:50 -0700 (PDT)","Subject":"Re: [PATCH v6 11/11] of: mdio: Prevent of_mdiobus_register from\n\tscanning mdio-mux nodes","To":"Andrew Lunn , Corentin Labbe ","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-12-clabbe.montjoie@gmail.com>\n\t<20170927141213.GF13516@lunn.ch>","From":"Florian Fainelli ","Message-ID":"","Date":"Wed, 27 Sep 2017 21:50:46 -0700","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170927141213.GF13516@lunn.ch>","Content-Language":"en-US","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_215113_836969_C79E4FF3 ","X-CRM114-Status":"GOOD ( 16.08 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (f.fainelli[at]gmail.com)\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2607:f8b0:4003:c06:0:0:0:241 listed in] [list.dnswl.org]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, devicetree@vger.kernel.org, alexandre.torgue@st.com,\n\tcatalin.marinas@arm.com, linux-sunxi@googlegroups.com,\n\twill.deacon@arm.com, \n\tlinux@armlinux.org.uk, linux-kernel@vger.kernel.org, wens@csie.org,\n\trobh+dt@kernel.org, netdev@vger.kernel.org, peppe.cavallaro@st.com,\n\tmaxime.ripard@free-electrons.com, frowand.list@gmail.com,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776767,"web_url":"http://patchwork.ozlabs.org/comment/1776767/","msgid":"<25e44368-2c07-9b7e-a0d3-a2a642286e5d@gmail.com>","list_archive_url":null,"date":"2017-09-28T04:53:15","subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","submitter":{"id":2800,"url":"http://patchwork.ozlabs.org/api/people/2800/","name":"Florian Fainelli","email":"f.fainelli@gmail.com"},"content":"On 09/27/2017 12:34 AM, Corentin Labbe wrote:\n> This patch add documentation about the MDIO switch used on sun8i-h3-emac\n> for integrated PHY.\n> \n> Signed-off-by: Corentin Labbe \n> ---\n> .../devicetree/bindings/net/dwmac-sun8i.txt | 138 +++++++++++++++++++--\n> 1 file changed, 126 insertions(+), 12 deletions(-)\n> \n> diff --git a/Documentation/devicetree/bindings/net/dwmac-sun8i.txt b/Documentation/devicetree/bindings/net/dwmac-sun8i.txt\n> index 725f3b187886..e2ef4683df08 100644\n> --- a/Documentation/devicetree/bindings/net/dwmac-sun8i.txt\n> +++ b/Documentation/devicetree/bindings/net/dwmac-sun8i.txt\n> @@ -4,18 +4,18 @@ This device is a platform glue layer for stmmac.\n> Please see stmmac.txt for the other unchanged properties.\n> \n> Required properties:\n> -- compatible: should be one of the following string:\n> +- compatible: must be one of the following string:\n> \t\t\"allwinner,sun8i-a83t-emac\"\n> \t\t\"allwinner,sun8i-h3-emac\"\n> \t\t\"allwinner,sun8i-v3s-emac\"\n> \t\t\"allwinner,sun50i-a64-emac\"\n> - reg: address and length of the register for the device.\n> - interrupts: interrupt for the device\n> -- interrupt-names: should be \"macirq\"\n> +- interrupt-names: must be \"macirq\"\n> - clocks: A phandle to the reference clock for this device\n> -- clock-names: should be \"stmmaceth\"\n> +- clock-names: must be \"stmmaceth\"\n> - resets: A phandle to the reset control for this device\n> -- reset-names: should be \"stmmaceth\"\n> +- reset-names: must be \"stmmaceth\"\n> - phy-mode: See ethernet.txt\n> - phy-handle: See ethernet.txt\n> - #address-cells: shall be 1\n> @@ -39,23 +39,38 @@ Optional properties for the following compatibles:\n> - allwinner,leds-active-low: EPHY LEDs are active low\n> \n> Required child node of emac:\n> -- mdio bus node: should be named mdio\n> +- mdio bus node: with compatible \"snps,dwmac-mdio\"\n> \n> Required properties of the mdio node:\n> - #address-cells: shall be 1\n> - #size-cells: shall be 0\n> \n> -The device node referenced by \"phy\" or \"phy-handle\" should be a child node\n> +The device node referenced by \"phy\" or \"phy-handle\" must be a child node\n> of the mdio node. See phy.txt for the generic PHY bindings.\n> \n> -Required properties of the phy node with the following compatibles:\n> +The following compatibles require that the mdio node have a mdio-mux child\n> +node called \"mdio-mux\":\n> + - \"allwinner,sun8i-h3-emac\"\n> + - \"allwinner,sun8i-v3s-emac\":\n> +Required properties for the mdio-mux node:\n> + - compatible = \"mdio-mux\"\n> + - one child mdio for the integrated mdio\n> + - one child mdio for the external mdio if present (V3s have none)\n> +Required properties for the mdio-mux children node:\n> + - reg: 1 for internal MDIO bus, 2 for external MDIO bus\n> +\n> +The following compatibles require a PHY node representing the integrated\n> +PHY, under the integrated MDIO bus node if an mdio-mux node is used:\n> - \"allwinner,sun8i-h3-emac\",\n> - \"allwinner,sun8i-v3s-emac\":\n> +\n> +Required properties of the integrated phy node:\n> - clocks: a phandle to the reference clock for the EPHY\n> - resets: a phandle to the reset control for the EPHY\n> +- phy-is-integrated\n> +- Must be a child of the integrated mdio\n> \n> -Example:\n> -\n> +Example with integrated PHY:\n> emac: ethernet@1c0b000 {\n> \tcompatible = \"allwinner,sun8i-h3-emac\";\n> \tsyscon = <&syscon>;\n> @@ -72,13 +87,112 @@ emac: ethernet@1c0b000 {\n> \tphy-handle = <&int_mii_phy>;\n> \tphy-mode = \"mii\";\n> \tallwinner,leds-active-low;\n> +\n> +\tmdio0: mdio {\n> +\t\t#address-cells = <1>;\n> +\t\t#size-cells = <0>;\n> +\t\tcompatible = \"snps,dwmac-mdio\";\n> +\n> +\t\tmdio-mux {\n> +\t\t\tcompatible = \"mdio-mux\";\n> +\t\t\t#address-cells = <1>;\n> +\t\t\t#size-cells = <0>;\n\nSorry for chiming in so late, but why don't we have the mdio-mux be the\nroot node here in the mdio bus hierarchy? I understand that with this\nbinding proposed here, we need to have patch 11 included (which btw,\nshould come before any DTS change), but this does not seem to accurately\nmodel the HW.\n\nThe mux itself is not a child node of the MDIO bus controller, it does\nnot really belong in that address space although it does mangle the MDIO\nbus controller address space between the two ends of the mux.\n\nIf this has been debated before, apologies for missing that part of the\ndiscussion.\n\n> +\n> +\t\t\tint_mdio: mdio@1 {\n> +\t\t\t\treg = <1>;\n> +\t\t\t\t#address-cells = <1>;\n> +\t\t\t\t#size-cells = <0>;\n> +\t\t\t\tint_mii_phy: ethernet-phy@1 {\n> +\t\t\t\t\treg = <1>;\n> +\t\t\t\t\tclocks = <&ccu CLK_BUS_EPHY>;\n> +\t\t\t\t\tresets = <&ccu RST_BUS_EPHY>;\n> +\t\t\t\t\tphy-is-integrated;\n> +\t\t\t\t};\n> +\t\t\t};\n> +\t\t\text_mdio: mdio@2 {\n> +\t\t\t\treg = <2>;\n> +\t\t\t\t#address-cells = <1>;\n> +\t\t\t\t#size-cells = <0>;\n> +\t\t\t};\n> +\t\t};\n> +\t};\n> +};\n> +\n> +Example with external PHY:\n> +emac: ethernet@1c0b000 {\n> +\tcompatible = \"allwinner,sun8i-h3-emac\";\n> +\tsyscon = <&syscon>;\n> +\treg = <0x01c0b000 0x104>;\n> +\tinterrupts = ;\n> +\tinterrupt-names = \"macirq\";\n> +\tresets = <&ccu RST_BUS_EMAC>;\n> +\treset-names = \"stmmaceth\";\n> +\tclocks = <&ccu CLK_BUS_EMAC>;\n> +\tclock-names = \"stmmaceth\";\n> +\t#address-cells = <1>;\n> +\t#size-cells = <0>;\n> +\n> +\tphy-handle = <&ext_rgmii_phy>;\n> +\tphy-mode = \"rgmii\";\n> +\tallwinner,leds-active-low;\n> +\n> +\tmdio0: mdio {\n> +\t\t#address-cells = <1>;\n> +\t\t#size-cells = <0>;\n> +\t\tcompatible = \"snps,dwmac-mdio\";\n> +\n> +\t\tmdio-mux {\n> +\t\t\tcompatible = \"mdio-mux\";\n> +\t\t\t#address-cells = <1>;\n> +\t\t\t#size-cells = <0>;\n> +\n> +\t\t\tint_mdio: mdio@1 {\n> +\t\t\t\treg = <1>;\n> +\t\t\t\t#address-cells = <1>;\n> +\t\t\t\t#size-cells = <0>;\n> +\t\t\t\tint_mii_phy: ethernet-phy@1 {\n> +\t\t\t\t\treg = <1>;\n> +\t\t\t\t\tclocks = <&ccu CLK_BUS_EPHY>;\n> +\t\t\t\t\tresets = <&ccu RST_BUS_EPHY>;\n> +\t\t\t\t\tphy-is-integrated;\n> +\t\t\t\t};\n> +\t\t\t};\n> +\t\t\text_mdio: mdio@2 {\n> +\t\t\t\treg = <2>;\n> +\t\t\t\t#address-cells = <1>;\n> +\t\t\t\t#size-cells = <0>;\n> +\t\t\t\text_rgmii_phy: ethernet-phy@1 {\n> +\t\t\t\t\treg = <1>;\n> +\t\t\t\t};\n> +\t\t\t}:\n> +\t\t};\n> +\t};\n> +};\n> +\n> +Example with SoC without integrated PHY\n> +\n> +emac: ethernet@1c0b000 {\n> +\tcompatible = \"allwinner,sun8i-a83t-emac\";\n> +\tsyscon = <&syscon>;\n> +\treg = <0x01c0b000 0x104>;\n> +\tinterrupts = ;\n> +\tinterrupt-names = \"macirq\";\n> +\tresets = <&ccu RST_BUS_EMAC>;\n> +\treset-names = \"stmmaceth\";\n> +\tclocks = <&ccu CLK_BUS_EMAC>;\n> +\tclock-names = \"stmmaceth\";\n> +\t#address-cells = <1>;\n> +\t#size-cells = <0>;\n> +\n> +\tphy-handle = <&ext_rgmii_phy>;\n> +\tphy-mode = \"rgmii\";\n> +\n> \tmdio: mdio {\n> +\t\tcompatible = \"snps,dwmac-mdio\";\n> \t\t#address-cells = <1>;\n> \t\t#size-cells = <0>;\n> -\t\tint_mii_phy: ethernet-phy@1 {\n> +\t\text_rgmii_phy: ethernet-phy@1 {\n> \t\t\treg = <1>;\n> -\t\t\tclocks = <&ccu CLK_BUS_EPHY>;\n> -\t\t\tresets = <&ccu RST_BUS_EPHY>;\n> \t\t};\n> \t};\n> };\n>","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"G5XUAHDp\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"nfwVJBgB\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2j570tm0z9sNc\n\tfor ;\n\tThu, 28 Sep 2017 14:53:47 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxQpU-0000b4-4u; Thu, 28 Sep 2017 04:53:44 +0000","from mail-oi0-x242.google.com ([2607:f8b0:4003:c06::242])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxQpQ-0000Vc-Rb for linux-arm-kernel@lists.infradead.org;\n\tThu, 28 Sep 2017 04:53:42 +0000","by mail-oi0-x242.google.com with SMTP id r20so417441oie.2\n\tfor ;\n\tWed, 27 Sep 2017 21:53:19 -0700 (PDT)","from ?IPv6:2001:470:d:73f:356d:fcbf:65d5:3e98?\n\t([2001:470:d:73f:356d:fcbf:65d5:3e98])\n\tby smtp.gmail.com with ESMTPSA id\n\tq10sm359140oth.64.2017.09.27.21.53.16\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 27 Sep 2017 21:53:17 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=Uabpr+XOlpgHQsQX29zhxg7qZO4ZM1GBa1uYk7Ida5g=;\n\tb=G5XUAHDpe7Emqo\n\tXhJtNhMAkdV058nFh899Y07LnIz4dKZ5AxSuCog++igpCunOfaE3BzwoL84I9jVhssopXAnmFHIza\n\tGFiJ3qQL+RUNEEwgKXWO99Em7m0LRGieVWw7rYI22wF8MkVw6NsecBX+Li9eMEPvLtWU1FFS7u2sn\n\tkJ3IzqJ8IBwvjyOo2yNMfwhz5/Pg69DzPsN1s6aZzoQtP0vr2zE2MokbIkAIoS4jucMyCi81bolUg\n\tuAEqNj2RduBaQGEo+GDj9ac0syjDwmWbCE2MEJLyRugRarPf5fl9QPLWfp0pd3vaeGXar0+nfMZq1\n\t509DeP9gEXhzhXTW515A==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=subject:to:cc:references:from:message-id:date:user-agent\n\t:mime-version:in-reply-to:content-language:content-transfer-encoding; \n\tbh=MkZ09e6ZpUkj07myk8PCLe2OZWtcTFYOElXPV1DGLoE=;\n\tb=nfwVJBgBDpnUy4GgmPNF3hEcKvwPE6+2hB4+rMharaqYy3RpRvdn4DS4KbTmp9Zr30\n\tonORYYYQEUqmBPivd9c5zDMxUEQDgkhyZVmmMlYR2IT6oHwIf8BcXucnmVqLrxX15GGM\n\tBRTt42dejccYeFmvl5DWFumqP0690q5BhJGv/naCIfW82kxByUeW980aA87kHhT42YnQ\n\tFA1OobQ+eI9VKYWtOOEMuqglY2Vk3v/rqgleAl44vXt1/sIXoxo+AinXas/fbOoJFqUr\n\tqnuptHM8GO/InZtuoqxWrmzueeX0PCkRIeoA75Bg0phfLz32RmGkNwhNDAvCifFf45ty\n\trzPA=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:subject:to:cc:references:from:message-id:date\n\t:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=MkZ09e6ZpUkj07myk8PCLe2OZWtcTFYOElXPV1DGLoE=;\n\tb=VE0/XLVVyvE8LNRxsPWmzT5seLDxjSqlAbMhu/HzlIbirUDbAKGxSuGRlgF1xNg2q6\n\tHabWlMIzeXSsl5LIZBBZMJv/Qb+/ycuQbguOv/ZRW2rUV6lCQmwq+R89T/St1INCE5JP\n\t63g6ailP41ZRoHFyuGESB+/zuY8Ro4fUqM7LNBl4AVsHBHplV3FQ+QFYtWNyG4WCyMED\n\tCtg9fk6EeFaYCC5RjB1vrlk5a7d3JUKya+Z5piEPOr8zVtBfzIQrvW5VnO4JjXQ8yTBD\n\tScIf9Q7eFqN59bPrB1x9E9kZsUaP1L7/8JMMgSEfuRFE8Q/6EQrqQNJD/4WH0AQJTuH7\n\ttC1A==","X-Gm-Message-State":"AHPjjUgKr+PbQPAKO1PcXCyj1ydGyzM55YCY3xSmid8s9KZpuwXeVVIw\n\tQl22UgnlzPwNu3cnUBKt85I=","X-Google-Smtp-Source":"AOwi7QAABrLQhtxuL74rvZVSYbHj8x0BSIxrp88mNku5k+WXLoTUlFWH1RDv6DUPAPeYuxkrMr/QDQ==","X-Received":"by 10.202.94.11 with SMTP id s11mr1966956oib.60.1506574398838;\n\tWed, 27 Sep 2017 21:53:18 -0700 (PDT)","Subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","To":"Corentin Labbe , robh+dt@kernel.org,\n\tmark.rutland@arm.com, maxime.ripard@free-electrons.com, wens@csie.org,\n\tlinux@armlinux.org.uk, catalin.marinas@arm.com, will.deacon@arm.com, \n\tpeppe.cavallaro@st.com, alexandre.torgue@st.com, andrew@lunn.ch,\n\tfrowand.list@gmail.com","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-6-clabbe.montjoie@gmail.com>","From":"Florian Fainelli ","Message-ID":"<25e44368-2c07-9b7e-a0d3-a2a642286e5d@gmail.com>","Date":"Wed, 27 Sep 2017 21:53:15 -0700","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170927073414.17361-6-clabbe.montjoie@gmail.com>","Content-Language":"en-US","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_215340_949050_9FC732E0 ","X-CRM114-Status":"GOOD ( 22.92 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (f.fainelli[at]gmail.com)\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"netdev@vger.kernel.org, linux-sunxi@googlegroups.com,\n\tlinux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,\n\tdevicetree@vger.kernel.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776807,"web_url":"http://patchwork.ozlabs.org/comment/1776807/","msgid":"<20170928070710.GA32676@Red>","list_archive_url":null,"date":"2017-09-28T07:07:10","subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","submitter":{"id":64152,"url":"http://patchwork.ozlabs.org/api/people/64152/","name":"Corentin Labbe","email":"clabbe.montjoie@gmail.com"},"content":"On Wed, Sep 27, 2017 at 09:53:15PM -0700, Florian Fainelli wrote:\n> \n> \n> On 09/27/2017 12:34 AM, Corentin Labbe wrote:\n> > This patch add documentation about the MDIO switch used on sun8i-h3-emac\n> > for integrated PHY.\n> > \n> > Signed-off-by: Corentin Labbe \n> > ---\n> > .../devicetree/bindings/net/dwmac-sun8i.txt | 138 +++++++++++++++++++--\n> > 1 file changed, 126 insertions(+), 12 deletions(-)\n> > \n> > diff --git a/Documentation/devicetree/bindings/net/dwmac-sun8i.txt b/Documentation/devicetree/bindings/net/dwmac-sun8i.txt\n> > index 725f3b187886..e2ef4683df08 100644\n> > --- a/Documentation/devicetree/bindings/net/dwmac-sun8i.txt\n> > +++ b/Documentation/devicetree/bindings/net/dwmac-sun8i.txt\n> > @@ -4,18 +4,18 @@ This device is a platform glue layer for stmmac.\n> > Please see stmmac.txt for the other unchanged properties.\n> > \n> > Required properties:\n> > -- compatible: should be one of the following string:\n> > +- compatible: must be one of the following string:\n> > \t\t\"allwinner,sun8i-a83t-emac\"\n> > \t\t\"allwinner,sun8i-h3-emac\"\n> > \t\t\"allwinner,sun8i-v3s-emac\"\n> > \t\t\"allwinner,sun50i-a64-emac\"\n> > - reg: address and length of the register for the device.\n> > - interrupts: interrupt for the device\n> > -- interrupt-names: should be \"macirq\"\n> > +- interrupt-names: must be \"macirq\"\n> > - clocks: A phandle to the reference clock for this device\n> > -- clock-names: should be \"stmmaceth\"\n> > +- clock-names: must be \"stmmaceth\"\n> > - resets: A phandle to the reset control for this device\n> > -- reset-names: should be \"stmmaceth\"\n> > +- reset-names: must be \"stmmaceth\"\n> > - phy-mode: See ethernet.txt\n> > - phy-handle: See ethernet.txt\n> > - #address-cells: shall be 1\n> > @@ -39,23 +39,38 @@ Optional properties for the following compatibles:\n> > - allwinner,leds-active-low: EPHY LEDs are active low\n> > \n> > Required child node of emac:\n> > -- mdio bus node: should be named mdio\n> > +- mdio bus node: with compatible \"snps,dwmac-mdio\"\n> > \n> > Required properties of the mdio node:\n> > - #address-cells: shall be 1\n> > - #size-cells: shall be 0\n> > \n> > -The device node referenced by \"phy\" or \"phy-handle\" should be a child node\n> > +The device node referenced by \"phy\" or \"phy-handle\" must be a child node\n> > of the mdio node. See phy.txt for the generic PHY bindings.\n> > \n> > -Required properties of the phy node with the following compatibles:\n> > +The following compatibles require that the mdio node have a mdio-mux child\n> > +node called \"mdio-mux\":\n> > + - \"allwinner,sun8i-h3-emac\"\n> > + - \"allwinner,sun8i-v3s-emac\":\n> > +Required properties for the mdio-mux node:\n> > + - compatible = \"mdio-mux\"\n> > + - one child mdio for the integrated mdio\n> > + - one child mdio for the external mdio if present (V3s have none)\n> > +Required properties for the mdio-mux children node:\n> > + - reg: 1 for internal MDIO bus, 2 for external MDIO bus\n> > +\n> > +The following compatibles require a PHY node representing the integrated\n> > +PHY, under the integrated MDIO bus node if an mdio-mux node is used:\n> > - \"allwinner,sun8i-h3-emac\",\n> > - \"allwinner,sun8i-v3s-emac\":\n> > +\n> > +Required properties of the integrated phy node:\n> > - clocks: a phandle to the reference clock for the EPHY\n> > - resets: a phandle to the reset control for the EPHY\n> > +- phy-is-integrated\n> > +- Must be a child of the integrated mdio\n> > \n> > -Example:\n> > -\n> > +Example with integrated PHY:\n> > emac: ethernet@1c0b000 {\n> > \tcompatible = \"allwinner,sun8i-h3-emac\";\n> > \tsyscon = <&syscon>;\n> > @@ -72,13 +87,112 @@ emac: ethernet@1c0b000 {\n> > \tphy-handle = <&int_mii_phy>;\n> > \tphy-mode = \"mii\";\n> > \tallwinner,leds-active-low;\n> > +\n> > +\tmdio0: mdio {\n> > +\t\t#address-cells = <1>;\n> > +\t\t#size-cells = <0>;\n> > +\t\tcompatible = \"snps,dwmac-mdio\";\n> > +\n> > +\t\tmdio-mux {\n> > +\t\t\tcompatible = \"mdio-mux\";\n> > +\t\t\t#address-cells = <1>;\n> > +\t\t\t#size-cells = <0>;\n> \n> Sorry for chiming in so late, but why don't we have the mdio-mux be the\n> root node here in the mdio bus hierarchy? I understand that with this\n> binding proposed here, we need to have patch 11 included (which btw,\n> should come before any DTS change), but this does not seem to accurately\n> model the HW.\n> \n> The mux itself is not a child node of the MDIO bus controller, it does\n> not really belong in that address space although it does mangle the MDIO\n> bus controller address space between the two ends of the mux.\n> \n> If this has been debated before, apologies for missing that part of the\n> discussion.\n> \n\nI have done it as asked by Rob.\nhttps://lkml.org/lkml/2017/9/13/422\nhttps://lkml.org/lkml/2017/9/19/849\n\nRegards","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"j+mZhRDI\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"mBwrfuNi\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2m444w73z9t5C\n\tfor ;\n\tThu, 28 Sep 2017 17:08:04 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxSvQ-0003R9-H1; Thu, 28 Sep 2017 07:08:00 +0000","from mail-wm0-x244.google.com ([2a00:1450:400c:c09::244])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxSv6-0002Ts-Vp for linux-arm-kernel@lists.infradead.org;\n\tThu, 28 Sep 2017 07:07:46 +0000","by mail-wm0-x244.google.com with SMTP id f4so618553wmh.4\n\tfor ;\n\tThu, 28 Sep 2017 00:07:20 -0700 (PDT)","from Red ([2a01:cb1d:16e:1300:2e56:dcff:fed2:c6d6])\n\tby smtp.googlemail.com with ESMTPSA id\n\tr25sm730971edi.42.2017.09.28.00.07.17\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tThu, 28 Sep 2017 00:07:18 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=LaKjmayJemosxJOxx8nMNaNERDT3Sbe6+CNfFhlVByQ=;\n\tb=j+mZhRDIXpK352\n\ttpqKrgsbGNMle/3dLEaewrByT7tT715ZW9lF99c5pu8DUw+mfd/hLfG0H19HiNx78aO6SFOMtZkBs\n\tcy2OCFSNFL3LFwajEImseK1vHbN4NBQvR26fL45H24FKWV+uccHlU7XCH83TQzmlV019dDZC0a/Fr\n\tIfdAjRLjyxNxQY8BklS+3//UlHL2xrPQ1JdSNAYCb4V+e3TPgTRwK0syWKCW4ctqUDbfE7BK4BOaN\n\tIn6vtyfyc4wwa8LcJ+P0Hab+BeUodZInXFvV5tBoK5AgK//1jHfu5sTQIlXWQH/3KvA3dR3YW8imy\n\tu5B23TU7Jx3uOkY31WNw==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:in-reply-to:user-agent;\n\tbh=/uvjS4Ph3aUlHqb3iciCc9ZaPl8Pww+kZ7E+fXHZvSY=;\n\tb=mBwrfuNiBhA8wTqarSCjTKWJNcrH+CB8QCsSJE/HkZnmwchOoSWrRJSoJR9S2RI1tR\n\te6WeOdksSvLpqmJ9FRnHrqpgVQRhXT3atRBW8rpQUVDBoRP1O/xuZOxv7tp66kxhfLiv\n\tchxMGJG1bZF4lznrgutwdjjcKdPSJB5OLKIdW7cyTnUMhre7dWJ42tB8wEaD0xLg053e\n\tY+JUlDQjGftVprXk17rJF+I0BDQ6JCd/X6u+q/MXvDX10BrYmMogVEzhH3rHMRCr7v5g\n\tD1Dz2v0A/JLp5Q1CtIvFzdBBXYm7wE6a8ktDVWFWQ/FDD9xxI9k+c5KY8LS8LCXfQpdu\n\t3IZA=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=/uvjS4Ph3aUlHqb3iciCc9ZaPl8Pww+kZ7E+fXHZvSY=;\n\tb=kwLwjBtr3bkvlZncq6mHEirgL0zL4LIGilHvhVWZKE7jxkqv9RNWGyPzBzhjbktzMI\n\toUGX4FyBSaG5/BLOeO5Yiwa4bsGh3TPhe24wmMZRb0rXCcoYWLdfJOPLG6TiV73lSC1Y\n\tC6U3p9lmXI4W9FidhXxIyoe1RoMoWFdgwEXOvcH697o8+bQRx38YzkYSdt62hFHM0zBk\n\tW3td/3ytVfDIBjQ39MUlRAadtm0qN2tKUTPsWCL9Gs4LBvejWyBneWaqQ3Cj2JarOLkH\n\tg/FMHXlXF/KLKVDUb4d2316mjTi2Yuz4MBjP5x4xUQTu3uShe/psOMBYKc5MhVjCUECp\n\tS6Iw==","X-Gm-Message-State":"AHPjjUj2rHqVo9rqGdork2oW6d9YRpEJPYtGa0UwYetqw2jeCeJQTzoP\n\tzkPiPGLDAQUjHnopA37r9gs=","X-Google-Smtp-Source":"AOwi7QAlOflQWxyoVpYBHe7buX8qmzr//OyIOy5wB8x9KPYxf4vyMSpQ83uoZDWBSD6a+dZGgzb+1g==","X-Received":"by 10.80.184.165 with SMTP id l34mr4473058ede.53.1506582438538; \n\tThu, 28 Sep 2017 00:07:18 -0700 (PDT)","Date":"Thu, 28 Sep 2017 09:07:10 +0200","From":"Corentin Labbe ","To":"Florian Fainelli ","Subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","Message-ID":"<20170928070710.GA32676@Red>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-6-clabbe.montjoie@gmail.com>\n\t<25e44368-2c07-9b7e-a0d3-a2a642286e5d@gmail.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<25e44368-2c07-9b7e-a0d3-a2a642286e5d@gmail.com>","User-Agent":"Mutt/1.7.2 (2016-11-26)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170928_000741_242585_45AA98BA ","X-CRM114-Status":"GOOD ( 27.36 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a00:1450:400c:c09:0:0:0:244 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (clabbe.montjoie[at]gmail.com)\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, andrew@lunn.ch, alexandre.torgue@st.com,\n\tdevicetree@vger.kernel.org, catalin.marinas@arm.com,\n\tlinux-sunxi@googlegroups.com, will.deacon@arm.com,\n\tlinux@armlinux.org.uk, \n\tlinux-kernel@vger.kernel.org, wens@csie.org, robh+dt@kernel.org,\n\tnetdev@vger.kernel.org, peppe.cavallaro@st.com,\n\tmaxime.ripard@free-electrons.com, frowand.list@gmail.com,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776828,"web_url":"http://patchwork.ozlabs.org/comment/1776828/","msgid":"<20170928073708.GB32676@Red>","list_archive_url":null,"date":"2017-09-28T07:37:08","subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","submitter":{"id":64152,"url":"http://patchwork.ozlabs.org/api/people/64152/","name":"Corentin Labbe","email":"clabbe.montjoie@gmail.com"},"content":"On Wed, Sep 27, 2017 at 04:02:10PM +0200, Andrew Lunn wrote:\n> Hi Corentin\n> \n> > +Required properties for the mdio-mux node:\n> > + - compatible = \"mdio-mux\"\n> \n> This is too generic. Please add a more specific compatible for this\n> particular mux. You can keep \"mdio-mux\", since that is what the MDIO\n> subsystem will look for.\n> \n\nI will add allwinner,sun8i-h3-mdio-mux\n\n> > +Required properties of the integrated phy node:\n> > - clocks: a phandle to the reference clock for the EPHY\n> > - resets: a phandle to the reset control for the EPHY\n> > +- phy-is-integrated\n> \n> So the last thing you said is that the mux is not the problem\n> here. Something else is locking up. Did you discover what?\n> \n> I really would like phy-is-integrated to go away.\n> \n\nI have found the problem: by enabling ephy clk/reset the timeout does not occur anymore.\nSo we could remove phy-is-integrated by:\nMoving internal phy clk/reset handling in mdio_mux_syscon_switch_fn()\nBut this means:\n- getting internalphy node always by manually get internal_mdio/internal_phy (and not by the given phyhandle)\n- doing some unnecessary tasks (enable/scan/disable) when external_phy is needed\n\nRegards","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"VlDpTBTH\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"Q6LBOITu\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2mkL2FZGz9t5x\n\tfor ;\n\tThu, 28 Sep 2017 17:37:44 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxTO9-0000i6-GR; Thu, 28 Sep 2017 07:37:41 +0000","from mail-wm0-x232.google.com ([2a00:1450:400c:c09::232])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxTO5-0000Pk-Kp for linux-arm-kernel@lists.infradead.org;\n\tThu, 28 Sep 2017 07:37:39 +0000","by mail-wm0-x232.google.com with SMTP id m127so318782wmm.3\n\tfor ;\n\tThu, 28 Sep 2017 00:37:16 -0700 (PDT)","from Red ([2a01:cb1d:16e:1300:2e56:dcff:fed2:c6d6])\n\tby smtp.googlemail.com with ESMTPSA id\n\tf20sm773442edm.46.2017.09.28.00.37.14\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tThu, 28 Sep 2017 00:37:14 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=VgR1JO+vj04ziGD3dYCMwuX3223d8TxNG5qJZm1i/d4=;\n\tb=VlDpTBTHi+vVD8\n\tG6Tb8z1Fkln6IF+TqnFj6nozG50ExsOYGghMc+tIQoQXlnmzaav2FacXxgVmZ1S/9e608gIiB/RGX\n\t4PMTty2assDWBRZyNTdZGmnd73B3apSsyRxpWb4KBs/+NhtLol0nFFiO6CBUGEH4UjVV4EK3lCszp\n\t65WCERDdCmEie5mMyRTB35YZG2n6dX7t7QA6JF25FIfst2XqoQOUaW73oOUsP2GLE/XplkxxdBt3D\n\tLQlRc6sxjX4Cser7rto0go5HxdWmxRdKaylw4BzOExt2sm13z5uxKt0lW2PESXOQ+XqmkZ8gakOQN\n\tzC33Y7BFN4T1ze7emYbw==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:in-reply-to:user-agent;\n\tbh=WIsKWtPuX6YpbO05Dgt7S9cmHporb41cOglNI2I1VnQ=;\n\tb=Q6LBOITuMKnxlkTTfYR3CkHj+eFjijj3b175QhQmXs0T520lkKbEXxbaqm0Cw6D1WW\n\tkhMb+2yG2i0Lrpt7wdGZ5XcVOXkA/Y7ra6FUPPqjaRwNG681Hh7Hns/CAZivmwHPPs8W\n\tT7oMGrbHVunE/7/vpvyddYhEV+YlOS09b6GgP+ic0y7B79gW6OkTnksWDsm5SpVX9dT1\n\toWA6LaBrKshTmZdNRWR2VImzV4X6/FFeT7lbzo3ct/GxFL466OU5oIcatseR9aYMGdeF\n\tJ/LOhSHWIbqD+DByI4tGT1mXtWEiMTgyNsvQJAKRRRX8wtjHo6aKrOTPEH+KEpNwk3Z3\n\tPKOQ=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=WIsKWtPuX6YpbO05Dgt7S9cmHporb41cOglNI2I1VnQ=;\n\tb=tPcCLlBRtYQkk9PhgS1uByGoQ6wo+Gwtz78i5VIeZ2v/QsYdi9FoL/q/xIVZbQCkR+\n\tHZtHR7Vex/XZPR3WXSpUsjLvKqrtwoB59ZSfeFhYBIqa3lE6y5D3qN2d7eNLcHfiIUL9\n\tEvudJ2W5ndVJM641p0N3UBXQTLNE0mOzrmi5itl5ZAM9dFlqh0qWAvEujo6Eekty6oGt\n\t/98F18tDqz3tWgozx5ia3wPYEjJp/Vc/WODB2u2jRPuf2PnQ5zSOYTfs5vmOqGq9XpCq\n\tFlU8PMvIpwN9lPT1mCavgVRPrY6VOtv7NpTFEUalr5+IYtpOlCDSWUlbSODqGkmaj7YS\n\t6fGQ==","X-Gm-Message-State":"AHPjjUjd5OaGnuIR50oAjVD4o6xraeQgk5KCPmDuX7DosDrHHRd7fnlR\n\tYTTS/35QzyGdmxk645T2YFY=","X-Google-Smtp-Source":"AOwi7QDiHyQtM1VF1aYulleAYxodFUwtFA8AXh9PRzeaxjdXWxOXdLg34oAbTwazwXZxmPN9RAlTDw==","X-Received":"by 10.80.163.101 with SMTP id 92mr4719983edn.100.1506584235260; \n\tThu, 28 Sep 2017 00:37:15 -0700 (PDT)","Date":"Thu, 28 Sep 2017 09:37:08 +0200","From":"Corentin Labbe ","To":"Andrew Lunn ","Subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","Message-ID":"<20170928073708.GB32676@Red>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-6-clabbe.montjoie@gmail.com>\n\t<20170927140210.GE13516@lunn.ch>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170927140210.GE13516@lunn.ch>","User-Agent":"Mutt/1.7.2 (2016-11-26)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170928_003737_856984_BEF99509 ","X-CRM114-Status":"GOOD ( 15.57 )","X-Spam-Score":"-2.7 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.7 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/,\n\tlow\n\ttrust [2a00:1450:400c:c09:0:0:0:232 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (clabbe.montjoie[at]gmail.com)\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, devicetree@vger.kernel.org, f.fainelli@gmail.com, \n\talexandre.torgue@st.com, catalin.marinas@arm.com,\n\tlinux-sunxi@googlegroups.com, \n\twill.deacon@arm.com, linux@armlinux.org.uk,\n\tlinux-kernel@vger.kernel.org, \n\twens@csie.org, robh+dt@kernel.org, netdev@vger.kernel.org,\n\tpeppe.cavallaro@st.com, maxime.ripard@free-electrons.com,\n\tfrowand.list@gmail.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1780081,"web_url":"http://patchwork.ozlabs.org/comment/1780081/","msgid":"<20171004190028.GA9208@Red>","list_archive_url":null,"date":"2017-10-04T19:00:28","subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","submitter":{"id":64152,"url":"http://patchwork.ozlabs.org/api/people/64152/","name":"Corentin Labbe","email":"clabbe.montjoie@gmail.com"},"content":"On Thu, Sep 28, 2017 at 09:37:08AM +0200, Corentin Labbe wrote:\n> On Wed, Sep 27, 2017 at 04:02:10PM +0200, Andrew Lunn wrote:\n> > Hi Corentin\n> > \n> > > +Required properties for the mdio-mux node:\n> > > + - compatible = \"mdio-mux\"\n> > \n> > This is too generic. Please add a more specific compatible for this\n> > particular mux. You can keep \"mdio-mux\", since that is what the MDIO\n> > subsystem will look for.\n> > \n> \n> I will add allwinner,sun8i-h3-mdio-mux\n> \n> > > +Required properties of the integrated phy node:\n> > > - clocks: a phandle to the reference clock for the EPHY\n> > > - resets: a phandle to the reset control for the EPHY\n> > > +- phy-is-integrated\n> > \n> > So the last thing you said is that the mux is not the problem\n> > here. Something else is locking up. Did you discover what?\n> > \n> > I really would like phy-is-integrated to go away.\n> > \n> \n> I have found the problem: by enabling ephy clk/reset the timeout does not occur anymore.\n> So we could remove phy-is-integrated by:\n> Moving internal phy clk/reset handling in mdio_mux_syscon_switch_fn()\n> But this means:\n> - getting internalphy node always by manually get internal_mdio/internal_phy (and not by the given phyhandle)\n> - doing some unnecessary tasks (enable/scan/disable) when external_phy is needed\n> \n\nHello\n\nI have get rid of phy-is-integrated, but mdio_mux_syscon_switch_fn need to enable/disable ephy clk/reset.\nAnd so access to internal PHY node.\nBut current DT made this ugly: (need to find mdio-mux then internalmdio then internal PHY)\n\nSince MAC cannot reset/choose internal MDIO without ephy clk/rst, could we interpret this as thoses clk/rst must be set in emac node.\nThis will simplify a lot the code.\n\nRegards","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"UAi/o6WO\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"jaPrVEUk\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y6lw308wmz9t6m\n\tfor ;\n\tThu, 5 Oct 2017 06:14:59 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dzp8B-0006m3-KH; Wed, 04 Oct 2017 19:14:55 +0000","from mail-wm0-x243.google.com ([2a00:1450:400c:c09::243])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dzoug-00006j-Bv for linux-arm-kernel@lists.infradead.org;\n\tWed, 04 Oct 2017 19:01:57 +0000","by mail-wm0-x243.google.com with SMTP id u138so7952046wmu.0\n\tfor ;\n\tWed, 04 Oct 2017 12:00:37 -0700 (PDT)","from Red ([2a01:cb1d:16e:1300:2e56:dcff:fed2:c6d6])\n\tby smtp.googlemail.com with ESMTPSA id\n\ti16sm12706145edj.77.2017.10.04.12.00.35\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 04 Oct 2017 12:00:35 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=QZOOBDGPL/iWF9MdJyq1Oo1c3u4z48qv7FaDTQ4PBBA=;\n\tb=UAi/o6WO81/IIO\n\toAYgtTkjxu5S91if5qc5weYA0g9sRziqH7ZvIFXcmJGdfo64618mxunSR0riXBo8x1lyOAAfCeX7Y\n\thmZxQy2IIGDtk7itrVArSthKcEioeWDIwxpdn3DY12QgqbdFne56/25efLLES8e4lIIHqoCObcImT\n\tX4bDJkc7wc5zI1XUP4aZQcCl/833M9JUDOqXowLqppMLdHbWpntPw3cLZFY4L3qLOgr9Xt+3u9GAZ\n\tGsqG2eBnYnWYECfAH282sifw7/szyhEe84ZLgTAsSij10KEI8aqZTLRpLFmL/NiYW2Phw7UCterBt\n\t0YMFreWSaI75MDTzGd/Q==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:in-reply-to:user-agent;\n\tbh=5MUUta0y1CcIAOCmaC0/yTSsRnP1ATXRxjJwPxpIuxg=;\n\tb=jaPrVEUkGtwb0ekDAqg02sgbB9gpJqmufeaELhRcnEHK6HET40yNMmiM4zGeUrMbNI\n\tyy27R9WNRbnxtDmPI4S56qop7C36ScFsSFVlqg7x3bNeSbkXlmRpEcbLt7OxBRL7Xjkj\n\t7BSHMeVsWHI7y8bI3mp0efeET6j1OT3bMztJfdGJpSijPTHjOgNPkx7ozZFHA5zJX+tH\n\to9Dz43IyPiGEJlOiiwMVs56DPJS8pyxUqrHlxKPX7hKNKVXJfE9xZuPUPj/TKMP5LeVK\n\tKhP9xdoKVIzxMSAaVbykxU6P7v0z+4vnABzRT6UF9WQyhamB6d8ma2aNDifLU0gtIj0d\n\tVhBA=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=5MUUta0y1CcIAOCmaC0/yTSsRnP1ATXRxjJwPxpIuxg=;\n\tb=mYiL8ofI7hurOtRTU/4I1xMsHcnZdkfyX8dnxWIkBGIp2T3SNx6f6dsuViR9pGA8Gt\n\t+ZBB8kbFQDC4NjyL3OR8Y1hrJdrifSKI8PVYCbYkkGd8gcz/iT8IxYcPvCWOGMz6hkPe\n\tZfafGHT6YfUF3mPz1l/tAgvnz2z6Bri1SK6TbJnTN2i+CVLA9sQPqe+qa1SBZRrvZHhC\n\tr2sAOahPKGTn4TiSEIjlB0NhPTQhJURXnRz8ea623bEbC4NwtHeuEqLQVqNByfppoYwI\n\tEle94dw98YQ0uoGdSD/of0HHT1FDLQIrpPDO9KYERR/Zv68QU6+xgMzqcfxphzduQlvk\n\tSCCg==","X-Gm-Message-State":"AHPjjUjqeS+X5Trh1LGxO2qJcflyxEG64Laz90jpxgOOqXulVZjkec6t\n\tpYqExfqD+twbdGJ8/IoRipU=","X-Google-Smtp-Source":"AOwi7QAQMoU4E47yr2AC9kpzED6IowvXm5Lg1veKCFHfn/KwghgAO+sBC5tfMNJc7e2IMVS/XrkVlg==","X-Received":"by 10.80.183.148 with SMTP id h20mr29977131ede.178.1507143636183;\n\tWed, 04 Oct 2017 12:00:36 -0700 (PDT)","Date":"Wed, 4 Oct 2017 21:00:28 +0200","From":"Corentin Labbe ","To":"Andrew Lunn , robh+dt@kernel.org,\n\tmaxime.ripard@free-electrons.com, f.fainelli@gmail.com, wens@csie.org","Subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","Message-ID":"<20171004190028.GA9208@Red>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-6-clabbe.montjoie@gmail.com>\n\t<20170927140210.GE13516@lunn.ch> <20170928073708.GB32676@Red>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170928073708.GB32676@Red>","User-Agent":"Mutt/1.7.2 (2016-11-26)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171004_120059_232021_A6B73FD8 ","X-CRM114-Status":"GOOD ( 17.94 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a00:1450:400c:c09:0:0:0:243 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (clabbe.montjoie[at]gmail.com)\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, devicetree@vger.kernel.org, alexandre.torgue@st.com,\n\tcatalin.marinas@arm.com, will.deacon@arm.com, linux@armlinux.org.uk, \n\tlinux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com,\n\tnetdev@vger.kernel.org, peppe.cavallaro@st.com, frowand.list@gmail.com,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1781001,"web_url":"http://patchwork.ozlabs.org/comment/1781001/","msgid":"<20171005205956.2wy26t4mugbowucn@rob-hp-laptop>","list_archive_url":null,"date":"2017-10-05T20:59:56","subject":"Re: [PATCH v6 01/11] dt-bindings: net: Restore sun8i dwmac binding","submitter":{"id":62529,"url":"http://patchwork.ozlabs.org/api/people/62529/","name":"Rob Herring (Arm)","email":"robh@kernel.org"},"content":"On Wed, Sep 27, 2017 at 09:34:04AM +0200, Corentin Labbe wrote:\n> This patch restore dt-bindings documentation about dwmac-sun8i\n> This reverts commit 8aa33ec2f481 (\"dt-bindings: net: Revert sun8i dwmac binding\")\n\nWhat's missing here is why you are reverting?\n\n> \n> Signed-off-by: Corentin Labbe \n> ---\n> .../devicetree/bindings/net/dwmac-sun8i.txt | 84 ++++++++++++++++++++++\n> 1 file changed, 84 insertions(+)\n> create mode 100644 Documentation/devicetree/bindings/net/dwmac-sun8i.txt\n\nOtherwise,\n\nAcked-by: Rob Herring ","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"WlGajKQQ\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y7QCK3fQzz9s3w\n\tfor ;\n\tFri, 6 Oct 2017 08:00:29 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e0DFn-0004S8-Mm; Thu, 05 Oct 2017 21:00:23 +0000","from mail-oi0-f68.google.com ([209.85.218.68])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e0DFj-0002rd-Iy for linux-arm-kernel@lists.infradead.org;\n\tThu, 05 Oct 2017 21:00:21 +0000","by mail-oi0-f68.google.com with SMTP id h200so2924509oib.1\n\tfor ;\n\tThu, 05 Oct 2017 13:59:58 -0700 (PDT)","from localhost (216-188-254-6.dyn.grandenetworks.net.\n\t[216.188.254.6]) by smtp.gmail.com with ESMTPSA id\n\ts14sm10182599ots.22.2017.10.05.13.59.57\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tThu, 05 Oct 2017 13:59:57 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=/Vx1bwoMfmnl9FC6CQKwSHOKfMb+N1yPQhbC78Unbww=;\n\tb=WlGajKQQKegQUy\n\tX5KCu2R8o0T5oAlfBieA2PCZqgrvKypqMrhGkMG02Lm5Fmq5EC2WhFIiLE0a+21H2TYLmXEWLcGqu\n\tghbNCGzXxy22BJmkX96L+AT6Z4Jw6J1tAY/cAgKTuiEAzlfp33mEggO3/gyHp710hG800Zixrd/6Z\n\tMG1P+gz56asQSVsAOrvNS/LM2h3hBA6TElMwepiZbNF6nfm4a6p9DD+XRAJXRaCUqKA0vmkaKnKRA\n\tV+zdFhdI/WbBKbuOpVic1uQi5tSodbzmPp0mjiDKrMlRz0Ntae10WcgG6lq029Iw1V3EdjND2mw01\n\tRhqiFcorDuWaSG8b2lJQ==;","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=rZbZ1bq9hYAZAL1FiEx0mMPbidh440edpBqfGNXgp6w=;\n\tb=f6yRJFBEsG4bTtnY6Qfy7tfoY+Thj3oo0be9dwppx4IB7B1A/8pq36btrWgHi2O6cu\n\tIAGmi4Jh9CBOosseknvBv/8tcnW6r32D+uFLReYBBjK1yfqwyPOxeXzWjMzO0XVW8UCA\n\tMcfq2g+afXwQSqyBg3LUxClMezvYml9N72+BZIDsAM2S//bPF/VEa8EOUn2lcpuYHLU9\n\tdThD1owj6dIoTks/qQHfpRuVPPifz48+NrlqqSzORp8jHQri7DzP+qy7c7daej5gkHa0\n\tkBoQkzIfVRsvafa1RNjPP7uXQAPE5kw43OSK8BEvMIZu1Gu5x9vs6oJ0zD4qsIFvbbFB\n\to8iQ==","X-Gm-Message-State":"AMCzsaUIy6+7cCfr/I370hk4i9STvkJIIno+l8cIVLNAe8VO3engey8V\n\t1zVs88SYtsHq8Bu7d0JUHQ==","X-Google-Smtp-Source":"AOwi7QAU5UKAp6kNhtQ4I+DzQbKydLoCPztBEzpjMkUmvaxXodDeNd1mVXJVtP6YDT3MtauIYVgmbA==","X-Received":"by 10.157.57.132 with SMTP id y4mr12983491otb.283.1507237198051; \n\tThu, 05 Oct 2017 13:59:58 -0700 (PDT)","Date":"Thu, 5 Oct 2017 15:59:56 -0500","From":"Rob Herring ","To":"Corentin Labbe ","Subject":"Re: [PATCH v6 01/11] dt-bindings: net: Restore sun8i dwmac binding","Message-ID":"<20171005205956.2wy26t4mugbowucn@rob-hp-laptop>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-2-clabbe.montjoie@gmail.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170927073414.17361-2-clabbe.montjoie@gmail.com>","User-Agent":"NeoMutt/20170113 (1.7.2)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171005_140019_744575_B7C543E2 ","X-CRM114-Status":"GOOD ( 10.38 )","X-Spam-Score":"-1.0 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [209.85.218.68 listed in list.dnswl.org]\n\t0.5 RCVD_IN_SORBS_SPAM RBL: SORBS: sender is a spam source\n\t[209.85.218.68 listed in dnsbl.sorbs.net]\n\t-0.0 RCVD_IN_MSPIKE_H3 RBL: Good reputation (+3)\n\t[209.85.218.68 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.2 FREEMAIL_ENVFROM_END_DIGIT Envelope-from freemail username ends\n\tin digit (robherring2[at]gmail.com)\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (robherring2[at]gmail.com)\n\t0.0 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level\n\tmail domains are different\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.0 RCVD_IN_MSPIKE_WL Mailspike good senders\n\t0.2 FREEMAIL_FORGED_FROMDOMAIN 2nd level domains in From and\n\tEnvelopeFrom freemail headers are different","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, andrew@lunn.ch, f.fainelli@gmail.com,\n\talexandre.torgue@st.com, devicetree@vger.kernel.org,\n\tcatalin.marinas@arm.com, \n\tlinux-sunxi@googlegroups.com, will.deacon@arm.com,\n\tlinux@armlinux.org.uk, \n\tlinux-kernel@vger.kernel.org, wens@csie.org, netdev@vger.kernel.org, \n\tpeppe.cavallaro@st.com, maxime.ripard@free-electrons.com,\n\tfrowand.list@gmail.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1781075,"web_url":"http://patchwork.ozlabs.org/comment/1781075/","msgid":"<20171005220927.7rabn3ftvow253xj@rob-hp-laptop>","list_archive_url":null,"date":"2017-10-05T22:09:27","subject":"Re: [PATCH v6 11/11] of: mdio: Prevent of_mdiobus_register from\n\tscanning mdio-mux nodes","submitter":{"id":62529,"url":"http://patchwork.ozlabs.org/api/people/62529/","name":"Rob Herring (Arm)","email":"robh@kernel.org"},"content":"On Wed, Sep 27, 2017 at 09:50:46PM -0700, Florian Fainelli wrote:\n> \n> \n> On 09/27/2017 07:12 AM, Andrew Lunn wrote:\n> > On Wed, Sep 27, 2017 at 09:34:14AM +0200, Corentin Labbe wrote:\n> >> Each child node of an MDIO node is scanned as a PHY when calling\n> >> of_mdiobus_register() givint the following result:\n> >> [ 18.175379] mdio_bus stmmac-0: /soc/ethernet@1c30000/mdio/mdio-mux has invalid PHY address\n> >> [ 18.175408] mdio_bus stmmac-0: scan phy mdio-mux at address 0\n> >> [ 18.175450] mdio_bus stmmac-0: scan phy mdio-mux at address 1\n> >> [...]\n> >> [ 18.176420] mdio_bus stmmac-0: scan phy mdio-mux at address 30\n> >> [ 18.176452] mdio_bus stmmac-0: scan phy mdio-mux at address 31\n> >>\n> >> Since mdio-mux nodes are not PHY, this patch a way to to not scan\n> >> them.\n> > \n> > Hi Corentin\n> > \n> > I still don't like this, but ...\n> \n> Me neither, even more so as I don't understand the reasoning behind\n> putting the mux as a child node of the MDIO bus controller in the first\n> place.\n\nIf no one agrees with me, then you shouldn't just accept my answers. I'm \nnot always right.\n\nLooking at this some more, why can't mdio-mux-mmio be used? Then there \nis nothing to review or discuss. If you want to extend that then make it \nclear this is just extending/inheriting already defined bindings.\n\nThat being said, if we were starting over I would probably do mux \ndesigns differently with DT hierarchy reflecting the mux hierarchy, but \nthat ship has sailed.\n\nRob","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"CvbYjCjK\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y7RlW22Kyz9t5Y\n\tfor ;\n\tFri, 6 Oct 2017 09:09:59 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e0EL4-00026J-Cr; Thu, 05 Oct 2017 22:09:54 +0000","from mail-oi0-f43.google.com ([209.85.218.43])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e0EL0-0001sv-Bm for linux-arm-kernel@lists.infradead.org;\n\tThu, 05 Oct 2017 22:09:52 +0000","by mail-oi0-f43.google.com with SMTP id m198so14532456oig.5\n\tfor ;\n\tThu, 05 Oct 2017 15:09:29 -0700 (PDT)","from localhost (216-188-254-6.dyn.grandenetworks.net.\n\t[216.188.254.6]) by smtp.gmail.com with ESMTPSA id\n\th124sm33184oib.34.2017.10.05.15.09.27\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tThu, 05 Oct 2017 15:09:27 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=LdyT58SxVjI3pom7lhGkXk94oTUKMEuia3Yh/PY3E6A=;\n\tb=CvbYjCjKn+/1i2\n\tHBPXh1zrI0x0KAUxMp3enyqCONtBge2wBMCAd8Q4xlPZ6EQtjIiyUjZMjhdoS/sLq3QF3BsD0ezkD\n\tJA0uxaTrWnXn2J6h/E+GeVIpyDEgAOrF63M6fNQjtVWSgCUWLBhWeSXs/yNzAaQPFDuAtYhWeZx4D\n\tKhiA0FLJ3olFOUqItH1u2rB0Sitl8+kbJq6Dwu5Fl7NvTAT7VSsBlTvBSHvA1Lh+0aERoBU+H2Ujn\n\tuP94kr/6EXX/oUqewerKQW30/ac1bsb6iIqxJX8lXYuqYZiB4HS7AgP7XkLGPUDY33UW9eUkQzFHA\n\tjQxPBuj3eVUeb5x2OGkA==;","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=1RCN6xC8xFRao7A8uSNOdvC+lX7CucwQZP5OaoiOGIM=;\n\tb=J42xOVbp2iD2ZxCFFsohX3LNL66Tnq0kUYMDLlI4eYJu1K3ttXlG34lMrEVnzJ00/o\n\tukQIOsF+xDSX9GGHrRBMfy1yUpAzWLplPSaVj0YakPwrie8PsE6ydvKgy3fS7y0pnABA\n\tj6aIGGFr1TEftQLX9PzXmvWNWcAt25zimNQxUOY/UCvH8kI6N2msGlDg07I3sDF0e0V5\n\tCZiEWsXTBVoZR3vWYNr4xRSJQNBBY250008PCHJqOnTI339l2JhRWZJXz/hQasNdWLA7\n\t0SvThOHCgTSOlUmXjEgHvGqvR8NJgs4JXJHsKBZw4o5kowaTMJzWqBkJplRiZ3z9+DxI\n\t4W4Q==","X-Gm-Message-State":"AMCzsaUSaDWyaJEuwW7E5916LlmNHsRz2t7iSCZuT0jSIeXOjqEPdJ4o\n\tP3xFFEnOZtGQ5I/vfXC2rg==","X-Google-Smtp-Source":"AOwi7QDPUahKbsVHGeHRlwyw1j1WAAJqRFcgA0ij/YHvnGmGEJhVF0hCkigHa8le0iWUjpO5uNyVPw==","X-Received":"by 10.157.46.207 with SMTP id w73mr78092ota.227.1507241368458;\n\tThu, 05 Oct 2017 15:09:28 -0700 (PDT)","Date":"Thu, 5 Oct 2017 17:09:27 -0500","From":"Rob Herring ","To":"Florian Fainelli ","Subject":"Re: [PATCH v6 11/11] of: mdio: Prevent of_mdiobus_register from\n\tscanning mdio-mux nodes","Message-ID":"<20171005220927.7rabn3ftvow253xj@rob-hp-laptop>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-12-clabbe.montjoie@gmail.com>\n\t<20170927141213.GF13516@lunn.ch>\n\t","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"","User-Agent":"NeoMutt/20170113 (1.7.2)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171005_150950_481404_0795585B ","X-CRM114-Status":"GOOD ( 15.37 )","X-Spam-Score":"-3.8 (---)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-3.8 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t0.5 RCVD_IN_SORBS_SPAM RBL: SORBS: sender is a spam source\n\t[209.85.218.43 listed in dnsbl.sorbs.net]\n\t-2.8 RCVD_IN_MSPIKE_H2 RBL: Average reputation (+2)\n\t[209.85.218.43 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.2 FREEMAIL_ENVFROM_END_DIGIT Envelope-from freemail username ends\n\tin digit (robherring2[at]gmail.com)\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (robherring2[at]gmail.com)\n\t0.0 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level\n\tmail domains are different\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [209.85.218.43 listed in list.dnswl.org]\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t0.2 FREEMAIL_FORGED_FROMDOMAIN 2nd level domains in From and\n\tEnvelopeFrom freemail headers are different","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, Andrew Lunn ,\n\talexandre.torgue@st.com, \n\tdevicetree@vger.kernel.org, catalin.marinas@arm.com,\n\tlinux-sunxi@googlegroups.com, will.deacon@arm.com,\n\tlinux@armlinux.org.uk, linux-kernel@vger.kernel.org, wens@csie.org,\n\tCorentin Labbe , netdev@vger.kernel.org,\n\tpeppe.cavallaro@st.com, maxime.ripard@free-electrons.com,\n\tfrowand.list@gmail.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1781092,"web_url":"http://patchwork.ozlabs.org/comment/1781092/","msgid":"<20171005222159.GO13247@lunn.ch>","list_archive_url":null,"date":"2017-10-05T22:21:59","subject":"Re: [PATCH v6 11/11] of: mdio: Prevent of_mdiobus_register from\n\tscanning mdio-mux nodes","submitter":{"id":13608,"url":"http://patchwork.ozlabs.org/api/people/13608/","name":"Andrew Lunn","email":"andrew@lunn.ch"},"content":"> Looking at this some more, why can't mdio-mux-mmio be used?\n\nHi Rob\n\nThat was my first thought. And then i dug into the details. The mux is\nmore than just a couple of bits in an address space. Clocks needs to\nbe enabled/disabled, and the MAC needs a reset, when the mux changes.\nSo the mux needs intimate knowledge of MAC.\n\n Andrew","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"Wj6jLLd0\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y7S2K32Tqz9t63\n\tfor ;\n\tFri, 6 Oct 2017 09:22:49 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e0EXU-0001Ex-C2; Thu, 05 Oct 2017 22:22:44 +0000","from vps0.lunn.ch ([185.16.172.187])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e0EXQ-0000cq-7v for linux-arm-kernel@lists.infradead.org;\n\tThu, 05 Oct 2017 22:22:42 +0000","from andrew by vps0.lunn.ch with local (Exim 4.84_2)\n\t(envelope-from )\n\tid 1e0EWl-0004WR-Pf; Fri, 06 Oct 2017 00:21:59 +0200"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=alRsVcwxLd40LcowCHHKnZXhagesGFywVXkkQWszIMU=;\n\tb=Wj6jLLd0L2NhUq\n\tcTMFA6gYTyoUHRMADHCPgy44xpNHePhbfTkpMMIHepo2H4WjtNpRbETMQa+clsJWb/69Af+RJOnld\n\tXE1r129oYbwbJVIc2vZviqDQmXqUYNjm1VeH0GKQ4tFxp5bXL3t5tl5pEycSOeZE4nyRULewMBKzD\n\tbJDC94xaeIeLxn96MSpspfcVE5ywJ9UbtjOykwITtcZIr23rB42MG6BtvVbTujV2AUA9+LXfU9AZO\n\tgMJVb/hFCaON2iv76x3NiqXyZ9T/HmHA5Z+uQ4Y82Y8pO3jSdU0pY1bKND4nbPSGmEjjGcmnd4A+4\n\tIf4XHZdcAM6NBIhTuLUA==;","Date":"Fri, 6 Oct 2017 00:21:59 +0200","From":"Andrew Lunn ","To":"Rob Herring ","Subject":"Re: [PATCH v6 11/11] of: mdio: Prevent of_mdiobus_register from\n\tscanning mdio-mux nodes","Message-ID":"<20171005222159.GO13247@lunn.ch>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-12-clabbe.montjoie@gmail.com>\n\t<20170927141213.GF13516@lunn.ch>\n\t\n\t<20171005220927.7rabn3ftvow253xj@rob-hp-laptop>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20171005220927.7rabn3ftvow253xj@rob-hp-laptop>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171005_152240_553215_85A1B4B1 ","X-CRM114-Status":"UNSURE ( 5.72 )","X-CRM114-Notice":"Please train this message.","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, devicetree@vger.kernel.org,\n\tFlorian Fainelli , alexandre.torgue@st.com,\n\tcatalin.marinas@arm.com, linux-sunxi@googlegroups.com,\n\twill.deacon@arm.com, \n\tlinux@armlinux.org.uk, linux-kernel@vger.kernel.org, wens@csie.org,\n\tCorentin Labbe , netdev@vger.kernel.org,\n\tpeppe.cavallaro@st.com, maxime.ripard@free-electrons.com,\n\tfrowand.list@gmail.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1783250,"web_url":"http://patchwork.ozlabs.org/comment/1783250/","msgid":"<20171009210831.qc2cgxr4cx2qz5x7@flea.home>","list_archive_url":null,"date":"2017-10-09T21:08:31","subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","submitter":{"id":12916,"url":"http://patchwork.ozlabs.org/api/people/12916/","name":"Maxime Ripard","email":"maxime.ripard@free-electrons.com"},"content":"On Sun, Oct 08, 2017 at 06:33:40PM +0000, Corentin Labbe wrote:\n> On Thu, Sep 28, 2017 at 09:37:08AM +0200, Corentin Labbe wrote:\n> > On Wed, Sep 27, 2017 at 04:02:10PM +0200, Andrew Lunn wrote:\n> > > Hi Corentin\n> > > \n> > > > +Required properties for the mdio-mux node:\n> > > > + - compatible = \"mdio-mux\"\n> > > \n> > > This is too generic. Please add a more specific compatible for this\n> > > particular mux. You can keep \"mdio-mux\", since that is what the MDIO\n> > > subsystem will look for.\n> > > \n> > \n> > I will add allwinner,sun8i-h3-mdio-mux\n> > \n> > > > +Required properties of the integrated phy node:\n> > > > - clocks: a phandle to the reference clock for the EPHY\n> > > > - resets: a phandle to the reset control for the EPHY\n> > > > +- phy-is-integrated\n> > > \n> > > So the last thing you said is that the mux is not the problem\n> > > here. Something else is locking up. Did you discover what?\n> > > \n> > > I really would like phy-is-integrated to go away.\n> > > \n> > \n> > I have found the problem: by enabling ephy clk/reset the timeout does not occur anymore.\n> > So we could remove phy-is-integrated by:\n> > Moving internal phy clk/reset handling in mdio_mux_syscon_switch_fn()\n> > But this means:\n> > - getting internalphy node always by manually get internal_mdio/internal_phy (and not by the given phyhandle)\n> > - doing some unnecessary tasks (enable/scan/disable) when external_phy is needed\n> > \n> > Regards\n> \n> Hello all\n> \n> Below is the current patch, as you can read, it does not use anymore the phy-is-integrated property.\n> So now, the mdio-mux must always enable the internal mdio when switch_fn ask for it and so reset MAC and so need to enable ephy clk/reset.\n> But for this I need a reference to thoses clock and reset. (this is done in get_ephy_nodes)\n> The current version set those clock in mdio-mux node, and as you can see it is already ugly (lots of get next node),\n> if the clk/rst nodes were as it should be, in phy nodes, it will be more bad.\n> \n> So, since the MAC have a dependency on thoses clk/rst nodes for\n> doing reset(), I seek a proper way to get references on it.\n>\n> OR do you agree that putting ephy clk/rst in emac is acceptable ?\n\nWhy not just parsing the DT child nodes looking for resets and clocks\nproperties? The usual PHY don't have that.\n\nMaxime","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"ciDXcm4q\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y9tCM0nyhz9t5x\n\tfor ;\n\tTue, 10 Oct 2017 08:09:03 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e1fII-0005Qt-NO; Mon, 09 Oct 2017 21:08:58 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e1fIE-0005MR-8v for linux-arm-kernel@lists.infradead.org;\n\tMon, 09 Oct 2017 21:08:56 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid C9A2A207D4; Mon, 9 Oct 2017 23:08:30 +0200 (CEST)","from localhost (LFbn-TOU-1-209-191.w86-201.abo.wanadoo.fr\n\t[86.201.56.191])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id 9EE2E2076D;\n\tMon, 9 Oct 2017 23:08:30 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type:Cc:\n\tList-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:\n\tIn-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To:\n\tContent-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner;\n\tbh=3t2i7y8xyAn/1Tg7H0DptKlKU0s9STfvJovFpJ3kXu4=;\n\tb=ciDXcm4qvdhQm17VjHJ/kSdkL\n\txHMlMASpZwQGq9OPUP8EWGfee46C5TjUNW7OIjo7/86+i+yS+7s0zHUpORHIv5fGqyaR6gsuFKPQ+\n\tjAMPyNU/CXDslGmVuZlDRE1xJF3eNVcvxK3MAPUb2KsuaS9ZAPNXOeIRywmrtiN0V+FDxa27lWs0v\n\t6d51uuN1IEtMHfAiHmIxz0Qfo7Jzq8C97GjAfaz+IAQOoqRYy2Az2TfznBG8D/dIQyiy0OZqXDdxL\n\tYg13yqmA5bo7EpTWBsCXuCq7rHE6oejDYtvI1Mr3r25Q0qM3nFxCtXqE3BcRKv3va1C4fAbxZSmG0\n\t214jSCh3w==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT,\n\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","Date":"Mon, 9 Oct 2017 23:08:31 +0200","From":"Maxime Ripard ","To":"Corentin Labbe ","Subject":"Re: [PATCH v6 05/11] dt-bindings: net: dwmac-sun8i: update\n\tdocumentation about integrated PHY","Message-ID":"<20171009210831.qc2cgxr4cx2qz5x7@flea.home>","References":"<20170927073414.17361-1-clabbe.montjoie@gmail.com>\n\t<20170927073414.17361-6-clabbe.montjoie@gmail.com>\n\t<20170927140210.GE13516@lunn.ch> <20170928073708.GB32676@Red>\n\t<20171008183340.GA21531@Red>","MIME-Version":"1.0","In-Reply-To":"<20171008183340.GA21531@Red>","User-Agent":"NeoMutt/20170914 (1.9.0)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171009_140854_616857_8F739606 ","X-CRM114-Status":"GOOD ( 23.74 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, Andrew Lunn , f.fainelli@gmail.com,\n\talexandre.torgue@st.com, devicetree@vger.kernel.org,\n\tcatalin.marinas@arm.com, \n\tlinux-sunxi@googlegroups.com, will.deacon@arm.com,\n\tlinux@armlinux.org.uk, \n\tlinux-kernel@vger.kernel.org, wens@csie.org, robh+dt@kernel.org,\n\tnetdev@vger.kernel.org, peppe.cavallaro@st.com, frowand.list@gmail.com,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"multipart/mixed;\n\tboundary=\"===============7080117887554230610==\"","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]