Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 29 Nov 2022 01:20:21 -0800
From:   Colin Foster <colin.foster@in-advantage.com>
To:     Linus Walleij <linus.walleij@linaro.org>
Cc:     linux-renesas-soc@vger.kernel.org,
        linux-mediatek@lists.infradead.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org, netdev@vger.kernel.org,
        John Crispin <john@phrozen.org>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Marek Vasut <marex@denx.de>,
        Sean Wang <sean.wang@mediatek.com>,
        DENG Qingfang <dqfext@gmail.com>,
        Landen Chao <Landen.Chao@mediatek.com>,
        =?utf-8?B?bsOnIMOcTkFM?= <arinc.unal@arinc9.com>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        =?utf-8?B?Q2zDqW1lbnQgTMOpZ2Vy?= <clement.leger@bootlin.com>,
        Alvin =?utf-8?Q?=C5=A0ipraga?= <alsi@bang-olufsen.dk>,
        UNGLinuxDriver@microchip.com,
        Woojung Huh <woojung.huh@microchip.com>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        Kurt Kanzenbach <kurt@linutronix.de>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        "David S. Miller" <davem@davemloft.net>,
        Vladimir Oltean <olteanv@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>,
        George McCollister <george.mccollister@gmail.com>,
        Rob Herring <robh@kernel.org>
Subject: Re: [PATCH v3 net-next 03/10] dt-bindings: net: dsa: utilize base
 definitions for standard dsa switches
Message-ID: <Y4XO1T/M47bXqIBY@COLIN-DESKTOP1.localdomain>
References: <20221127224734.885526-1-colin.foster@in-advantage.com>
 <20221127224734.885526-4-colin.foster@in-advantage.com>
 <CACRpkdYkD-=y0mCnVkkQ9+4m9-nx6jQu=TfL-TPePbps6x+7Xw@mail.gmail.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CACRpkdYkD-=y0mCnVkkQ9+4m9-nx6jQu=TfL-TPePbps6x+7Xw@mail.gmail.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?23tJppmi52bea75EECEjEozSS7pDTpRD6mHdYDaRMlIZNKYwL03BMpDTZZuC?=
 =?us-ascii?Q?OPg+PRty64RdiZRsLll7GnjxMqB8pAaOccR/WMBWtVpSZHGjBy7Tujthh1jU?=
 =?us-ascii?Q?VogbHa3rgCsQxHppldmjalRnWm9PGdR2vByCZiGcyOXuiDWkkcZIUyqcFc+L?=
 =?us-ascii?Q?86021MckFiZriGANsyB4orgC1dn8hpQyAJChg6xjr9QP47qm2vHijkkeFENx?=
 =?us-ascii?Q?tEMzQRZYkfZI15xNAyF/5OqbKj5da1TYgEVydZfOdwoh3hKIkIco4v+oRan4?=
 =?us-ascii?Q?meW59bD2PIiVnRvkRQ44QwkMW8IZzhz4EcwvgudHsuKNjRZaQ91VISirweeq?=
 =?us-ascii?Q?aAIBKPTGSIGNNlTkLMaFCjDSuGGZClv7ebOjydUc7as8EGc4fRoJV/j9w92K?=
 =?us-ascii?Q?l9jWXNM/bD5qw4koS5H7yLI+tSHmcIoMxcIdagB303bX+4dIrSOgWLtp6w5J?=
 =?us-ascii?Q?rFoXzzlkUnQGnI9LqA3KwlSYZDIz9re1MdYTIJ6BFKtdcrdYilVAHAICjGv2?=
 =?us-ascii?Q?l2fcx+eyhAzRt/UuJ3YaqUFqp9EPWKjDWfGyxSCwn2lkdoXnb3bfQZs13KWs?=
 =?us-ascii?Q?siTSm1r/+gFjcYmxstUSpV+KSnLozN3eQs3qEm0o9ab4+hYttzBBZCTsx5Rh?=
 =?us-ascii?Q?REazU6OTQihfMYbkafzr3bYF3iwxJTMo375pHr9IOa6l23Kt7gT1w/In0+By?=
 =?us-ascii?Q?Gm6VvPFj5makDBaotoNH6pia2LAlKq5+5+zy9tDui7J5m8IBsiU7uSMgQKUw?=
 =?us-ascii?Q?7R/P1D2fuL/R2E8dQuJDreWttlcDBLbgSua3bBmuE9NEI1CwSLHNMPAUE1xV?=
 =?us-ascii?Q?wRFG/yI1zgT/hA75IfEyCxSozaydGmtTDVX/MmJJRFHjCkRGp0w9G3BKJQEn?=
 =?us-ascii?Q?48uVV3HX9yd5yvAXj38BJ6ubc08Lse+nKmNGRSHBf65JM44wcygCMvDFp4c+?=
 =?us-ascii?Q?OOI+NUPobDMjMthC6J6vDM+WxVfPkb1bJ6IIXbcY9ZMLx9/wRiD2XOezAPHY?=
 =?us-ascii?Q?zOhe7bfcAs7O4nNn08JtefB4wAY9FH8gOfnVNqAxZPQv+NCKrsfP5GrSGBnk?=
 =?us-ascii?Q?2jMGXcai4sBOEa3ZRPyZjm4k0CcjUiemB5Th59Gm7DkXiW5Wd1Ev63lXt+AY?=
 =?us-ascii?Q?V1jEiK0Zjsdoo6SHARWkga5F0kGJha8Er7Trtr1VKkGJV1OOnxmzsubWAC6u?=
 =?us-ascii?Q?MxhrZst/A91QErSMHYsoSDnQH6QtA7Hot9Q+UMQCUu7hW27R96ORBXcIBDVD?=
 =?us-ascii?Q?0dey3yM9KaWL8Oz/6H33LKnKWdLMADcbq9jYx4prPGTpkWf2yFYzacr898du?=
 =?us-ascii?Q?0NCLEjTr5IJ0n3jfacj93d/V4WWaE7lOcvjCHybzCFGCNcRw9/0pWgwXTZkm?=
 =?us-ascii?Q?h+ZzfYsFRhA8nCU3LyQ5ok01BTfZ4aaj7E3VfMi4XqbCCUt5igxXa4AQlISS?=
 =?us-ascii?Q?kgc1HYHX5V7dniD9K4v0x/Ubn47orDPyvrY13UP5dfo829Ctsj59Lg6MI9c8?=
 =?us-ascii?Q?tNW6iyM6K68E41D7E0rAGU11BVCxp80q4/LdV2zuEpEzpSv5lwItQ+fl7ArT?=
 =?us-ascii?Q?x1wifGhRnI+m9WkoTLsboBS8Nj/PNTn9Knh/W7cJpH3B1AKemi1zdIjKi+xq?=
 =?us-ascii?Q?Uw=3D=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 10df13f4-5c1d-4965-34a6-08dad1b04a20
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Nov 2022 02:20:31.6920
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 48e842ca-fbd8-4633-a79d-0c955a7d3aae
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: QoBaS/uq4o4AlMDkKVgryxisDOfhtlp3gcLajIB2o5qVp0pGiBn8IqIaKrs2nvOEcPZtYPKFOjZ5FnsuCtUHdkdySMmpia/9qv6a3MriSc8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR10MB5084
X-Spam-Status: No, score=0.0 required=5.0 tests=BAYES_00,DATE_IN_FUTURE_06_12,
        DKIM_SIGNED,DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,
        SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Nov 28, 2022 at 09:21:26PM +0100, Linus Walleij wrote:
> On Sun, Nov 27, 2022 at 11:47 PM Colin Foster
> <colin.foster@in-advantage.com> wrote:
> 
> > DSA switches can fall into one of two categories: switches where all ports
> > follow standard '(ethernet-)?port' properties, and switches that have
> > additional properties for the ports.
> >
> > The scenario where DSA ports are all standardized can be handled by
> > swtiches with a reference to 'dsa.yaml#'.
> >
> > The scenario where DSA ports require additional properties can reference
> > the new '$dsa.yaml#/$defs/base'. This will allow switches to reference
> > these base defitions of the DSA switch, but add additional properties under
> > the port nodes.
> >
> > Suggested-by: Rob Herring <robh@kernel.org>
> > Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
> 
> This is neat.
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Thanks. All credit goes to Rob (except the bugs - those are mine)

> 
> Yours,
> Linus Walleij