diff mbox series

MAINTAINERS: add Bertrand to the ARM reviewers

Message ID alpine.DEB.2.22.394.2111041357580.284830@ubuntu-linux-20-04-desktop (mailing list archive)
State Superseded
Headers show
Series MAINTAINERS: add Bertrand to the ARM reviewers | expand

Commit Message

Stefano Stabellini Nov. 4, 2021, 9 p.m. UTC
Acked-by: Bertrand Marquis <bertrand.marquis@arm.com>
Acked-by: Stefano Stabellini <sstabellini@kernel.org>

Comments

Julien Grall Nov. 4, 2021, 10:59 p.m. UTC | #1
Hi Stefano,

On 04/11/2021 21:00, Stefano Stabellini wrote:
> 
> Acked-by: Bertrand Marquis <bertrand.marquis@arm.com>

I am not sure where the Acked-by is coming from. Can Bertrand confirm on 
the ML that the is happy to be a reviewer?

> Acked-by: Stefano Stabellini <sstabellini@kernel.org>

There is a missing signed-off-by on the patch.

I am happy with the proposal, although I will wait for the two remarks 
to be resolved before giving my acked-by.

Cheers,
Stefano Stabellini Nov. 4, 2021, 11:04 p.m. UTC | #2
On Thu, 4 Nov 2021, Julien Grall wrote:
> Hi Stefano,
> 
> On 04/11/2021 21:00, Stefano Stabellini wrote:
> > 
> > Acked-by: Bertrand Marquis <bertrand.marquis@arm.com>
> 
> I am not sure where the Acked-by is coming from. Can Bertrand confirm on the
> ML that the is happy to be a reviewer?

Yes, he gave it to me verbally but I was assuming Bertrand would also
confirm on xen-devel.


> > Acked-by: Stefano Stabellini <sstabellini@kernel.org>
> 
> There is a missing signed-off-by on the patch.
> 
> I am happy with the proposal, although I will wait for the two remarks to be
> resolved before giving my acked-by.

Ahah oh man I somehow forgot it
Bertrand Marquis Nov. 5, 2021, 8:35 a.m. UTC | #3
Hi Julien,

> On 4 Nov 2021, at 22:59, Julien Grall <julien@xen.org> wrote:
> 
> Hi Stefano,
> 
> On 04/11/2021 21:00, Stefano Stabellini wrote:
>> Acked-by: Bertrand Marquis <bertrand.marquis@arm.com>
> 
> I am not sure where the Acked-by is coming from. Can Bertrand confirm on the ML that the is happy to be a reviewer?

I can confirm I am happy to be a reviewer :-)

So I can say:
Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>

> 
>> Acked-by: Stefano Stabellini <sstabellini@kernel.org>
> 
> There is a missing signed-off-by on the patch.
> 
> I am happy with the proposal, although I will wait for the two remarks to be resolved before giving my acked-by.

Then it might be easier to give my ack by on the v2 signed off by Stefano.

Cheers
Bertrand

> 
> Cheers,
> 
> -- 
> Julien Grall
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 9be4ad653a..4956db1011 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -232,6 +232,7 @@  ARM (W/ VIRTUALISATION EXTENSIONS) ARCHITECTURE
 M:	Stefano Stabellini <sstabellini@kernel.org>
 M:	Julien Grall <julien@xen.org>
 R:	Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>
+R:	Bertrand Marquis <bertrand.marquis@arm.com>
 S:	Supported
 L:	xen-devel@lists.xenproject.org
 F:	docs/misc/arm/