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 537638F68 for ; Wed, 13 Sep 2023 14:36:52 +0000 (UTC) Received: from relay5-d.mail.gandi.net (relay5-d.mail.gandi.net [217.70.183.197]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0AC7D98; Wed, 13 Sep 2023 07:36:50 -0700 (PDT) Received: by mail.gandi.net (Postfix) with ESMTPSA id C56931C0005; Wed, 13 Sep 2023 14:36:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arinc9.com; s=gm1; t=1694615809; 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=kkIQnIuFCF1TTp1ea3vLPJG6FNuTS8tRKjMxldrO4oM=; b=HEBFCtkjbRZFJkev8Pf8mhxM3RpoVAvD4QVoBNYxzrOaDscDxTFNA+yN7T81EUckVol9u4 xjgKo9Q2xVQt3oM3BBz7rNoFGOOjvyYNOl0F5adqeTjK0UACKAcGc4926K4ZdnBlqjJ08o TpIrTeJB8xnY7NWS0l094yS+bhkU6fqnQUiGeuRrM2KJxHJwFpeCvR9WyU4wa4Wwh1B6hQ 2Qre22XS5evB8RsPzIkTLVjNiZHZfV7EkTDVZUkl8/RjO128yfKq/8wwffdLy05kzQ6GgG m4eAKhrIq4COyT9WCMl2GYyyhWow3rwrabnUMW24Rcu4ES+LLBYao4WeKApkJQ== Message-ID: <58b87aea-c7f5-4e7d-9736-74a2ed997306@arinc9.com> Date: Wed, 13 Sep 2023 17:36:36 +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: Vladimir Oltean Cc: Andrew Lunn , 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: <676d1a2b-6ffa-4aff-8bed-a749c373f5b3@arinc9.com> <87325ce9-595a-4dda-a6a1-b5927d25719b@arinc9.com> <20230911225126.rk23g3u3bzo3agby@skbuf> <036c0763-f1b2-49ff-bc82-1ff16eec27ab@arinc9.com> <20230912193450.h5s6miubag46z623@skbuf> <6cec079e-991e-4222-a76d-d6156de0daca@arinc9.com> <20230913074231.5azwxqjuv2wp5nik@skbuf> <89c9b84c-574c-4071-9524-9207597a3f0a@arinc9.com> <20230913110404.co7earmnbzf6hhoe@skbuf> <137fd54d-7d2d-4d0b-a50b-cca69875a814@arinc9.com> <20230913130049.ivvl4vzjcfedsddr@skbuf> From: =?UTF-8?B?QXLEsW7DpyDDnE5BTA==?= In-Reply-To: <20230913130049.ivvl4vzjcfedsddr@skbuf> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-GND-Sasl: arinc.unal@arinc9.com On 13.09.2023 16:00, Vladimir Oltean wrote: > On Wed, Sep 13, 2023 at 02:35:11PM +0300, Arınç ÜNAL wrote: >> On 13.09.2023 14:04, Vladimir Oltean wrote: >>> I don't think they're for switch ports only. Any driver which uses >>> phylink_fwnode_phy_connect() or its derivatives gets subject to the same >>> bindings. But putting the sub-schema in ethernet-controller.yaml makes >>> sense, just maybe not naming it "phylink-switch". >> >> Got it. Should we disallow managed altogether when fixed-link is also >> defined, or just with in-band-status value? > > Just with the "in-band-status" value - just like phylink_parse_mode() > implies. If not possible, just leave that condition out. I can rewrite the property to allow values other than in-band-status. - if: required: [ fixed-link ] then: properties: managed: const: auto Arınç