On 24.07.2023 19:25, Simone Ballarin wrote: > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -305,6 +305,12 @@ F: xen/include/xen/libfdt/ > F: xen/include/xen/device_tree.h > F: xen/drivers/passthrough/device_tree.c > > +ECLAIR > +M: Simone Ballarin <[email protected]> > +S: Supported > +F: xen/automation/eclair_analysis/ > +F: xen/automation/scripts/eclair > + > EFI > M: Jan Beulich <[email protected]> > S: Supported
Please follow surrounding style (tab padding, not blank). It also looks as if the xen/ prefixes in the names were wrong. Finally, following Stefano's suggestion to split - decisions to make people maintainers of certain pieces of the code are usually separate from accepting any code changes / additions. Hence even if tiny, this change may want splitting out and putting last. Content- wise personally I think we want to start out with R:, not M:, but this would need discussing among committers plus maintainers of automation/. Jan
