Re: [edk2] [PATCH] BaseTools/CommonLib: drop the use of MAX_ADDRESS

2018-12-05 Thread Ard Biesheuvel
On Thu, 6 Dec 2018 at 00:56, Gao, Liming wrote: > > Reviewed-by: Liming Gao > Thanks all Pushed as 6e8cad49a09d..67938bcc9d9e > > -Original Message- > > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard > > Biesheuvel > > Sent: Wednesday, December 5, 2018

Re: [edk2] [PATCH] Maintainers.txt: Remove DuetPkg

2018-12-05 Thread Ni, Ruiyu
On 12/6/2018 11:11 AM, Shenglei Zhang wrote: -DuetPkg -W:https://github.com/tianocore/tianocore.github.io/wiki/DuetPkg -M: Ruiyu Ni -M: Hao Wu Reviewed-by: Ruiyu Ni -- Thanks, Ray ___ edk2-devel mailing list edk2-devel@lists.01.org

[edk2] [PATCH] CryptoPkg/IntrinsicLib: add missing BaseLib declaration

2018-12-05 Thread Jian J Wang
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=596 BaseLib interfaces are used in this library but not declared in module's inf file. This patch fix this situation to keep inf and its code in consistency. No functionality or interface change are involved. Cc: Qin Long Cc: Ting Ye

Re: [edk2] [PATCH v2] Maintainers.txt: Change package maintainer of IntelFsp*Pkg

2018-12-05 Thread Yao, Jiewen
Reviewed-by: jiewen@intel.com > -Original Message- > From: Chiu, Chasel > Sent: Wednesday, December 5, 2018 8:49 AM > To: edk2-devel@lists.01.org > Cc: Yao, Jiewen ; Desimone, Nathaniel L > ; Zeng, Star ; Chiu, > Chasel > Subject: [PATCH v2] Maintainers.txt: Change package maintainer

[edk2] [PATCH] Maintainers.txt: Remove DuetPkg

2018-12-05 Thread Shenglei Zhang
Since DuetPkg is due to be removed, Maintainers.txt should also be updated. https://bugzilla.tianocore.org/show_bug.cgi?id=1322 Cc: Ruiyu Ni Cc: Hao Wu Cc: Andrew Fish Cc: Laszlo Ersek Cc: Leif Lindholm Cc: Michael D Kinney Contributed-under: TianoCore Contribution Agreement 1.1

Re: [edk2] [PATCH v4 0/2] Remove DuetPkg and unused tools

2018-12-05 Thread Gao, Liming
Reviewed-by: Liming Gao for BaseTools part. Besides, please also update Maintainers.txt to remove DuetPkg. >-Original Message- >From: Ni, Ruiyu >Sent: Thursday, December 06, 2018 10:58 AM >To: Zhang, Shenglei ; edk2-devel@lists.01.org >Cc: Wu, Hao A ; Zhu, Yonghong >; Gao, Liming

Re: [edk2] [PATCH v4 0/2] Remove DuetPkg and unused tools

2018-12-05 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni Thanks/Ray > -Original Message- > From: Zhang, Shenglei > Sent: Monday, December 3, 2018 10:18 AM > To: edk2-devel@lists.01.org > Cc: Ni, Ruiyu ; Wu, Hao A ; Zhu, > Yonghong ; Gao, Liming > Subject: [PATCH v4 0/2] Remove DuetPkg and unused tools > > DuetPkg

Re: [edk2] [PATCH v2] ArmPkg/ArmScmiDxe: Add clock enable function

2018-12-05 Thread Jeff Brasen
Leif/Ard, Any comments on this v2 patch for this? Thanks, Jeff From: Jeff Brasen Sent: Wednesday, November 28, 2018 1:36:16 PM To: edk2-devel@lists.01.org Cc: Jeff Brasen; Ard Biesheuvel; Leif Lindholm; Grish Pathak Subject: [PATCH v2] ArmPkg/ArmScmiDxe:

Re: [edk2] [Patch] UefiCpuPkg/MpLib: Fix PEI Services Table pointer on AP

2018-12-05 Thread Felix Polyudov
Andrew, I think there are two aspects here: 1) What's the final solution we would like to see? I think DebugLibMpSafe and a fake PEI Services Table that you are proposing are good ideas. Ideally, it should still be possible to use the real PEI Services table by overriding a library class.

Re: [edk2] Build fail using latest UDK2018

2018-12-05 Thread Gao, Liming
How reproduce this issue? Could you list your step? >-Original Message- >From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of >Prem Kumar >Sent: Thursday, December 06, 2018 2:37 AM >To: edk2-devel@lists.01.org >Subject: [edk2] Build fail using latest UDK2018 > >Hi All, >

Re: [edk2] [PATCH] BaseTools/CommonLib: drop the use of MAX_ADDRESS

2018-12-05 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard > Biesheuvel > Sent: Wednesday, December 5, 2018 4:24 PM > To: edk2-devel@lists.01.org > Cc: ler...@redhat.com; Gao, Liming > Subject: [edk2] [PATCH]

Re: [edk2] [PATCH edk2-platforms 5/6] Platform: add acpiview to Seattle/Styx platforms

2018-12-05 Thread Carsey, Jaben
You didn't CC anyone, but that is the right syntax for adding the shell command. Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Ard Biesheuvel > Sent: Wednesday, December 05, 2018 10:50 AM > To:

[edk2] [PATCH edk2-platforms] Platform/AMD/OverdriveBoard: fix byte order of default MAC addresses

2018-12-05 Thread Ard Biesheuvel
The PCDs containing the default MAC addresses are of type UINT64, and so the byte order needs to be inverted. As they are currently, both default MAC addresses are invalid since they have the multicast bit set. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel

[edk2] [PATCH edk2-platforms 5/6] Platform: add acpiview to Seattle/Styx platforms

2018-12-05 Thread Ard Biesheuvel
Enable the 'acpiview' UEFI shell command so we can inspect the ACPI tables at boot time. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel --- Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 1 + Platform/LeMaker/CelloBoard/CelloBoard.dsc

[edk2] [PATCH edk2-platforms 3/6] Silicon/AMD/Styx: move XGBE declarations to separate SSDT table

2018-12-05 Thread Ard Biesheuvel
Move the XGBE out of the DSDT, and along with it the logic that patches the correct MAC address into the device nodes. However, this time we patch the SSDT binary directly rather than relying on intermediate output of an outdated version of the iasl compiler. Contributed-under: TianoCore

[edk2] [PATCH edk2-platforms 6/6] Platform/AMD/OverdriveBoard: use default resolution for GOP

2018-12-05 Thread Ard Biesheuvel
Set the GOP resolution to 0x0 so that the resolution will be chosen by the driver, usually based on the capabilities of the connected display. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel --- Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 3 +++ 1 file

[edk2] [PATCH edk2-platforms 1/6] Silicon/AMD/Styx: move SOC version macros to common header

2018-12-05 Thread Ard Biesheuvel
Instead of adding yet another redefinition in the next patch, move the silicon revision testing macros into a shared header file. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel --- Silicon/AMD/Styx/Common/SocVersion.h | 19

[edk2] [PATCH edk2-platforms 4/6] Silicon/AMD/Styx: emit DSDT as aml directly

2018-12-05 Thread Ard Biesheuvel
Instead of emitting the DSDT by incorporating the intermediate output of [some version of] the iasl compiler, move the DSDT source file to the ACPI platform driver, which will install it directly. This permits us to drop a lot of cruft related to handling of this intermediate output.

[edk2] [PATCH edk2-platforms 2/6] Silicon/AMD/Styx: move B1 revision peripherals to separate SSDT table

2018-12-05 Thread Ard Biesheuvel
Instead of poking DSDT _STA method bytecode to make it return something else depending on whether we are running on B1 silicon, move the B1 only peripherals to a separate SSDT and only install it when running on compatible hardware. Contributed-under: TianoCore Contribution Agreement 1.1

[edk2] [PATCH edk2-platforms 0/6] Styx: fixes for IASL abuse and more

2018-12-05 Thread Ard Biesheuvel
Primarily, this series gets rid of the hacked up way this platform patches the DSDT at build time, by #include'ing intermediate output of the iasl compiler [or some version of it, at least] While at it, apply some other cleanups/improvements. Ard Biesheuvel (6): Silicon/AMD/Styx: move SOC

[edk2] Build fail using latest UDK2018

2018-12-05 Thread Prem Kumar
Hi All, When I try to build UDK2018 package I'm facing below build error. *Note:* When I delete the build folder the same build error is not happening. The same build error is not happening when build is happening for first time. Any pointers is appreciated. build.py... : error C0DE:

Re: [edk2] [edk2-announce] Research Request

2018-12-05 Thread Rebecca Cran via edk2-devel
On Wednesday, 5 December 2018 05:55:41 MST Laszlo Ersek wrote: > Can you assist with the following please? Also, a couple of notes: Go to https://code.bluestop.org/settings/user/lersek/ to configure preferences related to emails (http/plain), diffs etc. Install the arcanist package on your

Re: [edk2] [edk2-announce] Research Request

2018-12-05 Thread Rebecca Cran via edk2-devel
On Wednesday, 5 December 2018 05:55:41 MST Laszlo Ersek wrote: > (1) Pls. explain to me how I can create an edk2 clone at > . :) You don't. In a production system it may be possible to clone from either GitHub or code.bluestop.org (which mirrors github), but the clone URL given when you click

[edk2] Community Meeting Reminder

2018-12-05 Thread stephano
Hey All, Quick reminder that our community meetings will held in about 24 hours or so. Remember that there are two: one at 17:00 GMT and one at 3:00 GMT (+1 day), so please attend the one that is most convenient for your timezone (or come to both! :) ). Details and calendar invites here:

Re: [edk2] [edk2-announce] Research Request

2018-12-05 Thread stephano
On 12/4/2018 10:20 AM, Philippe Mathieu-Daudé wrote: Hi Stephano, I'll run your checklist [*] with GitLab on Thursday. Perfect, thank you Philippe. Cheers, Stephano ___ edk2-devel mailing list edk2-devel@lists.01.org

[edk2] [Patch] edk2 wiki: remove edk2-stable201811 tag planning in EDK-II-Release-Planning

2018-12-05 Thread Liming Gao
edk2-stable201811 tag has been created. its planning can be removed. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- EDK-II-Release-Planning.md | 24 1 file changed, 24 deletions(-) diff --git a/EDK-II-Release-Planning.md

[edk2] [Patch] edk2 Readme.md: Add edk2 release tag and edk2 release plan

2018-12-05 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=1364 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- Readme.md | 4 1 file changed, 4 insertions(+) diff --git a/Readme.md b/Readme.md index 1ef0780ee0..bf7c97cd2b 100644 --- a/Readme.md +++ b/Readme.md

Re: [edk2] [edk2-announce] Research Request

2018-12-05 Thread Laszlo Ersek
On 12/03/18 22:39, Rebecca Cran wrote: > On Monday, 3 December 2018 02:29:28 MST Laszlo Ersek wrote: >> On 11/29/18 22:20, Rebecca Cran wrote: >>> Would you be interested in going through this process with Phabricator, >>> too? >> Sure! Just tell me where to create an account. > > Go to

Re: [edk2] Shell EFI (ipconfig.efi) for IPConfig2 Protocol

2018-12-05 Thread Ye, Ting
Hi, You may try "ifconfig.efi" under shell, which supports Ip4Config2 protocol. The source code is available under ShellPkg\Library\UefiShellNetwork1CommandsLib\Ifconfig.c Thanks, Ting -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of

Re: [edk2] [PATCH] BaseTools/CommonLib: drop the use of MAX_ADDRESS

2018-12-05 Thread Laszlo Ersek
On 12/05/18 09:24, Ard Biesheuvel wrote: > The macro MAX_ADDRESS represents the largest virtual address that > is valid for a certain architecture. For the BaseTools, this quantity > is irrelevant, since the same tools can be used to build for different > targets. > > Since we only refer to it in

Re: [edk2] [PATCH] BaseTools/CommonLib: drop the use of MAX_ADDRESS

2018-12-05 Thread Philippe Mathieu-Daudé
On 5/12/18 9:24, Ard Biesheuvel wrote: > The macro MAX_ADDRESS represents the largest virtual address that > is valid for a certain architecture. For the BaseTools, this quantity > is irrelevant, since the same tools can be used to build for different > targets. > > Since we only refer to it in a

[edk2] [PATCH v1 1/1] ArmPkg/OpteeLib: Add dummy RPC handler

2018-12-05 Thread Sumit Garg
Add dummy RPC handler for RPCs that are not implemented as control should be returned back to OP-TEE in case any RPC is invoked. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Sumit Garg --- ArmPkg/Library/OpteeLib/OpteeSmc.h | 3

Re: [edk2] [PATCH edk2-platforms 13/27] Silicon/NXP: Add support for iMX SDHC

2018-12-05 Thread Leif Lindholm
On Fri, Sep 21, 2018 at 08:26:04AM +, Chris Co wrote: > This adds support for using the SD host controller on > NXP i.MX platforms. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Christopher Co > Cc: Ard Biesheuvel > Cc: Leif Lindholm > Cc: Michael D Kinney >

Re: [edk2] [edk2-platforms] [PATCH v6 2/2] Platform/ARM: Add Readme.md

2018-12-05 Thread Nariman Poushin
On Wed, 5 Dec 2018 at 09:45, Leif Lindholm wrote: > Hi Nariman, > > Sorry, one more round: can you replace the tags for 'text` and > ```text > more text > even more text > ``` > please? > No problem, will scrub and resend! > > And the with **text**? > > (And if I missed any more inline

Re: [edk2] [edk2-platforms] [PATCH v6 2/2] Platform/ARM: Add Readme.md

2018-12-05 Thread Leif Lindholm
Hi Nariman, Sorry, one more round: can you replace the tags for 'text` and ```text more text even more text ``` please? And the with **text**? (And if I missed any more inline HTML, please convert that too.) Regards, Leif On Fri, Nov 30, 2018 at 01:56:36PM +, Nariman Poushin wrote: >

[edk2] Shell EFI (ipconfig.efi) for IPConfig2 Protocol

2018-12-05 Thread Ankit.Singh3
Hi, Do we have any ipconfig.efi type utility to see the IP address details on shell (all the existing utilities are old and does not support new Ipconfig2 protocol) Regards, Ankit Singh ___ edk2-devel mailing list edk2-devel@lists.01.org

[edk2] [PATCH] MdeModulePkg/PciBus: Shadow option ROM after BARs are programmed

2018-12-05 Thread Ruiyu Ni
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1376 Today's implementation reuses the 32bit MMIO resource requested by all PCI devices MMIO BARs when shadowing the option ROM. Take a simple example, a system has only one PCI device. It requires 8MB 32bit MMIO and contains a 4MB option ROM.

[edk2] [PATCH] BaseTools/CommonLib: drop the use of MAX_ADDRESS

2018-12-05 Thread Ard Biesheuvel
The macro MAX_ADDRESS represents the largest virtual address that is valid for a certain architecture. For the BaseTools, this quantity is irrelevant, since the same tools can be used to build for different targets. Since we only refer to it in a single place, which is an ASSERT() that doesn't

Re: [edk2] [PATCH v2 0/6] BaseTools: get rid of MAX_UINTN

2018-12-05 Thread Ard Biesheuvel
On Wed, 5 Dec 2018 at 01:04, Gao, Liming wrote: > > Reviewed-by: Liming Gao for this serials. > Thanks all Series pushed as 64ab2c82e8f6..8efc6d84ca41 (with the requested MAX_UINT16 -> MAX_UINT32 change applied) > On patch 4, I have the same comments to Laszlo. > > >-Original