On 2015-02-16 08:25:48, Laszlo Ersek wrote:
> On 02/16/15 03:06, Jordan Justen wrote:
> > Since the protocol is in the depex, there is no reason to expect we
> > might fail to locate the protocol.
> >
> > Contributed-under: TianoCore Contribution Agreement 1.0
> > Signed-off-by: Jordan Justen <jordan.l.jus...@intel.com>
> > ---
> > OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c | 4 +---
> > 1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c
> > b/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c
> > index 11f0ca8..331cdc4 100644
> > --- a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c
> > +++ b/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.c
> > @@ -250,9 +250,7 @@ AcpiPlatformEntryPoint (
> > NULL,
> > (VOID**)&AcpiTable
> > );
> > - if (EFI_ERROR (Status)) {
> > - return EFI_ABORTED;
> > - }
> > + ASSERT_EFI_ERROR (Status);
> >
> > if (XenDetected ()) {
> > Status = InstallXenTables (AcpiTable);
> >
>
> This is true of the depexes of both INF files, hence the patch should
> modify both entry point functions, I think.
r-b with that change?
-Jordan
------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel
