From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE,SPF_PASS, USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2273AC433F5 for ; Sat, 18 Sep 2021 12:34:23 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 7A5CD6127A for ; Sat, 18 Sep 2021 12:34:22 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 7A5CD6127A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=nxp.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id BF89482D85; Sat, 18 Sep 2021 14:34:19 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=nxp.com header.i=@nxp.com header.b="BKnGndjW"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 7F4F283238; Sat, 18 Sep 2021 14:34:16 +0200 (CEST) Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05on20612.outbound.protection.outlook.com [IPv6:2a01:111:f400:7d00::612]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 1560682999 for ; Sat, 18 Sep 2021 14:34:11 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=vladimir.oltean@nxp.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=aNJRQSrz1n3mVhXf9eyWJ0JMyyHkk5OGG0B4pBhhVPxpo5MIfBG542gsBgz5jxk1HuT5fEALTqnrqSf29lVzPI1M06+htEJbTHuHAUh2HFRgoT9qs5QQKwLcEvC6HONG3eF+41BSKfAmz57u8ouV56/tvfzpy1zu61wlP3tG6Gxv37B1W8RLj1nQsAAotJZXUgggN0mRYmFxEoDPpZF2+Bm53oiVUxvHxBKhYWfwsll6+lnxKAjFrogY6kGpdmh/hB6Tv/YgyaMOkXZVqOIFZ9k5h6Nnp1Uvhl8Jl2UFUfuX9tFOYvydcdw0lkIVLq/U7OfutThOgidF/VsEyWpQ0g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=MbZguSkpI//h4z+GJJP5aW1HOqjCu/N2P33Z6vX8rLA=; b=DtaJ10Rh++TPHNflBUGOk6tjxHD8P7nU0AP/l1nkCrlKCsN1yXlTmwi2ZwjmfxDUqCEFCN7VJEMWP7vVl8I0JIY7+SGsOlwywpRam4ojYjDn1+HGPxDK3fJsUSh29hW1UDxVaE3ZqK/qBcwBoh4CzK0yCRlrtHYCi2tZsqk9SwNtMpmr8RLzoG4t7Yd294clcDMZKVpbPxhuaXtIclJ7Bs0HwhvlLeX1ZGicJwntPQjkwH9qFIt+2v1+RPqla7JYKlyE54PNkFO8R1UAOPyRuGa48AoyCX6iXSCcSVDmC119htv2mbXF+AqdFCLK9zVIR/1uH/sREIxQ1w9s07qPNg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=MbZguSkpI//h4z+GJJP5aW1HOqjCu/N2P33Z6vX8rLA=; b=BKnGndjWpYHbj0uw74L+0+05dIXnqKW9XTGsIdtYhmlBBqkcCINPxpsG9hsXHrk9SFFtXPlzp78pSmB3eFe5YiFzfPCIVC2elgaEst9icNiA9b0/9lhOXJo9qwRfzdixem4WpSnszjXsP3NTIG40ZRPQzeaVbvPmeRA/c1qhmCo= Authentication-Results: lists.denx.de; dkim=none (message not signed) header.d=none;lists.denx.de; dmarc=none action=none header.from=nxp.com; Received: from VI1PR04MB5136.eurprd04.prod.outlook.com (2603:10a6:803:55::19) by VI1PR04MB4687.eurprd04.prod.outlook.com (2603:10a6:803:72::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4523.18; Sat, 18 Sep 2021 12:34:09 +0000 Received: from VI1PR04MB5136.eurprd04.prod.outlook.com ([fe80::109:1995:3e6b:5bd0]) by VI1PR04MB5136.eurprd04.prod.outlook.com ([fe80::109:1995:3e6b:5bd0%2]) with mapi id 15.20.4500.021; Sat, 18 Sep 2021 12:34:09 +0000 From: Vladimir Oltean To: u-boot@lists.denx.de Cc: Joe Hershberger , Ramon Fried , Priyanka Jain , Madalin Bucur , Ioana Ciornei , Camelia Groza , Scott Wood , =?UTF-8?q?Pali=20Roh=C3=A1r?= , Marcin Wojtas , Stefan Chulski Subject: [PATCH 0/4] phy-mode cleanup Date: Sat, 18 Sep 2021 15:32:33 +0300 Message-Id: <20210918123237.1081530-1-vladimir.oltean@nxp.com> X-Mailer: git-send-email 2.25.1 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: VI1PR08CA0218.eurprd08.prod.outlook.com (2603:10a6:802:15::27) To VI1PR04MB5136.eurprd04.prod.outlook.com (2603:10a6:803:55::19) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from localhost.localdomain (82.78.148.104) by VI1PR08CA0218.eurprd08.prod.outlook.com (2603:10a6:802:15::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4523.16 via Frontend Transport; Sat, 18 Sep 2021 12:34:08 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0aae0e21-7462-40dc-bec1-08d97aa09c82 X-MS-TrafficTypeDiagnostic: VI1PR04MB4687: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:4941; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: TgwAuC90v87AIyd/8/H3ZUL9kNntMdeJCbG9zR8H3XLwy/5WdvzEEECO94bfyGsYmt4pEmwkkTvHuae7q/TmzPE87ixcy3j76qHY//eJXJPxrcSoc/6t3gNEYPTWVrTDEFs9OgtmwG6emjVDVR/eqIvSdRPLwDTCGfpePjxjydvaGovp48V06jVAkQrt0MpnjsLBudLzKAbsT44urCuuKkoomlP5G8GmeU3bT2KioVp7aYyejoRWg2dFKfL4FyDiTulJ54L/4VkMcF+ibrFEebP8C21dN1pinWSoEky3iIm4/LxTuIi+roQJupQRC0oiLhw1iURtW9Mk99S0Oyaib7pXrNJXeoW9xWhi1tMFZWChDqJCwfDkg2JekV0dAEQg/Uvq8oAeb/EX2HDFQNTwUTT96K37+dYkPnWC/gU/kaletWr3+EjL1dwpB0RUhYtH8fNXU2mcuxBG7cWb/JMkKdwAZJ2eALt3O/zabviViM6oYUBs5CXae6Bj3DV3vosTxAQRol3nOYrDgmvuh+qhT6/NAGtVXBrepQeAtMHqNylFSBBqYvIvo0PmKxtBojBPJuhRmm5sGtuXfNr376+EFOdpH2rmfeHFRtNk4eqPcuFPWGVLe2ouFi91dNt94avECJJxwsv3Er6oKFQHAzXEM0aURv24h4IDi/yydaWKoTLZLDSNw77tiiiPUddCyuZO8lQRoLQA5WKZCY2r4w1H1mNxQ+rFMIqv2SjyrPEcyukk5zXKgrZ4vI5XHs1ac3nZaKpIbMv7dxB/QU2UGI+RVYOTbGBqZvV/EHql/2l1LnA= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR04MB5136.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(136003)(346002)(376002)(396003)(39860400002)(36756003)(6512007)(26005)(38350700002)(38100700002)(6486002)(2906002)(6506007)(6666004)(83380400001)(66946007)(66476007)(66556008)(4326008)(52116002)(86362001)(44832011)(6916009)(54906003)(5660300002)(966005)(1076003)(8676002)(186003)(8936002)(956004)(316002)(2616005)(478600001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?t25jq2+amlROM/BwVGuSPyNHTY4Tu8DeMp3VT3ryzwUqCQLLc3bjQ/M6o342?= =?us-ascii?Q?IbtB61p7lUKcREpME8zu2BXMsoUKbQvt575Gjl6zX85hZkOmJO1vHjYS2NSD?= =?us-ascii?Q?d+H7GOuOOX5rVhJnGBBwS75AHnU2+UufWdUQJEt1q4y1TnsSTTBI2Uh/T29L?= =?us-ascii?Q?+O/W+Bnd70QXkL2wDr3OSCwTLJPZ3GxPibCC/BoUOgHuAJUfSNrqa+pJL4+B?= =?us-ascii?Q?laD1KmzW1GdP1V5r/mdGTnwq/lNJNAC0814s/c8xVaIeIUFhIOuNCNH9QF2k?= =?us-ascii?Q?vBpeUbr2N0f2ciGfjdu21rx6ar8rJypuRw05FM4dSGH0k6M0Fiavb7DHl27z?= =?us-ascii?Q?O7YpU5w6kJbec/nBMXKZXNEmVkKKWDgIdd/FA05GD2JnAfx1L2yBQ/tF7kwG?= =?us-ascii?Q?qUWUa+QSMj2h1wRkLAKOuBMnQxBc7nVN6UVI1vBckLuV2N5qiotXfNpDVj/S?= =?us-ascii?Q?+vVEj4cFb6r2ciUx4Yz1/+ss68GWUUYobbOKaI/f/DKjeL9e5T07WmkJsWd9?= =?us-ascii?Q?Iqbo1ngW0+vS0uLySCqcpB6QgDIqJptKdyu/pX/N1SNNkBH/TGwSptckYxvG?= =?us-ascii?Q?IzjdAH38eth4rMySN64YRjyvl6IQbpQh+HOX0WJNsIzR6q1p0+esWot79R5n?= =?us-ascii?Q?o81FzhJYlFWCQRZyOuPFS1TiXttCtyOqwmuflA4kGB9a9hTic1F/ULQZRxjU?= =?us-ascii?Q?4E5GigQz1ADH18At8dUtjzblBeBMATsRFrhoP3/oBGDInk0hsx5FIP55Q6Xh?= =?us-ascii?Q?K4M3NX1o0AwxckJlxiJcFwbpTrhp43zj2H6JziL53YnNVLaKJMHnrCBbuMWu?= =?us-ascii?Q?2pTgq4wVecuc81uG0F7kxPwZRV34Se6U/qZsIvFGa+OI0sCDmDzosUSmz/db?= =?us-ascii?Q?NF9RK37a3pMrUr2EPpRzR/nCHmulgo1Zmuz2tz5GberZunH2xL/fI/qkHHJi?= =?us-ascii?Q?1P9Ktbw+yEAmWkJu/760W5xDt8hb6p+8H16FXFCl5DbDrEdx+y7s6Mkm4+ny?= =?us-ascii?Q?WMvI8qEHNZ1mQQHm0PKGrQDSe303ibMPmecEJdtEklhNs24mGdd2inxmgQ0i?= =?us-ascii?Q?PT9xVHGKU60yQNu7iqqQdjwEXh+oTWC9HtoU8vjNdr2GqsmAHtFJu2JXkVxX?= =?us-ascii?Q?J8GPifRhtxbzS0xAlizSCOMJAp7+A4bE+Oh09/SwQM/itQmxBn9PCqJvjGAl?= =?us-ascii?Q?1/hirn9RbgLnnoDLHSmEpXqXy0xiwwZDq03Vkg/AYuVQHwhFhpzCrS9zpQsz?= =?us-ascii?Q?N2fGLVsJrciIPuS1qhqp3+k6sRDgB631f0qNSMdDvpZQji5Pk/uq/MYVmXBw?= =?us-ascii?Q?VRCyw1gX/bOBmZjdYBVHpiB2?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0aae0e21-7462-40dc-bec1-08d97aa09c82 X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Sep 2021 12:34:09.0095 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 6d8ioMJ+nYRfpI2JkUbLDXgqqG7OX9JL4FGO/U0AF7xGFxJUW8I5twbvEi8sFSjd0fMAA7+rediZr6SwRjEFPQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB4687 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean Michael Walle did an excellent job in starting the process of bringing U-Boot and Linux device tree bindings in sync for the NXP LS1028A: https://patchwork.ozlabs.org/project/uboot/cover/20210902164558.1920849-1-michael@walle.cc/ but there still are some hairy issues which need to be addressed head on, and which Michael said he'd rather leave for NXP to do, so here we are /shrug/ NXP LS1028A has 2 Ethernet controllers: drivers/net/fsl_enetc.c for the endpoint ports and DSA master drivers/net/mscc_eswitch/felix_switch.c for the switch ports but in fact, the MAC-side Physical Coding Sublayer (PCS) device used for translating the MAC layer data into a serial bit stream is the same kind between the enetc and the switch, as well as the same kind as the PCS used in the other NXP SoCs (those based on DPAA1 and those based on DPAA2). So most of the conclusions we draw about the LS1028A PCS apply equally to the other NXP SoCs from the family. There are 3 general directions in this patch series: - the "sgmii-2500" idea was rejected in Linux due to the inability to explain what it is. Linux has standardized on "2500base-x" as a way to describe both fiber ports and MAC-to-PHY links. Convert the NXP devices to use the phy-mode accepted in Linux. - the "xfi" phy-mode was rejected in Linux because it is an electrical standard covering more than a single SERDES protocol. For Ethernet things were standardized as "10gbase-r" which describes the 64b/66b coding scheme used there (the "R" in 10GBase-R). - the "xgmii" phy-mode is pretty much bogus, there are no NXP devices with this pinout. This pretty much stems from the idea that the device tree should describe an internal set of wires between the MAC and the PCS (the much speculated semantic difference between "phy-mode" and "phy-connection-type"), which it should not. Even though I'm pretty sure that no NXP devices have an external pinout of XGMII, I have only replaced this usage in the LS1028A networking drivers, because I am not completely sure of all the implications my changes would have in the DPAA1 and DPAA2 drivers. Vladimir Oltean (4): net: replace the "xfi" phy-mode with "10gbase-r" net: freescale: replace usage of phy-mode = "sgmii-2500" with "2500base-x" net: enetc: remove support for "xgmii" phy-mode net: dsa: felix: remove "xgmii" phy-mode arch/arm/cpu/armv8/fsl-layerscape/cpu.c | 2 +- .../cpu/armv8/fsl-layerscape/doc/README.soc | 8 +++---- .../cpu/armv8/fsl-layerscape/ls1088a_serdes.c | 2 +- .../dts/fsl-ls1028a-qds-6xxx-sch-30842.dtsi | 2 +- .../dts/fsl-ls1028a-qds-7777-sch-30841.dtsi | 8 +++---- .../dts/fsl-ls1028a-qds-7xx7-sch-30841R.dtsi | 4 ++-- .../dts/fsl-ls1028a-qds-x7xx-sch-30842.dtsi | 2 +- .../dts/fsl-ls1028a-qds-xx7x-sch-30842.dtsi | 2 +- arch/arm/dts/fsl-ls1088a-qds-sd1-21.dtsi | 4 ++-- arch/arm/dts/fsl-ls1088a-qds-sd1-29.dtsi | 4 ++-- arch/arm/dts/fsl-ls2080a-qds-sd1-42.dtsi | 16 +++++++------- arch/arm/dts/fsl-ls2088a-rdb-qspi.dts | 16 +++++++------- arch/arm/dts/fsl-sch-30841.dtsi | 2 +- arch/arm/dts/fsl-sch-30842.dtsi | 2 +- board/Marvell/octeon_ebb7304/board.c | 6 ++--- board/freescale/ls1012aqds/eth.c | 4 ++-- board/freescale/ls1012aqds/ls1012aqds.c | 4 ++-- board/freescale/ls1012aqds/ls1012aqds_pfe.h | 2 +- board/freescale/ls1012ardb/eth.c | 4 ++-- board/freescale/ls1043aqds/README | 2 +- board/freescale/ls1043aqds/eth.c | 12 +++++----- board/freescale/ls1043ardb/README | 2 +- board/freescale/ls1043ardb/eth.c | 2 +- board/freescale/ls1046aqds/README | 2 +- board/freescale/ls1046aqds/eth.c | 8 +++---- board/freescale/ls1046ardb/README | 4 ++-- board/freescale/ls1046ardb/eth.c | 2 +- board/freescale/ls1088a/README | 4 ++-- board/freescale/ls1088a/eth_ls1088ardb.c | 6 ++--- board/freescale/ls2080aqds/README | 2 +- board/freescale/ls2080aqds/eth.c | 13 +++++------ board/freescale/ls2080ardb/README | 2 +- board/freescale/t102xrdb/README | 2 +- board/freescale/t102xrdb/eth_t102xrdb.c | 8 +++---- board/freescale/t208xqds/README | 18 +++++++-------- board/freescale/t208xqds/eth_t208xqds.c | 22 +++++++++---------- board/freescale/t208xqds/t208xqds.c | 8 +++---- board/freescale/t208xrdb/README | 4 ++-- board/freescale/t4rdb/eth.c | 2 +- doc/device-tree-bindings/net/ethernet.txt | 12 +++++++++- drivers/net/fm/b4860.c | 2 +- drivers/net/fm/eth.c | 10 ++++----- drivers/net/fm/ls1043.c | 4 ++-- drivers/net/fm/ls1046.c | 2 +- drivers/net/fm/memac.c | 6 ++--- drivers/net/fm/t1024.c | 2 +- drivers/net/fsl_enetc.c | 10 ++++----- drivers/net/mscc_eswitch/felix_switch.c | 7 +++--- drivers/net/pfe_eth/pfe_mdio.c | 4 ++-- drivers/net/phy/aquantia.c | 18 +++++++-------- include/phy.h | 2 +- include/phy_interface.h | 4 ++-- 52 files changed, 154 insertions(+), 148 deletions(-) -- 2.25.1