From patchwork Tue Aug 24 12:00:37 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vladimir Oltean X-Patchwork-Id: 1520250 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=nxp.com header.i=@nxp.com header.a=rsa-sha256 header.s=selector2 header.b=X7MEzlJM; dkim-atps=neutral Received: from phobos.denx.de (phobos.denx.de [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 4Gv74f0m5Gz9sSs for ; Tue, 24 Aug 2021 22:01:38 +1000 (AEST) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 1EC6281E3D; Tue, 24 Aug 2021 14:01:29 +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="X7MEzlJM"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id E512682DB6; Tue, 24 Aug 2021 14:01:06 +0200 (CEST) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO, MSGID_FROM_MTA_HEADER,SPF_HELO_PASS,T_SPF_PERMERROR autolearn=no autolearn_force=no version=3.4.2 Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-am5eur02on0615.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe07::615]) (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 2DC14805DF for ; Tue, 24 Aug 2021 14:00:59 +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=JrY2QGkgUaIUvgJWdU5z1lrona6bhF/5t6fAkbXJ1TeS+Cu3dqeEkKNAr7xXhZ6jwuSiMUpAA/FQTyQjH0e6ciRtlyoBU5d0UCSiiTB0K3krLEDV6dqsQn4VnYl9/U4E+oxQejx9bTh7co0AFJxLvpKFpSF1eHtxB3skKtsLZe9BaSsr1DppZMzKNVULBDsclq07tRvQ/eEjDBkd7f7ZiBC255J0dl5jrY1LEGHrFqfANeCkJym8CNUz8JD5QzvW3TaNQNWROaUIbfHF66/bwnvDmKns9j2wTiJbK7BomSkh7loD1/1l80jDi+QNLYx9WKLsFFYH8V0S+Jzkctrx9A== 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-SenderADCheck; bh=xkAWJF+27SytDcDk7XrBMqTiGevso5AGtHs+UWJZZUY=; b=e9xjEVX80/08TQttqp54WBwyeZ6JgphiN542UsrzuWckyGDlU20mrWcKkN2foet5zFvFuO5jJdUGj610cTMRwV3WsNBkK1nj5ynFhBdC87usA1CCDH69O3sclguP673qS/fQAQLqtajczTHtDQ0R6GmiylWb2JAUZF1m+3E6HGi7V8HhhuPiqVykegZr/PX1r2UgS+Rj2fIXhQpIaHN3nER6wHWlTbqyfo2EWPlzQDF3fRZzWw3TOzbR1KdHCwlLGebB3O/1Gt7oQwWx5RCVGBdpejKjDYlyIEj6rTc17M72oygaEFsO7y5obJ5X/hGs6xpcwI/ySayuFZrONb5Stw== 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=xkAWJF+27SytDcDk7XrBMqTiGevso5AGtHs+UWJZZUY=; b=X7MEzlJM48d0NovQJTG82dn4zHmVgBqom2T/2+Kx9nkfeAHpMmYMzT4j7L8TNa5UAAh8+UTkHnqlTWt+A1Utxs4D4he/6DoQWOcsHOqdOx5R2lrVqUTp1iQHbag6tbBCSVG0akjTRgYA/MeGMYYkW/a93T+/+gLBcA+4bNvAB30= Authentication-Results: ni.com; dkim=none (message not signed) header.d=none;ni.com; dmarc=none action=none header.from=nxp.com; Received: from VI1PR04MB5136.eurprd04.prod.outlook.com (2603:10a6:803:55::19) by VE1PR04MB7374.eurprd04.prod.outlook.com (2603:10a6:800:1ac::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4436.22; Tue, 24 Aug 2021 12:00:57 +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.4436.025; Tue, 24 Aug 2021 12:00:57 +0000 From: Vladimir Oltean To: Joe Hershberger , Ramon Fried , u-boot@lists.denx.de Cc: Claudiu Manoil , Michael Walle , Priyanka Jain Subject: [PATCH v2 0/6] Call phy_config at port probe time for the Felix DSA driver Date: Tue, 24 Aug 2021 15:00:37 +0300 Message-Id: <20210824120043.3823931-1-vladimir.oltean@nxp.com> X-Mailer: git-send-email 2.25.1 X-ClientProxiedBy: PR3P192CA0010.EURP192.PROD.OUTLOOK.COM (2603:10a6:102:56::15) To VI1PR04MB5136.eurprd04.prod.outlook.com (2603:10a6:803:55::19) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from localhost.localdomain (188.25.144.60) by PR3P192CA0010.EURP192.PROD.OUTLOOK.COM (2603:10a6:102:56::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4457.17 via Frontend Transport; Tue, 24 Aug 2021 12:00:57 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: a422a9df-5a17-44e1-7a7f-08d966f6d54a X-MS-TrafficTypeDiagnostic: VE1PR04MB7374: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:9508; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: pA1YW7SBQL12C78xWl+wQRWqaFseMRibZxUENeD92+Q0ETy36ZJuipigu+FdbgqZXaqt9mt8LhGLJCLYS+V1eZHGojPb6C6d811ul+a3vY8+TPGJob4bR9CKyj0BARsY9pLhAhgAD8Gf08BQbeQ5hTbGjG1knT3E5hhNYJhrTmHf3u7J9+kuUSNyz4w+2JZ+2NeyGmQqT/LWbUjT4SMEW366/dOXiRCKJ2AZUo1NhYzoJvTqS4Vjq9uy66qA25LkzCHt40RtkH3P/R1ktfkkdGvqZx3CWoN6EKgoQVrDeHcWr4eMTRSZeegQfNxCT5mGFOoT1mluEOktS6gua/VN3J2hn7bwuvS0i40jQCwv17MjuNDo6aJEtkOcEkcVGx4fkp2xZ+gvviBcu0BpdbkQPtAHtZiPoxOiNe7hCqxTzi64F7EBFMqCyOuT8Eiwfjyb6lgskRT9z3GROmohuzIrF1uwZOK9NnB2EQofx6zEcii4NVSmFULdSrRlaEkoQYAjVezE1Jduiq7LTJ1H4MQw2U5Iz/iiMhDBRuxD1rOfTxDEzAbJfIhbgVPNhL61Bfj3dW8jRa5yjqXBoZ4g/mUBjd0dRgR9t/8b5qPK7q1NaPBsdZBYx+/jp9X7ucJT7Jt9PIlKVollN+6Lr+1TPNmq6J8hsOVk3YkFpgVjEbV23KaR1VI370QCpa+LZIN07Zyv73q/gLcMxCXPKh/ycYKEzet6cofR4PP2lftlS1qe90EbRKJ82Wysbp8PP/2OBZECC3xONYJEw0ot3afPdJiOl1ajzGrgCO4CtV1kB54w8fQ= 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)(396003)(376002)(39850400004)(346002)(110136005)(8936002)(54906003)(316002)(956004)(2906002)(6512007)(966005)(6506007)(83380400001)(36756003)(44832011)(478600001)(4326008)(2616005)(66946007)(66556008)(1076003)(6486002)(66476007)(86362001)(6666004)(186003)(38100700002)(38350700002)(26005)(8676002)(5660300002)(52116002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 8XAawQnJAeHsIAiFlhW238bmLw5/6Lx0ntQfn1/KLpb/MahNMOq8Ys2qV4vSVns8OWm807JxdmD5zzo5GiKP2FKfRdjJ8lPkmSFZOsNaKEWa5lugj2Jkj9f2WbcomMTN3H8HXHaFVdfF4hr9sNLYNOHYyZgtR2wT2kLUFnbhnkV3SxyM96QhoQVcNSyk/fs4IM4knOEysu0J0JchSyNOHyteERb3SzxfQ42cs0yBXISWYuPESabsU73MMHOfbdZDTHcrlxEP4rdwMRIqa7elpSRFvdEFdzQnp3LOw4aej1SFhNHnFGSwOkoOakrngWYZnxUG/PTCIKZLG2mfTuz9CG/aoeR4yiWOLVGrYCDKqXXkS+vaFLqeBuIJGXVINRxJZVgWvR9TCYQdmnjTibYC9qXlSrzA6LhTkeXToVzNkbFuA9TMKlYxo+2qu0yilnot1NfM3QqjRvdwdb5V46C15jhTw0HDuiXfBUlZofGE7h5ECsyHuRBe3nHpDMuhFrOINV1ldQLtbtCCNYgfWomcTGIEO5/fdocdpLRt9x70N+9NDxOXeWMU9rbzLm2w4bBwMiy3AX705XLojCdDR/+cHRtHw+NR4GyL/zW7n6MbdaP4PpVuXVp2jegNRw2DdefsbufgK7wEO8VcvdQwnDiDj5zK1w0vNcEiNNDdrMJ1iat8vzufss4gYbhdUNaG9Z5yhybxrvripoj9MYbyh25IZ5SCXwGmF4jLtDcGOTVUGwSfD0y2wu8vm7hL+CrfbLDxkqgd1kzgkuwAXLcTxzRrqRNBDO8ItgdxXlUPh3uP5XTCN0UbRbIqUl8+r51BgkntD2BUyEjBTtBHpyzlV4EZUgwTnxUeDP6jkd/R3OuiDWradUqPGf6TGv6pV0UaLhLjEOuPwFYCVDsnjWGdNLvEa/o4KEKMDCyZZqUNx1Ds6fmx87iV5Z4UYMjQwuwtexJAw/cd4V56Ua1VIqo3jz/zOJ7KZgoYo3fdN7yYcWQdNWn/tZ2kU35FGdMRgLGvDUeOZ4v0vA4SJhEfslWuwoTwb38BGEFE9QJpJDUTnpBi7oQ9rkpQsW/hoQqAFE96cHX8RCl/RZGygs9/GM8EtJRJztd7juqOR5Ro+G9PntDXeXMkFUHxrLZe+0kfZgZFnEFYD1h5zRU0zAJ5JWmt8kD5swPdcRldth1wqGHLuSXXTKeiQm9OibsXRTZBJENWYvS1WK2mQPdySWAi10lnmuo2lz67nlpEwK8JUIaNRW6zhTeILtja8NP2Rl9dqAWENeg2KEriPKaqvcsDklmJ9G93QXKarb0A7Cp1mqQFk99TgUQAXHgHnF47um1mwLEvlMRY X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: a422a9df-5a17-44e1-7a7f-08d966f6d54a X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Aug 2021 12:00:57.7670 (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: VcU1lDW1kFJI3H0dNs81RSZO18zgbPcpp9K6g2eBdSURoLf9DFTZiAcCUHDuzLb3RU1TxJp+aWebTs3W31j/xg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR04MB7374 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 Changes in v2: Rebased on top of the master branch (solves this conflict: https://patchwork.ozlabs.org/project/uboot/patch/20210629170839.2583797-4-olteanv@gmail.com/ with Tim Harvey's patch to make DSA masters promiscuous) This series makes the Felix DSA driver initialize all its connected PHYs regardless of whether those will be used for networking or not. This is in order to satisfy the expectations of some software in later boot stages. To make this work, it is necessary to introduce a new method in struct dsa_ops: .port_probe(). There is some further refactoring/cleanup along the way. Vladimir Oltean (6): net: dsa: felix: felix_init() can be static net: dsa: use "err" instead of "ret" in dsa_port_probe net: dsa: refactor the code to set the port MAC address into a dedicated function net: dsa: introduce a .port_probe() method in struct dsa_ops net: dsa: felix: call phy_config at .port_probe() time net: dsa: felix: propagate the error code from phy_startup() drivers/net/mscc_eswitch/felix_switch.c | 28 ++++++----- include/net/dsa.h | 5 +- net/dsa-uclass.c | 62 +++++++++++++++---------- 3 files changed, 59 insertions(+), 36 deletions(-)