On Mon, 16 Dec 2019 at 17:11, Leif Lindholm <leif.lindh...@linaro.org> wrote:
>
> On Wed, Dec 11, 2019 at 17:04:01 +0000, Pete Batard wrote:
> > From: Ard Biesheuvel <ard.biesheu...@linaro.org>
>
> Oops - sorry, I missed the From: - thanks for the poke.
>
> Yeah, I'm good with this change:
> Reviewed-by: Leif Lindholm <leif.lindh...@linaro.org>
>

Thanks

Pushed as e9395c53ba2c..479821a9677c


> > This is an improvement of e9db04631b63574b090aeab769cc47dcb75a29f7
> > where we inhibit serial output of MMIO mapped UARTs to all runtime
> > drivers rather than just RTC, as other drivers may crash the OS
> > just the same.
> >
> > Also add it to the Pi 4 platform where it was missing altogether.
> >
> > Signed-off-by: Pete Batard <p...@akeo.ie>
> > ---
> >  Platform/RaspberryPi/RPi3/RPi3.dsc | 3 ++-
> >  Platform/RaspberryPi/RPi4/RPi4.dsc | 2 ++
> >  2 files changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/Platform/RaspberryPi/RPi3/RPi3.dsc 
> > b/Platform/RaspberryPi/RPi3/RPi3.dsc
> > index 945baa5021f8..2368b04f40a4 100644
> > --- a/Platform/RaspberryPi/RPi3/RPi3.dsc
> > +++ b/Platform/RaspberryPi/RPi3/RPi3.dsc
> > @@ -200,6 +200,8 @@ [LibraryClasses.common.UEFI_DRIVER]
> >    
> > MemoryAllocationLib|MdePkg/Library/UefiMemoryAllocationLib/UefiMemoryAllocationLib.inf
> >
> >  [LibraryClasses.common.DXE_RUNTIME_DRIVER]
> > +  # Runtime debug messages may crash an OS unless serial output to MMIO 
> > mapped UARTs is inhibited
> > +  
> > DebugLib|MdePkg/Library/DxeRuntimeDebugLibSerialPort/DxeRuntimeDebugLibSerialPort.inf
> >    
> > MemoryAllocationLib|MdePkg/Library/UefiMemoryAllocationLib/UefiMemoryAllocationLib.inf
> >    CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
> >    EfiResetSystemLib|Platform/RaspberryPi/Library/ResetLib/ResetLib.inf
> > @@ -505,7 +507,6 @@ [Components.common]
> >    EmbeddedPkg/ResetRuntimeDxe/ResetRuntimeDxe.inf
> >    EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClockRuntimeDxe.inf {
> >      <LibraryClasses>
> > -      
> > DebugLib|MdePkg/Library/DxeRuntimeDebugLibSerialPort/DxeRuntimeDebugLibSerialPort.inf
> >        
> > RealTimeClockLib|EmbeddedPkg/Library/VirtualRealTimeClockLib/VirtualRealTimeClockLib.inf
> >    }
> >    EmbeddedPkg/MetronomeDxe/MetronomeDxe.inf
> > diff --git a/Platform/RaspberryPi/RPi4/RPi4.dsc 
> > b/Platform/RaspberryPi/RPi4/RPi4.dsc
> > index 3764d1b3d794..03139e57a8db 100644
> > --- a/Platform/RaspberryPi/RPi4/RPi4.dsc
> > +++ b/Platform/RaspberryPi/RPi4/RPi4.dsc
> > @@ -202,6 +202,8 @@ [LibraryClasses.common.UEFI_DRIVER]
> >    
> > MemoryAllocationLib|MdePkg/Library/UefiMemoryAllocationLib/UefiMemoryAllocationLib.inf
> >
> >  [LibraryClasses.common.DXE_RUNTIME_DRIVER]
> > +  # Runtime debug messages may crash an OS unless serial output to MMIO 
> > mapped UARTs is inhibited
> > +  
> > DebugLib|MdePkg/Library/DxeRuntimeDebugLibSerialPort/DxeRuntimeDebugLibSerialPort.inf
> >    
> > MemoryAllocationLib|MdePkg/Library/UefiMemoryAllocationLib/UefiMemoryAllocationLib.inf
> >    CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
> >    EfiResetSystemLib|Platform/RaspberryPi/Library/ResetLib/ResetLib.inf
> > --
> > 2.21.0.windows.1
> >

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#52435): https://edk2.groups.io/g/devel/message/52435
Mute This Topic: https://groups.io/mt/68154411/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to