Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Platform GOP Policy #10

Merged
merged 1 commit into from
May 14, 2020
Merged

Add Platform GOP Policy #10

merged 1 commit into from
May 14, 2020

Conversation

liujunming
Copy link

GOP driver has dependency on PlatformGopPolicy protocol,
especially the GetVbtData() interface.

Tracked-On: projectacrn/acrn-hypervisor#4365

Signed-off-by: Xiong Zhang xiong.y.zhang@intel.com
Signed-off-by: Junming Liu junming.liu@intel.com

@liujunming
Copy link
Author

@peterfang Hi, peter. The GUID has changed, please review it, thanks.

@peterfang peterfang requested a review from yuchuyang May 13, 2020 08:26
@peterfang
Copy link

Approved. Handing over to @yuchuyang for SDL review.

Copy link

@peterfang peterfang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just noticed a couple minor things I'd suggest changing. Thanks.

@@ -0,0 +1,50 @@
#
#
# Copyright (c) 1999 - 2014, Intel Corporation. All rights reserved

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we update this to 2020?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK.

#
# The following information is for reference only and not required by the build tools.
#
# VALID_ARCHITECTURES = IA32

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just for reference but can we update this to X64? Or "IA32 X64"?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just for reference but can we update this to X64? Or "IA32 X64"?

ok.

@liujunming
Copy link
Author

@peterfang Hi, peter. The minor changes have done. Please review it, thanks.

@peterfang peterfang merged commit 0ff86f6 into projectacrn:ovmf-acrn May 14, 2020
GOP driver has dependency on PlatformGopPolicy protocol,
especially the GetVbtData() interface.

v2 -> v1:
change GUID

Tracked-On: projectacrn/acrn-hypervisor#4365

Signed-off-by: Xiong Zhang <xiong.y.zhang@intel.com>
Signed-off-by: Junming Liu <junming.liu@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants