| Message ID | 20170704110017.32042-1-ard.biesheuvel@linaro.org |
|---|---|
| State | New |
| Headers | show |
Hi, 2017-07-04 13:00 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: > Switch to the generic implementation of the ResetSystem() runtime > service call, which is preferred over the one in EmbeddedPkg. This > involves selecting another DXE runtime driver, and morphing the > existing EfiResetSystemLib implementation into a ResetSystemLib one > (which is what the generic driver depends on for platform glue) > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> > --- > Platforms/Marvell/Armada/Armada.dsc.inc | 4 +- > Platforms/Marvell/Armada/Armada70x0.fdf | 2 +- > Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c | 110 ++++++++++++++------ > Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf | 7 +- > 4 files changed, 87 insertions(+), 36 deletions(-) > > diff --git a/Platforms/Marvell/Armada/Armada.dsc.inc b/Platforms/Marvell/Armada/Armada.dsc.inc > index 475fc6d05379..32eaa2942564 100644 > --- a/Platforms/Marvell/Armada/Armada.dsc.inc > +++ b/Platforms/Marvell/Armada/Armada.dsc.inc > @@ -94,7 +94,7 @@ > # Reset and Time libraries > EfiTimeBaseLib|OpenPlatformPkg/Library/EfiTimeBaseLib/EfiTimeBaseLib.inf > RealTimeClockLib|OpenPlatformPkg/Platforms/Marvell/Armada/Library/RealTimeClockLib/RealTimeClockLib.inf > - EfiResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf > + ResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf Isn't it better to switch to generic PSCI: EfiResetSystemLib|ArmPkg/Library/ArmPsciResetSystemLib/ArmPsciResetSystemLib.inf and remove MvResetSystemLib whatsoever? Best regards, Marcin > > # Network support > NetLib|MdeModulePkg/Library/DxeNetLib/DxeNetLib.inf > @@ -416,7 +416,7 @@ > MdeModulePkg/Universal/Variable/EmuRuntimeDxe/EmuVariableRuntimeDxe.inf > > EmbeddedPkg/EmbeddedMonotonicCounter/EmbeddedMonotonicCounter.inf > - EmbeddedPkg/ResetRuntimeDxe/ResetRuntimeDxe.inf > + MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystemRuntimeDxe.inf > EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClockRuntimeDxe.inf > EmbeddedPkg/MetronomeDxe/MetronomeDxe.inf > > diff --git a/Platforms/Marvell/Armada/Armada70x0.fdf b/Platforms/Marvell/Armada/Armada70x0.fdf > index 280b40be1d39..8091ea260261 100644 > --- a/Platforms/Marvell/Armada/Armada70x0.fdf > +++ b/Platforms/Marvell/Armada/Armada70x0.fdf > @@ -99,7 +99,7 @@ FvNameGuid = 5eda4200-2c5f-43cb-9da3-0baf74b1b30c > INF MdeModulePkg/Universal/CapsuleRuntimeDxe/CapsuleRuntimeDxe.inf > INF MdeModulePkg/Universal/Variable/EmuRuntimeDxe/EmuVariableRuntimeDxe.inf > INF EmbeddedPkg/EmbeddedMonotonicCounter/EmbeddedMonotonicCounter.inf > - INF EmbeddedPkg/ResetRuntimeDxe/ResetRuntimeDxe.inf > + INF MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystemRuntimeDxe.inf > INF EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClockRuntimeDxe.inf > INF EmbeddedPkg/MetronomeDxe/MetronomeDxe.inf > INF OpenPlatformPkg/Drivers/I2c/MvI2cDxe/MvI2cDxe.inf > diff --git a/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c b/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c > index c0787aceac9c..f349c5c5387b 100644 > --- a/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c > +++ b/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c > @@ -66,51 +66,99 @@ LibResetSystemVirtualNotifyEvent ( > } > > /** > - Resets the entire platform. > - > - @param ResetType The type of reset to perform. > - @param ResetStatus The status code for the reset. > - @param DataSize The size, in bytes, of WatchdogData. > - @param ResetData For a ResetType of EfiResetCold, EfiResetWarm, > - or EfiResetShutdown the data buffer starts with > - a Null-terminated Unicode string, optionally > - followed by additional binary data. > + This function causes a system-wide reset (cold reset), in which > + all circuitry within the system returns to its initial state. This type of reset > + is asynchronous to system operation and operates without regard to > + cycle boundaries. > + > + If this function returns, it means that the system does not support cold reset. > **/ > -EFI_STATUS > +VOID > EFIAPI > -LibResetSystem ( > - IN EFI_RESET_TYPE ResetType, > - IN EFI_STATUS ResetStatus, > - IN UINTN DataSize, > - IN VOID *ResetData OPTIONAL > +ResetCold ( > + VOID > ) > { > UINT32 Data; > > - switch (ResetType) { > - case EfiResetCold: > - case EfiResetWarm: > - Data = MmioRead32 (mAddress); > - Data &= ~PcdGet32 (PcdResetRegMask); > - MmioWrite32 (mAddress, Data); > - break; > - case EfiResetShutdown: > + Data = MmioRead32 (mAddress); > + Data &= ~PcdGet32 (PcdResetRegMask); > + MmioWrite32 (mAddress, Data); > +} > + > +/** > + This function causes a system-wide initialization (warm reset), in which all processors > + are set to their initial state. Pending cycles are not corrupted. > + > + If this function returns, it means that the system does not support warm reset. > +**/ > +VOID > +EFIAPI > +ResetWarm ( > + VOID > + ) > +{ > + // Map a warm reset into a cold reset > + ResetCold (); > +} > + > +/** > + This function causes the system to enter a power state equivalent > + to the ACPI G2/S5 or G3 states. > + > + If this function returns, it means that the system does not support shutdown reset. > +**/ > +VOID > +EFIAPI > +ResetShutdown ( > + VOID > + ) > +{ > // > // Currently there is no support for power-off platform > // > - break; > - default: > - break; > - } > +} > + > +/** > + This function causes the system to enter S3 and then wake up immediately. > + > + If this function returns, it means that the system does not support S3 feature. > +**/ > +VOID > +EFIAPI > +EnterS3WithImmediateWake ( > + VOID > + ) > +{ > + // not implemented > +} > > - return EFI_DEVICE_ERROR; > +/** > + This function causes a systemwide reset. The exact type of the reset is > + defined by the EFI_GUID that follows the Null-terminated Unicode string passed > + into ResetData. If the platform does not recognize the EFI_GUID in ResetData > + the platform must pick a supported reset type to perform.The platform may > + optionally log the parameters from any non-normal reset that occurs. > + > + @param[in] DataSize The size, in bytes, of ResetData. > + @param[in] ResetData The data buffer starts with a Null-terminated string, > + followed by the EFI_GUID. > +**/ > +VOID > +EFIAPI > +ResetPlatformSpecific ( > + IN UINTN DataSize, > + IN VOID *ResetData > + ) > +{ > + // Map the platform specific reset as reboot > + ResetCold (); > } > > -EFI_STATUS > +RETURN_STATUS > EFIAPI > LibInitializeResetSystem ( > - IN EFI_HANDLE ImageHandle, > - IN EFI_SYSTEM_TABLE *SystemTable > + VOID > ) > { > UINT64 Alignment; > diff --git a/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf b/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf > index 87fff5707909..0d5951d7e9a0 100644 > --- a/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf > +++ b/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf > @@ -1,4 +1,5 @@ > # Copyright (C) 2016 Marvell International Ltd. > +# Copyright (C) 2017 Linaro Ltd. > # > # Marvell BSD License Option > # > @@ -32,16 +33,18 @@ > > [Defines] > INF_VERSION = 0x00010019 > - BASE_NAME = Reset > + BASE_NAME = MvResetSystemLib > FILE_GUID = 9d1373c0-6fac-432c-88e7-818744dc45d9 > MODULE_TYPE = BASE > VERSION_STRING = 1.0 > - LIBRARY_CLASS = EfiResetSystemLib > + LIBRARY_CLASS = ResetSystemLib > + CONSTRUCTOR = LibInitializeResetSystem > > [Sources.common] > MvResetSystemLib.c > > [Packages] > + MdeModulePkg/MdeModulePkg.dec > MdePkg/MdePkg.dec > OpenPlatformPkg/Platforms/Marvell/Marvell.dec > > -- > 2.9.3 > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
On 4 July 2017 at 12:04, Marcin Wojtas <mw@semihalf.com> wrote: > Hi, > > 2017-07-04 13:00 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: >> Switch to the generic implementation of the ResetSystem() runtime >> service call, which is preferred over the one in EmbeddedPkg. This >> involves selecting another DXE runtime driver, and morphing the >> existing EfiResetSystemLib implementation into a ResetSystemLib one >> (which is what the generic driver depends on for platform glue) >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> >> --- >> Platforms/Marvell/Armada/Armada.dsc.inc | 4 +- >> Platforms/Marvell/Armada/Armada70x0.fdf | 2 +- >> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c | 110 ++++++++++++++------ >> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf | 7 +- >> 4 files changed, 87 insertions(+), 36 deletions(-) >> >> diff --git a/Platforms/Marvell/Armada/Armada.dsc.inc b/Platforms/Marvell/Armada/Armada.dsc.inc >> index 475fc6d05379..32eaa2942564 100644 >> --- a/Platforms/Marvell/Armada/Armada.dsc.inc >> +++ b/Platforms/Marvell/Armada/Armada.dsc.inc >> @@ -94,7 +94,7 @@ >> # Reset and Time libraries >> EfiTimeBaseLib|OpenPlatformPkg/Library/EfiTimeBaseLib/EfiTimeBaseLib.inf >> RealTimeClockLib|OpenPlatformPkg/Platforms/Marvell/Armada/Library/RealTimeClockLib/RealTimeClockLib.inf >> - EfiResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf >> + ResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf > > Isn't it better to switch to generic PSCI: > EfiResetSystemLib|ArmPkg/Library/ArmPsciResetSystemLib/ArmPsciResetSystemLib.inf > and remove MvResetSystemLib whatsoever? > Yes. But Leif tells me that 7040 does not support PSCI? _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Leif, where do you have this information from? 7040 differs from 8040 only with the numeber of peripherals. It has same AP806 HW block (CPUs, GIC, DDR controller, etc.). And, what's most important ArmPsciResetSystemLib.inf on the other branch works on my 7040-db board as well. Best regards, Marcin 2017-07-04 13:10 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: > On 4 July 2017 at 12:04, Marcin Wojtas <mw@semihalf.com> wrote: >> Hi, >> >> 2017-07-04 13:00 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: >>> Switch to the generic implementation of the ResetSystem() runtime >>> service call, which is preferred over the one in EmbeddedPkg. This >>> involves selecting another DXE runtime driver, and morphing the >>> existing EfiResetSystemLib implementation into a ResetSystemLib one >>> (which is what the generic driver depends on for platform glue) >>> >>> Contributed-under: TianoCore Contribution Agreement 1.0 >>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> >>> --- >>> Platforms/Marvell/Armada/Armada.dsc.inc | 4 +- >>> Platforms/Marvell/Armada/Armada70x0.fdf | 2 +- >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c | 110 ++++++++++++++------ >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf | 7 +- >>> 4 files changed, 87 insertions(+), 36 deletions(-) >>> >>> diff --git a/Platforms/Marvell/Armada/Armada.dsc.inc b/Platforms/Marvell/Armada/Armada.dsc.inc >>> index 475fc6d05379..32eaa2942564 100644 >>> --- a/Platforms/Marvell/Armada/Armada.dsc.inc >>> +++ b/Platforms/Marvell/Armada/Armada.dsc.inc >>> @@ -94,7 +94,7 @@ >>> # Reset and Time libraries >>> EfiTimeBaseLib|OpenPlatformPkg/Library/EfiTimeBaseLib/EfiTimeBaseLib.inf >>> RealTimeClockLib|OpenPlatformPkg/Platforms/Marvell/Armada/Library/RealTimeClockLib/RealTimeClockLib.inf >>> - EfiResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf >>> + ResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf >> >> Isn't it better to switch to generic PSCI: >> EfiResetSystemLib|ArmPkg/Library/ArmPsciResetSystemLib/ArmPsciResetSystemLib.inf >> and remove MvResetSystemLib whatsoever? >> > > Yes. But Leif tells me that 7040 does not support PSCI? _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
On Tue, Jul 04, 2017 at 01:19:10PM +0200, Marcin Wojtas wrote: > 2017-07-04 13:10 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: > > On 4 July 2017 at 12:04, Marcin Wojtas <mw@semihalf.com> wrote: > >> Hi, > >> > >> 2017-07-04 13:00 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: > >>> Switch to the generic implementation of the ResetSystem() runtime > >>> service call, which is preferred over the one in EmbeddedPkg. This > >>> involves selecting another DXE runtime driver, and morphing the > >>> existing EfiResetSystemLib implementation into a ResetSystemLib one > >>> (which is what the generic driver depends on for platform glue) > >>> > >>> Contributed-under: TianoCore Contribution Agreement 1.0 > >>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> > >>> --- > >>> Platforms/Marvell/Armada/Armada.dsc.inc | 4 +- > >>> Platforms/Marvell/Armada/Armada70x0.fdf | 2 +- > >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c | 110 ++++++++++++++------ > >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf | 7 +- > >>> 4 files changed, 87 insertions(+), 36 deletions(-) > >>> > >>> diff --git a/Platforms/Marvell/Armada/Armada.dsc.inc b/Platforms/Marvell/Armada/Armada.dsc.inc > >>> index 475fc6d05379..32eaa2942564 100644 > >>> --- a/Platforms/Marvell/Armada/Armada.dsc.inc > >>> +++ b/Platforms/Marvell/Armada/Armada.dsc.inc > >>> @@ -94,7 +94,7 @@ > >>> # Reset and Time libraries > >>> EfiTimeBaseLib|OpenPlatformPkg/Library/EfiTimeBaseLib/EfiTimeBaseLib.inf > >>> RealTimeClockLib|OpenPlatformPkg/Platforms/Marvell/Armada/Library/RealTimeClockLib/RealTimeClockLib.inf > >>> - EfiResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf > >>> + ResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf > >> > >> Isn't it better to switch to generic PSCI: > >> EfiResetSystemLib|ArmPkg/Library/ArmPsciResetSystemLib/ArmPsciResetSystemLib.inf > >> and remove MvResetSystemLib whatsoever? > >> > > > > Yes. But Leif tells me that 7040 does not support PSCI? > > Leif, where do you have this information from? From the source code in OpenPlatformPkg, which does not use it and the documentation that does not mention it. If this is incorrect, please submit patches to make the code in the tree reflect reality. / Leif _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
2017-07-04 13:41 GMT+02:00 Leif Lindholm <leif.lindholm@linaro.org>: > On Tue, Jul 04, 2017 at 01:19:10PM +0200, Marcin Wojtas wrote: >> 2017-07-04 13:10 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: >> > On 4 July 2017 at 12:04, Marcin Wojtas <mw@semihalf.com> wrote: >> >> Hi, >> >> >> >> 2017-07-04 13:00 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: >> >>> Switch to the generic implementation of the ResetSystem() runtime >> >>> service call, which is preferred over the one in EmbeddedPkg. This >> >>> involves selecting another DXE runtime driver, and morphing the >> >>> existing EfiResetSystemLib implementation into a ResetSystemLib one >> >>> (which is what the generic driver depends on for platform glue) >> >>> >> >>> Contributed-under: TianoCore Contribution Agreement 1.0 >> >>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> >> >>> --- >> >>> Platforms/Marvell/Armada/Armada.dsc.inc | 4 +- >> >>> Platforms/Marvell/Armada/Armada70x0.fdf | 2 +- >> >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c | 110 ++++++++++++++------ >> >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf | 7 +- >> >>> 4 files changed, 87 insertions(+), 36 deletions(-) >> >>> >> >>> diff --git a/Platforms/Marvell/Armada/Armada.dsc.inc b/Platforms/Marvell/Armada/Armada.dsc.inc >> >>> index 475fc6d05379..32eaa2942564 100644 >> >>> --- a/Platforms/Marvell/Armada/Armada.dsc.inc >> >>> +++ b/Platforms/Marvell/Armada/Armada.dsc.inc >> >>> @@ -94,7 +94,7 @@ >> >>> # Reset and Time libraries >> >>> EfiTimeBaseLib|OpenPlatformPkg/Library/EfiTimeBaseLib/EfiTimeBaseLib.inf >> >>> RealTimeClockLib|OpenPlatformPkg/Platforms/Marvell/Armada/Library/RealTimeClockLib/RealTimeClockLib.inf >> >>> - EfiResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf >> >>> + ResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf >> >> >> >> Isn't it better to switch to generic PSCI: >> >> EfiResetSystemLib|ArmPkg/Library/ArmPsciResetSystemLib/ArmPsciResetSystemLib.inf >> >> and remove MvResetSystemLib whatsoever? >> >> >> > >> > Yes. But Leif tells me that 7040 does not support PSCI? >> >> Leif, where do you have this information from? > > From the source code in OpenPlatformPkg, which does not use it and the > documentation that does not mention it. Ok, long time ago in early SoC revision, we PSCI was indeed broken and later, we were not aware of the ArmPsciResetSystemLib existence:) On the WIP branch, we have a replacement commit: https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commit/b3fe877fe7207475ba293a1fc9b0a5c8f873d2c9 which will be submitted of course. Best regards, Marcin > > If this is incorrect, please submit patches to make the code in the > tree reflect reality. > > / > Leif _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
On 4 July 2017 at 12:47, Marcin Wojtas <mw@semihalf.com> wrote: > 2017-07-04 13:41 GMT+02:00 Leif Lindholm <leif.lindholm@linaro.org>: >> On Tue, Jul 04, 2017 at 01:19:10PM +0200, Marcin Wojtas wrote: >>> 2017-07-04 13:10 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: >>> > On 4 July 2017 at 12:04, Marcin Wojtas <mw@semihalf.com> wrote: >>> >> Hi, >>> >> >>> >> 2017-07-04 13:00 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: >>> >>> Switch to the generic implementation of the ResetSystem() runtime >>> >>> service call, which is preferred over the one in EmbeddedPkg. This >>> >>> involves selecting another DXE runtime driver, and morphing the >>> >>> existing EfiResetSystemLib implementation into a ResetSystemLib one >>> >>> (which is what the generic driver depends on for platform glue) >>> >>> >>> >>> Contributed-under: TianoCore Contribution Agreement 1.0 >>> >>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> >>> >>> --- >>> >>> Platforms/Marvell/Armada/Armada.dsc.inc | 4 +- >>> >>> Platforms/Marvell/Armada/Armada70x0.fdf | 2 +- >>> >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c | 110 ++++++++++++++------ >>> >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf | 7 +- >>> >>> 4 files changed, 87 insertions(+), 36 deletions(-) >>> >>> >>> >>> diff --git a/Platforms/Marvell/Armada/Armada.dsc.inc b/Platforms/Marvell/Armada/Armada.dsc.inc >>> >>> index 475fc6d05379..32eaa2942564 100644 >>> >>> --- a/Platforms/Marvell/Armada/Armada.dsc.inc >>> >>> +++ b/Platforms/Marvell/Armada/Armada.dsc.inc >>> >>> @@ -94,7 +94,7 @@ >>> >>> # Reset and Time libraries >>> >>> EfiTimeBaseLib|OpenPlatformPkg/Library/EfiTimeBaseLib/EfiTimeBaseLib.inf >>> >>> RealTimeClockLib|OpenPlatformPkg/Platforms/Marvell/Armada/Library/RealTimeClockLib/RealTimeClockLib.inf >>> >>> - EfiResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf >>> >>> + ResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf >>> >> >>> >> Isn't it better to switch to generic PSCI: >>> >> EfiResetSystemLib|ArmPkg/Library/ArmPsciResetSystemLib/ArmPsciResetSystemLib.inf >>> >> and remove MvResetSystemLib whatsoever? >>> >> >>> > >>> > Yes. But Leif tells me that 7040 does not support PSCI? >>> >>> Leif, where do you have this information from? >> >> From the source code in OpenPlatformPkg, which does not use it and the >> documentation that does not mention it. > > Ok, long time ago in early SoC revision, we PSCI was indeed broken and > later, we were not aware of the ArmPsciResetSystemLib existence:) > On the WIP branch, we have a replacement commit: > https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commit/b3fe877fe7207475ba293a1fc9b0a5c8f873d2c9 > which will be submitted of course. > OK. But EfiSystemResetLib and EmbeddedPkg/ResetRuntimeDxe are going away, so this commit cannot be merged as-is. But since you are saying 7040 should use PSCI, I can update the patch and drop MvResetSystemLib entirely. _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
2017-07-04 13:49 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: > On 4 July 2017 at 12:47, Marcin Wojtas <mw@semihalf.com> wrote: >> 2017-07-04 13:41 GMT+02:00 Leif Lindholm <leif.lindholm@linaro.org>: >>> On Tue, Jul 04, 2017 at 01:19:10PM +0200, Marcin Wojtas wrote: >>>> 2017-07-04 13:10 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: >>>> > On 4 July 2017 at 12:04, Marcin Wojtas <mw@semihalf.com> wrote: >>>> >> Hi, >>>> >> >>>> >> 2017-07-04 13:00 GMT+02:00 Ard Biesheuvel <ard.biesheuvel@linaro.org>: >>>> >>> Switch to the generic implementation of the ResetSystem() runtime >>>> >>> service call, which is preferred over the one in EmbeddedPkg. This >>>> >>> involves selecting another DXE runtime driver, and morphing the >>>> >>> existing EfiResetSystemLib implementation into a ResetSystemLib one >>>> >>> (which is what the generic driver depends on for platform glue) >>>> >>> >>>> >>> Contributed-under: TianoCore Contribution Agreement 1.0 >>>> >>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> >>>> >>> --- >>>> >>> Platforms/Marvell/Armada/Armada.dsc.inc | 4 +- >>>> >>> Platforms/Marvell/Armada/Armada70x0.fdf | 2 +- >>>> >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c | 110 ++++++++++++++------ >>>> >>> Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf | 7 +- >>>> >>> 4 files changed, 87 insertions(+), 36 deletions(-) >>>> >>> >>>> >>> diff --git a/Platforms/Marvell/Armada/Armada.dsc.inc b/Platforms/Marvell/Armada/Armada.dsc.inc >>>> >>> index 475fc6d05379..32eaa2942564 100644 >>>> >>> --- a/Platforms/Marvell/Armada/Armada.dsc.inc >>>> >>> +++ b/Platforms/Marvell/Armada/Armada.dsc.inc >>>> >>> @@ -94,7 +94,7 @@ >>>> >>> # Reset and Time libraries >>>> >>> EfiTimeBaseLib|OpenPlatformPkg/Library/EfiTimeBaseLib/EfiTimeBaseLib.inf >>>> >>> RealTimeClockLib|OpenPlatformPkg/Platforms/Marvell/Armada/Library/RealTimeClockLib/RealTimeClockLib.inf >>>> >>> - EfiResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf >>>> >>> + ResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf >>>> >> >>>> >> Isn't it better to switch to generic PSCI: >>>> >> EfiResetSystemLib|ArmPkg/Library/ArmPsciResetSystemLib/ArmPsciResetSystemLib.inf >>>> >> and remove MvResetSystemLib whatsoever? >>>> >> >>>> > >>>> > Yes. But Leif tells me that 7040 does not support PSCI? >>>> >>>> Leif, where do you have this information from? >>> >>> From the source code in OpenPlatformPkg, which does not use it and the >>> documentation that does not mention it. >> >> Ok, long time ago in early SoC revision, we PSCI was indeed broken and >> later, we were not aware of the ArmPsciResetSystemLib existence:) >> On the WIP branch, we have a replacement commit: >> https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commit/b3fe877fe7207475ba293a1fc9b0a5c8f873d2c9 >> which will be submitted of course. >> > > OK. But EfiSystemResetLib and EmbeddedPkg/ResetRuntimeDxe are going > away, so this commit cannot be merged as-is. I understand, I wasn't aware of those plans. Thanks, Marcin > > But since you are saying 7040 should use PSCI, I can update the patch > and drop MvResetSystemLib entirely. _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
diff --git a/Platforms/Marvell/Armada/Armada.dsc.inc b/Platforms/Marvell/Armada/Armada.dsc.inc index 475fc6d05379..32eaa2942564 100644 --- a/Platforms/Marvell/Armada/Armada.dsc.inc +++ b/Platforms/Marvell/Armada/Armada.dsc.inc @@ -94,7 +94,7 @@ # Reset and Time libraries EfiTimeBaseLib|OpenPlatformPkg/Library/EfiTimeBaseLib/EfiTimeBaseLib.inf RealTimeClockLib|OpenPlatformPkg/Platforms/Marvell/Armada/Library/RealTimeClockLib/RealTimeClockLib.inf - EfiResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf + ResetSystemLib|OpenPlatformPkg/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf # Network support NetLib|MdeModulePkg/Library/DxeNetLib/DxeNetLib.inf @@ -416,7 +416,7 @@ MdeModulePkg/Universal/Variable/EmuRuntimeDxe/EmuVariableRuntimeDxe.inf EmbeddedPkg/EmbeddedMonotonicCounter/EmbeddedMonotonicCounter.inf - EmbeddedPkg/ResetRuntimeDxe/ResetRuntimeDxe.inf + MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystemRuntimeDxe.inf EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClockRuntimeDxe.inf EmbeddedPkg/MetronomeDxe/MetronomeDxe.inf diff --git a/Platforms/Marvell/Armada/Armada70x0.fdf b/Platforms/Marvell/Armada/Armada70x0.fdf index 280b40be1d39..8091ea260261 100644 --- a/Platforms/Marvell/Armada/Armada70x0.fdf +++ b/Platforms/Marvell/Armada/Armada70x0.fdf @@ -99,7 +99,7 @@ FvNameGuid = 5eda4200-2c5f-43cb-9da3-0baf74b1b30c INF MdeModulePkg/Universal/CapsuleRuntimeDxe/CapsuleRuntimeDxe.inf INF MdeModulePkg/Universal/Variable/EmuRuntimeDxe/EmuVariableRuntimeDxe.inf INF EmbeddedPkg/EmbeddedMonotonicCounter/EmbeddedMonotonicCounter.inf - INF EmbeddedPkg/ResetRuntimeDxe/ResetRuntimeDxe.inf + INF MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystemRuntimeDxe.inf INF EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClockRuntimeDxe.inf INF EmbeddedPkg/MetronomeDxe/MetronomeDxe.inf INF OpenPlatformPkg/Drivers/I2c/MvI2cDxe/MvI2cDxe.inf diff --git a/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c b/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c index c0787aceac9c..f349c5c5387b 100644 --- a/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c +++ b/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c @@ -66,51 +66,99 @@ LibResetSystemVirtualNotifyEvent ( } /** - Resets the entire platform. - - @param ResetType The type of reset to perform. - @param ResetStatus The status code for the reset. - @param DataSize The size, in bytes, of WatchdogData. - @param ResetData For a ResetType of EfiResetCold, EfiResetWarm, - or EfiResetShutdown the data buffer starts with - a Null-terminated Unicode string, optionally - followed by additional binary data. + This function causes a system-wide reset (cold reset), in which + all circuitry within the system returns to its initial state. This type of reset + is asynchronous to system operation and operates without regard to + cycle boundaries. + + If this function returns, it means that the system does not support cold reset. **/ -EFI_STATUS +VOID EFIAPI -LibResetSystem ( - IN EFI_RESET_TYPE ResetType, - IN EFI_STATUS ResetStatus, - IN UINTN DataSize, - IN VOID *ResetData OPTIONAL +ResetCold ( + VOID ) { UINT32 Data; - switch (ResetType) { - case EfiResetCold: - case EfiResetWarm: - Data = MmioRead32 (mAddress); - Data &= ~PcdGet32 (PcdResetRegMask); - MmioWrite32 (mAddress, Data); - break; - case EfiResetShutdown: + Data = MmioRead32 (mAddress); + Data &= ~PcdGet32 (PcdResetRegMask); + MmioWrite32 (mAddress, Data); +} + +/** + This function causes a system-wide initialization (warm reset), in which all processors + are set to their initial state. Pending cycles are not corrupted. + + If this function returns, it means that the system does not support warm reset. +**/ +VOID +EFIAPI +ResetWarm ( + VOID + ) +{ + // Map a warm reset into a cold reset + ResetCold (); +} + +/** + This function causes the system to enter a power state equivalent + to the ACPI G2/S5 or G3 states. + + If this function returns, it means that the system does not support shutdown reset. +**/ +VOID +EFIAPI +ResetShutdown ( + VOID + ) +{ // // Currently there is no support for power-off platform // - break; - default: - break; - } +} + +/** + This function causes the system to enter S3 and then wake up immediately. + + If this function returns, it means that the system does not support S3 feature. +**/ +VOID +EFIAPI +EnterS3WithImmediateWake ( + VOID + ) +{ + // not implemented +} - return EFI_DEVICE_ERROR; +/** + This function causes a systemwide reset. The exact type of the reset is + defined by the EFI_GUID that follows the Null-terminated Unicode string passed + into ResetData. If the platform does not recognize the EFI_GUID in ResetData + the platform must pick a supported reset type to perform.The platform may + optionally log the parameters from any non-normal reset that occurs. + + @param[in] DataSize The size, in bytes, of ResetData. + @param[in] ResetData The data buffer starts with a Null-terminated string, + followed by the EFI_GUID. +**/ +VOID +EFIAPI +ResetPlatformSpecific ( + IN UINTN DataSize, + IN VOID *ResetData + ) +{ + // Map the platform specific reset as reboot + ResetCold (); } -EFI_STATUS +RETURN_STATUS EFIAPI LibInitializeResetSystem ( - IN EFI_HANDLE ImageHandle, - IN EFI_SYSTEM_TABLE *SystemTable + VOID ) { UINT64 Alignment; diff --git a/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf b/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf index 87fff5707909..0d5951d7e9a0 100644 --- a/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf +++ b/Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf @@ -1,4 +1,5 @@ # Copyright (C) 2016 Marvell International Ltd. +# Copyright (C) 2017 Linaro Ltd. # # Marvell BSD License Option # @@ -32,16 +33,18 @@ [Defines] INF_VERSION = 0x00010019 - BASE_NAME = Reset + BASE_NAME = MvResetSystemLib FILE_GUID = 9d1373c0-6fac-432c-88e7-818744dc45d9 MODULE_TYPE = BASE VERSION_STRING = 1.0 - LIBRARY_CLASS = EfiResetSystemLib + LIBRARY_CLASS = ResetSystemLib + CONSTRUCTOR = LibInitializeResetSystem [Sources.common] MvResetSystemLib.c [Packages] + MdeModulePkg/MdeModulePkg.dec MdePkg/MdePkg.dec OpenPlatformPkg/Platforms/Marvell/Marvell.dec
Switch to the generic implementation of the ResetSystem() runtime service call, which is preferred over the one in EmbeddedPkg. This involves selecting another DXE runtime driver, and morphing the existing EfiResetSystemLib implementation into a ResetSystemLib one (which is what the generic driver depends on for platform glue) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> --- Platforms/Marvell/Armada/Armada.dsc.inc | 4 +- Platforms/Marvell/Armada/Armada70x0.fdf | 2 +- Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.c | 110 ++++++++++++++------ Platforms/Marvell/Library/ResetSystemLib/MvResetSystemLib.inf | 7 +- 4 files changed, 87 insertions(+), 36 deletions(-) -- 2.9.3 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel