On Thu, Apr 26, 2018 at 02:01:21PM +0100, Sami Mujawar wrote: > Acpiview is a program that allows examination of ACPI table > contents from the UEFI Shell. > > Enable acpiview support as this can help with investigations, > especially at that stage where the tables are not enabling an > OS to boot.
I have no objections to this, but I would prefer to see a comment from Arvind, Thomas or Daniil before merging. Adding Nariman to cc as well for good measure :) However, please use --subject-prefix="PATCH edk2-platforms“ in future. > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Sami Mujawar <[email protected]> > Reviewed-by: Evan Lloyd <[email protected]> > --- > The patch can be seen at: > https://github.com/samimujawar/edk2-platforms/tree/254_enable_acpiview_v1 > > Notes: > v1: > - Enable acpiview support in the UEFI shell for ARM platforms. [SAMI] > > Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc > b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc > index > 7cb47937329efa1944ec4c156e2be995a755ce6f..0badb24f030a58363e7bee22a7cc1e03040b60aa > 100644 > --- a/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc > +++ b/Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc > @@ -1,5 +1,5 @@ > # > -# Copyright (c) 2011-2015, ARM Limited. All rights reserved. > +# Copyright (c) 2011-2018, ARM Limited. All rights reserved. > # > # This program and the accompanying materials > # are licensed and made available under the terms and conditions of the BSD > License > @@ -450,6 +450,7 @@ [Components.common] > > NULL|ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsLib.inf > > NULL|ShellPkg/Library/UefiShellInstall1CommandsLib/UefiShellInstall1CommandsLib.inf > > NULL|ShellPkg/Library/UefiShellNetwork1CommandsLib/UefiShellNetwork1CommandsLib.inf > + > NULL|ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.inf Although I would probably move this line up a few to improve sorting. Reviewed-by: Leif Lindholm <[email protected]> > > HandleParsingLib|ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.inf > PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf > > BcfgCommandLib|ShellPkg/Library/UefiShellBcfgCommandLib/UefiShellBcfgCommandLib.inf > -- > 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' > > _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel

