On 13 April 2016 at 10:10, Ard Biesheuvel <ard.biesheu...@linaro.org> wrote: > PL031RealTimeClockLib is a base library that could potentially (although > unlikely) be incorporated into other modules than the DXE_RUNTIME_DRIVER > module that it was intended to complement. > > This means the library has no business whatsoever setting the Runtime > Service table pointers directly (since we have no way of knowing which > instance will 'win', and the pointers may end up referring to a module > that is not a DXE_RUNTIME_DRIVER). So remove the assignment altogether. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org> > --- > ArmPlatformPkg/Library/PL031RealTimeClockLib/PL031RealTimeClockLib.c | 6 > ------ > 1 file changed, 6 deletions(-) > > diff --git > a/ArmPlatformPkg/Library/PL031RealTimeClockLib/PL031RealTimeClockLib.c > b/ArmPlatformPkg/Library/PL031RealTimeClockLib/PL031RealTimeClockLib.c > index 52ba48992b83..516b45675c69 100644 > --- a/ArmPlatformPkg/Library/PL031RealTimeClockLib/PL031RealTimeClockLib.c > +++ b/ArmPlatformPkg/Library/PL031RealTimeClockLib/PL031RealTimeClockLib.c > @@ -650,12 +650,6 @@ LibRtcInitialize ( > return Status; > } > > - // Setup the setters and getters > - gRT->GetTime = LibGetTime; > - gRT->SetTime = LibSetTime; > - gRT->GetWakeupTime = LibGetWakeupTime; > - gRT->SetWakeupTime = LibSetWakeupTime; > - > mRT = gRT; > > // Install the protocol > -- > 2.5.0 >
Ping? (also for the next one) _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel