Hi Tom, On Thu, Jun 5, 2025 at 1:12 PM Tom Rini <tr...@konsulko.com> wrote: > > On Mon, Jun 02, 2025 at 03:44:48PM -0400, Greg Malysa wrote: > > > This adds missing maintainers entries for the ADI SC5xx defconfigs. > > > > Signed-off-by: Vasileios Bimpikas <vasileios.bimpi...@analog.com> > > Signed-off-by: Utsav Agarwal <utsav.agar...@analog.com> > > Signed-off-by: Arturs Artamonovs <arturs.artamon...@analog.com> > > Signed-off-by: Nathan Barrett-Morrison <nathan.morri...@timesys.com> > > Signed-off-by: Greg Malysa <malysag...@gmail.com> > > --- > > > > MAINTAINERS | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index d62dd35a385..cee5f1a9271 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -631,6 +631,7 @@ F: arch/arm/dts/sc5* > > F: arch/arm/include/asm/arch-adi/ > > F: arch/arm/mach-sc5xx/ > > F: board/adi/ > > +F: configs/sc5* > > F: doc/device-tree-bindings/arm/adi/adi,sc5xx.yaml > > F: doc/device-tree-bindings/clock/adi,sc5xx-clocks.yaml > > F: doc/device-tree-bindings/pinctrl/adi,adsp-pinctrl.yaml > > This doesn't also catch the other drivers and so forth added. Perhaps > the "N: " regex syntax would be helpful here, to make adding all of > those as well easier? Thanks.
Thanks for the feedback, I'll review the names again before v2 and look for any missed entries. Our names are a bit inconsistent: some adi, some adsp, some include sc5xx and some do not. In v2 of this series we could rename the defconfigs to adhere to a useful, specific naming convention, and then follow up by renaming drivers across different subsystems in a separate series so that we can have one N: line and catch them all. Do you have any tips on tracking down files that are missed? Using scripts/get_maintainer.pl with a list of what I think the files are could work but I won't know if I forgot something that way :-). Is there a particular way to invoke it that checks all files, perhaps, and reports on files that are missing supporters? Thanks, Greg
