Looks good.
Just have some comments mentioned Patch 0/4 about moving/renaming file/folder 
for synchronizing with NXP LSDK opensource 
https://source.codeaurora.org/external/qoriq/qoriq-components/edk2-platforms.

Best Regards,
Sunny Wang

-----Original Message-----
From: Vikas Singh <vikas.si...@puresoftware.com>
Sent: Friday, June 18, 2021 11:28 PM
To: devel@edk2.groups.io
Cc: Sami Mujawar <sami.muja...@arm.com>; l...@nuviainc.com; Meenakshi Aggarwal 
(meenakshi.aggar...@nxp.com) <meenakshi.aggar...@nxp.com>; Samer El-Haj-Mahmoud 
<samer.el-haj-mahm...@arm.com>; V Sethi (v.se...@nxp.com) <v.se...@nxp.com>; 
arokia.samy <arokia.s...@puresoftware.com>; kuldip.dwiv...@puresoftware.com; 
Ard Biesheuvel <ard.biesheu...@arm.com>; vikas.si...@nxp.com; Sunny Wang 
<sunny.w...@arm.com>
Subject: [PATCH V2 2/4] Silicon/NXP: Add support of SVR handling for LS1046A SoC

This patch adds a generic method to get Silicon Version Register (SVR)
on LS1046A SoC. This method will be generic for all platfroms based
on LS1046A SoC, like - LS1046AFRWY, LS1046ARDB.

Signed-off-by: Vikas Singh <vikas.si...@puresoftware.com>
---
 Silicon/NXP/LS1046A/Library/SocLib/SocLib.c | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/Silicon/NXP/LS1046A/Library/SocLib/SocLib.c 
b/Silicon/NXP/LS1046A/Library/SocLib/SocLib.c
index 8fa6a7dd00..003f5bd82f 100644
--- a/Silicon/NXP/LS1046A/Library/SocLib/SocLib.c
+++ b/Silicon/NXP/LS1046A/Library/SocLib/SocLib.c
@@ -2,6 +2,7 @@
   SoC specific Library containg functions to initialize various SoC components



   Copyright 2017-2020 NXP

+  Copyright 2021 Puresoftware Ltd



   SPDX-License-Identifier: BSD-2-Clause-Patent



@@ -64,6 +65,21 @@ SocGetClock (
   return ReturnValue;

 }



+/**

+  Function to get SoC's System Version Register(SVR)

+ **/

+UINT32

+SocGetSvr (

+  VOID

+  )

+{

+  LS1046A_DEVICE_CONFIG  *Dcfg;

+

+  Dcfg = (LS1046A_DEVICE_CONFIG  *)LS1046A_DCFG_ADDRESS;

+

+  return DcfgRead32 ((UINTN)&Dcfg->Svr);

+}

+

 /**

    Function to select pins depending upon pcd using supplemental

    configuration unit(SCFG) extended RCW controlled pinmux control

--
2.25.1

IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.


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


Reply via email to