Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2
Michal Simek <[email protected]>
| Newsgroups | org.u-boot-project.lists.u-boot |
|---|---|
| Message-ID | <[email protected]> |
On 8/12/26 19:51, Marek Vasut wrote: > 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 ? No issue. Let me prepare a patch for it. Thanks, Michal