On 25/09/2018 10:43, Udit Kumar wrote:
Hi Grant

-----Original Message-----
From: [email protected] <arm.ebbr-discuss-
[email protected]> On Behalf Of Grant Likely
Sent: Monday, September 24, 2018 7:24 PM
To: [email protected]; [email protected]
Cc: [email protected]
Subject: [Arm.ebbr-discuss] [PATCH 4/7] GetTime()/SetTime() should return
EFI_UNSUPPORTED when unsupported

UEFI spec is adding ability for all runtime services to return
EFI_UNSUPPORTED if not implemented. Use this return code instead of
EFI_DEVICE_ERROR.

Could we wait to push this patch, until UEFI spec is not updated with
this error code.
AFAIK, Spec 2.7, Section 8.3 does not have EFI_UNSUPPORTED  error.

The ECR is as an errata against 2.7, and it will be discussed in the USWG meeting this week. I will wait until there is high confidence that the ECR will be accepted, and I won't publish a release with this change until the errata is published.


Thanks
Udit
Resolves: #14
Signed-off-by: Grant Likely <[email protected]>
---
  source/chapter2-uefi.rst | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/source/chapter2-uefi.rst b/source/chapter2-uefi.rst index
6f7d260..239fd8c 100644
--- a/source/chapter2-uefi.rst
+++ b/source/chapter2-uefi.rst
@@ -151,9 +151,9 @@ it may not be possible to access the RTC from runtime
services.
  e.g., The RTC may be on a shared I2C bus which runtime services cannot
access  because it will conflict with the OS.

-Firmware still must provide the UEFI GetTime() and SetTime() runtime service
-calls, but if an RTC isn't present, or cannot be accessed at runtime, then 
both -
calls shall return EFI_DEVICE_ERROR.
+If firmware does not support access to the RTC, then GetTime() and
+SetTime() shall return EFI_UNSUPPORTED, and the OS must use a device
+driver to control the RTC.


  UEFI Reset and Shutdown
  -----------------------
--
2.13.0

_______________________________________________
Arm.ebbr-discuss mailing list
[email protected]

_______________________________________________
boot-architecture mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/boot-architecture

Reply via email to