Re: [edk2-devel] [PATCH v3 1/1] OvmfPkg: Add QemuFwCfgLibNull

2020-05-04 Thread Philippe Mathieu-Daudé
On 5/4/20 1:09 AM, Rebecca Cran wrote: Add a null implementation library for QemuFwCfgLib, in order to support building PciHostBridgeLib for bhyve. Signed-off-by: Rebecca Cran Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Philippe Mathieu-Daudé --- I think I've addressed all

Re: [edk2-devel] [PATCH edk2-platforms v3 7/8] Platform/ARM/VExpressPkg: incorporate Lan9118 driver

2020-05-04 Thread Philippe Mathieu-Daudé
On 5/4/20 1:42 PM, Philippe Mathieu-Daudé wrote: On 4/30/20 7:16 PM, Ard Biesheuvel wrote: Incorporate the Lan91x driver from EmbeddedPkg, which is only used on obsolete ARM development platforms and does not follow the UEFI driver model. This will allow us to drop if from the core EDK2

Re: [edk2-devel] [PATCH edk2-platforms v3 6/8] Platform/ARM/VExpressPkg: incorporate Lan91x driver

2020-05-04 Thread Philippe Mathieu-Daudé
On 5/4/20 1:41 PM, Philippe Mathieu-Daudé wrote: On 4/30/20 7:16 PM, Ard Biesheuvel wrote: Incorporate the Lan91x driver from EmbeddedPkg, which is only used on emulated ARM development platforms and does not follow the UEFI driver model. This will allow us to drop if from the core EDK2

Re: [edk2-devel] [PATCH edk2-platforms v3 4/8] Silicon/Synopsys/DesignWare: import eMMC DXE driver from EmbeddedPkg

2020-05-04 Thread Philippe Mathieu-Daudé
On 5/4/20 1:40 PM, Philippe Mathieu-Daudé wrote: [...] Reviewed-by: Philippe Mathieu-Daudé Oops, without acute on EDK2: Reviewed-by: Philippe Mathieu-Daude -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#58547): https://edk2

Re: [edk2-devel] [PATCH edk2-platforms v3 7/8] Platform/ARM/VExpressPkg: incorporate Lan9118 driver

2020-05-04 Thread Philippe Mathieu-Daudé
-by: Philippe Mathieu-Daudé -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#58546): https://edk2.groups.io/g/devel/message/58546 Mute This Topic: https://groups.io/mt/73380299/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https

Re: [edk2-devel] [PATCH edk2-platforms v3 6/8] Platform/ARM/VExpressPkg: incorporate Lan91x driver

2020-05-04 Thread Philippe Mathieu-Daudé
(+), 6 deletions(-) Reviewed-by: Philippe Mathieu-Daudé -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#58545): https://edk2.groups.io/g/devel/message/58545 Mute This Topic: https://groups.io/mt/73380297/21656 Group Owner: devel+ow...@edk2

Re: [edk2-devel] [PATCH edk2-platforms v3 4/8] Silicon/Synopsys/DesignWare: import eMMC DXE driver from EmbeddedPkg

2020-05-04 Thread Philippe Mathieu-Daudé
.eed5fc57fc22 --- /dev/null +++ b/Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmcDxe.c @@ -0,0 +1,693 @@ +/** @file + This file implement the MMC Host Protocol for the DesignWare eMMC. + + Copyright (c) 2014-2017, Linaro Limited. All rights reserved. + + SPDX-License-Identifier: BSD-2-Clause-Pa

Re: [edk2-devel] [edk2-platforms][PATCH 3/3] Platform/RPi: Report core clock frequency during early init

2020-05-04 Thread Philippe Mathieu-Daudé
"VideoCore: 0x%ll08X (Size 0x%ll08X)\n", mVideoCoreBase, mVideoCoreSize)); + DEBUG ((DEBUG_INFO, "Core Freq: %d MHz\n", gSerialLibCoreClockFreq / 100)); ASSERT (mSystemMemoryBase == 0); ASSERT (VirtualMemoryMap != NULL); Reviewed-by: Philippe Mathieu-Dau

Re: [edk2-devel] [PATCH edk2-platforms v2 1/2] Platform/ARM/VExpressPkg: incorporate PL180 driver

2020-04-30 Thread Philippe Mathieu-Daudé
Hi Ard, On 4/30/20 10:25 AM, Ard Biesheuvel wrote: The PL180 SD host controller driver is only used on emulated ARM platforms, uses an obsolete version of the MMC host protocol and does not adhere to the UEFI driver model. Given the above, let's just move it into VExpressPkg where it belongs.

Re: [edk2-devel] [PATCH v2 1/1] OvmfPkg: replace old EFI_D_ debug levels with new DEBUG_ ones

2020-04-29 Thread Philippe Mathieu-Daudé
Hi Laszlo, On 4/29/20 2:20 AM, Rebecca Cran wrote: In case you don't mind modifying when applying, I think Rebecca missed here "Generated mechanically with find/sed." Personally I'd also add "$ sed -i -e s/EFI_D_INFO/DEBUG_INFO/ $(git grep -l EFI_D_INFO OvmfPkg/)". Signed-off-by: Rebecca

Re: [edk2-devel] [PATCH 0/7] OvmfPkg, ArmVirtPkg: control PXE v4/v6 boot support from the QEMU cmdline

2020-04-29 Thread Philippe Mathieu-Daudé
PcdSetXX APIs.", 2015-04-10). Got it, thanks for the pointer! See also EFI_PCD_PROTOCOL in the Platform Init spec: <https://uefi.org/specifications>. Thanks! Laszlo Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Leif Lindholm Cc: Per Sundstrom Cc: Philippe Mathieu-Daudé Thanks L

Re: [edk2-devel] [PATCH v2 0/1] OvmfPkg: replace old EFI_D_ debug levels with

2020-04-29 Thread Philippe Mathieu-Daudé
On 4/29/20 2:20 AM, Rebecca Cran wrote: Generated mechanically with find/sed. Signed-off-by: Rebecca Cran ^ I suppose this was meant for the patch description, since the description is empty. Rebecca Cran (1): OvmfPkg: replace old EFI_D_ debug levels with new DEBUG_ ones

Re: [edk2-devel] [PATCH] OvmfPkg: replace old EFI_D_ debug levels with new DEBUG_ ones

2020-04-28 Thread Philippe Mathieu-Daudé
On 4/28/20 3:38 PM, Rebecca Cran wrote: On 4/28/20 6:44 AM, Laszlo Ersek wrote: Rebecca: thanks for the contribution. Personally I'd have preferred a finer-grained approach here, but Phil removed the review burden on me. Regarding the git history, edk2 unfortunately alreay carries a bunch of

Re: [edk2-devel] [PATCH] OvmfPkg: replace old EFI_D_ debug levels with new DEBUG_ ones

2020-04-28 Thread Philippe Mathieu-Daudé
On 4/28/20 2:44 PM, Laszlo Ersek wrote: On 04/27/20 10:56, Philippe Mathieu-Daudé wrote: On 4/25/20 12:17 AM, Rebecca Cran wrote: Signed-off-by: Rebecca Cran Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel ---   OvmfPkg/AcpiPlatformDxe/EntryPoint.c  |  8 +-   OvmfPkg

Re: [edk2-devel] [PATCH 6/7] ArmVirtPkg/QemuFwCfgLib: allow UEFI_DRIVER modules

2020-04-28 Thread Philippe Mathieu-Daudé
NULL class libraries into UEFI drivers, e.g. in order to set dynamic PCDs. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Per Sundstrom Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2681 Signed-off-by: Laszlo Ersek --- ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.inf

Re: [edk2-devel] [PATCH 4/7] OvmfPkg/QemuFwCfgDxeLib: allow UEFI_DRIVER modules

2020-04-28 Thread Philippe Mathieu-Daudé
libraries into UEFI drivers, e.g. in order to set dynamic PCDs. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Per Sundstrom Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2681 Signed-off-by: Laszlo Ersek --- OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgDxeLib.inf | 2 +- 1

Re: [edk2-devel] [PATCH 7/7] ArmVirtPkg: control PXEv4 / PXEv6 boot support from the QEMU command line

2020-04-28 Thread Philippe Mathieu-Daudé
On 4/24/20 9:53 AM, Laszlo Ersek wrote: Port the DSC file changes from the similarly titled OvmfPkg patch in this series to ArmVirtPkg. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Per Sundstrom Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2681 Signed-off

Re: [edk2-devel] [PATCH 5/7] OvmfPkg: control PXEv4 / PXEv6 boot support from the QEMU command line

2020-04-28 Thread Philippe Mathieu-Daudé
/org.tianocore" prefix follows the "opt/RFQDN/" recommendation from QEMU's "docs/specs/fw_cfg.txt". Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Per Sundstrom Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2681 Signed-off-by: Laszlo Ersek

Re: [edk2-devel] [PATCH v4 2/6] OvmfPkg: Add QemuFwCfgLibNull

2020-04-28 Thread Philippe Mathieu-Daudé
On 4/27/20 7:33 PM, Rebecca Cran wrote: On 4/27/20 3:03 AM, Philippe Mathieu-Daudé wrote: +#include +#include +#include Don't you need ? Hmm, you're right. I'm not sure where I got "IndustryStandard/QemuFwCfg.h" from, though I noticed QemuFwCfg.h itself includes a n

Re: [edk2-devel] [edk2-platforms][PATCH] Platform/RaspberryPi: Fix comments in Bcm2836Pwm.h

2020-04-28 Thread Philippe Mathieu-Daudé
On 4/28/20 7:08 AM, GH Cao wrote: The original comment incorrectly named the PWM controller as Power Management Contoller, this commit fixes it. Signed-off-by: GH Cao --- Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836Pwm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [edk2-devel] [PATCH v4 2/6] OvmfPkg: Add QemuFwCfgLibNull

2020-04-27 Thread Philippe Mathieu-Daudé
Hi Rebecca, On 4/24/20 8:29 PM, Rebecca Cran wrote: Add a null implementation library for QemuFwCfgLib, in order to support building PciHostBridgeLib for bhyve. Signed-off-by: Rebecca Cran --- .../Library/QemuFwCfgLib/QemuFwCfgLibNull.inf | 37

Re: [edk2-devel] [PATCH] OvmfPkg: replace old EFI_D_ debug levels with new DEBUG_ ones

2020-04-27 Thread Philippe Mathieu-Daudé
On 4/25/20 12:17 AM, Rebecca Cran wrote: Signed-off-by: Rebecca Cran Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel --- OvmfPkg/AcpiPlatformDxe/EntryPoint.c | 8 +- OvmfPkg/AcpiPlatformDxe/PciDecoding.c | 10 +-- OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c | 32

Re: [edk2-devel] [PATCH] ShellPkg: smbiosview - print field values as unsigned integers

2020-04-27 Thread Philippe Mathieu-Daudé
Hi Rebecca, On 4/25/20 12:55 AM, Rebecca Cran wrote: This prevents overflow when printing DWORD fields such as the type 17 tables's extended DIMM size. Similar issue with type 14: ShellPrintEx(-1,-1,L"ItemType %d: %d\n", Index + 1, Struct->Type14->Group[Index].ItemType);

Re: [edk2-devel] [PATCH 1/7] OvmfPkg: introduce QemuFwCfgSimpleParserLib

2020-04-24 Thread Philippe Mathieu-Daudé
Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2681 Signed-off-by: Laszlo Ersek --- OvmfPkg/OvmfPkg.dec | 4 + OvmfPkg/OvmfPkgIa32.dsc | 1 + OvmfPkg/OvmfPkgIa32X64.dsc

Re: [edk2-devel] [PATCH 3/7] OvmfPkg/PlatformPei: use QemuFwCfgParseBool in UPDATE_BOOLEAN_PCD_FROM_...

2020-04-24 Thread Philippe Mathieu-Daudé
previously. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Per Sundstrom Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2681 Signed-off-by: Laszlo Ersek --- OvmfPkg/PlatformPei/Platform.c | 47 +--- 1 file changed, 2 insertions(+), 45 deletions(-) diff

Re: [edk2-devel] [PATCH 2/7] OvmfPkg/PlatformPei: parse "X-PciMmio64Mb" with QemuFwCfgSimpleParserLib

2020-04-24 Thread Philippe Mathieu-Daudé
previously. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Per Sundstrom Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2681 Signed-off-by: Laszlo Ersek --- OvmfPkg/PlatformPei/PlatformPei.inf | 1 + OvmfPkg/PlatformPei/MemDetect.c | 36

Re: [edk2-devel] [PATCH 1/1] BaseTools: convert diff.order to LF-only

2020-04-23 Thread Philippe Mathieu-Daudé
On 4/22/20 5:46 PM, Leif Lindholm wrote: SetupGit.py sets the git config option diff.orderFile to {edk2 directory}/BaseTools/Conf/diff.order, to override the default order in which files are shown in a diff/patch/whatever. This is in imitation of what is done manually in Laszlo's Unkempt Guide.

Re: [edk2-devel] [PATCH edk2-platforms 1/1] Silicon/NXP: rework IoAccessLib to use single function pointer struct

2020-04-23 Thread Philippe Mathieu-Daudé
On 4/23/20 12:09 PM, Leif Lindholm wrote: For whatever reason, the function pointer structs (and their retrieval functions) were split up on data size boundary - separate for 16-bit, 32-bit, and 64-bit. This makes the already tedious boilerplate require to deal with these hardware quirks even

Re: [edk2-devel] [PATCH v2 1/5] OvmfPkg/OvmfXen: Remove DEBUG_ON_SERIAL_PORT

2020-04-23 Thread Philippe Mathieu-Daudé
On 4/23/20 11:53 AM, Anthony PERARD wrote: Remove support for DEBUG_ON_SERIAL_PORT because OvmfXen can't be build with it due to a circular dependency: DebugLib: BaseDebugLibSerialPort -> SerialPortLib : XenConsoleSerialPortLib -> XenHypercallLib : XenHypercallLib ->

Re: [edk2-devel] [PATCH v2 3/5] OvmfPkg/PlatformDebugLibIoPort: factor out debug port detection

2020-04-23 Thread Philippe Mathieu-Daudé
On 4/23/20 11:53 AM, Anthony PERARD wrote: Factor out debug port detection in PlatformDebugLibIoPort. Signed-off-by: Anthony PERARD --- .../PlatformDebugLibIoPort.inf| 1 + .../PlatformRomDebugLibIoPort.inf | 1 + .../PlatformDebugLibIoPort/DebugLibDetect.h

Re: [edk2-devel] [PATCH 5/6] OvmfPkg/ResetSystemLib: rename to BaseResetSystemLib

2020-04-21 Thread Philippe Mathieu-Daudé
SC diffing simple.) Cc: Anthony Perard Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Julien Grall Cc: Philippe Mathieu-Daudé Cc: Rebecca Cran Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2675 Signed-off-by: Laszlo Ersek --- OvmfPkg/OvmfP

Re: [edk2-devel] [PATCH 0/6] OvmfPkg/ResetSystemLib: clean up, refactor, fix

2020-04-20 Thread Philippe Mathieu-Daudé
On 4/20/20 11:48 AM, Laszlo Ersek wrote: On 04/17/20 18:19, Philippe Mathieu-Daudé wrote: On 4/17/20 5:59 PM, Ard Biesheuvel wrote: On 4/17/20 5:37 PM, Laszlo Ersek wrote: Ref:    https://bugzilla.tianocore.org/show_bug.cgi?id=2675 Repo:   https://pagure.io/lersek/edk2.git Branch: rsl_cleanup

Re: [edk2-devel] [PATCH 6/6] OvmfPkg/ResetSystemLib: introduce the DxeResetSystemLib instance

2020-04-17 Thread Philippe Mathieu-Daudé
4+0x5f/0x1a0 ? entry_SYSCALL_64_after_hwframe+0x44/0xa9 ---[ end trace d5bee708166d198b ]--- efi: efi_reset_system() buggy! Reboot through BIOS Cc: Anthony Perard Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Julien Grall Cc: Philippe Mathieu-Daudé Cc: Rebecca Cran Ref: https://bugzilla.tianocore.o

Re: [edk2-devel] [PATCH 0/6] OvmfPkg/ResetSystemLib: clean up, refactor, fix

2020-04-17 Thread Philippe Mathieu-Daudé
t us add a simple bhyve-specific instance (later), and also allows us to fix a long time dormant bug (now). Cc: Anthony Perard Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Julien Grall Cc: Philippe Mathieu-Daudé Cc: Rebecca Cran Thanks, Laszlo Laszlo Ersek (6):    OvmfPkg/ResetSystemLib: wrap lo

Re: [edk2-devel] [PATCH 4/6] OvmfPkg/ResetSystemLib: factor out ResetShutdown()

2020-04-17 Thread Philippe Mathieu-Daudé
"reset -s" UEFI shell command, on both i440fx and q35. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Philippe Mathieu-Daudé Cc: Rebecca Cran Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2675 Signed-off-by: Laszlo Ersek --- OvmfPkg/Library/ResetSystemLib/ResetSystemLib.inf | 1

Re: [edk2-devel] [PATCH 3/6] OvmfPkg/ResetSystemLib: improve coding style in ResetSystem()

2020-04-17 Thread Philippe Mathieu-Daudé
", 2019-04-28.) Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Philippe Mathieu-Daudé Cc: Rebecca Cran Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2675 Signed-off-by: Laszlo Ersek --- OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c | 6 +++--- 1 file changed, 3 insertions(+), 3 de

Re: [edk2-devel] [PATCH 2/6] OvmfPkg/ResetSystemLib: clean up library dependencies

2020-04-17 Thread Philippe Mathieu-Daudé
: Philippe Mathieu-Daudé Cc: Rebecca Cran Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2675 Signed-off-by: Laszlo Ersek --- OvmfPkg/Library/ResetSystemLib/ResetSystemLib.inf | 2 ++ OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c | 15 --- 2 files changed, 10 insert

Re: [edk2-devel] [PATCH 1/6] OvmfPkg/ResetSystemLib: wrap long lines

2020-04-17 Thread Philippe Mathieu-Daudé
On 4/17/20 5:37 PM, Laszlo Ersek wrote: Wrap the source code and the INF file at 79 characters. Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Philippe Mathieu-Daudé Cc: Rebecca Cran Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2675 Signed-off-by: Laszlo Ersek --- OvmfPkg/Library

Re: [edk2-devel] [PATCH] MdeModulePkg/NonDiscoverablePciDeviceDxe: use standard PCI_MAX_BAR macro

2020-04-15 Thread Philippe Mathieu-Daudé
e value and the type remain unchanged (the standard PCI_MAX_BAR macro expands to 0x0006). "NonDiscoverablePciDeviceIo.h" gets the definition of PCI_MAX_BAR via its existent #include directive. Build-tested only. Cc: Ard Biesheuvel Cc: Hao A Wu Cc: Jian J Wang Cc: Philippe Mathieu

Re: [edk2-devel] [PATCH] OvmfPkg: supply missing lib class declarations in the DEC file

2020-04-07 Thread Philippe Mathieu-Daudé
) - VirtioMmioDeviceLib (one instance: VirtioMmioDeviceLib) Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Philippe Mathieu-Daudé Cc: Sean Brogan Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2662 Signed-off-by: Laszlo Ersek Reviewed-by: Philippe Mathieu-Daude --- Notes: Repo: https

Re: [edk2-devel] [PATCH] OvmfPkg/X86QemuLoadImageLib: fix "unused variable" error in X64 DXE builds

2020-03-06 Thread Philippe Mathieu-Daudé
Cc: Jordan Justen Cc: Philippe Mathieu-Daudé Fixes: 7c47d89003a6f8f7f6f0ce8ca7d3e87c630d14cc Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2572 Signed-off-by: Laszlo Ersek Reviewed-by: Philippe Mathieu-Daude Tested-by: Philippe Mathieu-Daude --- Notes: Ard, if you get to it first,

Re: [edk2-devel] [edk2-platforms][PATCH 1/1] Platform/RaspberryPi/RPi4: gain 2MB of RAM back

2020-03-06 Thread Philippe Mathieu-Daudé
On 3/5/20 11:46 PM, Andrei Warkentin wrote: From: Andrei Warkentin The RPi4 TF-A is much smaller than RPi3 TF-A, and doesn't need an extra 2MB region. The note ...: ---vvv--- Note: this depends on the edk2 ArmPlatformPkg/PrePi: fix IS_XIP. ---^^^--- Signed-off-by: Andrei Warkentin

Re: [edk2-devel] [edk2-platforms][PATCH 1/2] Platform/RaspberryPi/Drivers/ConfigDxe: fix bug in 3GB RAM logic

2020-03-05 Thread Philippe Mathieu-Daudé
On 3/5/20 12:38 AM, Pete Batard wrote: On 2020.03.04 22:31, Andrei Warkentin wrote: The original change*** used positive logic (PcdPi4GBEnabled), while the upstreamed change uses negative logic (PcdRamLimitTo3GB), which requires an additional condition, or it blows up on 1GiB and 2GiB boards.

Re: [edk2-devel] [PATCH 0/2] OvmfPkg, ArmVirtPkg: sync Timeout with PcdPlatformBootTimeOut

2020-03-04 Thread Philippe Mathieu-Daudé
Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Leif Lindholm Cc: Philippe Mathieu-Daudé Thanks Laszlo Laszlo Ersek (2): OvmfPkg/PlatformBootManagerLib: sync Timeout with PcdPlatformBootTimeOut ArmVirtPkg/PlatformBootManagerLib: sync Timeout with PcdPlatformBootTimeOut ArmVirtPkg/

Re: [edk2-devel] [PATCH v2 00/16] OvmfPkg: support VCPU hotplug with -D SMM_REQUIRE

2020-03-04 Thread Philippe Mathieu-Daudé
On 3/2/20 8:59 PM, Laszlo Ersek wrote: On 02/26/20 23:11, Laszlo Ersek wrote: Supersedes: <20200223172537.28464-1-ler...@redhat.com> Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=1512 Repo: https://github.com/lersek/edk2.git Branch: vcpu_hotplug_smm_bz_1512_v2 V1 was

Re: [edk2-devel] [PATCH v2 09/16] OvmfPkg/CpuHotplugSmm: add function for collecting CPUs with events

2020-03-02 Thread Philippe Mathieu-Daudé
block. Cc: Ard Biesheuvel Cc: Igor Mammedov Cc: Jiewen Yao Cc: Jordan Justen Cc: Michael Kinney Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1512 Signed-off-by: Laszlo Ersek Acked-by: Ard Biesheuvel --- Notes: v2: - Pick up Ard's Acked-by, which

Re: [edk2-devel] [PATCH v2 03/16] OvmfPkg: clone SmmCpuPlatformHookLib from UefiCpuPkg

2020-03-02 Thread Philippe Mathieu-Daudé
: Jordan Justen Cc: Michael Kinney Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1512 Signed-off-by: Laszlo Ersek Acked-by: Ard Biesheuvel --- Notes: v2: - Pick up Ard's Acked-by, which is conditional on approval from Intel reviewers on Cc

Re: [edk2-devel] [PATCH v2 01/16] MdeModulePkg/PiSmmCore: log SMM image start failure

2020-03-02 Thread Philippe Mathieu-Daudé
%r\n" Do similarly in the SMM Core (update the message slightly). Cc: Ard Biesheuvel Cc: Eric Dong Cc: Hao A Wu Cc: Igor Mammedov Cc: Jian J Wang Cc: Jiewen Yao Cc: Jordan Justen Cc: Michael Kinney Cc: Philippe Mathieu-Daudé Cc: Ray Ni Ref: https://bugzilla.tianocore.org/show_bug.c

Re: [edk2-devel] [PATCH v2 14/16] OvmfPkg: clone CpuS3DataDxe from UefiCpuPkg

2020-03-02 Thread Philippe Mathieu-Daudé
Justen Cc: Michael Kinney Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1512 Signed-off-by: Laszlo Ersek Acked-by: Ard Biesheuvel --- Notes: v2: - Pick up Ard's Acked-by, which is conditional on approval from Intel reviewers on Cc. (I'd like t

Re: [edk2-devel] [PATCH v2 07/16] OvmfPkg/CpuHotplugSmm: add hotplug register block helper functions

2020-03-02 Thread Philippe Mathieu-Daudé
encounter an internal failure. Cc: Ard Biesheuvel Cc: Igor Mammedov Cc: Jiewen Yao Cc: Jordan Justen Cc: Michael Kinney Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1512 Signed-off-by: Laszlo Ersek Acked-by: Ard Biesheuvel --- Notes: v2: - Pic

Re: [edk2-devel] [PATCH v2 08/16] OvmfPkg/CpuHotplugSmm: define the QEMU_CPUHP_CMD_GET_ARCH_ID macro

2020-03-02 Thread Philippe Mathieu-Daudé
dd a macro for this command value, because we'll need it later. At the same time, add a sanity check for the modern hotplug interface to CpuHotplugSmm. Cc: Ard Biesheuvel Cc: Igor Mammedov Cc: Jiewen Yao Cc: Jordan Justen Cc: Michael Kinney Cc: Philippe Mathieu-Daudé Ref: https://bugzilla.tia

Re: [edk2-devel] [edk2-platforms][PATCH 01/15] Platform/RPi: Move DW USB base address to Silicon

2020-03-02 Thread Philippe Mathieu-Daudé
On 2/28/20 11:38 AM, Pete Batard wrote: The official BCM2835 ARM Peripherals guide lists the DW USB controller as standard SoC device. Treat is as such by adding its base address to Silicon. Signed-off-by: Pete Batard --- Platform/RaspberryPi/Drivers/DwUsbHostDxe/DwUsbHostDxe.c| 3 ++-

Re: [edk2-devel] [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs

2020-02-28 Thread Philippe Mathieu-Daudé
. The first would increase inconsistency, the second would make "PatchCheck.py" yell. Convert these files to CRLF uniformly (and while at it, expand hard TABS, because "PatchCheck.py" catches those too). Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Philippe Mathieu-Daudé Thanks, Las

Re: [edk2-devel] [PATCH v4 02/11] ArmVirtPkg/PlatformPeiLib: make PcdLib dependency explicit in .INF

2020-02-27 Thread Philippe Mathieu-Daudé
On 2/27/20 3:40 PM, Ard Biesheuvel wrote: We currently include PcdLib.h in PlatformPeiLib, without declaring this dependency in its .INF description. Since all the PCDs we use resolve to fixed type in practice, this does not really matter at the moment, but since we will be adding dynamic PCD

Re: [edk2-devel] [PATCH edk2-stable202002] OvmfPkg/QemuVideoDxe: unbreak "secondary-vga" and "bochs-display" support

2020-02-26 Thread Philippe Mathieu-Daudé
On 2/25/20 11:02 PM, Laszlo Ersek wrote: On 02/25/20 21:51, Philippe Mathieu-Daudé wrote: Hi Laszlo, On 2/24/20 6:17 PM, Laszlo Ersek wrote: In edk2 commit 333f32ec23dd, QemuVideoDxe gained support for QEMU's "secondary-vga" device model (originally introduced in QEMU commit 63

Re: [edk2-devel] [PATCH edk2-stable202002] OvmfPkg/QemuVideoDxe: unbreak "secondary-vga" and "bochs-display" support

2020-02-25 Thread Philippe Mathieu-Daudé
6 -- even if the device wasn't actually VGA compatible. Restrict the IO Ports requirement to VGA compatible devices. Cc: Ard Biesheuvel Cc: Gerd Hoffmann Cc: Jordan Justen Cc: Marc W Chen Cc: Philippe Mathieu-Daudé Fixes: 662bd0da7fd77e4d2cf9ef4a78015af5cad7d9db Ref: https://bugzilla.tianocore.o

Re: [edk2-devel] [RFC PATCH 1/1] MdeModulePkg/PiDxeS3BootScriptLib: Use SafeIntLib to avoid truncation

2020-02-13 Thread Philippe Mathieu-Daudé
On 2/13/20 7:29 PM, Philippe Mathieu-Daude wrote: Math expressions written in terms of SafeIntLib function calls are easily readable, making review trivial. Convert the truncation checks added by commit 322ac05f8 to SafeIntLib calls. Cc: Jian J Wang Cc: Hao A Wu Cc: Eric Dong Suggested-by:

[edk2-devel] [RFC PATCH 0/1] MdeModulePkg/PiDxeS3BootScriptLib: Use SafeIntLib to avoid truncation

2020-02-13 Thread Philippe Mathieu-Daudé
which have pretty readable calls. This is my try at using it. Regards, Phil. Cc: Jian J Wang Cc: Hao A Wu Cc: Eric Dong Cc: Laszlo Ersek Philippe Mathieu-Daudé (1): MdeModulePkg/PiDxeS3BootScriptLib: Use SafeIntLib to avoid truncation .../DxeS3BootScriptLib.inf

[edk2-devel] [RFC PATCH 1/1] MdeModulePkg/PiDxeS3BootScriptLib: Use SafeIntLib to avoid truncation

2020-02-13 Thread Philippe Mathieu-Daudé
Math expressions written in terms of SafeIntLib function calls are easily readable, making review trivial. Convert the truncation checks added by commit 322ac05f8 to SafeIntLib calls. Cc: Jian J Wang Cc: Hao A Wu Cc: Eric Dong Suggested-by: Laszlo Ersek Signed-off-by: Philippe Mathieu-Daude

Re: [edk2-devel] [PATCH 8/9] SecurityPkg/DxeImageVerificationLib: plug Data leak in IsForbiddenByDbx()(CVE-2019-14575)

2020-02-13 Thread Philippe Mathieu-Daudé
s = gRT->GetVariable (EFI_IMAGE_SECURITY_DATABASE1, , NULL, , (VOID *) Data); if (EFI_ERROR (Status)) { -return IsForbidden; +goto Done; } // Reviewed-by: Philippe Mathieu-Daudé -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online

Re: [edk2-devel] [PATCH 1/9] SecurityPkg/DxeImageVerificationLib: Fix memory leaks(CVE-2019-14575)

2020-02-13 Thread Philippe Mathieu-Daudé
NT8 *) DbxList + SiglistHeaderSize); CertHashCount = (DbxList->SignatureListSize - SiglistHeaderSize) / DbxList->SignatureSize; Reviewed-by: Philippe Mathieu-Daudé -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#5436

Re: [edk2-devel] [PATCH v3 00/78] Fix a bunch of typos (part 1)

2020-02-11 Thread Philippe Mathieu-Daudé
On 2/10/20 11:54 PM, Laszlo Ersek wrote: On 02/07/20 11:51, Laszlo Ersek wrote: On 02/07/20 02:07, Philippe Mathieu-Daude wrote: Around February of this year, Antoine posted various patches fixing a huge quantity of typos, over 8000+ lines (such [1]). This series salvages a third of his work.

Re: [edk2-devel] [PATCH 1/2] IntelFsp2Pkg/GenCfgOpt.py: Coverity scan flags issues.

2020-02-10 Thread Philippe Mathieu-Daudé
On 2/10/20 4:53 AM, Chiu, Chasel wrote: REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1706 Issue was: invalid_operation: Invalid operation on null-like value "Base". Fixed it by initializing Base to 0 before entering while loop. Test: Compared script output before and after this patch

[edk2-devel] [PATCH v3 73/78] SourceLevelDebugPkg: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Correctly write 'definitions' in comments and documentation. Cc: Hao A Wu Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Hao A Wu Signed-off-by: Philippe Mathieu-Daude --- SourceLevelDebugPkg/Include/Ia32/ProcessorContext.h | 2 +-

[edk2-devel] [PATCH v3 71/78] SourceLevelDebugPkg/DebugAgent: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in documentation, comments and debug strings. Cc: Hao A Wu Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Hao A Wu Signed-off-by: Philippe Mathieu-Daude --- .../DebugAgent/DebugAgentCommon/DebugMp.h | 6 +--

[edk2-devel] [PATCH v3 77/78] UefiCpuPkg/PiSmm: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Laszlo Ersek Reviewed-by: Eric Dong Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 72/78] SourceLevelDebugPkg/DebugCommunicationLib: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Hao A Wu Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Hao A Wu Signed-off-by: Philippe Mathieu-Daude --- .../DebugCommunicationLibSerialPort.inf | 2 +-

[edk2-devel] [PATCH v3 78/78] UefiCpuPkg/Smm: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Laszlo Ersek Reviewed-by: Eric Dong Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 76/78] UefiCpuPkg/CpuDxe: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix few typos in comments and documentation. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Laszlo Ersek Reviewed-by: Eric Dong Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 74/78] StandaloneMmPkg: Fix a typo

2020-02-06 Thread Philippe Mathieu-Daudé
Fix a typo in a comment. Cc: Achin Gupta Cc: Jiewen Yao Cc: Supreeth Venkatesh Reviewed-by: Jiewen Yao Signed-off-by: Philippe Mathieu-Daude --- StandaloneMmPkg/Core/Dispatcher.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/StandaloneMmPkg/Core/Dispatcher.c

[edk2-devel] [PATCH v3 75/78] StandaloneMmPkg: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix few typos in comments. Cc: Achin Gupta Cc: Jiewen Yao Cc: Supreeth Venkatesh Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Jiewen Yao Signed-off-by: Philippe Mathieu-Daude --- .../StandaloneMmPeCoffExtraActionLib.inf

[edk2-devel] [PATCH v3 70/78] SignedCapsulePkg/Universal: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in documentation, comments and debug strings. Cc: Jiewen Yao Cc: Chao Zhang Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Jiewen Yao Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 67/78] ShellPkg/Application: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Ray Ni Cc: Zhichao Gao Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Zhichao Gao Signed-off-by: Philippe Mathieu-Daude --- ShellPkg/Application/Shell/ConsoleLogger.h| 4 +--

[edk2-devel] [PATCH v3 64/78] SecurityPkg/Library/Tpm: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jiewen Yao Cc: Jian J Wang Cc: Chao Zhang Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Jiewen Yao Signed-off-by: Philippe Mathieu-Daude --- SecurityPkg/Include/Library/TpmCommLib.h

[edk2-devel] [PATCH v3 69/78] SignedCapsulePkg: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jiewen Yao Cc: Chao Zhang Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Jiewen Yao Signed-off-by: Philippe Mathieu-Daude --- SignedCapsulePkg/SignedCapsulePkg.dec | 4 +--

[edk2-devel] [PATCH v3 68/78] ShellPkg/DynamicCommand: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Ray Ni Cc: Zhichao Gao Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Zhichao Gao Signed-off-by: Philippe Mathieu-Daude --- ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h |

[edk2-devel] [PATCH v3 65/78] SecurityPkg/Tcg: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in documentation, comments and debug strings. Cc: Jiewen Yao Cc: Jian J Wang Cc: Chao Zhang Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Jiewen Yao Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 66/78] SecurityPkg/VariableAuthenticated: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix few typos in comments and documentation. Cc: Jiewen Yao Cc: Jian J Wang Cc: Chao Zhang Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Jiewen Yao Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 39/78] NetworkPkg/Ip4Dxe: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/Ip4Dxe/Ip4Common.h |

[edk2-devel] [PATCH v3 62/78] SecurityPkg: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix few typos in comments and documentation. Cc: Jiewen Yao Cc: Jian J Wang Cc: Chao Zhang Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Jiewen Yao Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 36/78] NetworkPkg/HttpBootDxe: Fix a typo in variable name

2020-02-06 Thread Philippe Mathieu-Daudé
Correctly write 'Initialized'. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/HttpBootDxe/HttpBootConfig.h | 2 +- NetworkPkg/HttpBootDxe/HttpBootConfig.c | 4 ++-- 2 files changed, 3 insertions(+), 3

[edk2-devel] [PATCH v3 46/78] NetworkPkg/SnpDxe: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix few typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/SnpDxe/Snp.h | 8

[edk2-devel] [PATCH v3 37/78] NetworkPkg/HttpBootDxe: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/HttpBootDxe/HttpBootDhcp4.h

[edk2-devel] [PATCH v3 59/78] OvmfPkg/Xen: Fix a typo

2020-02-06 Thread Philippe Mathieu-Daudé
Fix a typo in a comment. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Anthony Perard Cc: Julien Grall Acked-by: Anthony PERARD Reviewed-by: Laszlo Ersek Signed-off-by: Philippe Mathieu-Daude --- OvmfPkg/XenBusDxe/GrantTable.h | 2 +- 1 file changed, 1 insertion(+), 1

[edk2-devel] [PATCH v3 60/78] OvmfPkg/Xen: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Anthony Perard Cc: Julien Grall Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Acked-by: Anthony PERARD Acked-by: Laszlo Ersek Signed-off-by:

[edk2-devel] [PATCH v3 51/78] NetworkPkg/Udp6Dxe: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/Udp6Dxe/Udp6Driver.h | 6

[edk2-devel] [PATCH v3 35/78] NetworkPkg/DpcDxe: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix few typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/DpcDxe/Dpc.h | 2 +-

[edk2-devel] [PATCH v3 47/78] NetworkPkg/TcpDxe/Sock: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix few typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/TcpDxe/SockImpl.h | 2 +-

[edk2-devel] [PATCH v3 42/78] NetworkPkg/Library: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/Include/Library/HttpLib.h

[edk2-devel] [PATCH v3 38/78] NetworkPkg/HttpDxe: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/HttpDxe/HttpDriver.h | 8

[edk2-devel] [PATCH v3 56/78] OvmfPkg/Csm/LegacyBios: Fix a typo

2020-02-06 Thread Philippe Mathieu-Daudé
Fix a typo in the header documentation. Reviewed-by: Laszlo Ersek Signed-off-by: Philippe Mathieu-Daude --- OvmfPkg/Csm/Include/Protocol/LegacyBios.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/OvmfPkg/Csm/Include/Protocol/LegacyBios.h

[edk2-devel] [PATCH v3 43/78] NetworkPkg/MnpDxe: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in documentation, comments and debug strings. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 52/78] NetworkPkg/UefiPxeBcDxe: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/UefiPxeBcDxe/PxeBcDhcp4.h

[edk2-devel] [PATCH v3 40/78] NetworkPkg/Ip6Dxe: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude --- NetworkPkg/Ip6Dxe/Ip6ConfigImpl.h | 4

[edk2-devel] [PATCH v3 44/78] NetworkPkg/Mtftp4Dxe: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in documentation, comments and debug strings. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 48/78] NetworkPkg/TcpDxe/Tcp: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in documentation, comments and debug strings. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 63/78] SecurityPkg/Hash2DxeCrypto: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix few typos in documentation, comments and debug strings. Cc: Jiewen Yao Cc: Jian J Wang Cc: Chao Zhang Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Jiewen Yao Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 54/78] OvmfPkg/Acpi: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix few typos in comments and documentation. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Laszlo Ersek Signed-off-by: Philippe Mathieu-Daude ---

[edk2-devel] [PATCH v3 55/78] OvmfPkg/Csm: Fix various typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Fix various typos in documentation, comments and strings. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: David Woodhouse Acked-by: Laszlo Ersek Signed-off-by: Philippe Mathieu-Daude

[edk2-devel] [PATCH v3 61/78] PcAtChipsetPkg: Fix few typos

2020-02-06 Thread Philippe Mathieu-Daudé
From: Antoine Coeur Correctly write 'serial', remove duplicated 'the'. Cc: Ray Ni Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Acked-by: Liming Gao Signed-off-by: Philippe Mathieu-Daude --- PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c | 2 +-

<    1   2   3   4   5   6   7   8   9   10   >