Hello Vivek,
I just have one minor comment,

On 2/14/22 13:13, Vivek Kumar Gautam via groups.io wrote:
From: Shriram K <shrira...@arm.com>

The IO virtualization block on reference design platforms allow
connecting non-discoverable devices such as PL011 UART. On platforms
that support this, initialize the UART controller connected to the
IO virtualization block.

Signed-off-by: Shriram K <shrira...@arm.com>
Signed-off-by: Vivek Gautam <vivek.gau...@arm.com>
---
  Platform/ARM/SgiPkg/SgiPlatform.dec                      |  1 +
  Platform/ARM/SgiPkg/RdN2/RdN2.dsc                        |  4 ++
  Platform/ARM/SgiPkg/RdN2Cfg1/RdN2Cfg1.dsc                |  6 +-
  Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.inf  |  7 +-
  Platform/ARM/SgiPkg/Library/PlatformLib/PlatformLib.inf  |  8 ++-
  Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.c    | 73 
+++++++++++++++++++-
  Platform/ARM/SgiPkg/Library/PlatformLib/PlatformLibMem.c | 19 ++++-
  7 files changed, 112 insertions(+), 6 deletions(-)


[...]

diff --git a/Platform/ARM/SgiPkg/Library/PlatformLib/PlatformLibMem.c 
b/Platform/ARM/SgiPkg/Library/PlatformLib/PlatformLibMem.c
index 8139b75d8ee4..26fd873fa647 100644
--- a/Platform/ARM/SgiPkg/Library/PlatformLib/PlatformLibMem.c
+++ b/Platform/ARM/SgiPkg/Library/PlatformLib/PlatformLibMem.c
@@ -1,6 +1,6 @@
  /** @file
  *
-*  Copyright (c) 2018-2020, ARM Limited. All rights reserved.
+*  Copyright (c) 2018-2022, ARM Limited. All rights reserved.
  *
  *  SPDX-License-Identifier: BSD-2-Clause-Patent
  *
@@ -17,7 +17,8 @@
// Total number of descriptors, including the final "end-of-table" descriptor.
  #define MAX_VIRTUAL_MEMORY_MAP_DESCRIPTORS                 \
-          (14 + (FixedPcdGet32 (PcdChipCount) * 2))
+          (14 + (FixedPcdGet32 (PcdChipCount) * 2)) +      \
+          (FeaturePcdGet (PcdIoVirtBlkNonDiscoverable) * 2)

Shouldn't it be dependent on PcdIoVirtBlkNonDiscoverable's value ?


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97075): https://edk2.groups.io/g/devel/message/97075
Mute This Topic: https://groups.io/mt/89134664/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to