On 02/16/15 22:24, Jordan Justen wrote:
> 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?
Yes.
Thanks!
Laszlo
------------------------------------------------------------------------------
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
