executing
under the OS.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
Reviewed-by: Jian J Wang
---
MdeModulePkg/Core/Dxe/Gcd/Gcd.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/MdeModulePkg/Core/Dxe/Gcd/Gcd.c b/MdeModulePkg
macro instead of MAX_ADDRESS.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
Reviewed-by: Laszlo Ersek
---
ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuLibCore.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ArmPkg/Library/ArmMmuLib/AA
below this limit to above
it should be split.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
Reviewed-by: Laszlo Ersek
---
ArmVirtPkg/Library/ArmVirtMemoryInitPeiLib/ArmVirtMemoryInitPeiLib.c | 8
1 file changed, 4 insertions(+), 4 deletions
-by: Ard Biesheuvel
---
MdeModulePkg/Core/Dxe/Mem/Page.c | 52 ++--
1 file changed, 26 insertions(+), 26 deletions(-)
diff --git a/MdeModulePkg/Core/Dxe/Mem/Page.c b/MdeModulePkg/Core/Dxe/Mem/Page.c
index 961c5b833546..5ad8e1171ef7 100644
--- a/MdeModulePkg/Core/Dxe/Mem/Page.c
+++ b
-by: Ard Biesheuvel
---
MdePkg/Include/AArch64/ProcessorBind.h | 5 +
MdePkg/Include/Arm/ProcessorBind.h | 5 +
MdePkg/Include/Ebc/ProcessorBind.h | 5 +
MdePkg/Include/Ia32/ProcessorBind.h| 5 +
MdePkg/Include/X64/ProcessorBind.h | 5 +
5 files changed, 25
: Eric Auger
Cc: Andrew Jones
Cc: Philippe Mathieu-Daude
Ard Biesheuvel (6):
MdePkg/Base: introduce MAX_ALLOC_ADDRESS
MdeModulePkg/Dxe/Gcd: disregard memory above MAX_ALLOC_ADDRESS
MdeModulePkg/Dxe/Page: take MAX_ALLOC_ADDRESS into account
ArmPkg/ArmMmuLib: take MAX_ALLOC_ADDRESS into
eement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPlatformPkg/ArmPlatformPkg.dec| 1 +
ArmPlatformPkg/Drivers/SP805WatchdogDxe/SP805WatchdogDxe.inf | 6 +-
ArmPlatformPkg/Drivers/SP805WatchdogDxe/SP805Watchdog.c | 100
+++-
3 files changed, 80
Clean up the code, by adding missing STATIC modifiers, drop
redundant casts, and get rid of the 'success handling' anti
pattern in the entry point code.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
Reviewed-by: Leif Lindholm
---
ArmP
Before fixing the SP805 driver, let's clean it up a bit. No
functional changes.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
Reviewed-by: Leif Lindholm
---
ArmPlatformPkg/Drivers/SP805WatchdogDxe/SP805WatchdogDxe.inf | 11 +--
ArmPlatformPkg/Dr
registered handler
returns
- WARN() if running the SP805 driver with interrupt handling disabled
- add some R-bs
Cc: Leif Lindholm
Cc: Sami Mujawar
Cc: Thomas Panakamattam Abraham
Cc: Meenakshi Aggarwal
Cc: Udit Kumar
Cc: Matteo Carlini
Cc: Nariman Poushin
Ard Biesheuvel (4):
ArmPlatformPk
ff-by: Ard Biesheuvel
Reviewed-by: Leif Lindholm
---
ArmPkg/Drivers/GenericWatchdogDxe/GenericWatchdogDxe.c | 34
++--
1 file changed, 25 insertions(+), 9 deletions(-)
diff --git a/ArmPkg/Drivers/GenericWatchdogDxe/GenericWatchdogDxe.c
b/ArmPkg/Drivers/GenericWatchd
On Wed, 12 Dec 2018 at 13:36, Leif Lindholm wrote:
>
> On Wed, Dec 12, 2018 at 01:19:31PM +0100, Ard Biesheuvel wrote:
> > The ARM linker may emit veneers, i.e., trampolines, when ordinary
> > direct relative branches cannot be used, e.g., for Thumb interworking
> > or bra
On Wed, 19 Dec 2018 at 20:11, Leif Lindholm wrote:
>
> On Wed, Dec 19, 2018 at 06:39:07PM +0100, Ard Biesheuvel wrote:
> > Ensure that we prevent the CPU from proceeding after having taken an
> > unhandled exception on a RELEASE build, which does not contain the
> > ASSERT
-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
v2: remove pointless 'if (!PcdAdjust)' conditional
ArmPkg/Library/DefaultExceptionHandlerLib/Arm/DefaultExceptionHandler.c | 2 ++
1 file changed, 2 insertions(+)
diff --git
a/ArmPkg/Library/DefaultException
On Wed, 19 Dec 2018 at 18:26, Leif Lindholm wrote:
>
> On Wed, Dec 19, 2018 at 06:06:14PM +0100, Ard Biesheuvel wrote:
> > Remove the DO_CAPSULE conditionals so that the capsules are
> > always built. This fixes a build issue that exists currently,
> > since the IPv6 ena
: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 9 -
Platform/AMD/OverdriveBoard/OverdriveBoard.fdf | 9 -
2 files changed, 18 deletions(-)
diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
b/Platform/AMD
irtio network device
>
> Vijayenthiran Subramaniam (4):
> SgiClark.Ares: AcpiTables: Use PCDs for virtio disk
> SgiClark.Ares: AcpiTables: Add entry for virtio network device
> SgiClark.Helios: AcpiTables: Use PCDs for virtio disk
> SgiClark.Helios: AcpiTables: Add entry for virtio
On Wed, 19 Dec 2018 at 16:30, Ard Biesheuvel wrote:
>
> On Wed, 19 Dec 2018 at 16:20, Leif Lindholm wrote:
> >
> > On Wed, Dec 12, 2018 at 03:17:29PM +, Leif Lindholm wrote:
> > > > Considering the commit log from its introduction
>
On Tue, 18 Dec 2018 at 14:05, Ard Biesheuvel wrote:
>
> On Tue, 18 Dec 2018 at 14:04, Sumit Garg wrote:
> >
> > On Tue, 18 Dec 2018 at 18:30, Ard Biesheuvel
> > wrote:
> > >
> > > On Tue, 18 Dec 2018 at 13:58, Sumit Garg wrote:
> > > >
buted-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
BaseTools/Conf/tools_def.template | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/BaseTools/Conf/tools_def.template
b/BaseTools/Conf/tools_def.template
index 2ba833e1fb06..f82bc72327d8 1
; Signed-off-by: Leif Lindholm
Reviewed-by: Ard Biesheuvel
> ---
> Silicon/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.inf | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/Silicon/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.inf
> b/Silicon/AMD/Styx/Librar
fined, without actual users.
> So get rid of them.
>
> One reference to be deleted separately from edk2-platforms.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Leif Lindholm
Reviewed-by: Ard Biesheuvel
> ---
> ArmPkg/ArmPkg.dec | 20 ---
since the default timeout set by the DXE core is
5 minutes, which is not representable in 32-bit at this clock rate.
So switch to the SBSA watchdog instead, which is wired up to the
generic timer, and ticks at the correct rate.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-o
On Tue, 18 Dec 2018 at 14:39, Leif Lindholm wrote:
>
> On Tue, Dec 18, 2018 at 02:10:12PM +0100, Ard Biesheuvel wrote:
> > The SP805 watchdog driver doesn't implement the PI watchdog protocol
> > fully, but always simply resets the system if the watchdog time runs
> &
On Tue, 18 Dec 2018 at 16:58, Udit Kumar wrote:
>
>
>
> > -Original Message-----
> > From: Ard Biesheuvel
> > Sent: Tuesday, December 18, 2018 6:40 PM
> > To: edk2-devel@lists.01.org
> > Cc: Ard Biesheuvel ; Leif Lindholm
> > ; Sami Mujawar
On Tue, 18 Dec 2018 at 14:43, Leif Lindholm wrote:
>
> On Tue, Dec 18, 2018 at 02:10:13PM +0100, Ard Biesheuvel wrote:
> > Clean up the code, by adding missing STATIC modifiers, drop
> > redundant casts, and get rid of the 'success handling' anti
> &g
Clean up the code, by adding missing STATIC modifiers, drop
redundant casts, and get rid of the 'success handling' anti
pattern in the entry point code.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPkg/Drivers/GenericW
eement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPlatformPkg/ArmPlatformPkg.dec| 1 +
ArmPlatformPkg/Drivers/SP805WatchdogDxe/SP805Watchdog.c | 95
++--
ArmPlatformPkg/Drivers/SP805WatchdogDxe/SP805WatchdogDxe.inf | 6 +-
3 files changed, 75 inserti
Before fixing the SP805 driver, let's clean it up a bit. No
functional changes.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPlatformPkg/Drivers/SP805WatchdogDxe/SP805Watchdog.c | 97
++--
ArmPlatformPkg/Dr
hi Aggarwal
Cc: Udit Kumar
Cc: Matteo Carlini
Cc: Nariman Poushin
Ard Biesheuvel (4):
ArmPlatformPkg/SP805WatchdogDxe: cosmetic cleanup
ArmPlatformPkg/SP805WatchdogDxe: switch to interrupt mode
ArmPkg/GenericWatchdogDxe: clean up the code
ArmPkg/GenericWatchdogDxe: implement RegisterH
ff-by: Ard Biesheuvel
---
ArmPkg/Drivers/GenericWatchdogDxe/GenericWatchdogDxe.c | 34
++--
1 file changed, 25 insertions(+), 9 deletions(-)
diff --git a/ArmPkg/Drivers/GenericWatchdogDxe/GenericWatchdogDxe.c
b/ArmPkg/Drivers/GenericWatchdogDxe/GenericWatchdogDxe.c
index 717a180
On Tue, 18 Dec 2018 at 14:04, Sumit Garg wrote:
>
> On Tue, 18 Dec 2018 at 18:30, Ard Biesheuvel
> wrote:
> >
> > On Tue, 18 Dec 2018 at 13:58, Sumit Garg wrote:
> > >
> > > On Tue, 18 Dec 2018 at 18:16, Ard Biesheuvel
> > > wrote:
> >
On Tue, 18 Dec 2018 at 13:58, Sumit Garg wrote:
>
> On Tue, 18 Dec 2018 at 18:16, Ard Biesheuvel
> wrote:
> >
> > On Tue, 18 Dec 2018 at 13:09, Sumit Garg wrote:
> > >
> > > Include a prebuilt binary of OP-TEE OS built from commit
> > &
On Tue, 18 Dec 2018 at 13:17, Sumit Garg wrote:
>
> Hi Ard,
>
> On Wed, 12 Dec 2018 at 11:05, Sumit Garg wrote:
> >
> > On Tue, 11 Dec 2018 at 21:37, Ard Biesheuvel
> > wrote:
> > >
> > > On Tue, 11 Dec 2018 at 10:06, Ard Biesheuvel
> >
On Tue, 18 Dec 2018 at 13:09, Sumit Garg wrote:
>
> Include a prebuilt binary of OP-TEE OS built from commit
> a5d528c7e54fd7726230483bd4cd5c4786d7703f.
> (https://github.com/OP-TEE/optee_os.git master)
>
> Also update ARM-TF RELEASE build to commit 47577cbaaf4b.
>
igned-off-by: Ard Biesheuvel
---
Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
index 7094e57ee13a..7db1c67
console driver clearing the screen)
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPlatformPkg/Library/PL011SerialPortLib/PL011SerialPortLib.c | 14
+++---
ArmPlatformPkg/Library/PL011SerialPortLib/PL011SerialPortLib.inf | 4 +++-
2 files
On Fri, 21 Sep 2018 at 10:26, Chris Co wrote:
>
> This adds baseline ACPI table support for booting Windows on
> NXP i.MX6 SoCs. Platforms may override these tables as necessary.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Christopher Co
&g
--Original Message-
> > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> > Ard Biesheuvel
> > Sent: Monday, December 17, 2018 4:35 PM
> > To: Yao, Jiewen
> > Cc: edk2-devel@lists.01.org; Gao, Liming ; Zhang,
> > Chao B
> > Subjec
l Message-
> > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> > Yao, Jiewen
> > Sent: Monday, December 17, 2018 4:25 PM
> > To: Ard Biesheuvel ; Leif Lindholm
> >
> > Cc: edk2-devel@lists.01.org; Zhang, Chao B ;
> > Gao, Limi
On Mon, 17 Dec 2018 at 08:45, Ard Biesheuvel wrote:
>
> On Mon, 17 Dec 2018 at 04:29, Yao, Jiewen wrote:
> >
> > I think we have below definition.
> > -- MemoryFence: Serialize load and store operations.
> > -- LoadFence: Serialize load operations.
> > -- St
tion here as well as some kind of other
barrier. Calling it LoadFence() makes no sense whatsoever.
>
> > -Original Message-
> > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> > Gao, Liming
> > Sent: Monday, December 17, 2018 10:04 AM
&
On Sat, 15 Dec 2018 at 12:03, Leif Lindholm wrote:
>
> On Sat, Dec 15, 2018 at 09:33:08AM +0100, Ard Biesheuvel wrote:
> > Our default is already armv7-a, so no need to rewrite the PLATFORM_FLAGS
> > for that. Also, setting -mfpu=neon is not entirely inappropriate, since
> &g
Our default is already armv7-a, so no need to rewrite the PLATFORM_FLAGS
for that. Also, setting -mfpu=neon is not entirely inappropriate, since
NEON is not mandatory under v7.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPkg/ArmPkg.dsc | 3 ---
1
On Mon, 10 Dec 2018 at 13:39, Pete Batard wrote:
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard
> ---
> Platform/Broadcom/Bcm283x/RaspberryPiPkg.dec | 63 ++
> Platform/Broadcom/Bcm283x/RaspberryPiPkg.dsc | 636
> Platform/Broadcom/
On Mon, 10 Dec 2018 at 13:39, Pete Batard wrote:
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard
Please use STATIC etc etc
> ---
> Platform/Broadcom/Bcm283x/Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c |
> 915
> Platform/Broadc
On Mon, 10 Dec 2018 at 13:39, Pete Batard wrote:
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard
> ---
> Platform/Broadcom/Bcm283x/Drivers/GraphicsConsoleDxe/ComponentName.c
> | 183 ++
> Platform/Broadcom/Bcm283x/Drivers/GraphicsConsoleDxe/
On Mon, 10 Dec 2018 at 13:39, Pete Batard wrote:
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard
Please fix the usual things like INF_VERSION and space before ( in
function or macro invocations.
Otherwise, this looks fine
'
> ---
> Platform/Broadcom/Bcm
On Tue, 11 Dec 2018 at 10:08, Vijayenthiran Subramaniam
wrote:
>
> On Fri, Dec 7, 2018 at 5:49 AM Vijayenthiran Subramaniam
> wrote:
> >
> > On Thu, Dec 6, 2018 at 5:18 PM Ard Biesheuvel
> > wrote:
> > >
> > > On Wed, 5 Dec 2018 at
On Fri, 14 Dec 2018 at 14:56, Laszlo Ersek wrote:
>
> On 12/14/18 12:22, Ard Biesheuvel wrote:
> > Commit 9a67ba261fe9 ("ArmVirtPkg: Replace obsoleted network drivers
> > from platform DSC/FDF") failed to take into account that the now
> > unconditionally inc
On Fri, 14 Dec 2018 at 13:13, Jagadeesh Ujja wrote:
>
> Variable service driver includes a call to AsmLfence. To reuse this
> driver on AArch64 based platforms, add an implementation of AsmLfence
> that acts as a wrapper on the AArch64 specific MemoryFence function.
>
> Contributed-under: TianoCor
at have no networking enabled, let's just fix things
by dropping the conditionals around it.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmVirtPkg/ArmVirt.dsc.inc | 2 --
1 file changed, 2 deletions(-)
diff --git a/ArmVirtPkg/ArmVirt.dsc.i
be removed.
>
> Cc: Jordan Justen
> Cc: Laszlo Ersek
> Cc: Ard Biesheuvel
> Cc: Anthony Perard
> Cc: Julien Grall
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Star Zeng
Reviewed-by: Ard Biesheuvel
> ---
> OvmfPkg/OvmfPkgIa32
On Fri, 14 Dec 2018 at 07:11, Fu, Siyuan wrote:
>
> Hi, Arb
>
> Do you think if it's ok to commit this patch to ArmVirtPkg now?
>
Sure, please go ahead
> > -Original Message-
> > From: Fu, Siyuan
> > Sent: Wednesday, November 7, 2018 8:59 AM
>
On Thu, 13 Dec 2018 at 12:42, Gao, Liming wrote:
>
> Yes. Reviewed-by: Liming Gao
>
Thanks all
Pushed as 580f4539dfbb..36deafb838d0
> > -Original Message-----
> > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> > Sent: Thursday, December 13, 2018
On Wed, 12 Dec 2018 at 23:23, Leif Lindholm wrote:
>
> On Tue, Dec 11, 2018 at 07:35:03PM +0100, Ard Biesheuvel wrote:
> > I promise :-)
>
> Yeah, right :)
>
> > This gets rid of the last build config options passed via the command
> > line into the preproc
On Wed, 12 Dec 2018 at 15:19, Ard Biesheuvel wrote:
>
> On Wed, 12 Dec 2018 at 15:19, Gao, Liming wrote:
> >
> > Make sense. So, CLANG35 tool chain can also be used by CLANG 3.5 and above
> > version compiler, like GCC49?
> >
>
> Exactly
>
Liming,
Are
On Sun, 9 Dec 2018 at 17:32, Achin Gupta wrote:
>
> Hi Leif,Ard,
>
> These patches are the last bits to complete support for Standalone MM on the
> FVP. All other patches have been merged now. Could you please review these?
>
Sure.
Could they be rebased onto current edk2-platforms and resent ple
adcom/Bcm283x/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.inf | 49 +
> Platform/Broadcom/Bcm283x/Include/Protocol/RaspberryPiFirmware.h| 131
> +++
> 3 files changed, 1265 insertions(+)
>
Reviewed-by: Ard Biesheuvel
Please move this patch forward in the series. It is depended upon by
earlier pa
On Mon, 10 Dec 2018 at 13:39, Pete Batard wrote:
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard
> ---
> Platform/Broadcom/Bcm283x/Drivers/Bcm2836InterruptDxe/Bcm2836InterruptDxe.c
> | 367 +++
>
> Platform/Broadcom/Bcm283x/Drivers/Bcm
On Mon, 10 Dec 2018 at 13:39, Pete Batard wrote:
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard
Reviewed-by: Ard Biesheuvel
> ---
> Platform/Broadcom/Bcm283x/Library/PlatformBootManagerLib/PlatformBm.c
On Mon, 10 Dec 2018 at 13:39, Pete Batard wrote:
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard
> ---
> Platform/Broadcom/Bcm283x/AcpiTables/AcpiTables.inf | 51 ++
> Platform/Broadcom/Bcm283x/AcpiTables/Csrt.aslc | 337 +
> Platform/Br
On Mon, 10 Dec 2018 at 13:39, Pete Batard wrote:
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard
> ---
> Platform/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2837Gpio.h
> | 50 +
> Platform/Broadcom/Bcm283x/Include/Library/Gpio
On Mon, 10 Dec 2018 at 13:39, Pete Batard wrote:
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pete Batard
> ---
> Platform/Broadcom/Bcm283x/Library/MemoryInitPeiLib/MemoryInitPeiLib.c |
> 183
> Platform/Broadcom/Bcm283x/Library/MemoryInitP
lignment requirement is as big as
> 0x_.
>
> The patch fixes this issue by only creating virtual BAR for option
> ROM when there is PCI option ROM.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ruiyu Ni
> Cc: Chiu Chasel
>
On Wed, 12 Dec 2018 at 16:02, Leif Lindholm wrote:
>
> On Wed, Dec 12, 2018 at 01:48:42PM +0100, Ard Biesheuvel wrote:
> > The GdbDebugAgent library is unused and unmaintained, and now it
> > turns out it doesn't build with Clang, so let's just get rid of it.
> >
On Wed, 12 Dec 2018 at 15:41, Leif Lindholm wrote:
>
> On Wed, Dec 12, 2018 at 01:45:17PM +0100, Ard Biesheuvel wrote:
> > > > > > @@ -198,9 +199,9 @@ ASM_PFX(AsmCommonExceptionEntry):
> > > > > >and R3, R1, #0x1f @ Check CPSR
nables
> > LTO, you need the LLVMgold plugin, which is not shipped for all
> > versions of Clang by the distros. So it is good to keep CLANG35 as a
> > fallback.
> >
> > > > -Original Message-
> > > > From: Ard Biesheuvel [mailto:ard.bi
not shipped for all
versions of Clang by the distros. So it is good to keep CLANG35 as a
fallback.
> > -Original Message-
> > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> > Sent: Wednesday, December 12, 2018 6:33 PM
> > To: edk2-devel@lists.01.org
>
The GdbDebugAgent library is unused and unmaintained, and now it
turns out it doesn't build with Clang, so let's just get rid of it.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
BeagleBoardPkg/BeagleBoardPkg.dsc
On Wed, 12 Dec 2018 at 13:31, Leif Lindholm wrote:
>
> On Wed, Dec 12, 2018 at 12:33:07PM +0100, Ard Biesheuvel wrote:
> > On Wed, 12 Dec 2018 at 12:30, Leif Lindholm
> > wrote:
> > >
> > > On Wed, Dec 12, 2018 at 10:12:11AM +0100, Ard Biesheuvel wrote:
>
On Wed, 12 Dec 2018 at 11:33, Ard Biesheuvel wrote:
>
> PE/COFF only has a very limited id space for runtime relocations, and
> so it defines only a single relocation for movw/movt instruction pairs,
> which can be combined to load a 32-bit symbol reference into a register.
> For
Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
BaseTools/Conf/tools_def.template | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/BaseTools/Conf/tools_def.template
b/BaseTools/Conf/tools_def.template
index 2ba833e1fb06..902680c24c85 100755
--- a/BaseTools/Conf/tools_def.template
On Wed, 12 Dec 2018 at 12:50, Laszlo Ersek wrote:
>
> On 12/12/18 11:33, Ard Biesheuvel wrote:
> > PE/COFF only has a very limited id space for runtime relocations, and
> > so it defines only a single relocation for movw/movt instruction pairs,
> > which can be combined
On Wed, 12 Dec 2018 at 12:30, Leif Lindholm wrote:
>
> On Wed, Dec 12, 2018 at 10:12:11AM +0100, Ard Biesheuvel wrote:
> > Clang does not like the legacy 32-bit assembler syntax used in the
> > gdbstub exception handling routines, so update them to something
> > more
the now redundant, and incompatible command line
overrides for the relocatable PrePi.
Cc: Michael D Kinney
Cc: Liming Gao
Cc: Bob Feng
Cc: Leif Lindholm
Cc: Laszlo Ersek
Ard Biesheuvel (3):
MdePkg/BaseMemoryLibOptDxe ARM: add missing function annotations
BaseTools/tools_def ARM CLANG35
this
function annotation, or they may end up getting called in the
wrong mode, crashing the program.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
MdePkg/Library/BaseMemoryLibOptDxe/Arm/CompareGuid.S | 1 +
MdePkg/Library/BaseMemoryLibOptDxe/Arm
/movt generation, along
with some other changes (e.g., long calls) which do affect code generation
but not in a undesirable manner.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
BaseTools/Conf/tools_def.template | 2 +-
1 file changed, 1 insertion(
Drop the -mno-movt command line option override, which is no longer
needed, and actually incompatible with versions of Clang before 3.6.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmVirtPkg/ArmVirtQemuKernel.dsc | 5 -
ArmVirtPkg
appropriate for an empty descending stack.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
EmbeddedPkg/Library/GdbDebugAgent/Arm/ExceptionSupport.ARMv6.S | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/EmbeddedPkg/Library
On Wed, 12 Dec 2018 at 01:24, Gao, Liming wrote:
>
> That's good. Reviewed-by: Liming Gao
>
Thanks all
Pushed as e07092edca84..0d68ce514b92
> >-Original Message-
> >From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> >Sent: Tuesday, December 11,
.g.
>> "uint32_t". Together with standard C macros expressing limits, such as
>> UINT_MAX (from ) and UINT32_MAX (from ).
>>
>> Clearly no-one has capacity to clean up BaseTools like this. For
>> starters, we should at least remove whatever actively causes confusi
On Tue, 11 Dec 2018 at 21:16, Pete Batard wrote:
>
> Hi Leif,
>
> On 2018.12.11 18:10, Leif Lindholm wrote:
> > Hi Pete,
> >
> > Many thanks for this.
> > I expect Ard will give more detailed review, since he has some
> > familiarity with the port. But I wanted to make a few comments.
> >
> > Coul
Align the ACPI platform driver with the DT patching code, which only
enables the IPMI/KCS device node on B1 silicon (or later).
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/AcpiPlatform.c | 3 ++-
1 file
Get rid of the CPP conditionals, and use the new boolean PCD for KCS
support instead.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Silicon/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.inf | 1 +
Silicon/AMD/Styx/Library/StyxDtbLoaderLib
Now that all code references the PCD instead, let's drop the macro
from all Styx platforms.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 8
Platform/LeMaker/Cello
In order to get rid of another CPP macro, introduce a boolean PCD
that we will set in the platform to enable IPMI/KCS support.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Silicon/AMD/Styx/AmdStyx.dec| 1 +
Platform
We already have a boolean PCD that tells us whether XGBE support should
be enabled, so use it instead of C preprocessor conditionals.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Silicon/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.inf | 1
We already have a boolean PCD that tells us whether XGBE support should
be enabled, so use it instead of C preprocessor conditionals.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Silicon/AMD/Styx/Drivers/PlatInitPei/PlatInitPei.inf | 1 +
Silicon
Move the IPMI/KCS device node from the DSDT into its own SSDT, and only
install it if we have support for KCS enabled in the platform.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/AcpiPlatformDxe.inf | 2
Now that all code relies on the PCD rather than the DO_XGBE macro,
get rid of it.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 9 ++---
Platform/AMD/OverdriveBoard/OverdriveBoard.fdf | 2 --
2
I promise :-)
This gets rid of the last build config options passed via the command
line into the preprocessor, and replaces them with PCD references.
Ard Biesheuvel (11):
Silicon/AMD/Styx/Iort: drop conditionally included XGBE nodes
Platform/SoftIron/Overdrive1000Board: remove dead XGBE
buted-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/Iort.aslc | 8
1 file changed, 8 deletions(-)
diff --git a/Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/Iort.aslc
b/Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/Iort.aslc
Remove the references to the XGBE SNP drivers, which are inside a
!ifdef() that never gets enabled anyway.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.fdf | 8
1 file changed, 8
We already have a boolean PCD that tells us whether XGBE support should
be enabled, so use it instead of C preprocessor conditionals.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/AcpiPlatformDxe.inf | 1
On Tue, 11 Dec 2018 at 13:19, Ard Biesheuvel wrote:
>
> Patch #1 reshuffles the .dsc contents of ArmVirtQemuKernel so that we
> can run on ARM targets as well.
>
> Patch #2 fixes the CLANG38 build for ArmVirtQemuKernel and ArmVirtXen,
> by tweaking the linker options passe
On Tue, 11 Dec 2018 at 16:53, Leif Lindholm wrote:
>
> On Tue, Dec 11, 2018 at 04:02:31PM +0100, Ard Biesheuvel wrote:
> > Another day, another round of cleanups for Styx. Patch #1 was sent out
> > earlier, and has been updated to switch to a VOID* type PCD. Remaining
>
On Thu, 6 Dec 2018 at 13:51, Wu, Hao A wrote:
>
> > -Original Message-
> > From: Ni, Ruiyu
> > Sent: Wednesday, December 05, 2018 5:25 PM
> > To: edk2-devel@lists.01.org
> > Cc: Wu, Hao A
> > Subject: [PATCH] MdeModulePkg/PciBus: Shadow option ROM after BARs
> > are programmed
> >
> > REF:
On Tue, 11 Dec 2018 at 10:06, Ard Biesheuvel wrote:
>
> On Tue, 11 Dec 2018 at 10:04, Sumit Garg wrote:
> >
> > On Tue, 11 Dec 2018 at 13:56, Ard Biesheuvel
> > wrote:
> > >
> > > On Tue, 11 Dec 2018 at 08:46, Sumit Garg wrote:
> > > >
properties are dropped, and
unused clocks and other redudant pieces are removed as well. The
existing prebuilt binary is deleted.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
401 - 500 of 5502 matches
Mail list logo