1. Patchew
  2. QEMU
  3. [PATCH v36 00/17] QEMU AVR 8 bit cores
  4. View patch

[PATCH v36 17/17] target/avr: Update MAINTAINERS file

Michael Rolnik posted 17 patches 5 years, 11 months ago
Maintainers: "Marc-André Lureau" <marcandre.lureau@redhat.com>, "Philippe Mathieu-Daudé" <philmd@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>
There is a newer version of this series
[PATCH v36 17/17] target/avr: Update MAINTAINERS file
Posted by Michael Rolnik 5 years, 11 months ago 
Include AVR maintaners in MAINTAINERS file

Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
---
 MAINTAINERS | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 5e5e3e52d6..ad2d9dd04a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -163,6 +163,15 @@ S: Maintained
 F: hw/arm/smmu*
 F: include/hw/arm/smmu*
 
+AVR TCG CPUs
+M: Michael Rolnik <mrolnik@gmail.com>
+S: Maintained
+F: target/avr/
+F: hw/misc/avr_mask.c
+F: hw/char/avr_usart.c
+F: hw/timer/avr_timer16.c
+F: hw/avr/
+
 CRIS TCG CPUs
 M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
 S: Maintained
-- 
2.17.2 (Apple Git-113)
Re: [PATCH v36 17/17] target/avr: Update MAINTAINERS file
Posted by Aleksandar Markovic 5 years, 11 months ago 
On Sunday, November 24, 2019, Michael Rolnik <mrolnik@gmail.com> wrote:

> Include AVR maintaners in MAINTAINERS file
>
> Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
> ---
>  MAINTAINERS | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5e5e3e52d6..ad2d9dd04a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -163,6 +163,15 @@ S: Maintained
>  F: hw/arm/smmu*
>  F: include/hw/arm/smmu*
>
> +AVR TCG CPUs
> +M: Michael Rolnik <mrolnik@gmail.com>
> +S: Maintained
> +F: target/avr/
> +F: hw/misc/avr_mask.c
> +F: hw/char/avr_usart.c
> +F: hw/timer/avr_timer16.c
> +F: hw/avr/
> +


If Cleber and Eduardo don't object, please add file:

tests/acceptance/machine_avr6.py

to the list. This means that the file would have a sort of "dual"
maintainrship: Cleber and Eduardo would take care of Python/Avocado/test
harness aspects, while you and Sarah would take care of AVR aspects.

Cleber and Eduardo, please note that AVR target will be significantly
different than others, since these CPUs are not meant to work with Linux
kernel, and there are other idiosyncracies where AVR folks would be really
needed. There is also a case of MIPS-specific Avocado test where, at least
from my point of view, dual maintainership worked well, so I think this one
would work well too.

With or without that file addition:

Reviewed-by: Aleksandar Markovic <amarkovic@wavecomp.com>



>  CRIS TCG CPUs
>  M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
>  S: Maintained
> --
> 2.17.2 (Apple Git-113)
>
>
Re: [PATCH v36 17/17] target/avr: Update MAINTAINERS file
Posted by Eduardo Habkost 5 years, 11 months ago 
On Tue, Nov 26, 2019 at 12:49:40AM +0100, Aleksandar Markovic wrote:
> On Sunday, November 24, 2019, Michael Rolnik <mrolnik@gmail.com> wrote:
> 
> > Include AVR maintaners in MAINTAINERS file
> >
> > Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
> > ---
> >  MAINTAINERS | 9 +++++++++
> >  1 file changed, 9 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 5e5e3e52d6..ad2d9dd04a 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -163,6 +163,15 @@ S: Maintained
> >  F: hw/arm/smmu*
> >  F: include/hw/arm/smmu*
> >
> > +AVR TCG CPUs
> > +M: Michael Rolnik <mrolnik@gmail.com>
> > +S: Maintained
> > +F: target/avr/
> > +F: hw/misc/avr_mask.c
> > +F: hw/char/avr_usart.c
> > +F: hw/timer/avr_timer16.c
> > +F: hw/avr/
> > +
> 
> 
> If Cleber and Eduardo don't object, please add file:
> 
> tests/acceptance/machine_avr6.py
> 
> to the list. This means that the file would have a sort of "dual"
> maintainrship: Cleber and Eduardo would take care of Python/Avocado/test
> harness aspects, while you and Sarah would take care of AVR aspects.

No objections.  Thanks!

> 
> Cleber and Eduardo, please note that AVR target will be significantly
> different than others, since these CPUs are not meant to work with Linux
> kernel, and there are other idiosyncracies where AVR folks would be really
> needed. There is also a case of MIPS-specific Avocado test where, at least
> from my point of view, dual maintainership worked well, so I think this one
> would work well too.
> 
> With or without that file addition:
> 
> Reviewed-by: Aleksandar Markovic <amarkovic@wavecomp.com>

-- 
Eduardo
Re: [PATCH v36 17/17] target/avr: Update MAINTAINERS file
Posted by Cleber Rosa 5 years, 11 months ago 

----- Original Message -----
> From: "Aleksandar Markovic" <aleksandar.m.mail@gmail.com>
> To: "Michael Rolnik" <mrolnik@gmail.com>, "Cleber Rosa" <crosa@redhat.com>, "Eduardo Habkost" <ehabkost@redhat.com>
> Cc: qemu-devel@nongnu.org, "richard henderson" <richard.henderson@linaro.org>, philmd@redhat.com, thuth@redhat.com,
> dovgaluk@ispras.ru, imammedo@redhat.com
> Sent: Monday, November 25, 2019 6:49:40 PM
> Subject: Re: [PATCH v36 17/17] target/avr: Update MAINTAINERS file
> 
> On Sunday, November 24, 2019, Michael Rolnik <mrolnik@gmail.com> wrote:
> 
> > Include AVR maintaners in MAINTAINERS file
> >
> > Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
> > ---
> >  MAINTAINERS | 9 +++++++++
> >  1 file changed, 9 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 5e5e3e52d6..ad2d9dd04a 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -163,6 +163,15 @@ S: Maintained
> >  F: hw/arm/smmu*
> >  F: include/hw/arm/smmu*
> >
> > +AVR TCG CPUs
> > +M: Michael Rolnik <mrolnik@gmail.com>
> > +S: Maintained
> > +F: target/avr/
> > +F: hw/misc/avr_mask.c
> > +F: hw/char/avr_usart.c
> > +F: hw/timer/avr_timer16.c
> > +F: hw/avr/
> > +
> 
> 
> If Cleber and Eduardo don't object, please add file:
> 
> tests/acceptance/machine_avr6.py
> 
> to the list. This means that the file would have a sort of "dual"
> maintainrship: Cleber and Eduardo would take care of Python/Avocado/test
> harness aspects, while you and Sarah would take care of AVR aspects.
> 

No objections, quite the contrary, I'd happy to be CC'd when such code
changes.

Thanks,
- Cleber.

> Cleber and Eduardo, please note that AVR target will be significantly
> different than others, since these CPUs are not meant to work with Linux
> kernel, and there are other idiosyncracies where AVR folks would be really
> needed. There is also a case of MIPS-specific Avocado test where, at least
> from my point of view, dual maintainership worked well, so I think this one
> would work well too.
> 
> With or without that file addition:
> 
> Reviewed-by: Aleksandar Markovic <amarkovic@wavecomp.com>
> 
> 
> 
> >  CRIS TCG CPUs
> >  M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
> >  S: Maintained
> > --
> > 2.17.2 (Apple Git-113)
> >
> >
>
[PATCH v36 17/17] target/avr: Update MAINTAINERS file
Posted by Aleksandar Markovic 5 years, 11 months ago 
On Sunday, November 24, 2019, Michael Rolnik <mrolnik@gmail.com> wrote:

> Include AVR maintaners in MAINTAINERS file
>
> Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
> ---
>  MAINTAINERS | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5e5e3e52d6..ad2d9dd04a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -163,6 +163,15 @@ S: Maintained
>  F: hw/arm/smmu*
>  F: include/hw/arm/smmu*
>
> +AVR TCG CPUs
> +M: Michael Rolnik <mrolnik@gmail.com>
> +S: Maintained
> +F: target/avr/
> +F: hw/misc/avr_mask.c
> +F: hw/char/avr_usart.c
> +F: hw/timer/avr_timer16.c
> +F: hw/avr/
> +


I had a strange feeling that something is missing here, and I finally
realized what that was:

R: Sarah Harris <S.E.Harris@kent.ac.uk>

https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg04225.html



 CRIS TCG CPUs
>  M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
>  S: Maintained
> --
> 2.17.2 (Apple Git-113)
>
>
Re: [PATCH v36 17/17] target/avr: Update MAINTAINERS file
Posted by Michael Rolnik 5 years, 11 months ago 
Aleksandar,

there was an email from Sarah, stating that she does not want to be a
maintainer.

On Tue, Nov 26, 2019 at 5:17 AM Aleksandar Markovic <
aleksandar.m.mail@gmail.com> wrote:

>
>
> On Sunday, November 24, 2019, Michael Rolnik <mrolnik@gmail.com> wrote:
>
>> Include AVR maintaners in MAINTAINERS file
>>
>> Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
>> ---
>>  MAINTAINERS | 9 +++++++++
>>  1 file changed, 9 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 5e5e3e52d6..ad2d9dd04a 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -163,6 +163,15 @@ S: Maintained
>>  F: hw/arm/smmu*
>>  F: include/hw/arm/smmu*
>>
>> +AVR TCG CPUs
>> +M: Michael Rolnik <mrolnik@gmail.com>
>> +S: Maintained
>> +F: target/avr/
>> +F: hw/misc/avr_mask.c
>> +F: hw/char/avr_usart.c
>> +F: hw/timer/avr_timer16.c
>> +F: hw/avr/
>> +
>
>
> I had a strange feeling that something is missing here, and I finally
> realized what that was:
>
> R: Sarah Harris <S.E.Harris@kent.ac.uk>
>
> https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg04225.html
>
>
>
>  CRIS TCG CPUs
>>  M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
>>  S: Maintained
>> --
>> 2.17.2 (Apple Git-113)
>>
>>

-- 
Best Regards,
Michael Rolnik
Re: [PATCH v36 17/17] target/avr: Update MAINTAINERS file
Posted by Aleksandar Markovic 5 years, 11 months ago 
On Tue, Nov 26, 2019 at 8:06 PM Michael Rolnik <mrolnik@gmail.com> wrote:
>
> Aleksandar,
>
> there was an email from Sarah, stating that she does not want to be a maintainer.
>

But this is for "reviewer" role, not "maintainer".

Sarah?

> On Tue, Nov 26, 2019 at 5:17 AM Aleksandar Markovic <aleksandar.m.mail@gmail.com> wrote:
>>
>>
>>
>> On Sunday, November 24, 2019, Michael Rolnik <mrolnik@gmail.com> wrote:
>>>
>>> Include AVR maintaners in MAINTAINERS file
>>>
>>> Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
>>> ---
>>>  MAINTAINERS | 9 +++++++++
>>>  1 file changed, 9 insertions(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 5e5e3e52d6..ad2d9dd04a 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -163,6 +163,15 @@ S: Maintained
>>>  F: hw/arm/smmu*
>>>  F: include/hw/arm/smmu*
>>>
>>> +AVR TCG CPUs
>>> +M: Michael Rolnik <mrolnik@gmail.com>
>>> +S: Maintained
>>> +F: target/avr/
>>> +F: hw/misc/avr_mask.c
>>> +F: hw/char/avr_usart.c
>>> +F: hw/timer/avr_timer16.c
>>> +F: hw/avr/
>>> +
>>
>>
>> I had a strange feeling that something is missing here, and I finally realized what that was:
>>
>> R: Sarah Harris <S.E.Harris@kent.ac.uk>
>>
>> https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg04225.html
>>
>>
>>
>>>  CRIS TCG CPUs
>>>  M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
>>>  S: Maintained
>>> --
>>> 2.17.2 (Apple Git-113)
>>>
>
>
> --
> Best Regards,
> Michael Rolnik
Re: [PATCH v36 17/17] target/avr: Update MAINTAINERS file
Posted by Michael Rolnik 5 years, 11 months ago 
Ah. I think Sarah was ok with reviewer role.

On Tue, Nov 26, 2019 at 9:39 PM Aleksandar Markovic <
aleksandar.m.mail@gmail.com> wrote:

> On Tue, Nov 26, 2019 at 8:06 PM Michael Rolnik <mrolnik@gmail.com> wrote:
> >
> > Aleksandar,
> >
> > there was an email from Sarah, stating that she does not want to be a
> maintainer.
> >
>
> But this is for "reviewer" role, not "maintainer".
>
> Sarah?
>
> > On Tue, Nov 26, 2019 at 5:17 AM Aleksandar Markovic <
> aleksandar.m.mail@gmail.com> wrote:
> >>
> >>
> >>
> >> On Sunday, November 24, 2019, Michael Rolnik <mrolnik@gmail.com> wrote:
> >>>
> >>> Include AVR maintaners in MAINTAINERS file
> >>>
> >>> Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
> >>> ---
> >>>  MAINTAINERS | 9 +++++++++
> >>>  1 file changed, 9 insertions(+)
> >>>
> >>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>> index 5e5e3e52d6..ad2d9dd04a 100644
> >>> --- a/MAINTAINERS
> >>> +++ b/MAINTAINERS
> >>> @@ -163,6 +163,15 @@ S: Maintained
> >>>  F: hw/arm/smmu*
> >>>  F: include/hw/arm/smmu*
> >>>
> >>> +AVR TCG CPUs
> >>> +M: Michael Rolnik <mrolnik@gmail.com>
> >>> +S: Maintained
> >>> +F: target/avr/
> >>> +F: hw/misc/avr_mask.c
> >>> +F: hw/char/avr_usart.c
> >>> +F: hw/timer/avr_timer16.c
> >>> +F: hw/avr/
> >>> +
> >>
> >>
> >> I had a strange feeling that something is missing here, and I finally
> realized what that was:
> >>
> >> R: Sarah Harris <S.E.Harris@kent.ac.uk>
> >>
> >> https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg04225.html
> >>
> >>
> >>
> >>>  CRIS TCG CPUs
> >>>  M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
> >>>  S: Maintained
> >>> --
> >>> 2.17.2 (Apple Git-113)
> >>>
> >
> >
> > --
> > Best Regards,
> > Michael Rolnik
>


-- 
Best Regards,
Michael Rolnik
Re: [PATCH v36 17/17] target/avr: Update MAINTAINERS file
Posted by Sarah Harris 5 years, 11 months ago 
Yes, I don't have time to do maintenance, but I can manage reviewing.

Kind regards,
Sarah Harris


On Tue, 26 Nov 2019 22:41:32 +0200
Michael Rolnik <mrolnik@gmail.com> wrote:

> Ah. I think Sarah was ok with reviewer role.
> 
> On Tue, Nov 26, 2019 at 9:39 PM Aleksandar Markovic <
> aleksandar.m.mail@gmail.com> wrote:
> 
> > On Tue, Nov 26, 2019 at 8:06 PM Michael Rolnik <mrolnik@gmail.com> wrote:
> > >
> > > Aleksandar,
> > >
> > > there was an email from Sarah, stating that she does not want to be a
> > maintainer.
> > >
> >
> > But this is for "reviewer" role, not "maintainer".
> >
> > Sarah?
> >
> > > On Tue, Nov 26, 2019 at 5:17 AM Aleksandar Markovic <
> > aleksandar.m.mail@gmail.com> wrote:
> > >>
> > >>
> > >>
> > >> On Sunday, November 24, 2019, Michael Rolnik <mrolnik@gmail.com> wrote:
> > >>>
> > >>> Include AVR maintaners in MAINTAINERS file
> > >>>
> > >>> Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
> > >>> ---
> > >>>  MAINTAINERS | 9 +++++++++
> > >>>  1 file changed, 9 insertions(+)
> > >>>
> > >>> diff --git a/MAINTAINERS b/MAINTAINERS
> > >>> index 5e5e3e52d6..ad2d9dd04a 100644
> > >>> --- a/MAINTAINERS
> > >>> +++ b/MAINTAINERS
> > >>> @@ -163,6 +163,15 @@ S: Maintained
> > >>>  F: hw/arm/smmu*
> > >>>  F: include/hw/arm/smmu*
> > >>>
> > >>> +AVR TCG CPUs
> > >>> +M: Michael Rolnik <mrolnik@gmail.com>
> > >>> +S: Maintained
> > >>> +F: target/avr/
> > >>> +F: hw/misc/avr_mask.c
> > >>> +F: hw/char/avr_usart.c
> > >>> +F: hw/timer/avr_timer16.c
> > >>> +F: hw/avr/
> > >>> +
> > >>
> > >>
> > >> I had a strange feeling that something is missing here, and I finally
> > realized what that was:
> > >>
> > >> R: Sarah Harris <S.E.Harris@kent.ac.uk>
> > >>
> > >> https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg04225.html
> > >>
> > >>
> > >>
> > >>>  CRIS TCG CPUs
> > >>>  M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
> > >>>  S: Maintained
> > >>> --
> > >>> 2.17.2 (Apple Git-113)
> > >>>
> > >
> > >
> > > --
> > > Best Regards,
> > > Michael Rolnik
> >
> 
> 
> -- 
> Best Regards,
> Michael Rolnik

Patchew 2.1-devel-b67e4c2

© 2016 - 2025 Red Hat, Inc.