From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D0EEF1A703 for ; Wed, 13 Sep 2023 12:44:18 +0000 (UTC) Received: from relay7-d.mail.gandi.net (relay7-d.mail.gandi.net [217.70.183.200]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6CD2D19B1; Wed, 13 Sep 2023 05:44:17 -0700 (PDT) Received: by mail.gandi.net (Postfix) with ESMTPSA id 7668420008; Wed, 13 Sep 2023 12:44:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arinc9.com; s=gm1; t=1694609055; 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=jOWHnup71AUlA2pdfGiWMAnf7L7N7R7hXAeXu+YYNpk=; b=LoVXclUT4ag/ho/8wurTwZ89GnA2gA+DpO1lpnb6s8GY4d5cD124trIxlVkUZ0egL/1LET sXeP8RWQPMYk/chORC/sv4Wo/pTkSwLQUV7+yC2nVNfv07CDYikZ6HIYPVjf8s35jtl3zn c4OeEWb0mOT9DVwFgbTo/TYSTiivS9zgKA0tinbGJ506oJ1h3h+HIpKN/VpWExKOv0t1De tzcfmiKZmJ3+a8IxyEtWNStEeyu9mNjCqWhZNVCMnU3gv8MrjqqcWgcr1i0azI/R7ecWH6 lHOIK0FYfDcTMHMPamOLZRJmJ+lR7xLMRcOJcpw/QH/hAKpcxgy+nxSgiqc66g== Message-ID: Date: Wed, 13 Sep 2023 15:44:03 +0300 Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 2/4] dt-bindings: net: dsa: document internal MDIO bus Content-Language: en-US To: Andrew Lunn Cc: Vladimir Oltean , Florian Fainelli , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Woojung Huh , UNGLinuxDriver@microchip.com, Linus Walleij , =?UTF-8?Q?Alvin_=C5=A0ipraga?= , Daniel Golle , Landen Chao , DENG Qingfang , Sean Wang , Matthias Brugger , AngeloGioacchino Del Regno , 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 References: <20230812091708.34665-3-arinc.unal@arinc9.com> <47b61929-5c2d-4906-b153-2046a94858c8@arinc9.com> <47b61929-5c2d-4906-b153-2046a94858c8@arinc9.com> <20230813112026.ohsx6srbt2staxma@skbuf> <8a8e14f1-0493-4298-a2cc-6e7ae7929334@arinc9.com> <20230813190157.4y3zoro53qsz43pe@skbuf> <617c51cf-2c09-4865-ac60-96599db597e7@lunn.ch> <563ac27a-22f2-463e-b5c1-9df721842976@arinc9.com> From: =?UTF-8?B?QXLEsW7DpyDDnE5BTA==?= In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-GND-Sasl: arinc.unal@arinc9.com On 13.09.2023 04:21, Andrew Lunn wrote: >>> The marvell switch can have up to 2 MDIO busses. If i remember >>> correctly, there is also one switch which has one MDIO bus per port. >> >> I'm writing the json-schema for Marvell switches. I've checked a few >> devicetree source files on Linus's Linux tree, I only see two buses used at >> the most. > > Sorry, i was ambiguous. Its not a Marvell switch which can have one > MDIO bus per port. I don't remember which switch it is, and it might > be a pure switchdev switch, not a DSA switch. > >> The internal bus and another bus with >> marvell,mv88e6xxx-mdio-external. I've never seen a devicetree with >> marvell,mv88e6250 though. Could the switch that has one MDIO bus per port >> be this one? Also, is every bus of this switch a >> marvell,mv88e6xxx-mdio-external bus or, one internal bus and the remaining >> are marvell mv88e6xxx-mdio-external buses? > > Only the 6390 family has two busses. It has an internal MDIO bus with > the same register API as all the other switches. However, unlike the > other families, it is not exposed on pins. And the 6390 has a second > MDIO bus using a slight variant of the registers, which is connected > to the outside world via pins. This second bus then has a compatible > to separate it from the normal MDIO bus. OK, I will disallow the external mdio bus for the compatible strings other than marvell,mv88e6190. Arınç