Cover Letter Detail
Show a cover letter.
GET /api/covers/811353/?format=api
{ "id": 811353, "url": "http://patchwork.ozlabs.org/api/covers/811353/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-imx/cover/20170908071156.5115-1-clabbe.montjoie@gmail.com/", "project": { "id": 19, "url": "http://patchwork.ozlabs.org/api/projects/19/?format=api", "name": "Linux IMX development", "link_name": "linux-imx", "list_id": "linux-imx-kernel.lists.patchwork.ozlabs.org", "list_email": "linux-imx-kernel@lists.patchwork.ozlabs.org", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20170908071156.5115-1-clabbe.montjoie@gmail.com>", "list_archive_url": null, "date": "2017-09-08T07:11:46", "name": "[v5,00/10] net: stmmac: dwmac-sun8i: Handle integrated PHY", "submitter": { "id": 64152, "url": "http://patchwork.ozlabs.org/api/people/64152/?format=api", "name": "Corentin Labbe", "email": "clabbe.montjoie@gmail.com" }, "mbox": "http://patchwork.ozlabs.org/project/linux-imx/cover/20170908071156.5115-1-clabbe.montjoie@gmail.com/mbox/", "series": [ { "id": 2115, "url": "http://patchwork.ozlabs.org/api/series/2115/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-imx/list/?series=2115", "date": "2017-09-08T07:11:46", "name": "net: stmmac: dwmac-sun8i: Handle integrated PHY", "version": 5, "mbox": "http://patchwork.ozlabs.org/series/2115/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/811353/comments/", "headers": { "Return-Path": "<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>", "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=<UNKNOWN>)", "ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"tU/gVE7t\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"dcE3SbdK\"; 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 3xpT9Z2qGxz9s83\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tFri, 8 Sep 2017 17:15:14 +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 1dqDVL-0003YF-HX; Fri, 08 Sep 2017 07:15:08 +0000", "from mail-wm0-x233.google.com ([2a00:1450:400c:c09::233])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dqDUj-000332-SA for linux-arm-kernel@lists.infradead.org;\n\tFri, 08 Sep 2017 07:14:31 +0000", "by mail-wm0-x233.google.com with SMTP id f199so9779613wme.0\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tFri, 08 Sep 2017 00:14:09 -0700 (PDT)", "from Red.local (LFbn-MAR-1-580-96.w90-118.abo.wanadoo.fr.\n\t[90.118.159.96]) by smtp.googlemail.com with ESMTPSA id\n\tl19sm684566wrl.47.2017.09.08.00.14.05\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128);\n\tFri, 08 Sep 2017 00:14:06 -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:MIME-Version:Cc:List-Subscribe:\n\tList-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date:\n\tSubject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:\n\tReferences:List-Owner; bh=3w8xxvIlcTZ3nrzwnt73v69+KrXgGKz0zOF/tV7R5CQ=;\n\tb=tU/\n\tgVE7tt4PTFNPz2XqUpvBcXWIMnbIo3vVB7N893jv91QXQ62MCgcRRQwellZqPnmJPhiU91+SVPvlR\n\tMV1IoIZhUwVrPCQnRFZwOTy1gA/heXGxfmD/QIkRSiRrQTgK9EGu0Le7bdqyoYT9Bdgyl0yM2DVdk\n\tnvtYRmm4eilSBrAoGOoTr+Hq4pFy/v6cftx2MZQ4Q/QoCgcvoHmjuOlylw3SFz/dpgoTnmo57CEHq\n\tTrr9jKjBZUboi+RUe37Q4nKJ6T5dclYzII70jGp1syc8SAv3DUR9EAEK43KGZKGi9umYcZQ//Nu91\n\tvg9Kod/NKHfiON1G69TGprjKYzQhfjQ==;", "v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=from:to:cc:subject:date:message-id;\n\tbh=w8RcbEOMKY/vjLYzKK6iAjKcvFVR8cve2hrcJlAqCbA=;\n\tb=dcE3SbdKCzW/XL7krCxC1tTbS1aq+DosWb8Ho2OO5UFbS8Nvq9Kt8xSwS7nrrtv/xK\n\tNrZOoshoV1Ioc4jv2NxjIZRIiq3Fs3GOiJCDqETXlKXCWbGIhn/me5otOsxzmfcHTe9C\n\tY7anqc22lVPgvDSSL71qOaSp7ExylkylifEYXV5dFkrJU9g2VA6WBmXpXAQS3FkNrqsD\n\t8gJojBDlPxyWXyu/bMi3Tn0R0LLC8CkqKTjvyF6bhLhtz9WbObIywix78y2ogh2t/tIE\n\t9Yuzttu9rvOF35+tK/VQyixxIMS4GpgMaeOB5yNo5nMOCqW6cpURzffXNCDh6vy6fOlq\n\tgKyA==" ], "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id;\n\tbh=w8RcbEOMKY/vjLYzKK6iAjKcvFVR8cve2hrcJlAqCbA=;\n\tb=nZCgpJml/BJEIbrjxM30rs5DHkGLNFmLXOnxKsIIdfkmSRT/6GKwkmPQEi0pVTTB0x\n\tdiwQQWoGfCbUU7+Gl3ktDf0BsVNEfu9ZGAWC7rByvppWp7pLd7BQ6R8q2MfN388B/t2B\n\tqEXza4KUKza2ANGLlEIFEfoZOxWjm2DEqZuUBQTJ+mfgjZSv1rYt2MW81F6Cn/Wgu0CC\n\tw/KOkFHMYKGvYJXsysdzrHqe1afE/jeDdrCc7iekVX8mFbK8rKag5S5ndaSRY/JODzkP\n\tnrMbGu4XnoKirpWjQd3FiR7Wmdt6zPwswUExUohtnBYZCPt6doxY5F6z7moT6nERurul\n\t7Nbw==", "X-Gm-Message-State": "AHPjjUjdfPAlZzXs22OqfOs9XNYKugYC5pDJE3lLXYII7T+W+PqoUzbd\n\tNhZN/4kbBxzTpzdyM7urwcc=", "X-Google-Smtp-Source": "AOwi7QDpUF3KpPm0N5WH0y5vLBDL+hcs7B++N4f/++2/PAQjBVmIbdv8FKT95LdBWdKHf+aU6UIp9A==", "X-Received": "by 10.28.111.196 with SMTP id c65mr853315wmi.18.1504854847595;\n\tFri, 08 Sep 2017 00:14:07 -0700 (PDT)", "From": "Corentin Labbe <clabbe.montjoie@gmail.com>", "To": "robh+dt@kernel.org, mark.rutland@arm.com,\n\tmaxime.ripard@free-electrons.com, \n\twens@csie.org, linux@armlinux.org.uk, catalin.marinas@arm.com,\n\twill.deacon@arm.com, peppe.cavallaro@st.com, alexandre.torgue@st.com, \n\tandrew@lunn.ch, f.fainelli@gmail.com", "Subject": "[PATCH v5 00/10] net: stmmac: dwmac-sun8i: Handle integrated PHY", "Date": "Fri, 8 Sep 2017 09:11:46 +0200", "Message-Id": "<20170908071156.5115-1-clabbe.montjoie@gmail.com>", "X-Mailer": "git-send-email 2.13.5", "X-CRM114-Version": "20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ", "X-CRM114-CacheID": "sfid-20170908_001430_081909_8AA62AB4 ", "X-CRM114-Status": "GOOD ( 19.16 )", "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:233 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": "<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>", "List-Archive": "<http://lists.infradead.org/pipermail/linux-arm-kernel/>", "List-Post": "<mailto:linux-arm-kernel@lists.infradead.org>", "List-Help": "<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>", "List-Subscribe": "<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>", "Cc": "netdev@vger.kernel.org, Corentin Labbe <clabbe.montjoie@gmail.com>,\n\tlinux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,\n\tdevicetree@vger.kernel.org", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Sender": "\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>", "Errors-To": "linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org", "List-Id": "linux-imx-kernel.lists.patchwork.ozlabs.org" }, "content": "Hello\n\nThe current way to find if the PHY is internal is to compare DT phy-mode\nand emac_variant/internal_phy.\nBut it will negate a possible future SoC where an external PHY use the\nsame phy mode than the integrated one.\n\nThis patchs series adds a new way to find if the PHY is integrated, via\nthe phy-is-integrated DT property.\n\nSince it exists both integrated and external ethernet-phy@1, they are merged in\nthe final DTB and so share all properties.\nFor avoiding this, and better represent the reality, we use a MDIO mux.\n\nThe first try was to create a new MDIO mux \"mdio-mux-syscon\".\nmdio-mux-syscon working the same way than mdio-mux-mmioreg with the exception\nthat the register is used via syscon/regmap.\nBut this solution does not work for two reason:\n- changing the MDIO selection need the reset of MAC which cannot be done by the\n mdio-mux-syscon driver\n- There were driver loading order problem:\n - mdio-mux-syscon needing that stmmac register the parent MDIO\n - stmmac needing that child MDIO was registered just after registering parent MDIO\n\nSo we cannot use any external MDIO-mux.\n\nThe final solution was to represent a mdio-mux and let the MAC handle all things.\nNote that phy-is-integrated is still needed (even if we use a MDIO mux) since\nsome properties apply only on integrated PHY and we need to know the final MDIO\nbus in mdio_mux_syscon_switch_fn().\n\nSince DT bits was reverted in 4.13, this patch series include the revert of the revert.\nSo\n- the first four patchs bring back DT/stmmac stuff that was in 4.13 (and reverted)\n- fifth patch document how DT MDIO mux is implemented\n- patch 6 and 7 modify DT\n- patch 8, 9, 10 Modify stmmac according to the new bindings\n\nI have let patch splited for easy review. (for seeing what's new)\nBut the final serie could have some patch squashed if someone want.\nLike squashing patch and 2 and 5 (documentation)\n\nSince DT worked well in 4.13, could it be targeted for 4.14 ?\nIf necessary I could split this serie in two:\n- bring back A64/A83T (patchs 1, 2, 4, 7, 9)\n- add MXIO-mux and H3 (patchs 3, 4, 5, 6, 8, 10)\n\nRegards\n\nChanges since v4:\n- Update documentation for new bindings\n- Added 4 patchs for bring back reverted stuff of 4.13\n- dwmac-sun8i now handle mdio-mux\n- MDIO use now compatible = \"snps,dwmac-mdio\";\n\nChanges since v3:\n- Added a patch for handling fixed-link\n- Updated documentation\n\nChanges since v2:\n- Add a MDIO mux for creating distinction between integrated and external MDIO.\n- phy-is-integrated is not set in dtsi.\n\nChanges since v1:\n- Dropped phy-is-integrated documentation patch since another same patch was already merged\n- Moved phy-is-integrated from SoC dtsi to final board DT.\n\n\nCorentin Labbe (10):\n arm64: dts: allwinner: Restore EMAC changes\n dt-bindings: net: Restore sun8i dwmac binding\n arm: dts: sunxi: Restore EMAC changes\n net: stmmac: sun8i: Restore the compatibles\n dt-bindings: net: dwmac-sun8i: update documentation about integrated\n PHY\n ARM: dts: sunxi: h3/h5: represent the mdio switch used by\n sun8i-h3-emac\n arm64: dts: allwinner: add snps,dwmac-mdio compatible to emac/mdio\n net: stmmac: dwmac-sun8i: choose internal PHY via phy-is-integrated\n net: stmmac: snps,dwmac-mdio MDIOs are automatically registered\n net: stmmac: dwmac-sun8i: Handle integrated/external MDIOs\n\n .../devicetree/bindings/net/dwmac-sun8i.txt | 197 +++++++++++++++++++++\n arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 9 +\n arch/arm/boot/dts/sun8i-h3-bananapi-m2-plus.dts | 19 ++\n arch/arm/boot/dts/sun8i-h3-nanopi-neo.dts | 7 +\n arch/arm/boot/dts/sun8i-h3-orangepi-2.dts | 8 +\n arch/arm/boot/dts/sun8i-h3-orangepi-one.dts | 8 +\n arch/arm/boot/dts/sun8i-h3-orangepi-pc-plus.dts | 5 +\n arch/arm/boot/dts/sun8i-h3-orangepi-pc.dts | 8 +\n arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts | 22 +++\n arch/arm/boot/dts/sun8i-h3-orangepi-plus2e.dts | 16 ++\n arch/arm/boot/dts/sunxi-h3-h5.dtsi | 46 +++++\n .../boot/dts/allwinner/sun50i-a64-bananapi-m64.dts | 16 ++\n .../boot/dts/allwinner/sun50i-a64-pine64-plus.dts | 15 ++\n .../arm64/boot/dts/allwinner/sun50i-a64-pine64.dts | 17 ++\n .../dts/allwinner/sun50i-a64-sopine-baseboard.dts | 16 ++\n arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 21 +++\n .../boot/dts/allwinner/sun50i-h5-nanopi-neo2.dts | 17 ++\n .../boot/dts/allwinner/sun50i-h5-orangepi-pc2.dts | 17 ++\n .../dts/allwinner/sun50i-h5-orangepi-prime.dts | 17 ++\n drivers/net/ethernet/stmicro/stmmac/Kconfig | 1 +\n drivers/net/ethernet/stmicro/stmmac/dwmac-sun8i.c | 140 ++++++++++++---\n .../net/ethernet/stmicro/stmmac/stmmac_platform.c | 4 -\n 22 files changed, 601 insertions(+), 25 deletions(-)\n create mode 100644 Documentation/devicetree/bindings/net/dwmac-sun8i.txt" }