Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2
Marek Vasut via U-Boot <[email protected]>
| Newsgroups | org.u-boot-project.lists.u-boot |
|---|---|
| Message-ID | <[email protected]> |
On 8/12/26 3:33 PM, Michal Simek wrote: >>> Before >>> $ ./scripts/get_maintainer.pl -f arch/arm/mach-versal2/ >>> Tom Rini <[email protected]> (maintainer:ARM) >>> Ilias Apalodimas <[email protected]> (maintainer:ARM) >>> Michal Simek <[email protected]> (maintainer:ARM >>> VERSAL,commit_signer:14/15=93%,authored:10/15=67%) >> There ^ you are , what's the problem ? > > Yes, I'm listed - but under ARM VERSAL, which is a different SoC family. > Versal Gen 2 is a completely different chip with different features. > It only matches the ARM VERSAL entry because of the "versal" substring in > the N: regex, not because they're the same chips. (FWIW keeping "Versal" > in every new SoC name is very confusing). > > Also Gen 2 specific drivers/files are coming, and those should be > attributed to the Versal Gen 2 not to the Versal. > Creating the section now is what makes that correct - otherwise every > new Versal2 file silently inherits the Versal. Why do we need 4 or 5 MAINTAINERS entries for Xilinx platforms, with the same maintainer, same git tree, same status, in the first place ? Why not have one XILINX entry that covers it all ?