* [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine @ 2018-11-06 18:17 Thomas Huth 2018-11-06 18:49 ` Philippe Mathieu-Daudé 2018-11-08 13:44 ` Peter Maydell 0 siblings, 2 replies; 9+ messages in thread From: Thomas Huth @ 2018-11-06 18:17 UTC (permalink / raw) To: qemu-arm, Peter Maydell; +Cc: qemu-devel, Dmitry Eremin-Solenikov There is no active maintainer, but since Peter is picking up patches via qemu-arm@nongnu.org, I think we could at least use "Odd Fixes" as status here. Signed-off-by: Thomas Huth <thuth@redhat.com> --- MAINTAINERS | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 0499e11..471cf72 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -592,6 +592,12 @@ F: hw/*/pxa2xx* F: hw/misc/mst_fpga.c F: include/hw/arm/pxa.h +Sharp SL-5500 (Collie) PDA +L: qemu-arm@nongnu.org +S: Odd Fixes +F: hw/arm/collie.c +F: hw/arm/strongarm* + Stellaris M: Peter Maydell <peter.maydell@linaro.org> L: qemu-arm@nongnu.org -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine 2018-11-06 18:17 [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine Thomas Huth @ 2018-11-06 18:49 ` Philippe Mathieu-Daudé 2018-11-06 18:52 ` Thomas Huth 2018-11-08 13:44 ` Peter Maydell 1 sibling, 1 reply; 9+ messages in thread From: Philippe Mathieu-Daudé @ 2018-11-06 18:49 UTC (permalink / raw) To: Thomas Huth, qemu-arm, Peter Maydell; +Cc: Dmitry Eremin-Solenikov, qemu-devel On 6/11/18 19:17, Thomas Huth wrote: > There is no active maintainer, but since Peter is picking up > patches via qemu-arm@nongnu.org, I think we could at least use > "Odd Fixes" as status here. This looks more as "Orphan" to me... Regardless the one choosed: Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > MAINTAINERS | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 0499e11..471cf72 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -592,6 +592,12 @@ F: hw/*/pxa2xx* > F: hw/misc/mst_fpga.c > F: include/hw/arm/pxa.h > > +Sharp SL-5500 (Collie) PDA > +L: qemu-arm@nongnu.org > +S: Odd Fixes > +F: hw/arm/collie.c > +F: hw/arm/strongarm* > + > Stellaris > M: Peter Maydell <peter.maydell@linaro.org> > L: qemu-arm@nongnu.org > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine 2018-11-06 18:49 ` Philippe Mathieu-Daudé @ 2018-11-06 18:52 ` Thomas Huth 2018-11-06 18:58 ` Peter Maydell 0 siblings, 1 reply; 9+ messages in thread From: Thomas Huth @ 2018-11-06 18:52 UTC (permalink / raw) To: Philippe Mathieu-Daudé, qemu-arm, Peter Maydell Cc: Dmitry Eremin-Solenikov, qemu-devel 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-arm@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... > Regardless the one choosed: > Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Thanks, Thomas ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine 2018-11-06 18:52 ` Thomas Huth @ 2018-11-06 18:58 ` Peter Maydell 2018-11-07 16:08 ` Markus Armbruster 0 siblings, 1 reply; 9+ messages in thread From: Peter Maydell @ 2018-11-06 18:58 UTC (permalink / raw) To: Thomas Huth Cc: Philippe Mathieu-Daudé, qemu-arm, Dmitry Eremin-Solenikov, QEMU Developers On 6 November 2018 at 18:52, Thomas Huth <thuth@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-arm@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" :-) thanks -- PMM ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine 2018-11-06 18:58 ` Peter Maydell @ 2018-11-07 16:08 ` Markus Armbruster 2018-11-07 19:39 ` Thomas Huth 0 siblings, 1 reply; 9+ messages in thread From: Markus Armbruster @ 2018-11-07 16:08 UTC (permalink / raw) To: Peter Maydell Cc: Thomas Huth, Dmitry Eremin-Solenikov, qemu-arm, Philippe Mathieu-Daudé, QEMU Developers Peter Maydell <peter.maydell@linaro.org> writes: > On 6 November 2018 at 18:52, Thomas Huth <thuth@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-arm@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! ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine 2018-11-07 16:08 ` Markus Armbruster @ 2018-11-07 19:39 ` Thomas Huth 2018-11-07 19:42 ` Peter Maydell 0 siblings, 1 reply; 9+ messages in thread From: Thomas Huth @ 2018-11-07 19:39 UTC (permalink / raw) To: Markus Armbruster, Peter Maydell Cc: Dmitry Eremin-Solenikov, qemu-arm, Philippe Mathieu-Daudé, QEMU Developers On 2018-11-07 17:08, Markus Armbruster wrote: > Peter Maydell <peter.maydell@linaro.org> writes: > >> On 6 November 2018 at 18:52, Thomas Huth <thuth@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-arm@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. "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-arm@nongnu.org" (big thanks for this, by the way!). Thomas ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine 2018-11-07 19:39 ` Thomas Huth @ 2018-11-07 19:42 ` Peter Maydell 2018-11-08 7:04 ` Markus Armbruster 0 siblings, 1 reply; 9+ messages in thread From: Peter Maydell @ 2018-11-07 19:42 UTC (permalink / raw) To: Thomas Huth Cc: Markus Armbruster, Dmitry Eremin-Solenikov, qemu-arm, Philippe Mathieu-Daudé, QEMU Developers On 7 November 2018 at 19:39, Thomas Huth <thuth@redhat.com> wrote: > On 2018-11-07 17:08, Markus Armbruster wrote: >> Peter Maydell <peter.maydell@linaro.org> writes: >> >>> On 6 November 2018 at 18:52, Thomas Huth <thuth@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-arm@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. "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-arm@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). thanks -- PMM ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine 2018-11-07 19:42 ` Peter Maydell @ 2018-11-08 7:04 ` Markus Armbruster 0 siblings, 0 replies; 9+ messages in thread From: Markus Armbruster @ 2018-11-08 7:04 UTC (permalink / raw) To: Peter Maydell Cc: Thomas Huth, Dmitry Eremin-Solenikov, qemu-arm, Philippe Mathieu-Daudé, QEMU Developers Peter Maydell <peter.maydell@linaro.org> writes: > On 7 November 2018 at 19:39, Thomas Huth <thuth@redhat.com> wrote: >> On 2018-11-07 17:08, Markus Armbruster wrote: >>> Peter Maydell <peter.maydell@linaro.org> writes: >>> >>>> On 6 November 2018 at 18:52, Thomas Huth <thuth@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-arm@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-arm@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. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine 2018-11-06 18:17 [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine Thomas Huth 2018-11-06 18:49 ` Philippe Mathieu-Daudé @ 2018-11-08 13:44 ` Peter Maydell 1 sibling, 0 replies; 9+ messages in thread From: Peter Maydell @ 2018-11-08 13:44 UTC (permalink / raw) To: Thomas Huth; +Cc: qemu-arm, QEMU Developers, Dmitry Eremin-Solenikov On 6 November 2018 at 18:17, Thomas Huth <thuth@redhat.com> wrote: > There is no active maintainer, but since Peter is picking up > patches via qemu-arm@nongnu.org, I think we could at least use > "Odd Fixes" as status here. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > MAINTAINERS | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 0499e11..471cf72 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -592,6 +592,12 @@ F: hw/*/pxa2xx* > F: hw/misc/mst_fpga.c > F: include/hw/arm/pxa.h > > +Sharp SL-5500 (Collie) PDA > +L: qemu-arm@nongnu.org > +S: Odd Fixes > +F: hw/arm/collie.c > +F: hw/arm/strongarm* > + > Stellaris > M: Peter Maydell <peter.maydell@linaro.org> > L: qemu-arm@nongnu.org Applied to target-arm.next with myself added as an M: contact, thanks. -- PMM ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-11-08 13:45 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-11-06 18:17 [Qemu-devel] [PATCH] MAINTAINERS: Add an entry for the 'collie' machine Thomas Huth 2018-11-06 18:49 ` Philippe Mathieu-Daudé 2018-11-06 18:52 ` Thomas Huth 2018-11-06 18:58 ` Peter Maydell 2018-11-07 16:08 ` Markus Armbruster 2018-11-07 19:39 ` Thomas Huth 2018-11-07 19:42 ` Peter Maydell 2018-11-08 7:04 ` Markus Armbruster 2018-11-08 13:44 ` Peter Maydell
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).