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, 6 Dec 2022 19:07:54 -0800
From:   Colin Foster <colin.foster@in-advantage.com>
To:     Vladimir Oltean <olteanv@gmail.com>
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>,
        Linus Walleij <linus.walleij@linaro.org>,
        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>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>,
        George McCollister <george.mccollister@gmail.com>
Subject: Re: [PATCH v4 net-next 0/9] dt-binding preparation for ocelot
 switches
Message-ID: <Y5ADis5/a0KrHM10@euler>
References: <20221202204559.162619-1-colin.foster@in-advantage.com>
 <20221206160430.4kiyrzrumcc6dp2g@skbuf>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20221206160430.4kiyrzrumcc6dp2g@skbuf>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?1NmU75uYeVvdJq4R538uT7lGO9MUBjTU2TwW7SJHUQorEexZoAYf2KZP8Tbm?=
 =?us-ascii?Q?8Pdt8rgAs0VCR67oFq1kVKipxtU+HRdS0e5FuEwZ5f2RjWML1t74NbwFCLPY?=
 =?us-ascii?Q?Y55TeJpKFU3DtKwbooGclQqPhI5X8gw72PqJz2hCgn5e3Z8cAbNKqlMf0qds?=
 =?us-ascii?Q?SCPp3KT/QP96AM2XiDKK6fmpVvKMmNmOz8QWNkXHWwFUusPn0wrqGUknx1Mw?=
 =?us-ascii?Q?R27C9qe287VJcg0GapjWMIDwjRQ/AymBM0AY+M+mEHalyWGBeBkVJnX6j8NU?=
 =?us-ascii?Q?JWT9Txk3Mlsvd0AAq2b+kO6K9wCHpjy3KJ+BSQZH2+BikqNKIGEEdQ0ehJ+N?=
 =?us-ascii?Q?+dYy54WzX6/AGJf+ZXeTRDc5iBrExV+p1FkhvWwE60NawmQOaJo6wYtKodpJ?=
 =?us-ascii?Q?fq1JesJfKy8LEhrnvDFo4VkO8AJL5QILPurVtNTaEcNcO9eWLnQMzxfBwf1P?=
 =?us-ascii?Q?ul6y4/VHMPHLO0RjQWIgjAUnyTUhfb2JC3ByDo2QA49yI1AS71oE2P5s4eR/?=
 =?us-ascii?Q?OMTIpYNltn3fRl9bc0QB2eKquuCiccwrvHU24GsIkYHA1j/j1A3ml6wA7ttF?=
 =?us-ascii?Q?4rlhoQv/O7u0hGhlryTK2R9VoqRP9TXRCi3T1XHXucadXH/ybkgQO7JoiaBf?=
 =?us-ascii?Q?BDUE+nezMx/d0B1UBwAFxY2dUcTULrOmDrL+v6RlcBjKaPjQY4ea0VJghhHY?=
 =?us-ascii?Q?K8trtomydrMBNldL3nGYegjDYBG80bdVl6jzR+qB/8BD3Qsa8NXT0Twog8Ug?=
 =?us-ascii?Q?k4ah7rFGWaL7i6P1xvQQS2CyhSr1q/aCQOEjsabxrrzuGROfzwrFtzjx8CWN?=
 =?us-ascii?Q?lJR42L4Fo2nzOJjdbZhAPp7yioJ/cGuhnYysdvZi4p2V2yoCAfkhvJ3bIWXG?=
 =?us-ascii?Q?iVqC76Ywi1mxFp34X3Rp4uS4Ehs8PFGzm3ITkLIRpNBLNt6LeSdTITd9w98Y?=
 =?us-ascii?Q?F6SLBUu81ZHxWU2Dfdpl7y1Qtip2YGVVlWEjBTVV5N8ZomdfvoL603BnSZF7?=
 =?us-ascii?Q?3ZDTGadLrfdkV1V4m4urIRuqwRbYnPz0bWXwYXeGrKEIrmCQpJaag7VZw+Xm?=
 =?us-ascii?Q?dMvTxtGkoe0p+Tncm/hBy38Y8FpkOcT+bJ2pSg2ybvx5HbtbojUQIVGVu5IT?=
 =?us-ascii?Q?sIAovWkW1grC0I30iy2gnbl+bbXDbtWsNY0PaiWD58Q6FPzch3FkolStN+Hl?=
 =?us-ascii?Q?127T//BWQgvUXkfFPxo9pWeeLd4AuQkahuTwIGpKMoGtmF/SPjK5PVJfP4s7?=
 =?us-ascii?Q?O2eZ5+ENLStG7nvVJA5ruCLzWBTuvuXztfSfinpU4+trie9RZ4ha0jt0nj8K?=
 =?us-ascii?Q?dBgZmCTMVcaG+5eraZoOAYhyW6qqh2bYmwNGhRyxJw4foPweNvsf5+nttMM5?=
 =?us-ascii?Q?yzA1APOLnfTZStRAYInoDnITRFJ1nuIbJB90JqDn1FVkodO7RGV3uFbwfoKS?=
 =?us-ascii?Q?mkBEk5sixDSmcqeLgOrKOTT7lcYvEKVSh1ClcEiU5YPuUxEw+qOdahxLZlHt?=
 =?us-ascii?Q?Nw1gH91ym6t7E0JG/KxmIk8ESr4XjJrCcsLpblXQ0whrKv1XdtLEX7YoJaR1?=
 =?us-ascii?Q?CSaFzurjpoDsvmig28s1bin4zx3vjfePnaklGl15vKkmYc8JZkD4t3Q9IW6S?=
 =?us-ascii?Q?zg=3D=3D?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 26b15179-be9f-46d7-e5ea-08dad8003e76
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Dec 2022 03:07:58.7644
 (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: p3/4rv9YhAYle8R0WPBOpDm3c3zwx8i2zOMoR2LS+gwtg8QlIGRQG21oFMtgtyiCIJWePTpEAI0hpQ0mWHBHt3Y+oNkZkQ+XRUc2nFu1uqo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR10MB6714
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,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 Tue, Dec 06, 2022 at 06:04:30PM +0200, Vladimir Oltean wrote:
> Hi Colin,
> 
> On Fri, Dec 02, 2022 at 12:45:50PM -0800, Colin Foster wrote:
> > Ocelot switches have the abilitiy to be used internally via
> > memory-mapped IO or externally via SPI or PCIe. This brings up issues
> > for documentation, where the same chip might be accessed internally in a
> > switchdev manner, or externally in a DSA configuration. This patch set
> > is perparation to bring DSA functionality to the VSC7512, utilizing as
> > much as possible with an almost identical VSC7514 chip.
> > 
> > This patch set changed quite a bit from v2, so I'll omit the background
> > of how those sets came to be. Rob offered a lot of very useful guidance.
> > My thanks.
> > 
> > At the end of the day, with this patch set, there should be a framework
> > to document Ocelot switches (and any switch) in scenarios where they can
> > be controlled internally (ethernet-switch) or externally (dsa-switch).
> > 
> > ---
> 
> This looks like a very clean implementation of what I had in mind
> (better than I could have done it). Sorry for not being able to help
> with the json-schema bits and thanks to Rob for doing so.

It seems like it worked out well. Thanks Rob for all the help on this!

> 
> Would you mind adding one more patch at the beginning of the series
> which syncs the maintainers from the DSA (and now also ethernet-switch)
> dt-bindings with the MAINTAINERS file? That would mean removing Vivien
> (see commit 6ce3df596be2 ("MAINTAINERS: Move Vivien to CREDITS")) and
> adding myself. This is in principle such that you don't carry around a
> not-up-to-date list of maintainers when adding new schemas.

Yep. I'll get that in the next set.

> 
> I don't know if we could do something about maintainer entries in
> schemas not becoming out of date w.r.t. the MAINTAINERS file.

Sounds like a nice feature. Maybe part of dt_binding_check / checkpatch?
I'm running out of hours in a day...