* [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 @ 2026-08-10 11:37 Michal Simek 2026-08-12 6:35 ` Michal Simek 0 siblings, 1 reply; 8+ messages in thread From: Michal Simek @ 2026-08-10 11:37 UTC (permalink / raw) To: u-boot, git; +Cc: Marek Vasut, Quentin Schulz, Tom Rini Versal Gen 2 and especially march-versal2 is not covered by own fragment that's why add it. Signed-off-by: Michal Simek <michal.simek@amd.com> --- MAINTAINERS | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index e1379a6a3e7e..eb48eea55c5a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -821,6 +821,12 @@ F: drivers/spi/cadence_ospi_versal.c F: drivers/watchdog/xilinx_wwdt.c N: (?<!uni)versal +ARM VERSAL GEN 2 +M: Michal Simek <michal.simek@amd.com> +S: Maintained +T: git https://git.u-boot-project.org/u-boot/custodians/u-boot-microblaze.git +F: arch/arm/mach-versal2/ + ARM VERSATILE EXPRESS DRIVERS M: Liviu Dudau <liviu.dudau@foss.arm.com> S: Maintained --- base-commit: 4b12b2532a904fcf1c5ed9a7b323f39717ee92e4 -- 2.43.0 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 2026-08-10 11:37 [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 Michal Simek @ 2026-08-12 6:35 ` Michal Simek 2026-08-12 9:11 ` Marek Vasut via U-Boot 0 siblings, 1 reply; 8+ messages in thread From: Michal Simek @ 2026-08-12 6:35 UTC (permalink / raw) To: u-boot, git; +Cc: Marek Vasut, Quentin Schulz, Tom Rini On 8/10/26 13:37, Michal Simek wrote: > Versal Gen 2 and especially march-versal2 is not covered by own fragment > that's why add it. > > Signed-off-by: Michal Simek <michal.simek@amd.com> > --- > > MAINTAINERS | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index e1379a6a3e7e..eb48eea55c5a 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -821,6 +821,12 @@ F: drivers/spi/cadence_ospi_versal.c > F: drivers/watchdog/xilinx_wwdt.c > N: (?<!uni)versal > > +ARM VERSAL GEN 2 > +M: Michal Simek <michal.simek@amd.com> > +S: Maintained > +T: git https://git.u-boot-project.org/u-boot/custodians/u-boot-microblaze.git > +F: arch/arm/mach-versal2/ > + > ARM VERSATILE EXPRESS DRIVERS > M: Liviu Dudau <liviu.dudau@foss.arm.com> > S: Maintained > --- > base-commit: 4b12b2532a904fcf1c5ed9a7b323f39717ee92e4 > Applied. M ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 2026-08-12 6:35 ` Michal Simek @ 2026-08-12 9:11 ` Marek Vasut via U-Boot 2026-08-12 11:28 ` Michal Simek 0 siblings, 1 reply; 8+ messages in thread From: Marek Vasut via U-Boot @ 2026-08-12 9:11 UTC (permalink / raw) To: Michal Simek, u-boot, git, Tom Rini; +Cc: Quentin Schulz On 8/12/26 8:35 AM, Michal Simek wrote: > > > On 8/10/26 13:37, Michal Simek wrote: >> Versal Gen 2 and especially march-versal2 is not covered by own fragment >> that's why add it. >> >> Signed-off-by: Michal Simek <michal.simek@amd.com> >> --- >> >> MAINTAINERS | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index e1379a6a3e7e..eb48eea55c5a 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -821,6 +821,12 @@ F: drivers/spi/cadence_ospi_versal.c >> F: drivers/watchdog/xilinx_wwdt.c >> N: (?<!uni)versal >> +ARM VERSAL GEN 2 >> +M: Michal Simek <michal.simek@amd.com> >> +S: Maintained >> +T: git https://git.u-boot-project.org/u-boot/custodians/u-boot- >> microblaze.git >> +F: arch/arm/mach-versal2/ Use "N: versal" instead for the globbing match . In fact, "N: (?<!uni)versal" above does not work ? ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 2026-08-12 9:11 ` Marek Vasut via U-Boot @ 2026-08-12 11:28 ` Michal Simek 2026-08-12 13:08 ` Marek Vasut via U-Boot 0 siblings, 1 reply; 8+ messages in thread From: Michal Simek @ 2026-08-12 11:28 UTC (permalink / raw) To: Marek Vasut, u-boot, git, Tom Rini; +Cc: Quentin Schulz On 8/12/26 11:11, Marek Vasut wrote: > On 8/12/26 8:35 AM, Michal Simek wrote: >> >> >> On 8/10/26 13:37, Michal Simek wrote: >>> Versal Gen 2 and especially march-versal2 is not covered by own fragment >>> that's why add it. >>> >>> Signed-off-by: Michal Simek <michal.simek@amd.com> >>> --- >>> >>> MAINTAINERS | 6 ++++++ >>> 1 file changed, 6 insertions(+) >>> >>> diff --git a/MAINTAINERS b/MAINTAINERS >>> index e1379a6a3e7e..eb48eea55c5a 100644 >>> --- a/MAINTAINERS >>> +++ b/MAINTAINERS >>> @@ -821,6 +821,12 @@ F: drivers/spi/cadence_ospi_versal.c >>> F: drivers/watchdog/xilinx_wwdt.c >>> N: (?<!uni)versal >>> +ARM VERSAL GEN 2 >>> +M: Michal Simek <michal.simek@amd.com> >>> +S: Maintained >>> +T: git https://git.u-boot-project.org/u-boot/custodians/u-boot- >>> microblaze.git >>> +F: arch/arm/mach-versal2/ > Use "N: versal" instead for the globbing match . > > In fact, "N: (?<!uni)versal" above does not work ? nope 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%) u-boot@lists.u-boot-project.org (open list) After: $ ./scripts/get_maintainer.pl -f arch/arm/mach-versal2/ Michal Simek <michal.simek@amd.com> (maintainer:ARM VERSAL GEN 2,commit_signer:14/15=93%,authored:10/15=67%) Tom Rini <trini@konsulko.com> (maintainer:ARM) Ilias Apalodimas <ilias.apalodimas@linaro.org> (maintainer:ARM) u-boot@lists.u-boot-project.org (open list) Thanks, Michal ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 2026-08-12 11:28 ` Michal Simek @ 2026-08-12 13:08 ` Marek Vasut via U-Boot 2026-08-12 13:33 ` Michal Simek 0 siblings, 1 reply; 8+ messages in thread From: Marek Vasut via U-Boot @ 2026-08-12 13:08 UTC (permalink / raw) To: Michal Simek, u-boot, git, Tom Rini; +Cc: Quentin Schulz On 8/12/26 1:28 PM, Michal Simek wrote: > > > On 8/12/26 11:11, Marek Vasut wrote: >> On 8/12/26 8:35 AM, Michal Simek wrote: >>> >>> >>> On 8/10/26 13:37, Michal Simek wrote: >>>> Versal Gen 2 and especially march-versal2 is not covered by own >>>> fragment >>>> that's why add it. >>>> >>>> Signed-off-by: Michal Simek <michal.simek@amd.com> >>>> --- >>>> >>>> MAINTAINERS | 6 ++++++ >>>> 1 file changed, 6 insertions(+) >>>> >>>> diff --git a/MAINTAINERS b/MAINTAINERS >>>> index e1379a6a3e7e..eb48eea55c5a 100644 >>>> --- a/MAINTAINERS >>>> +++ b/MAINTAINERS >>>> @@ -821,6 +821,12 @@ F: drivers/spi/cadence_ospi_versal.c >>>> F: drivers/watchdog/xilinx_wwdt.c >>>> N: (?<!uni)versal >>>> +ARM VERSAL GEN 2 >>>> +M: Michal Simek <michal.simek@amd.com> >>>> +S: Maintained >>>> +T: git https://git.u-boot-project.org/u-boot/custodians/u-boot- >>>> microblaze.git >>>> +F: arch/arm/mach-versal2/ >> Use "N: versal" instead for the globbing match . >> >> In fact, "N: (?<!uni)versal" above does not work ? > > nope > > 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 ? ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 2026-08-12 13:08 ` Marek Vasut via U-Boot @ 2026-08-12 13:33 ` Michal Simek 2026-08-12 17:51 ` Marek Vasut via U-Boot 0 siblings, 1 reply; 8+ messages in thread From: Michal Simek @ 2026-08-12 13:33 UTC (permalink / raw) To: Marek Vasut, u-boot, git, Tom Rini; +Cc: Quentin Schulz On 8/12/26 15:08, Marek Vasut wrote: > On 8/12/26 1:28 PM, Michal Simek wrote: >> >> >> On 8/12/26 11:11, Marek Vasut wrote: >>> On 8/12/26 8:35 AM, Michal Simek wrote: >>>> >>>> >>>> On 8/10/26 13:37, Michal Simek wrote: >>>>> Versal Gen 2 and especially march-versal2 is not covered by own fragment >>>>> that's why add it. >>>>> >>>>> Signed-off-by: Michal Simek <michal.simek@amd.com> >>>>> --- >>>>> >>>>> MAINTAINERS | 6 ++++++ >>>>> 1 file changed, 6 insertions(+) >>>>> >>>>> diff --git a/MAINTAINERS b/MAINTAINERS >>>>> index e1379a6a3e7e..eb48eea55c5a 100644 >>>>> --- a/MAINTAINERS >>>>> +++ b/MAINTAINERS >>>>> @@ -821,6 +821,12 @@ F: drivers/spi/cadence_ospi_versal.c >>>>> F: drivers/watchdog/xilinx_wwdt.c >>>>> N: (?<!uni)versal >>>>> +ARM VERSAL GEN 2 >>>>> +M: Michal Simek <michal.simek@amd.com> >>>>> +S: Maintained >>>>> +T: git https://git.u-boot-project.org/u-boot/custodians/u-boot- >>>>> microblaze.git >>>>> +F: arch/arm/mach-versal2/ >>> Use "N: versal" instead for the globbing match . >>> >>> In fact, "N: (?<!uni)versal" above does not work ? >> >> nope >> >> 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. Thanks, Michal ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 2026-08-12 13:33 ` Michal Simek @ 2026-08-12 17:51 ` Marek Vasut via U-Boot 2026-08-13 6:28 ` Michal Simek 0 siblings, 1 reply; 8+ messages in thread From: Marek Vasut via U-Boot @ 2026-08-12 17:51 UTC (permalink / raw) To: Michal Simek, u-boot, git, Tom Rini; +Cc: Quentin Schulz 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 ? ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 2026-08-12 17:51 ` Marek Vasut via U-Boot @ 2026-08-13 6:28 ` Michal Simek 0 siblings, 0 replies; 8+ messages in thread From: Michal Simek @ 2026-08-13 6:28 UTC (permalink / raw) To: Marek Vasut, u-boot, git, Tom Rini; +Cc: Quentin Schulz 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2026-08-13 6:28 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-08-10 11:37 [PATCH] MAINTAINERS: Add missing fragment for Versal Gen 2 Michal Simek 2026-08-12 6:35 ` Michal Simek 2026-08-12 9:11 ` Marek Vasut via U-Boot 2026-08-12 11:28 ` Michal Simek 2026-08-12 13:08 ` Marek Vasut via U-Boot 2026-08-12 13:33 ` Michal Simek 2026-08-12 17:51 ` Marek Vasut via U-Boot 2026-08-13 6:28 ` Michal Simek
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.