[edk2-devel] [PATCH] EmbeddedPkg/RealTimeClockRuntimeDxe: Improve GetWakeupTime

[Sunny Wang]

Looks good to me. Good catch!
Reviewed-by: Sunny Wang <sunny.wang at arm.com>

By the way, just FYI, I still run into the same problem (line-ending problem) with Windows Outlook and EDK2 email archive https://edk2.groups.io/g/devel/topic/patch/83043513?p=,,,20,0,0,0::recentpostdate%2Fsticky,,,20,2,0,83043513. Moreover, Gmail works well, so I used it to review this patch.

Best Regards,
Sunny Wang

-----Original Message-----
From: Marcin Wojtas <mw at semihalf.com>
Sent: Monday, May 24, 2021 11:55 AM
To: devel at edk2.groups.io
Cc: michael.d.kinney at intel.com; leif at nuviainc.com; ardb+tianocore at kernel.org; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud at arm.com>; Sunny Wang <Sunny.Wang at arm.com>; gjb at semihalf.com; upstream at semihalf.com; Marcin Wojtas <mw at semihalf.com>
Subject: [PATCH] EmbeddedPkg/RealTimeClockRuntimeDxe: Improve GetWakeupTime

GetWakeupTime should return full time information, including the daylight/timezone. Make use of the existing non-volatile variables for that purpose. Moreover add an error checking of possibly invalid parameters.
This partially fixes FWTS and SCT Set/GetWakeupTime tests on Marvell platforms.

Signed-off-by: Marcin Wojtas <mw at semihalf.com>
---
 EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClock.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClock.c b/EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClock.c
index e59036badc..85650a6ede 100644
--- a/EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClock.c
+++ b/EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClock.c
@@ -143,6 +143,17 @@ GetWakeupTime (
   OUT EFI_TIME    *Time   ) {+  if (Time == NULL || Enabled == NULL || Pending == NULL) {+    return EFI_INVALID_PARAMETER;+  }++  //+  // Set these first so the RealTimeClockLib implementation+  // can override them based on its own settings.+  //+  Time->TimeZone = mTimeSettings.TimeZone;+  Time->Daylight = mTimeSettings.Daylight;+   return LibGetWakeupTime (Enabled, Pending, Time); } --
2.29.0

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 (#75483): https://edk2.groups.io/g/devel/message/75483
Mute This Topic: https://groups.io/mt/83043513/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-