From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [205.139.110.120]) by mx.groups.io with SMTP id smtpd.web11.35014.1599212637064164456 for ; Fri, 04 Sep 2020 02:43:57 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 205.139.110.120, mailfrom: lersek@redhat.com) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-280-QG65n-4nMcCoEobri82_CA-1; Fri, 04 Sep 2020 05:43:52 -0400 X-MC-Unique: QG65n-4nMcCoEobri82_CA-1 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 23964801AF2; Fri, 4 Sep 2020 09:43:51 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-161.ams2.redhat.com [10.36.112.161]) by smtp.corp.redhat.com (Postfix) with ESMTP id 8971D7E424; Fri, 4 Sep 2020 09:43:43 +0000 (UTC) Subject: Re: [PATCH v2 1/1] Maintainers.txt: Update reviewers of */RiscV64 To: "Chang, Abner (HPS SW/FW Technologist)" , "devel@edk2.groups.io" Cc: Andrew Fish , Leif Lindholm , Michael D Kinney , Leif Lindholm References: <20200904071913.17295-1-abner.chang@hpe.com> From: "Laszlo Ersek" Message-ID: Date: Fri, 4 Sep 2020 11:43:41 +0200 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Mimecast-Spam-Score: 0.002 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Content-Language: en-US Hi Abner, On 09/04/20 10:51, Chang, Abner (HPS SW/FW Technologist) wrote: > > >> -----Original Message----- >> From: Laszlo Ersek [mailto:lersek@redhat.com] >> Sent: Friday, September 04, 2020 4:25 PM >> To: Chang, Abner (HPS SW/FW Technologist) ; >> devel@edk2.groups.io >> Cc: Andrew Fish ; Leif Lindholm ; >> Michael D Kinney ; Leif Lindholm >> >> Subject: Re: [PATCH v2 1/1] Maintainers.txt: Update reviewers of */RiscV64 >> >> On 09/04/20 09:19, Abner Chang wrote: >>> Add reviewers for all /RiscV64 folders. >>> >>> Signed-off-by: Abner Chang >>> Cc: Andrew Fish >>> Cc: Laszlo Ersek >>> Cc: Leif Lindholm >>> Cc: Michael D Kinney >>> Cc: Leif Lindholm >>> Acked-by: Laszlo Ersek >>> --- >>> Maintainers.txt | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/Maintainers.txt b/Maintainers.txt index >>> 57cd2fc662..6a22a14796 100644 >>> --- a/Maintainers.txt >>> +++ b/Maintainers.txt >>> @@ -98,6 +98,11 @@ F: */Arm/ >>> M: Leif Lindholm >>> M: Ard Biesheuvel >>> >>> +RISCV64 >>> +F: */RiscV64/ >>> +M: Abner Chang >>> +R: Daniel Schaefer >>> + >>> EDK II Continuous Integration: >>> ------------------------------ >>> .azurepipelines/ >>> >> >> What are the changes relative to v1? > R: for Abner changed to M: for Abner >> >> (Also, I believe Leif is away at the moment, and I think we should have his >> ACK on this patch, before we merge the patch.) > Yes, I had few conversations with Leif and Mike for this in this. Wait, that reminds me... OK, I've looked up the v1 discussion now. Is there any particular reason you didn't pick up Leif's Reviewed-by from the v1 thread? Leif gave his R-b conditional on the R->M change for your name: https://edk2.groups.io/g/devel/message/64807 and you have implemented that change. So I think Leif's R-b should have been picked up. Based on -- do you have push access to edk2 at this time? Because if that's the case, then we should merge this patch now. Hmm let me see. https://github.com/orgs/tianocore/teams/edk-ii-maintainers/members Yes, you are a member of this group. https://github.com/orgs/tianocore/teams?query=@changab So I'm going to apply Leif's R-b from the v1 thread now, and then merge this patch. Thanks, Laszlo > Sure we can wait for his Ack. > Thanks >> >> Thanks >> Laszlo >