From patchwork Sat Aug 6 14:10:49 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vladimir Oltean X-Patchwork-Id: 12938051 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6E5EFC25B07 for ; Sat, 6 Aug 2022 14:11:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231660AbiHFOLX (ORCPT ); Sat, 6 Aug 2022 10:11:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35350 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230407AbiHFOLU (ORCPT ); Sat, 6 Aug 2022 10:11:20 -0400 Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70041.outbound.protection.outlook.com [40.107.7.41]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6B1D4101FF; Sat, 6 Aug 2022 07:11:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ij/l8hsLIoFlxrHPJkqJAIJIreKm0BaqWmfFH+EEcfD48L9OR2Aqv6oBBRbxosZmrLfYyhO9BiXg8MNpAYVEALM8FyLAmNIlNuqy7cgWw/uW46JT30d5yeoblu5fgAzk4a+4vi7no4lGvXPesS6o1EpuwNM/QF0bhb15O2X51u1IzGLQxQEEnqOR5DWCs1BdJcIY4lIC4Ye+/sPM2eGWvcEc+napbhDeIgywpwehsChgX+vdtr1nUII5Vo3GaxP/rofdubgFs7oenXJoQjHVy13s0UsPP0KS2zg8+ccB0UzYUmQnEoY2YZQQWWPg3FLBBV+aEW8dLktQ/jHW+P8zXA== 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:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=BeyiDr+ONMlxayP+m1Q2qXUokQqLGwFXv3V21JU2yNI=; b=fXTwHkvIie4YMA/AEILvmHMXTXvl3sCx1Zbnt1EiYyJ2axFaOe16M71u23Jwx6Ckpz1xieKlCsEqsvfKObpe3TVn5+lCyW/x4txaFrPHqI+efTuw9AFGxXN4wadyQb9hvjKc8qjrZsTPp28PXvOBpCo+SIZANIn5qTfPv2ssCRMYr+whtdipWPCEA2LuEywe+S/UD6SnS4vYpemsOR/+21yAQOSc6cjMp0tA735+iN9rt5MzcG+s4M+ekGOLKuu+2cR9DuH4rrcMCANvJFQtmHvCD2eUYmi4yplyEQrrf7UcaugjWyLNBiz59bEttN5RR2MGEQ8WFAg9Pu4QF/Pxhw== 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=BeyiDr+ONMlxayP+m1Q2qXUokQqLGwFXv3V21JU2yNI=; b=oHSn+02vfCvbd3zGvbKkAc20oJH6vZr5DkzmlTntNRnDeFcr2L8ZwDqdp5ccUQGA5cYblu2IZxbt2u1+lYDV/OtBn4wy6SCDw5kHV9Sya/dpebodAh+9HlUPZRMf71LtgbJsO9kYU6/iYe6oApDwTGdlV04dAmazMU2w9eMf1dg= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from VI1PR04MB5136.eurprd04.prod.outlook.com (2603:10a6:803:55::19) by DB8PR04MB6988.eurprd04.prod.outlook.com (2603:10a6:10:117::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5504.17; Sat, 6 Aug 2022 14:11:14 +0000 Received: from VI1PR04MB5136.eurprd04.prod.outlook.com ([fe80::71b7:8ed1:e4e0:3857]) by VI1PR04MB5136.eurprd04.prod.outlook.com ([fe80::71b7:8ed1:e4e0:3857%4]) with mapi id 15.20.5504.017; Sat, 6 Aug 2022 14:11:14 +0000 From: Vladimir Oltean To: netdev@vger.kernel.org Cc: devicetree@vger.kernel.org, "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Andrew Lunn , Vivien Didelot , Florian Fainelli , Vladimir Oltean , Rob Herring , Krzysztof Kozlowski , Oleksij Rempel , Christian Marangi , John Crispin , Kurt Kanzenbach , Mans Rullgard , Arun Ramadoss , Woojung Huh , UNGLinuxDriver@microchip.com, Claudiu Manoil , Alexandre Belloni , George McCollister , DENG Qingfang , Sean Wang , Landen Chao , Matthias Brugger , Hauke Mehrtens , Martin Blumenstingl , Aleksander Jan Bajkowski , =?utf-8?q?Alvin_=C5=A0ipraga?= , Luiz Angelo Daros de Luca , Linus Walleij , Pawel Dembicki , =?utf-8?b?Q2zDqW1lbnQgTMOpZ2Vy?= , Geert Uytterhoeven , Russell King , =?utf-8?q?Marek_Beh=C3=BAn?= , Marcin Wojtas , Marek Vasut , linux-renesas-soc@vger.kernel.org, Frank Rowand Subject: [RFC PATCH v3 net-next 00/10] Validate OF nodes for DSA shared ports Date: Sat, 6 Aug 2022 17:10:49 +0300 Message-Id: <20220806141059.2498226-1-vladimir.oltean@nxp.com> X-Mailer: git-send-email 2.34.1 X-ClientProxiedBy: VI1PR0802CA0016.eurprd08.prod.outlook.com (2603:10a6:800:aa::26) To VI1PR04MB5136.eurprd04.prod.outlook.com (2603:10a6:803:55::19) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 3cdba6d1-e7e2-4a25-903b-08da77b5858a X-MS-TrafficTypeDiagnostic: DB8PR04MB6988:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: tGk7bix4SdkgHp501jEttQoE5ML9+xnXGF9qEzznstmgzFRIsg2w/m/nWAkMqx30cf057NdRpeR2Xnfpi2Op0pnfJ4lNbJL9DKg7CpOxbrQKttSNlvkMRris73NX8ceUDDnEMhc/iQzR5gsA6kLpijnsU4U3CIuaDvaidCD0X/9sg/nzspcTB6y8mVx1qQg5mAhrwvy2oTiK48yodN4ZbAIUlpn3n2grpSlQbCkq+AYFqjI52UQ63LTQrqfjfvKEvLQLhjrIgLV69HvxEHCptezS6Wc3FimKjdIXDea/ryvXNgJhSal1gNvPFEA6MPwRPtD4feHGIp+fdKft5TB4VmXENrtG8wHbpAIdkRArkoUu6a9fQuB+zjkACYLaZy1b3QlGpPzwIbDXKJXkrARUE+gIcTxo3EaWxNU+wHBmeFazVV9gy2DbUeK7GmQnEBpZ2lwYsVyiYXipHVGR+97yZBBx22msSACFITcWiD670RBG/sSLSNxb7/epIpNTnlZD01P6g2c/TIvHbCiHkb6L3LaYEeh1MSKAUPKXh+HoZjhEa+XL/VDt/lnjyYOaJcBKh+ZinnK0DmVNUxYORNQpmdaQXWaI/s6xx475QzJPmcWOPIwHplyFqQNz6b6aFeF7FaKhwgmF3ToKC4LFL4SEc8PMYp6VnsmycSjrKMlHqadL+a1fXTjNCepoxN+Y9cspcu508OYJZJWrYr0H6lI8gVqJ8H1/f39ozE1OTRvWVgSZadwPUM3fLbZX6Gp0PVGRmEVU69PI2L2DcpZvMfibeAb47SCrvvU0XHUWxau4fBqCYBCieXU+W57dUW6JaMZ4qu5HUprPe7vx7GfpMVr1jVz6Xu7YlMSeO9oo3LHb1L4= 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:(13230016)(4636009)(376002)(396003)(346002)(39860400002)(136003)(366004)(5660300002)(44832011)(7416002)(7406005)(186003)(1076003)(2616005)(38100700002)(38350700002)(83380400001)(6666004)(52116002)(966005)(6486002)(41300700001)(316002)(36756003)(6916009)(54906003)(6506007)(26005)(6512007)(66946007)(66556008)(66476007)(4326008)(8676002)(8936002)(478600001)(86362001)(2906002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?JV73s/7HsrOzReI9DO3ajIZONwMG?= =?utf-8?q?iLqnN26444qdCtBxQwUlLPtlvbtT0rVS5WRVqlFuRzvjanMORnIxA5Tw5bwLWSPzu?= =?utf-8?q?biX9B54an9/pugm5VfzxHfu37bQO2EJgrCrvqI+J1nZKksyMqLo70aP609KwAxzql?= =?utf-8?q?8ImVJBmVr/sTI2Tpao6pql4kQLpX9ApvGzEXFL8JKYYAPBd0TE/hzn+zLAvNI0aX2?= =?utf-8?q?oKWj7yOXuKQjhpDjSqRRIJewauvHAUHBqsEAxWa9DP70nn9Vghx7NO6kIskJ9VA3J?= =?utf-8?q?RVbOsROavHlJ/DsxYxNaMewiyx2ATNZpNGXC8zWJV76bDvpkZrdedAX5JeWOiMb4/?= =?utf-8?q?SaLvN7+uvcUMtjY6xqkArv8RNpsslUo5GCsw01gjqVEpFfWk7u6+sghXgV2PrcvFw?= =?utf-8?q?lF3RYBWtwlHl20S2C8kPk0QeOkI9195x6hILQ4ULHZKUDzUAfnUP14DJa40nLLlqp?= =?utf-8?q?LHvEya/DuiCWyfeqIS4dIIDcEE2+X3KjK6gBhY+Mdh4+kFZK1kGm/ozrzMv60fv2K?= =?utf-8?q?N1yMA8k4dmvHA6+xtqpN+7x9+guvAF4xzGrueg6QHfYF6KvFPKP2/du8Wb8B79Xxq?= =?utf-8?q?Y2SMyux94d4xmpKb1/Ojs8aMzmoh1BNUJftSUkt4g1jiO4H75yqGzPNrKqFfmINU7?= =?utf-8?q?7dJ8M7ebq1tF4imMfMvoEY04d+Bvr9pWdcx8EvhETBVsz/n6qmjNiroVYMWy9z23L?= =?utf-8?q?hd+XFD+LgJ5+daBtp0ILg34v0+x+hDGH5Y6wGNZtMmJgk1wOn6rjjF6SGiAgLckhz?= =?utf-8?q?r5GoayKrPqNujV3/sIUzwz84YwSTl7Yj3PuFJfE0THI7tVBCmfo88/Rif/cPXnd5u?= =?utf-8?q?OCEjb4OkVrJen9mUJ7SB1PESv86mWkXpFuTFPrxFEkyXBdpk/zKSN3lH1kyCOiXOH?= =?utf-8?q?iL4PjltuQWDeyX2QXBtiHThaW1gL5H7htnLdjHzH8gR/7BZ1eYjGJMj1u2ayNoHiR?= =?utf-8?q?Dis6SyQVToHeL8cTNH2TKkpYJs8kPQQV5+qwmdPOQ18Vx2tmb9tOrIMiDEdJl5ri8?= =?utf-8?q?FAA1dPx0BvakR+xhmscD6QeUcgO7jepfy1Y10mfRMIQGWpIdlEJ+RwPPdDesacjwD?= =?utf-8?q?NHwbfcLkGRP7BIJ/cLv8xdQxXU4KlY4lPAuTrjw7pisT/ZzhEsAHBe46YKzX8DTsk?= =?utf-8?q?BEULnc42sXZ+UjC0ZxEi1b7zfP0eRDFc9vmd8/0oqCdkavq7vLrlwv60mCEWConBO?= =?utf-8?q?3TKpNhqsb/z3oVBUB6Qvo+L7kIja2VrZlk4xzmiIu6/geVHgOKIc7MkHvWFdsFVii?= =?utf-8?q?fbTrRD64U0Cn2i7AsJANK7Oae1pwhc4W76mlXHf7Vs6DLpnw685uBE0+TvxfXd5Zg?= =?utf-8?q?EcEQAj4aYoWGPZ7xN2nWCgz6NwLBygjENeOx+q+0tTgCyt+OHPbJt6zBrad6oi+uv?= =?utf-8?q?K8S+WTXhc5j6a1ImzTsYQJdp4YYMFTdhZkom0yFDYbq5v9MMZE8bLMDqBqVCKfv0Z?= =?utf-8?q?NQrTI4RFTkIlVW8OJQDtA/7EO769iYTkoHI62fPz0UfqROKvpuUHEX19/Ou8W8ZFz?= =?utf-8?q?OIHz06DihdiYxcaz1dGg/njQ9bjD3qzivg=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 3cdba6d1-e7e2-4a25-903b-08da77b5858a X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Aug 2022 14:11:14.4270 (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: rUqq9IkDZ/L38jbXcjWjg7Z1GWMESvuZI7TJwFIralfgH8kiAseniADDfCbreJj3sVY+yzv0fc+Y75XrxL9u8g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR04MB6988 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org X-Patchwork-State: RFC This is the first set of measures taken so that more drivers can be transitioned towards phylink on shared (CPU and DSA) ports some time in the future. It consists of: - expanding the DT schema for DSA and related drivers to clarify the new requirements. - introducing warnings for drivers that currently skip phylink due to incomplete DT descriptions. - introducing warning for drivers that currently skip phylink due to using platform data (search for struct dsa_chip_data). - closing the possibility for new(ish) drivers to skip phylink, by validating their DT descriptions. - making the code paths used by shared ports more evident. - preparing the code paths used by shared ports for further work to fake a link description where that is possible. More details in patch 10/10. DT binding (patches 1-6) and kernel (7-10) are in principle separable, but are submitted together since they're part of the same story. Patches 8 and 9 are DSA cleanups, and patch 7 is a dependency for patch 10. Submitting as RFC because it's RFC season, but I'd like to resend this for proper inclusion as soon as possible once the merge window closes, so ACKs/NACKs are welcome. Change log in patches. v1 at https://patchwork.kernel.org/project/netdevbpf/patch/20220723164635.1621911-1-vladimir.oltean@nxp.com/ v2 at https://patchwork.kernel.org/project/netdevbpf/patch/20220729132119.1191227-5-vladimir.oltean@nxp.com/ Cc: Rob Herring Cc: Frank Rowand Vladimir Oltean (10): dt-bindings: net: dsa: xrs700x: add missing CPU port phy-mode to example dt-bindings: net: dsa: hellcreek: add missing CPU port phy-mode/fixed-link to example dt-bindings: net: dsa: b53: add missing CPU port phy-mode to example dt-bindings: net: dsa: microchip: add missing CPU port phy-mode to example dt-bindings: net: dsa: rzn1-a5psw: add missing CPU port phy-mode to example dt-bindings: net: dsa: make phylink bindings required for CPU/DSA ports of: base: export of_device_compatible_match() for use in modules net: dsa: avoid dsa_port_link_{,un}register_of() calls with platform data net: dsa: rename dsa_port_link_{,un}register_of net: dsa: make phylink-related OF properties mandatory on DSA and CPU ports .../bindings/net/dsa/arrow,xrs700x.yaml | 2 + .../devicetree/bindings/net/dsa/brcm,b53.yaml | 2 + .../devicetree/bindings/net/dsa/dsa-port.yaml | 17 ++ .../net/dsa/hirschmann,hellcreek.yaml | 6 + .../bindings/net/dsa/microchip,ksz.yaml | 4 + .../bindings/net/dsa/renesas,rzn1-a5psw.yaml | 2 + drivers/of/base.c | 1 + net/dsa/dsa2.c | 36 +++- net/dsa/dsa_priv.h | 4 +- net/dsa/port.c | 193 ++++++++++++++++-- 10 files changed, 240 insertions(+), 27 deletions(-)