From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Thu, 13 Jun 2019 08:33:55 -0700 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id DB2E5307D9D0; Thu, 13 Jun 2019 15:33:47 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-117-127.ams2.redhat.com [10.36.117.127]) by smtp.corp.redhat.com (Postfix) with ESMTP id 8885054216; Thu, 13 Jun 2019 15:33:44 +0000 (UTC) Subject: Re: [edk2-devel] [PATCH v2 0/4] Define EDKII_SERIAL_PORT_LIB_VENDOR_GUID From: "Laszlo Ersek" To: Ard Biesheuvel , Leif Lindholm , Maurice Ma , Guo Dong , Benjamin You Cc: Jian J Wang , Hao A Wu , Star Zeng , devel@edk2.groups.io, Anthony Perard , Julien Grall , Ray Ni References: <20190606131459.1464-1-anthony.perard@citrix.com> <71b7ea4d-59c2-67f8-faac-b72bca67e2bd@redhat.com> Message-ID: <59855a1a-890c-9799-9e7f-501cbbaaeee6@redhat.com> Date: Thu, 13 Jun 2019 17:33:43 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <71b7ea4d-59c2-67f8-faac-b72bca67e2bd@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.48]); Thu, 13 Jun 2019 15:33:53 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 06/13/19 17:32, Laszlo Ersek wrote: > On 06/06/19 15:14, Anthony PERARD wrote: >> The macro SERIAL_DXE_FILE_GUID is been defined twice and the GUID is been used >> once without defining the macro. This patch series define a macro in >> MdeModulePkg where the SerialDxe is, and replace all other use by this new one, >> EDKII_SERIAL_PORT_LIB_VENDOR_GUID. >> >> Note that I haven't build/test those changes, but I have test the first patch >> by applying a similar change to a patch series I'm working on. >> >> Patch series available in this git branch: >> https://xenbits.xen.org/git-http/people/aperard/ovmf.git br.serial-dxe-guid-v2 >> >> Thanks. >> >> Anthony PERARD (4): >> MdeModulePkg: Introduce EDKII_SERIAL_PORT_LIB_VENDOR_GUID >> ArmVirtPkg/PlatformBootManagerLib: Use >> EDKII_SERIAL_PORT_LIB_VENDOR_GUID from MdeModulePkg >> ArmPkg/PlatformBootManagerLib: Use EDKII_SERIAL_PORT_LIB_VENDOR_GUID >> from MdeModulePkg > > Leif, Ard, can you please review patch #3? Ooops, nevermind, I missed for a moment Ard's ACK for the whole series: http://mid.mail-archive.com/CAKv+Gu9jx+xgQBfD0DOc+ZLJ1XqKb6=TJDBh09_LUAwdSXs-hg@mail.gmail.com However, > >> UefiPayloadPkg/PlatformBootManagerLib: Use >> EDKII_SERIAL_PORT_LIB_VENDOR_GUID from MdeModulePkg > > Maurice, Guo, Benjamin, can you please review patch #4? the UefiPayloadPkg patch still needs a maintainer A-b or R-b. Thanks Laszlo