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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 624F7C61DA4 for ; Sun, 12 Mar 2023 02:54:20 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229578AbjCLCyR (ORCPT ); Sat, 11 Mar 2023 21:54:17 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38060 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229457AbjCLCyP (ORCPT ); Sat, 11 Mar 2023 21:54:15 -0500 Received: from vps0.lunn.ch (vps0.lunn.ch [156.67.10.101]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CA8DA3BDAD; Sat, 11 Mar 2023 18:54:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Disposition:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:From:Sender:Reply-To:Subject: Date:Message-ID:To:Cc:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Content-Disposition:In-Reply-To:References; bh=xLIgvG34Sc+lDohdVZWPEZo6FgjzStnEHWeCstzLe6k=; b=JJC1gzH4XjKAY1yMwNL6sqcv37 gSSSB8dGXc3gMl0PdFDUv5w3aSdiDh9830SEmmbma+yikri5UNuOesvefuHisH8/fpv0HNO4ICXC0 Pr97yR0lgntFbgpaWKDDaTvR4i/nOnW29hKU0w3QeXYDB7DkO1n6s7925UjcRlWlZ4eU=; Received: from andrew by vps0.lunn.ch with local (Exim 4.94.2) (envelope-from ) id 1pbBq8-0075xU-UV; Sun, 12 Mar 2023 03:53:40 +0100 Date: Sun, 12 Mar 2023 03:53:40 +0100 From: Andrew Lunn To: Klaus Kudielka Cc: Michael Walle , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Felix Fietkau , John Crispin , Sean Wang , Mark Lee , Lorenzo Bianconi , Matthias Brugger , Bryan Whitehead , UNGLinuxDriver@microchip.com, Giuseppe Cavallaro , Alexandre Torgue , Jose Abreu , Maxime Coquelin , Joel Stanley , Andrew Jeffery , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-aspeed@lists.ozlabs.org, Jesse Brandeburg Subject: Re: [PATCH net-next v2 4/6] net: mdio: scan bus based on bus capabilities for C22 and C45 Message-ID: <29ee3cc4-a1d6-4a07-8d90-4b2f26059e7d@lunn.ch> References: <0e10aa8492eadb587949d8744b56fccaabbd183b.camel@gmail.com> <72530e86-9ba9-4a01-9cd2-68835ecae7a0@lunn.ch> <09d65e1ee0679e1e74b4f3a5a4c55bd48332f043.camel@gmail.com> <70f5bca0-322c-4bae-b880-742e56365abe@lunn.ch> <10da10caea22a8f5da8f1779df3e13b948e8a363.camel@gmail.com> <4abd56aa-5b9f-4e16-b0ca-11989bb8c764@lunn.ch> <2f64385a350359c5755eb4d2479e2efef7a96216.camel@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2f64385a350359c5755eb4d2479e2efef7a96216.camel@gmail.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > Here's the definition of the switch in the Turris Omnia device tree. > > /* Switch MV88E6176 at address 0x10 */ > switch@10 { > pinctrl-names = "default"; > pinctrl-0 = <&swint_pins>; > compatible = "marvell,mv88e6085"; > #address-cells = <1>; > #size-cells = <0>; > > dsa,member = <0 0>; > reg = <0x10>; > > interrupt-parent = <&gpio1>; > interrupts = <13 IRQ_TYPE_LEVEL_LOW>; > > ports { > #address-cells = <1>; > #size-cells = <0>; > > ports@0 { > reg = <0>; > label = "lan0"; > }; > > ports@1 { > reg = <1>; > label = "lan1"; > }; > > ports@2 { > reg = <2>; > label = "lan2"; > }; > > ports@3 { > reg = <3>; > label = "lan3"; > }; > > ports@4 { > reg = <4>; > label = "lan4"; > }; > > ports@5 { > reg = <5>; > label = "cpu"; > ethernet = <ð1>; > phy-mode = "rgmii-id"; > > fixed-link { > speed = <1000>; > full-duplex; > }; > }; > > ports@6 { > reg = <6>; > label = "cpu"; > ethernet = <ð0>; > phy-mode = "rgmii-id"; > > fixed-link { > speed = <1000>; > full-duplex; > }; > }; > }; > > What you are proposing here would not show any improvement on the > Omnia, as only the 6 ports would be scanned - right? Correct. But their also should not of been any noticeable slow down, because there should not be any additional scanning when everything is described in DT. And the move of the MDIO bus registration from probe to setup should actually make it faster than before. Andrew