[PATCH] MAINTAINERS: add Bertrand to the ARM reviewers

Stefano Stabellini posted 1 patch 3 years ago
Test gitlab-ci failed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/alpine.DEB.2.22.394.2111041357580.284830@ubuntu-linux-20-04-desktop
There is a newer version of this series
[PATCH] MAINTAINERS: add Bertrand to the ARM reviewers
Posted by Stefano Stabellini 3 years ago

Acked-by: Bertrand Marquis <bertrand.marquis@arm.com>
Acked-by: Stefano Stabellini <sstabellini@kernel.org>

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/

Re: [PATCH] MAINTAINERS: add Bertrand to the ARM reviewers
Posted by Julien Grall 3 years ago
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,

-- 
Julien Grall

Re: [PATCH] MAINTAINERS: add Bertrand to the ARM reviewers
Posted by Stefano Stabellini 3 years ago
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

Re: [PATCH] MAINTAINERS: add Bertrand to the ARM reviewers
Posted by Bertrand Marquis 3 years ago
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