public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Leif Lindholm <leif.lindholm@linaro.org>
To: Pete Batard <pete@akeo.ie>
Cc: edk2-devel@lists.01.org, ard.biesheuvel@linaro.org
Subject: Re: [PATCH v4 edk2-platforms 06/23] Platform/Raspberry/Pi3: Add RTC library
Date: Wed, 30 Jan 2019 22:22:36 +0000	[thread overview]
Message-ID: <20190130222236.53jfh65ehhtk52ly@bivouac.eciton.net> (raw)
In-Reply-To: <20190129162655.3800-7-pete@akeo.ie>

First of all - this is something I would like to see contributed
directly to edk2 EmbeddedPkg. It's something me and Ard have discussed
adding at some point, but never getting around to.

I would also like to look into whether we could replace
PcdBootEpochSeconds with a -DBUILD_EPOCH=`date +%s` addition to the
CFLAGS in the .inf.

Regards,

Leif

On Tue, Jan 29, 2019 at 04:26:38PM +0000, Pete Batard wrote:
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard <pete@akeo.ie>
> ---
>  Platform/Raspberry/Pi3/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.c   | 221 ++++++++++++++++++++
>  Platform/Raspberry/Pi3/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.inf |  43 ++++
>  2 files changed, 264 insertions(+)
> 
> diff --git a/Platform/Raspberry/Pi3/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.c b/Platform/Raspberry/Pi3/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.c
> new file mode 100644
> index 000000000000..dea621ff03ef
> --- /dev/null
> +++ b/Platform/Raspberry/Pi3/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.c
> @@ -0,0 +1,221 @@
> +/** @file
> + *
> + *  Implement dummy EFI RealTimeClock runtime services.
> + *
> + *  Copyright (c) 2018, Andrei Warkentin <andrey.warkentin@gmail.com>
> + *  Copyright (c) Microsoft Corporation. All rights reserved.
> + *
> + *  This program and the accompanying materials
> + *  are licensed and made available under the terms and conditions of the BSD License
> + *  which accompanies this distribution.  The full text of the license may be found at
> + *  http://opensource.org/licenses/bsd-license.php
> + *
> + *  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> + *  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> + *
> + **/
> +
> +#include <PiDxe.h>
> +#include <Library/BaseLib.h>
> +#include <Library/DebugLib.h>
> +#include <Library/IoLib.h>
> +#include <Library/RealTimeClockLib.h>
> +#include <Library/TimerLib.h>
> +#include <Library/TimeBaseLib.h>
> +#include <Library/UefiRuntimeLib.h>
> +#include <Library/ArmGenericTimerCounterLib.h>
> +
> +/**
> +   Returns the current time and date information, and the time-keeping capabilities
> +   of the virtual RTC.
> +
> +   For simplicity, this LibGetTime does not report Years/Months, instead it will only report current
> +   Day, Hours, Minutes and Seconds starting from the begining of CPU up-time. Otherwise, a more
> +   complex logic will be required to account for leap years and days/month differences.
> +
> +   @param  Time                  A pointer to storage to receive a snapshot of the current time.
> +   @param  Capabilities          An optional pointer to a buffer to receive the real time clock
> +   device's capabilities.
> +
> +   @retval EFI_SUCCESS           The operation completed successfully.
> +   @retval EFI_INVALID_PARAMETER Time is NULL.
> +   @retval EFI_DEVICE_ERROR      The time could not be retrieved due to hardware error.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +LibGetTime (
> +  OUT EFI_TIME               *Time,
> +  OUT EFI_TIME_CAPABILITIES  *Capabilities
> +  )
> +{
> +  UINTN DataSize;
> +  UINT64 Counter;
> +  EFI_STATUS Status;
> +  UINTN ElapsedSeconds;
> +  UINT32 Remainder;
> +  UINT32 Freq = ArmGenericTimerGetTimerFreq ();
> +
> +  if (Time == NULL) {
> +    return EFI_INVALID_PARAMETER;
> +  }
> +
> +  //
> +  // Depend on ARM generic timer to report date/time relative to the
> +  // start of CPU timer counting where date and time will always
> +  // be relative to the date/time 1/1/1900 00H:00M:00S
> +  //
> +
> +  ASSERT (Freq != 0);
> +  if (Freq == 0) {
> +    return EFI_DEVICE_ERROR;
> +  }
> +
> +  if (Capabilities) {
> +    Capabilities->Accuracy = 0;
> +    Capabilities->Resolution = Freq;
> +    Capabilities->SetsToZero = FALSE;
> +  }
> +
> +  DataSize = sizeof (UINTN);
> +  ElapsedSeconds = 0;
> +  Status = EfiGetVariable (L"RtcEpochSeconds",
> +             &gEfiCallerIdGuid,
> +             NULL,
> +             &DataSize,
> +             &ElapsedSeconds);
> +  if (EFI_ERROR (Status)) {
> +    ElapsedSeconds = PcdGet64 (PcdBootEpochSeconds);
> +  }
> +  Counter = GetPerformanceCounter ();
> +  ElapsedSeconds += DivU64x32Remainder (Counter, Freq, &Remainder);
> +  EpochToEfiTime (ElapsedSeconds, Time);
> +
> +  //
> +  // Frequency < 0x100000000, so Remainder < 0x100000000, then (Remainder * 1,000,000,000)
> +  // will not overflow 64-bit.
> +  //
> +  Time->Nanosecond = DivU64x32 (MultU64x64 ((UINT64)Remainder, 1000000000U), Freq);
> +
> +  return EFI_SUCCESS;
> +}
> +
> +
> +/**
> +   Sets the current local time and date information.
> +
> +   @param  Time                  A pointer to the current time.
> +
> +   @retval EFI_SUCCESS           The operation completed successfully.
> +   @retval EFI_INVALID_PARAMETER A time field is out of range.
> +   @retval EFI_DEVICE_ERROR      The time could not be set due due to hardware error.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +LibSetTime (
> +  IN EFI_TIME  *Time
> +  )
> +{
> +  UINTN Epoch;
> +
> +  if (!IsTimeValid (Time)) {
> +    return EFI_INVALID_PARAMETER;
> +  }
> +
> +  Epoch = EfiTimeToEpoch (Time);
> +  return EfiSetVariable (L"RtcEpochSeconds", &gEfiCallerIdGuid,
> +           EFI_VARIABLE_BOOTSERVICE_ACCESS |
> +           EFI_VARIABLE_RUNTIME_ACCESS |
> +           EFI_VARIABLE_NON_VOLATILE,
> +           sizeof (Epoch),
> +           &Epoch);
> +}
> +
> +
> +/**
> +   Returns the current wakeup alarm clock setting.
> +
> +   @param  Enabled               Indicates if the alarm is currently enabled or disabled.
> +   @param  Pending               Indicates if the alarm signal is pending and requires acknowledgement.
> +   @param  Time                  The current alarm setting.
> +
> +   @retval EFI_SUCCESS           The alarm settings were returned.
> +   @retval EFI_INVALID_PARAMETER Any parameter is NULL.
> +   @retval EFI_DEVICE_ERROR      The wakeup time could not be retrieved due to a hardware error.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +LibGetWakeupTime (
> +  OUT BOOLEAN   *Enabled,
> +  OUT BOOLEAN   *Pending,
> +  OUT EFI_TIME  *Time
> +  )
> +{
> +  return EFI_UNSUPPORTED;
> +}
> +
> +
> +/**
> +   Sets the system wakeup alarm clock time.
> +
> +   @param  Enabled               Enable or disable the wakeup alarm.
> +   @param  Time                  If Enable is TRUE, the time to set the wakeup alarm for.
> +
> +   @retval EFI_SUCCESS           If Enable is TRUE, then the wakeup alarm was enabled. If
> +   Enable is FALSE, then the wakeup alarm was disabled.
> +   @retval EFI_INVALID_PARAMETER A time field is out of range.
> +   @retval EFI_DEVICE_ERROR      The wakeup time could not be set due to a hardware error.
> +   @retval EFI_UNSUPPORTED       A wakeup timer is not supported on this platform.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +LibSetWakeupTime (
> +  IN  BOOLEAN   Enabled,
> +  OUT EFI_TIME  *Time
> +  )
> +{
> +  return EFI_UNSUPPORTED;
> +}
> +
> +
> +/**
> +   This is the declaration of an EFI image entry point. This can be the entry point to an application
> +   written to this specification, an EFI boot service driver, or an EFI runtime driver.
> +
> +   @param  ImageHandle           Handle that identifies the loaded image.
> +   @param  SystemTable           System Table for this image.
> +
> +   @retval EFI_SUCCESS           The operation completed successfully.
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +LibRtcInitialize (
> +  IN EFI_HANDLE        ImageHandle,
> +  IN EFI_SYSTEM_TABLE  *SystemTable
> +  )
> +{
> +  return EFI_SUCCESS;
> +}
> +
> +
> +/**
> +   Fixup internal data so that EFI can be call in virtual mode.
> +   Call the passed in Child Notify event and convert any pointers in
> +   lib to virtual mode.
> +
> +   @param[in]    Event   The Event that is being processed
> +   @param[in]    Context Event Context
> +**/
> +VOID
> +EFIAPI
> +LibRtcVirtualNotifyEvent (
> +  IN EFI_EVENT  Event,
> +  IN VOID       *Context
> +  )
> +{
> +  return;
> +}
> diff --git a/Platform/Raspberry/Pi3/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.inf b/Platform/Raspberry/Pi3/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.inf
> new file mode 100644
> index 000000000000..5c39e4df4c34
> --- /dev/null
> +++ b/Platform/Raspberry/Pi3/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.inf
> @@ -0,0 +1,43 @@
> +#/** @file
> +#
> +#  Implement dummy EFI RealTimeClock runtime services.
> +#
> +#  Copyright (c) 2018, Andrei Warkentin <andrey.warkentin@gmail.com>
> +#  Copyright (c) Microsoft Corporation. All rights reserved.
> +#
> +#  This program and the accompanying materials
> +#  are licensed and made available under the terms and conditions of the BSD License
> +#  which accompanies this distribution.  The full text of the license may be found at
> +#  http://opensource.org/licenses/bsd-license.php
> +#
> +#  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +#  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +#
> +#**/
> +
> +[Defines]
> +  INF_VERSION                    = 0x0001001A
> +  BASE_NAME                      = VirtualRealTimeClockLib
> +  FILE_GUID                      = 1E27D461-78F3-4F7D-B1C2-F72384F13A6E
> +  MODULE_TYPE                    = BASE
> +  VERSION_STRING                 = 1.0
> +  LIBRARY_CLASS                  = RealTimeClockLib
> +
> +[Sources.common]
> +  VirtualRealTimeClockLib.c
> +
> +[Packages]
> +  ArmPkg/ArmPkg.dec
> +  MdePkg/MdePkg.dec
> +  EmbeddedPkg/EmbeddedPkg.dec
> +  Platform/Raspberry/Pi3/RPi3.dec
> +
> +[LibraryClasses]
> +  IoLib
> +  DebugLib
> +  TimerLib
> +  TimeBaseLib
> +  UefiRuntimeLib
> +
> +[Pcd]
> +  gRaspberryPiTokenSpaceGuid.PcdBootEpochSeconds
> -- 
> 2.17.0.windows.1
> 


  reply	other threads:[~2019-01-30 22:22 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-29 16:26 [PATCH v4 edk2-platforms 00/23] Platform/Raspberry: Add Raspberry Pi 3 support Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 01/23] Silicon/Broadcom/Bcm282x: Add interrupt driver Pete Batard
2019-01-31 15:24   ` Leif Lindholm
2019-01-31 17:19     ` Ard Biesheuvel
2019-01-31 19:57       ` Leif Lindholm
2019-01-31 21:01         ` Andrew Fish
2019-02-01  8:43           ` Laszlo Ersek
2019-02-01 10:28             ` Pete Batard
2019-02-01 15:18               ` Leif Lindholm
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 02/23] Silicon/Broadcom/Bcm283x: Add GpioLib Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 03/23] Platform/Raspberry/Pi3: Add ACPI tables Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 04/23] Platform/Raspberry/Pi3: Add reset and memory init libraries Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 05/23] Platform/Raspberry/Pi3: Add platform library Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 06/23] Platform/Raspberry/Pi3: Add RTC library Pete Batard
2019-01-30 22:22   ` Leif Lindholm [this message]
2019-01-31 12:31     ` Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 07/23] Platform/Raspberry/Pi3: Add firmware driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 08/23] Platform/Raspberry/Pi3: Add platform config driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 09/23] Platform/Raspberry/Pi3: Add SMBIOS driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 10/23] Platform/Raspberry/Pi3: Add display driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 11/23] Platform/Raspberry/Pi3: Add console driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 12/23] Platform/Raspberry/Pi3: Add NV storage driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 13/23] Platform/Raspberry/Pi3: Add Device Tree driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 14/23] Platform/Raspberry/Pi3: Add base MMC driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 15/23] Platform/Raspberry/Pi3: Add Arasan " Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 16/23] Platform/Raspberry/Pi3: Platform/Raspberry/Pi3: Add SD Host driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 17/23] Platform/Raspberry/Pi3: Add platform boot manager and helper libraries Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 18/23] Platform/Raspberry/Pi3: Add USB host driver Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 19/23] Platform/Raspberry/Pi3: Add platform Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 20/23] Platform/Raspberry/Pi3: Add platform readme Pete Batard
2019-01-30 21:50   ` Leif Lindholm
2019-01-31 12:30     ` Pete Batard
2019-01-31 14:13       ` Leif Lindholm
2019-01-31 14:36         ` Ard Biesheuvel
2019-01-31 14:44           ` Ard Biesheuvel
2019-01-31 17:19             ` Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 21/23] Platform/Raspberry/Pi3 *NON-OSI*: Add ATF binaries Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 22/23] Platform/Raspberry/Pi3 *NON-OSI*: Add Device Tree binaries Pete Batard
2019-01-29 16:26 ` [PATCH v4 edk2-platforms 23/23] Platform/Raspberry/Pi3 *NON-OSI*: Add logo driver Pete Batard
2019-01-29 17:40 ` [PATCH v4 edk2-platforms 00/23] Platform/Raspberry: Add Raspberry Pi 3 support Ard Biesheuvel
2019-01-29 21:09   ` Pete Batard
2019-01-30 19:38     ` Ard Biesheuvel
2019-01-30 19:42       ` Leif Lindholm
2019-01-30 19:45         ` Ard Biesheuvel
2019-01-30 21:59 ` Leif Lindholm
2019-01-30 22:28   ` Leif Lindholm
2019-01-31 12:31   ` Pete Batard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190130222236.53jfh65ehhtk52ly@bivouac.eciton.net \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox