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 DDC05CD37BC for ; Sat, 16 Sep 2023 15:32:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238693AbjIPPbb (ORCPT ); Sat, 16 Sep 2023 11:31:31 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53342 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233025AbjIPPbC (ORCPT ); Sat, 16 Sep 2023 11:31:02 -0400 Received: from relay6-d.mail.gandi.net (relay6-d.mail.gandi.net [IPv6:2001:4b98:dc4:8::226]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 92C74CE8; Sat, 16 Sep 2023 08:30:55 -0700 (PDT) Received: by mail.gandi.net (Postfix) with ESMTPSA id CC3C8C0003; Sat, 16 Sep 2023 15:30:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arinc9.com; s=gm1; t=1694878253; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=nNzjBvEdjVLIfZXjhwvUmOo9cBIhI+ah064hJXymrS0=; b=XelGTaolNJmnpe83jTCntCcgiRnplsFx18gQ0m1m920W/83H3auR3EqLSI29RuovlUfm/V VwKTF44wM+z8mX1yxBPlUy14aUVq5oF9z+FGwpth5iH2NG8/DfzKlic+7vllRWI5Zg+ero kGwOXqeK+ESrcQZrc4ONd+vpmap76/mCBOAbGgRN0ErUa5phiJwZfxOMi/dN+JpZITb9ek 66cPtW8p4GO5AEIRc9S4XJbtKZJOL86pCEOa8y3LTZ3a3FxgK1JJ85CLvmFy0AQ5KBpM9M vP9EUgcK9IdByP5RO2vJ3Y8XWnnQ7yKZcD/Rtytzt7vSoY5yKkhmTLZuL1BEaQ== Message-ID: Date: Sat, 16 Sep 2023 18:30:31 +0300 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next v2 05/10] dt-bindings: net: dsa: define MDIO bus child node Content-Language: en-US To: Andrew Lunn Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , George McCollister , Florian Fainelli , Vladimir Oltean , Kurt Kanzenbach , Matthias Brugger , AngeloGioacchino Del Regno , Woojung Huh , UNGLinuxDriver@microchip.com, Linus Walleij , =?UTF-8?Q?Alvin_=C5=A0ipraga?= , =?UTF-8?B?Q2zDqW1lbnQgTMOpZ2Vy?= , Marcin Wojtas , "Russell King (Oracle)" , Lars Povlsen , Steen Hegelund , Daniel Machon , Radhey Shyam Pandey , Daniel Golle , Landen Chao , DENG Qingfang , Sean Wang , Geert Uytterhoeven , Magnus Damm , Maxime Chevallier , Nicolas Ferre , Claudiu Beznea , Marek Vasut , Claudiu Manoil , Alexandre Belloni , John Crispin , Madalin Bucur , Ioana Ciornei , Lorenzo Bianconi , Felix Fietkau , Horatiu Vultur , Oleksij Rempel , Alexandre Torgue , Giuseppe Cavallaro , Jose Abreu , Grygorii Strashko , Sekhar Nori , Shyam Pandey , mithat.guner@xeront.com, erkin.bozoglu@xeront.com, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-renesas-soc@vger.kernel.org References: <20230916110902.234273-1-arinc.unal@arinc9.com> <20230916110902.234273-6-arinc.unal@arinc9.com> <445beba8-2499-44a3-9c36-b9ec761121fb@lunn.ch> From: =?UTF-8?B?QXLEsW7DpyDDnE5BTA==?= In-Reply-To: <445beba8-2499-44a3-9c36-b9ec761121fb@lunn.ch> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-GND-Sasl: arinc.unal@arinc9.com Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 16.09.2023 17:11, Andrew Lunn wrote: > On Sat, Sep 16, 2023 at 02:08:57PM +0300, Arınç ÜNAL wrote: >> Some DSA subdrivers register the MDIO bus of the switch they control. Or >> let the DSA driver register the MDIO bus. The node for these buses are >> either required or optional, depending on the subdriver. Document this on >> all of the affected DSA switch schemas. >> >> The attributes of a DSA subdriver that lets the DSA driver register the >> bus: >> - ds->ops->phy_read() and ds->ops->phy_write() are present. >> - ds->slave_mii_bus is not populated by the DSA subdriver. >> - The bus is registered non-OF-based or OF-based. Registered OF-based if >> "mdio" child node is defined. >> >> The affected DSA switch schemas are documented below. >> >> - brcm,b53.yaml >> >> drivers/net/dsa/b53/b53_common.c: >> - The DSA subdriver lets the DSA driver register the bus. >> >> --- > > git uses --- to separate the commit message from additional comments > for the reviewers. Anything after the --- will not be merged. Is that > your intention? No, they're vital information. I'll find something better to separate them, thanks. Arınç