Peter Maydell <peter.mayd...@linaro.org> writes: > On 7 November 2018 at 19:39, Thomas Huth <th...@redhat.com> wrote: >> On 2018-11-07 17:08, Markus Armbruster wrote: >>> Peter Maydell <peter.mayd...@linaro.org> writes: >>> >>>> On 6 November 2018 at 18:52, Thomas Huth <th...@redhat.com> wrote: >>>>> On 2018-11-06 19:49, Philippe Mathieu-Daudé wrote: >>>>>> On 6/11/18 19:17, Thomas Huth wrote: >>>>>>> There is no active maintainer, but since Peter is picking up >>>>>>> patches via qemu-...@nongnu.org, I think we could at least use >>>>>>> "Odd Fixes" as status here. >>>>>> >>>>>> This looks more as "Orphan" to me... >>>>> >>>>> I'll leave it up to Peter for the final decision... >>>> >>>> I think we're not very consistent[*] in our usage of the various >>>> statuses in the MAINTAINERS file. I guess "Odd Fixes" makes >>>> sense in that, well, if you send a patch to this >>>> code and cc me I'll review it and put it in the tree. (This >>>> is true of any of the arm boards we have.) >>>> >>>> [*] We have one thing tagged Orphan, which is bsd-user/, >>>> and some things tagged Odd Fixes with no listed maintainer, >>>> and some things tagged Odd Fixes which are in practice more >>>> like Orphan (for instance sh4), and we list "fpu/" as >>>> Odd Fixes despite having given it a pretty thorough >>>> overhaul very recently, and so on... >>>> >>>> If you wanted a mechanizable rule, you could try something >>>> like "every file which is in status Odd Fixes or better >>>> must list with M: at least one named individual who has >>>> submitted a pull request in the last nine months" :-) >>> >>> Sounds like an excellent idea to me! >> >> Well, for me "odd fixes" means that there is someone around who might >> pick up the patch and throw it into a PULL request.
That someone should be listed with M:. >> "Orphan" means it's >> mostly in vain to send patches for this subsystem, since there is nobody >> going to pick up your patch. So for the ARM boards, I'd say that "odd >> fixes" is a better choice, since Peter still picks up most of the >> patches via the "L: qemu-...@nongnu.org" (big thanks for this, by the way!). > > I think there should be a named person, though. A list > isn't a maintainer (at least not for the big whole > subsystem lists we have). Agree.