On 3/1/2018 6:20 PM, Laszlo Ersek wrote:
On 03/01/18 07:57, Heyi Guo wrote:
Use ZeroMem to initialize all fields in temporary
PCI_ROOT_BRIDGE_APERTURE variables to zero. This is not mandatory but
is helpful for future extension: when we add new fields to
PCI_ROOT_BRIDGE_APERTURE and the default value of these fields can
safely be zero, this code will not suffer from an additional
change.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Heyi Guo <[email protected]>
Cc: Jordan Justen <[email protected]>
Cc: Anthony Perard <[email protected]>
Cc: Julien Grall <[email protected]>
Cc: Ruiyu Ni <[email protected]>
Cc: Laszlo Ersek <[email protected]>
Cc: Ard Biesheuvel <[email protected]>
---
OvmfPkg/Library/PciHostBridgeLib/PciHostBridgeLib.c | 4 ++++
OvmfPkg/Library/PciHostBridgeLib/XenSupport.c | 5 +++++
2 files changed, 9 insertions(+)
I also suggest a different subject line:
OvmfPkg/PciHostBridgeLib: clear PCI_ROOT_BRIDGE_APERTURE vars for (re)init
(74 chars)
I sometimes tries very hard to make the subject line be <= 70 chars.
74 is acceptable?
Thanks
Laszlo
--
Thanks,
Ray
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel
