Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2

[Michal Simek <michal.simek@amd.com>]

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 <trini@konsulko.com> (maintainer:ARM)
>>>> Ilias Apalodimas <ilias.apalodimas@linaro.org> (maintainer:ARM)
>>>> Michal Simek <michal.simek@amd.com> (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