Re: [edk2-devel] [PATCH V3 2/6] IntelSiliconPkg/Include: Add Platform Device Security Policy protocol

2019-11-08 Thread Yao, Jiewen
Hi Ray I checked the UEFI spec and other protocol definition again. I believe Revision is a better name for minor update. I will update the structure field from Version to Revision. Thank you Yao Jiewen > -Original Message- > From: Ni, Ray > Sent: Friday, November 8, 2019 3:10 PM > To:

Re: [edk2-devel] [PATCH 0/2] Reserve page 0 for NULL pointer detection

2019-11-08 Thread Wang, Jian J
Pushed at fb92fe9e1817a53ca0fc985447f3c534201a62fa Regards, Jian > -Original Message- > From: Ni, Ray > Sent: Thursday, November 07, 2019 10:11 AM > To: Wang, Jian J ; devel@edk2.groups.io > Cc: Bi, Dandan ; Gao, Liming ; > Wu, Hao A ; Sean Brogan > Subject: RE: [PATCH 0/2] Reserve page

Re: [edk2-devel] [PATCH] MinPlatformPkg: Introduce library for customizing TPM platform configuration

2019-11-08 Thread Kubacki, Michael A
Also, the V1 patch was sent to edk2-de...@lists.01.org. Please use devel@edk2.groups.io. Thanks, Michael > -Original Message- > From: Kubacki, Michael A > Sent: Friday, November 8, 2019 12:11 PM > To: Gonzalez Del Cueto, Rodrigo ; > edk2-de...@lists.01.org > Cc: Chiu, Chasel ; Desimone,

Re: [edk2-devel] [edk2] OCP : Olympus project's UEFI BIOS code support

2019-11-08 Thread Oram, Isaac W
The Purley/Mt Olympus was just a proof of concept for a demo at OCP Global Summit. The priority was getting things working, not necessarily cleanly. I don't know the specific details, but we likely used a binary version because we required some non-public information for the Purley silicon. R

Re: [edk2-devel] EDK II Maintainers please test the pull request commit gate

2019-11-08 Thread rebecca
On 11/8/19 10:10 AM, Michael D Kinney wrote: 1) You can run the CI tests on your local dev system. Fix issues locally and rerun as needed before sending patch review email. This will only perform build tests with your local compiler. 2) You can push a branch with changes to your pe

Re: [edk2-devel] EDK II Maintainers please test the pull request commit gate

2019-11-08 Thread Michael D Kinney
Laszlo, Agreed. A GitHub fork is required. We can make that clear in the developer process wiki. Mike > -Original Message- > From: devel@edk2.groups.io On > Behalf Of Laszlo Ersek > Sent: Friday, November 8, 2019 9:12 AM > To: devel@edk2.groups.io; Kinney, Michael D > > Subject: Re:

Re: [edk2-devel] [Bug 2321] Add FitGen feature to support uCode Capsule Update

2019-11-08 Thread Chen, Kenji
Having problems in git send-email. Send it by outlook. -Original Message- From: bugzilla-dae...@bugzilla.tianocore.org Sent: Monday, November 4, 2019 10:14 AM To: Chen, Kenji Subject: [Bug 2321] Add FitGen feature to support uCode Capsule Update https://bugzilla.tianocore.org/show_bug

Re: [edk2-devel] EDK II Maintainers please test the pull request commit gate

2019-11-08 Thread Laszlo Ersek
Hi Mike, On 11/07/19 20:26, Michael D Kinney wrote: > Laszlo, > > The hub command line utility also shows an option to > create a fork without using the GitHub UI. But since I > already have all the relevant forks I needed, I have not > tried this feature yet. sorry, I was unclear with my "fork

Re: [edk2-devel] EDK II Maintainers please test the pull request commit gate

2019-11-08 Thread Michael D Kinney
Phil, Yes. There are two methods available 1) You can run the CI tests on your local dev system. Fix issues locally and rerun as needed before sending patch review email. This will only perform build tests with your local compiler. 2) You can push a branch with changes to your persona

Re: [edk2-devel] [Patch v4 03/22] requirements.txt: Add python pip requirements file

2019-11-08 Thread Michael D Kinney
Laszlo, Thank you for the extra effort on this evaluation. I will change to pip-requirements.txt Mike > -Original Message- > From: Laszlo Ersek > Sent: Friday, November 8, 2019 5:13 AM > To: Kinney, Michael D ; > devel@edk2.groups.io > Cc: Sean Brogan ; Andrew Fish > ; Leif Lindholm >

Re: [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-11-08 Thread Laszlo Ersek
On 11/07/19 18:46, Jeff Brasen wrote: > Fixing UiApp seems reasonable, I do think we would want a hook to the > platform library in here as the enumeration that occurs in the UiApp is > intended to do a full enumeration of the system and there may be platform > specifics to how that occurs. Ful

[edk2-devel] [staging/branch]: CdePkg - added unit test for clock() function

2019-11-08 Thread Kilian Kegel
Hi UEFI community, I’d like to inform you about an update in the CdePkg branch of edk2-staging. https://github.com/tianocore/edk2-staging/tree/CdePkg#cdepkg The goal of the CdePkg is to provide an ANSI C Interface for UEFI POST drivers, that enables BIOS engineers to (more) easily port standard

Re: [edk2-devel] EDK II Maintainers please test the pull request commit gate

2019-11-08 Thread Philippe Mathieu-Daudé
Hi Michael, On 11/7/19 3:24 AM, Michael D Kinney wrote: EDK II Maintainers, Please take a few minutes to test the GitHub pull request commit gate feature under review in the edk2-staging/edk2-ci branch. This will be the only method to push patches to the edk2 repo in the near future, and we wa

Re: [edk2-devel] [Patch v4 17/22] UefiCpuPkg: Add YAML file for CI builds

2019-11-08 Thread Laszlo Ersek
On 11/07/19 20:33, Sean Brogan wrote: > Laszlo, > > Our legal team has requested that we don't include the copyright year. This sounds unusual, per https://en.wikipedia.org/wiki/Copyright_notice#Technical_requirements and also per https://edk2-docs.gitbooks.io/edk-ii-c-coding-standards-spec

Re: [edk2-devel] [PATCH V3 0/6] Add Device Security driver

2019-11-08 Thread Javeed, Ashraf
True, thought PCI is a vast topic, could be many more sample drivers in future, thus having under one" Pci" folder would be better. I know this is your third version already, and I could have reviewed it earlier and made this point. No issues now, could be moved in future. Regards Ashraf > ---

Re: [edk2-devel] [edk2-staging/UEFI_PCI_ENHANCE-2 PATCH V2 00/12] New PCI features - MPS, MRRS, RO, NS, CTO

2019-11-08 Thread Javeed, Ashraf
These patches are also updated in the following EDK2 repo:- https://github.com/ashrafj/edk2-staging/commits/UEFI_PCI_ENHANCE-2 Please let me know if you need anything else with this regard. Thanks Ashraf > -Original Message- > From: Ni, Ray > Sent: Friday, November 8, 2019 11:42 AM > To:

[edk2-devel] [edk2] AMD : Styx/Overdrive board

2019-11-08 Thread Tiger Liu(BJ-RD)
Dear All: I found edk2-platforms\Platform\AMD\OverdriveBoard indicated it is related with OverdriveBoard. But I could not find this Motherboard introduction. Thanks 保密声明: 本邮件含有保密或专有信息,仅供指定收件人使用。严禁对本邮件或其内容做任何未经授权的查阅、使用、复制或转发。 CONFIDENTIAL NOTE: This email contains confidential or legally privil

Re: [edk2-devel] [Patch v4 22/22] Readme.md: Add CI build status badges

2019-11-08 Thread Leif Lindholm
Hi Mike, On Thu, Nov 07, 2019 at 07:42:27PM +, Kinney, Michael D wrote: > Hi Leif, > > I really like the build status information in Readme.md > so it is visible whenever anyone navigates to the edk2 > repo. This matches what many other open source projects > provide. > > GitHub markdown do

Re: [edk2-devel] [PATCH] IntelSiliconPkg/Library:Add BaseConfigBlockLib Library

2019-11-08 Thread Ni, Ray
When a API needs enough buffer prepared by caller, there are two styles of APIs: 1. Status FillBuffer (VOID *Buffer, UINTN BufferSize) BufferSize is passed by value so the API has no way to return back the required size. It's ok as long as the caller is able to calculate the proper size

Re: [edk2-devel] [PATCH 1/1] BaseTools/GenFw AARCH64: disregard ADRP instructions that are patched already

2019-11-08 Thread Ard Biesheuvel
On Fri, 8 Nov 2019 at 01:39, Gao, Liming wrote: > > Acked-by: Liming Gao > Thanks all Pushed as 9e9f0be353d4..f55c76b301f1 > >-Original Message- > >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ard > >Biesheuvel > >Sent: Thursday, November 07, 2019 5:06 PM > >To