Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
On Thu, 28 Mar 2019 at 11:46, Ryszard Knop wrote: > > On Wed, 2019-03-27 at 16:32 +0100, Ard Biesheuvel wrote: > > On Tue, 29 Jan 2019 at 14:55, Ryszard Knop < > > ryszard.k...@linux.intel.com> wrote: > > > +Team > > > > > > > As it turns out, this driver is still broken for non-1:1 mapped DMA. > > > > In particular, I am hitting a crash on > > > > E1000MemCopy ( > > (UINT8 *) (UINTN) CpbReceive->BufferAddr, > > (UINT8 *) (UINTN) ReceiveDescriptor->buffer_addr, > > TempLen > > ); > > > > (around line 676 in e1000.c), which uses the DMA address > > 'ReceiveDescriptor->buffer_addr' in a memory copy operation performed > > by the CPU. This causes a crash on systems where the DMA address is > > not also a valid CPU address. > > Huh, this is new... I don't have access to any system behaving this > way, so I can't test this, but E1000.c -> E1000TxRxConfigure links > RxDesc->buffer_addr to the physical addresses, that descriptor is used > by the hardware to DMA data where needed, and we try to copy from that > same physical address later, while we should copy from unmapped > addresses instead. > Indeed. > This probably should be solved by having a separate array/something > with CurRxInd -> unmapped addresses, but I'll have to talk with my team > to solve this in a sensible way. > > In the meantime, maybe you know if there's a way to simulate this > situation under QEMU or something? > I am using an arm64 board with modified firmware to emulate different PCIe host bridge configurations. I don't know whether QEMU has support for non-1:1 mapped DMA on x86, but it does emulate various boards (such as the raspberry pi 2 iirc) where the CPU and device addressing is not 1:1. ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
On Wed, 2019-03-27 at 16:32 +0100, Ard Biesheuvel wrote: > On Tue, 29 Jan 2019 at 14:55, Ryszard Knop < > ryszard.k...@linux.intel.com> wrote: > > +Team > > > > As it turns out, this driver is still broken for non-1:1 mapped DMA. > > In particular, I am hitting a crash on > > E1000MemCopy ( > (UINT8 *) (UINTN) CpbReceive->BufferAddr, > (UINT8 *) (UINTN) ReceiveDescriptor->buffer_addr, > TempLen > ); > > (around line 676 in e1000.c), which uses the DMA address > 'ReceiveDescriptor->buffer_addr' in a memory copy operation performed > by the CPU. This causes a crash on systems where the DMA address is > not also a valid CPU address. Huh, this is new... I don't have access to any system behaving this way, so I can't test this, but E1000.c -> E1000TxRxConfigure links RxDesc->buffer_addr to the physical addresses, that descriptor is used by the hardware to DMA data where needed, and we try to copy from that same physical address later, while we should copy from unmapped addresses instead. This probably should be solved by having a separate array/something with CurRxInd -> unmapped addresses, but I'll have to talk with my team to solve this in a sensible way. In the meantime, maybe you know if there's a way to simulate this situation under QEMU or something? > > On Tue, 2019-01-29 at 14:13 +0100, Ryszard Knop wrote: > > > Hi Ard, > > > > > > I've finally got some time to review and merge all of this. A bit > > > problematic thing is that we internally have a separate tree that > > > we > > > need to merge those commits into, then generate the open source > > > tree > > > and related commits from that. This will result in somewhat > > > broken > > > history, so sorry about that in advance - we're still figuring > > > out > > > the > > > proper way to handle multiple source trees on our end without > > > messing > > > it up. I'll push these changes to edk2-staging once we've got it > > > all > > > ready. > > > > > > On Tue, 2018-11-06 at 18:58 +0100, ard.biesheuvela wrote: > > > > This series fixes the GigUndiDxe in the edk2-staging/Intel_UNDI > > > > branch > > > > at github.com/tianocore so it can be built with GCC on Linux > > > > for > > > > ARM > > > > and AARCH64 (as well as X64) > > > > > > > > Ard Biesheuvel (19): > > > > IntelOpenSourceUndiPkg.dsc: add AARCH64 and ARM to supported > > > > architectures > > > > IntelUndiPkg: remove EOF markers > > > > IntelUndiPkg/GigUndiDxe: consistently use lowercase for e1000 > > > > in > > > > filenames > > > > IntelUndiPkg/GigUndiDxe: consistently use forward slashes as > > > > path > > > > separators > > > > IntelUndiPkg/GigUndiDxe: move BRAND_STRUCT declaration after > > > > type > > > > definition > > > > IntelUndiPkg/GigUndiDxe: use intermediate UINTN casts for > > > > pointers > > > > IntelUndiPkg/GigUndiDxe: create GCC alternatives for MSFT > > > > build > > > > options > > > > IntelUndiPkg/GigUndiDxe: add missing VOID** cast > > > > IntelUndiPkg/GigUndiDxe: add missing UINT8* cast > > > > IntelUndiPkg/GigUndiDxe: add missing braces to GUID literals > > > > IntelUndiPkg/GigUndiDxe: fix incorrect use of CPP token > > > > pasting > > > > IntelUndiPkg/GigUndiDxe: cast E1000MemCopy () args to correct > > > > pointer > > > > type > > > > IntelUndiPkg/GigUndiDxe: don't take address of cast > > > > expression > > > > IntelUndiPkg/GigUndiDxe: redefine UNREFERENCED_nPARAMETER > > > > macros > > > > for > > > > GCC > > > > IntelUndiPkg/GigUndiDxe: remove forward declaration of non- > > > > existent > > > > function > > > > IntelUndiPkg/GigUndiDxe: fix incorrect indentation > > > > IntelUndiPkg/GigUndiDxe: move MSFT warning overrides to INF > > > > file > > > > IntelUndiPkg/GigUndiDxe: add missing EFIAPI modifiers > > > > IntelUndiPkg/GigUndiDxe: remove or reorganize unused > > > > variables > > > > > > > > IntelUndiPkg/GigUndiDxe/AdapterInformation.c | 6 ++- > > > > IntelUndiPkg/GigUndiDxe/AdapterInformation.h | 1 - > > > > IntelUndiPkg/GigUndiDxe/Brand.c | 1 - > > > > IntelUndiPkg/GigUndiDxe/ComponentName.c | 5 ++- > > > > IntelUndiPkg/GigUndiDxe/ComponentName.h | 2 +- > > > > IntelUndiPkg/GigUndiDxe/Decode.c | 5 +-- > > > > IntelUndiPkg/GigUndiDxe/Decode.h | 1 - > > > > IntelUndiPkg/GigUndiDxe/DeviceSupport.c | 1 - > > > > IntelUndiPkg/GigUndiDxe/DeviceSupport.h | 9 ++--- > > > > IntelUndiPkg/GigUndiDxe/Dma.c | 11 +++--- > > > > IntelUndiPkg/GigUndiDxe/Dma.h | 1 - > > > > IntelUndiPkg/GigUndiDxe/DriverConfiguration.c | 6 ++- > > > > IntelUndiPkg/GigUndiDxe/DriverConfiguration.h | 1 - > > > > IntelUndiPkg/GigUndiDxe/DriverDiagnostics.c | 12 +++--- > > > > IntelUndiPkg/GigUndiDxe/DriverDiagnostics.h | 1 - > > > > IntelUndiPkg/GigUndiDxe/DriverHealth.c| 5 ++- > > > > IntelUndiPkg/GigUndiDxe/EepromConfig.c| 1
Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
On Tue, 29 Jan 2019 at 14:55, Ryszard Knop wrote:
>
> +Team
>
As it turns out, this driver is still broken for non-1:1 mapped DMA.
In particular, I am hitting a crash on
E1000MemCopy (
(UINT8 *) (UINTN) CpbReceive->BufferAddr,
(UINT8 *) (UINTN) ReceiveDescriptor->buffer_addr,
TempLen
);
(around line 676 in e1000.c), which uses the DMA address
'ReceiveDescriptor->buffer_addr' in a memory copy operation performed
by the CPU. This causes a crash on systems where the DMA address is
not also a valid CPU address.
> On Tue, 2019-01-29 at 14:13 +0100, Ryszard Knop wrote:
> > Hi Ard,
> >
> > I've finally got some time to review and merge all of this. A bit
> > problematic thing is that we internally have a separate tree that we
> > need to merge those commits into, then generate the open source tree
> > and related commits from that. This will result in somewhat broken
> > history, so sorry about that in advance - we're still figuring out
> > the
> > proper way to handle multiple source trees on our end without messing
> > it up. I'll push these changes to edk2-staging once we've got it all
> > ready.
> >
> > On Tue, 2018-11-06 at 18:58 +0100, ard.biesheuvela wrote:
> > > This series fixes the GigUndiDxe in the edk2-staging/Intel_UNDI
> > > branch
> > > at github.com/tianocore so it can be built with GCC on Linux for
> > > ARM
> > > and AARCH64 (as well as X64)
> > >
> > > Ard Biesheuvel (19):
> > > IntelOpenSourceUndiPkg.dsc: add AARCH64 and ARM to supported
> > > architectures
> > > IntelUndiPkg: remove EOF markers
> > > IntelUndiPkg/GigUndiDxe: consistently use lowercase for e1000 in
> > > filenames
> > > IntelUndiPkg/GigUndiDxe: consistently use forward slashes as path
> > > separators
> > > IntelUndiPkg/GigUndiDxe: move BRAND_STRUCT declaration after type
> > > definition
> > > IntelUndiPkg/GigUndiDxe: use intermediate UINTN casts for
> > > pointers
> > > IntelUndiPkg/GigUndiDxe: create GCC alternatives for MSFT build
> > > options
> > > IntelUndiPkg/GigUndiDxe: add missing VOID** cast
> > > IntelUndiPkg/GigUndiDxe: add missing UINT8* cast
> > > IntelUndiPkg/GigUndiDxe: add missing braces to GUID literals
> > > IntelUndiPkg/GigUndiDxe: fix incorrect use of CPP token pasting
> > > IntelUndiPkg/GigUndiDxe: cast E1000MemCopy () args to correct
> > > pointer
> > > type
> > > IntelUndiPkg/GigUndiDxe: don't take address of cast expression
> > > IntelUndiPkg/GigUndiDxe: redefine UNREFERENCED_nPARAMETER macros
> > > for
> > > GCC
> > > IntelUndiPkg/GigUndiDxe: remove forward declaration of non-
> > > existent
> > > function
> > > IntelUndiPkg/GigUndiDxe: fix incorrect indentation
> > > IntelUndiPkg/GigUndiDxe: move MSFT warning overrides to INF file
> > > IntelUndiPkg/GigUndiDxe: add missing EFIAPI modifiers
> > > IntelUndiPkg/GigUndiDxe: remove or reorganize unused variables
> > >
> > > IntelUndiPkg/GigUndiDxe/AdapterInformation.c | 6 ++-
> > > IntelUndiPkg/GigUndiDxe/AdapterInformation.h | 1 -
> > > IntelUndiPkg/GigUndiDxe/Brand.c | 1 -
> > > IntelUndiPkg/GigUndiDxe/ComponentName.c | 5 ++-
> > > IntelUndiPkg/GigUndiDxe/ComponentName.h | 2 +-
> > > IntelUndiPkg/GigUndiDxe/Decode.c | 5 +--
> > > IntelUndiPkg/GigUndiDxe/Decode.h | 1 -
> > > IntelUndiPkg/GigUndiDxe/DeviceSupport.c | 1 -
> > > IntelUndiPkg/GigUndiDxe/DeviceSupport.h | 9 ++---
> > > IntelUndiPkg/GigUndiDxe/Dma.c | 11 +++---
> > > IntelUndiPkg/GigUndiDxe/Dma.h | 1 -
> > > IntelUndiPkg/GigUndiDxe/DriverConfiguration.c | 6 ++-
> > > IntelUndiPkg/GigUndiDxe/DriverConfiguration.h | 1 -
> > > IntelUndiPkg/GigUndiDxe/DriverDiagnostics.c | 12 +++---
> > > IntelUndiPkg/GigUndiDxe/DriverDiagnostics.h | 1 -
> > > IntelUndiPkg/GigUndiDxe/DriverHealth.c| 5 ++-
> > > IntelUndiPkg/GigUndiDxe/EepromConfig.c| 1 -
> > > IntelUndiPkg/GigUndiDxe/EepromConfig.h| 3 +-
> > > IntelUndiPkg/GigUndiDxe/GigUndiDxe.inf| 39 +--
> > >
> > > IntelUndiPkg/GigUndiDxe/Hii.c | 11 +++---
> > > IntelUndiPkg/GigUndiDxe/Hii.h | 1 -
> > > IntelUndiPkg/GigUndiDxe/HiiInternalLib.c | 3 --
> > > IntelUndiPkg/GigUndiDxe/HiiInternalLib.h | 1 -
> > > IntelUndiPkg/GigUndiDxe/Init.c| 11 +++---
> > > IntelUndiPkg/GigUndiDxe/Init.h| 1 -
> > > IntelUndiPkg/GigUndiDxe/Inventory.vfr | 1 -
> > > IntelUndiPkg/GigUndiDxe/NVDataStruc.h | 7 ++--
> > > IntelUndiPkg/GigUndiDxe/StartStop.c | 5 ++-
> > > IntelUndiPkg/GigUndiDxe/StartStop.h | 7 ++--
> > > IntelUndiPkg/GigUndiDxe/Version.h | 1 -
> > > IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} | 37 ---
> > > --
> > > -
> > > IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} | 5 +--
> > > IntelUndiPkg/GigUndiDxe/e1000_80
Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
+Team
On Tue, 2019-01-29 at 14:13 +0100, Ryszard Knop wrote:
> Hi Ard,
>
> I've finally got some time to review and merge all of this. A bit
> problematic thing is that we internally have a separate tree that we
> need to merge those commits into, then generate the open source tree
> and related commits from that. This will result in somewhat broken
> history, so sorry about that in advance - we're still figuring out
> the
> proper way to handle multiple source trees on our end without messing
> it up. I'll push these changes to edk2-staging once we've got it all
> ready.
>
> On Tue, 2018-11-06 at 18:58 +0100, ard.biesheuvela wrote:
> > This series fixes the GigUndiDxe in the edk2-staging/Intel_UNDI
> > branch
> > at github.com/tianocore so it can be built with GCC on Linux for
> > ARM
> > and AARCH64 (as well as X64)
> >
> > Ard Biesheuvel (19):
> > IntelOpenSourceUndiPkg.dsc: add AARCH64 and ARM to supported
> > architectures
> > IntelUndiPkg: remove EOF markers
> > IntelUndiPkg/GigUndiDxe: consistently use lowercase for e1000 in
> > filenames
> > IntelUndiPkg/GigUndiDxe: consistently use forward slashes as path
> > separators
> > IntelUndiPkg/GigUndiDxe: move BRAND_STRUCT declaration after type
> > definition
> > IntelUndiPkg/GigUndiDxe: use intermediate UINTN casts for
> > pointers
> > IntelUndiPkg/GigUndiDxe: create GCC alternatives for MSFT build
> > options
> > IntelUndiPkg/GigUndiDxe: add missing VOID** cast
> > IntelUndiPkg/GigUndiDxe: add missing UINT8* cast
> > IntelUndiPkg/GigUndiDxe: add missing braces to GUID literals
> > IntelUndiPkg/GigUndiDxe: fix incorrect use of CPP token pasting
> > IntelUndiPkg/GigUndiDxe: cast E1000MemCopy () args to correct
> > pointer
> > type
> > IntelUndiPkg/GigUndiDxe: don't take address of cast expression
> > IntelUndiPkg/GigUndiDxe: redefine UNREFERENCED_nPARAMETER macros
> > for
> > GCC
> > IntelUndiPkg/GigUndiDxe: remove forward declaration of non-
> > existent
> > function
> > IntelUndiPkg/GigUndiDxe: fix incorrect indentation
> > IntelUndiPkg/GigUndiDxe: move MSFT warning overrides to INF file
> > IntelUndiPkg/GigUndiDxe: add missing EFIAPI modifiers
> > IntelUndiPkg/GigUndiDxe: remove or reorganize unused variables
> >
> > IntelUndiPkg/GigUndiDxe/AdapterInformation.c | 6 ++-
> > IntelUndiPkg/GigUndiDxe/AdapterInformation.h | 1 -
> > IntelUndiPkg/GigUndiDxe/Brand.c | 1 -
> > IntelUndiPkg/GigUndiDxe/ComponentName.c | 5 ++-
> > IntelUndiPkg/GigUndiDxe/ComponentName.h | 2 +-
> > IntelUndiPkg/GigUndiDxe/Decode.c | 5 +--
> > IntelUndiPkg/GigUndiDxe/Decode.h | 1 -
> > IntelUndiPkg/GigUndiDxe/DeviceSupport.c | 1 -
> > IntelUndiPkg/GigUndiDxe/DeviceSupport.h | 9 ++---
> > IntelUndiPkg/GigUndiDxe/Dma.c | 11 +++---
> > IntelUndiPkg/GigUndiDxe/Dma.h | 1 -
> > IntelUndiPkg/GigUndiDxe/DriverConfiguration.c | 6 ++-
> > IntelUndiPkg/GigUndiDxe/DriverConfiguration.h | 1 -
> > IntelUndiPkg/GigUndiDxe/DriverDiagnostics.c | 12 +++---
> > IntelUndiPkg/GigUndiDxe/DriverDiagnostics.h | 1 -
> > IntelUndiPkg/GigUndiDxe/DriverHealth.c| 5 ++-
> > IntelUndiPkg/GigUndiDxe/EepromConfig.c| 1 -
> > IntelUndiPkg/GigUndiDxe/EepromConfig.h| 3 +-
> > IntelUndiPkg/GigUndiDxe/GigUndiDxe.inf| 39 +--
> >
> > IntelUndiPkg/GigUndiDxe/Hii.c | 11 +++---
> > IntelUndiPkg/GigUndiDxe/Hii.h | 1 -
> > IntelUndiPkg/GigUndiDxe/HiiInternalLib.c | 3 --
> > IntelUndiPkg/GigUndiDxe/HiiInternalLib.h | 1 -
> > IntelUndiPkg/GigUndiDxe/Init.c| 11 +++---
> > IntelUndiPkg/GigUndiDxe/Init.h| 1 -
> > IntelUndiPkg/GigUndiDxe/Inventory.vfr | 1 -
> > IntelUndiPkg/GigUndiDxe/NVDataStruc.h | 7 ++--
> > IntelUndiPkg/GigUndiDxe/StartStop.c | 5 ++-
> > IntelUndiPkg/GigUndiDxe/StartStop.h | 7 ++--
> > IntelUndiPkg/GigUndiDxe/Version.h | 1 -
> > IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} | 37 ---
> > --
> > -
> > IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} | 5 +--
> > IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.c | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.h | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_82571.c | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_82571.h | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_82575.c | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_82575.h | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_api.c | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_api.h | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_defines.h | 10 -
> > IntelUndiPkg/GigUndiDxe/e1000_hw.h| 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_i210.c | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_i210.h | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_ich8lan.c | 18 ---
Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
Hi Ard,
I've finally got some time to review and merge all of this. A bit
problematic thing is that we internally have a separate tree that we
need to merge those commits into, then generate the open source tree
and related commits from that. This will result in somewhat broken
history, so sorry about that in advance - we're still figuring out the
proper way to handle multiple source trees on our end without messing
it up. I'll push these changes to edk2-staging once we've got it all
ready.
On Tue, 2018-11-06 at 18:58 +0100, ard.biesheuvela wrote:
> This series fixes the GigUndiDxe in the edk2-staging/Intel_UNDI
> branch
> at github.com/tianocore so it can be built with GCC on Linux for ARM
> and AARCH64 (as well as X64)
>
> Ard Biesheuvel (19):
> IntelOpenSourceUndiPkg.dsc: add AARCH64 and ARM to supported
> architectures
> IntelUndiPkg: remove EOF markers
> IntelUndiPkg/GigUndiDxe: consistently use lowercase for e1000 in
> filenames
> IntelUndiPkg/GigUndiDxe: consistently use forward slashes as path
> separators
> IntelUndiPkg/GigUndiDxe: move BRAND_STRUCT declaration after type
> definition
> IntelUndiPkg/GigUndiDxe: use intermediate UINTN casts for pointers
> IntelUndiPkg/GigUndiDxe: create GCC alternatives for MSFT build
> options
> IntelUndiPkg/GigUndiDxe: add missing VOID** cast
> IntelUndiPkg/GigUndiDxe: add missing UINT8* cast
> IntelUndiPkg/GigUndiDxe: add missing braces to GUID literals
> IntelUndiPkg/GigUndiDxe: fix incorrect use of CPP token pasting
> IntelUndiPkg/GigUndiDxe: cast E1000MemCopy () args to correct
> pointer
> type
> IntelUndiPkg/GigUndiDxe: don't take address of cast expression
> IntelUndiPkg/GigUndiDxe: redefine UNREFERENCED_nPARAMETER macros
> for
> GCC
> IntelUndiPkg/GigUndiDxe: remove forward declaration of non-existent
> function
> IntelUndiPkg/GigUndiDxe: fix incorrect indentation
> IntelUndiPkg/GigUndiDxe: move MSFT warning overrides to INF file
> IntelUndiPkg/GigUndiDxe: add missing EFIAPI modifiers
> IntelUndiPkg/GigUndiDxe: remove or reorganize unused variables
>
> IntelUndiPkg/GigUndiDxe/AdapterInformation.c | 6 ++-
> IntelUndiPkg/GigUndiDxe/AdapterInformation.h | 1 -
> IntelUndiPkg/GigUndiDxe/Brand.c | 1 -
> IntelUndiPkg/GigUndiDxe/ComponentName.c | 5 ++-
> IntelUndiPkg/GigUndiDxe/ComponentName.h | 2 +-
> IntelUndiPkg/GigUndiDxe/Decode.c | 5 +--
> IntelUndiPkg/GigUndiDxe/Decode.h | 1 -
> IntelUndiPkg/GigUndiDxe/DeviceSupport.c | 1 -
> IntelUndiPkg/GigUndiDxe/DeviceSupport.h | 9 ++---
> IntelUndiPkg/GigUndiDxe/Dma.c | 11 +++---
> IntelUndiPkg/GigUndiDxe/Dma.h | 1 -
> IntelUndiPkg/GigUndiDxe/DriverConfiguration.c | 6 ++-
> IntelUndiPkg/GigUndiDxe/DriverConfiguration.h | 1 -
> IntelUndiPkg/GigUndiDxe/DriverDiagnostics.c | 12 +++---
> IntelUndiPkg/GigUndiDxe/DriverDiagnostics.h | 1 -
> IntelUndiPkg/GigUndiDxe/DriverHealth.c| 5 ++-
> IntelUndiPkg/GigUndiDxe/EepromConfig.c| 1 -
> IntelUndiPkg/GigUndiDxe/EepromConfig.h| 3 +-
> IntelUndiPkg/GigUndiDxe/GigUndiDxe.inf| 39 +--
>
> IntelUndiPkg/GigUndiDxe/Hii.c | 11 +++---
> IntelUndiPkg/GigUndiDxe/Hii.h | 1 -
> IntelUndiPkg/GigUndiDxe/HiiInternalLib.c | 3 --
> IntelUndiPkg/GigUndiDxe/HiiInternalLib.h | 1 -
> IntelUndiPkg/GigUndiDxe/Init.c| 11 +++---
> IntelUndiPkg/GigUndiDxe/Init.h| 1 -
> IntelUndiPkg/GigUndiDxe/Inventory.vfr | 1 -
> IntelUndiPkg/GigUndiDxe/NVDataStruc.h | 7 ++--
> IntelUndiPkg/GigUndiDxe/StartStop.c | 5 ++-
> IntelUndiPkg/GigUndiDxe/StartStop.h | 7 ++--
> IntelUndiPkg/GigUndiDxe/Version.h | 1 -
> IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} | 37 -
> -
> IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} | 5 +--
> IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_82571.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_82571.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_82575.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_82575.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_api.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_api.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_defines.h | 10 -
> IntelUndiPkg/GigUndiDxe/e1000_hw.h| 1 -
> IntelUndiPkg/GigUndiDxe/e1000_i210.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_i210.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_ich8lan.c | 18 -
> IntelUndiPkg/GigUndiDxe/e1000_ich8lan.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_mac.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_mac.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_manage.c| 1 -
> IntelUndiPkg/GigUndiDxe/e1000_manage.h
Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
On 9 November 2018 at 18:54, Knop, Ryszard wrote: > Hey Ard, I'm from the Preboot team responsible for this component. We're too > busy to review and merge this at the moment, so this'll have to stay on the > list for now, but we'll come back to it in a few weeks once we've finished > some of our current tasks. > Thanks, Richard. > Thank you. No rush. > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard > Biesheuvel > Sent: Wednesday, November 7, 2018 0:04 > To: Kinney, Michael D > Cc: Jin, Eric ; edk2-devel@lists.01.org; Kacperski, Kamil > ; Orlowski, Pawel > Subject: Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build > fixes for AARCH64/ARM/GCC > > On 6 November 2018 at 23:10, Kinney, Michael D > wrote: >> Hi Ard, >> >> Can you please add CC lines to the commit message for the developers >> that have contributed to the edk2-staging/Intel_UNDI branch? >> >> This would include: >> >> Cc: Maciej Rabeda > > Maciej was already on cc. I hope he can forward the emails to his colleagues, > if they can't find them in the archives. > ___ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel > > > Intel Technology Poland sp. z o.o. > ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII > Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP > 957-07-52-316 | Kapital zakladowy 200.000 PLN. > > Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i > moze zawierac informacje poufne. W razie przypadkowego otrzymania tej > wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; > jakiekolwiek > przegladanie lub rozpowszechnianie jest zabronione. > This e-mail and any attachments may contain confidential material for the > sole use of the intended recipient(s). If you are not the intended recipient, > please contact the sender and delete all copies; any review or distribution by > others is strictly prohibited. > ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
Hey Ard, I'm from the Preboot team responsible for this component. We're too busy to review and merge this at the moment, so this'll have to stay on the list for now, but we'll come back to it in a few weeks once we've finished some of our current tasks. Thanks, Richard. -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard Biesheuvel Sent: Wednesday, November 7, 2018 0:04 To: Kinney, Michael D Cc: Jin, Eric ; edk2-devel@lists.01.org; Kacperski, Kamil ; Orlowski, Pawel Subject: Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC On 6 November 2018 at 23:10, Kinney, Michael D wrote: > Hi Ard, > > Can you please add CC lines to the commit message for the developers > that have contributed to the edk2-staging/Intel_UNDI branch? > > This would include: > > Cc: Maciej Rabeda Maciej was already on cc. I hope he can forward the emails to his colleagues, if they can't find them in the archives. ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel Intel Technology Poland sp. z o.o. ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN. Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione. This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited. ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
On Tue, Nov 06, 2018 at 10:10:16PM +, Kinney, Michael D wrote:
> Hi Ard,
>
> Can you please add CC lines to the commit message
> for the developers that have contributed to the
> edk2-staging/Intel_UNDI branch?
>
> This would include:
>
> Cc: Maciej Rabeda
> Cc: Kamil Kacperski
Could these two be added to the top-level Readme.MD?
(Pawel is already listed as the branch owner.)
Regards,
Leif
> Cc: Pawel Orlowski
>
> Thanks,
>
> Mike
>
>
> > -Original Message-
> > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> > Sent: Tuesday, November 6, 2018 9:58 AM
> > To: edk2-devel@lists.01.org
> > Cc: Rabeda, Maciej ; Kinney,
> > Michael D ; Jin, Eric
> > ; leif.lindh...@linaro.org; Ard
> > Biesheuvel
> > Subject: [PATCH edk2-staging 00/19]
> > IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
> >
> > This series fixes the GigUndiDxe in the edk2-
> > staging/Intel_UNDI branch
> > at github.com/tianocore so it can be built with GCC on
> > Linux for ARM
> > and AARCH64 (as well as X64)
> >
> > Ard Biesheuvel (19):
> > IntelOpenSourceUndiPkg.dsc: add AARCH64 and ARM to
> > supported
> > architectures
> > IntelUndiPkg: remove EOF markers
> > IntelUndiPkg/GigUndiDxe: consistently use lowercase
> > for e1000 in
> > filenames
> > IntelUndiPkg/GigUndiDxe: consistently use forward
> > slashes as path
> > separators
> > IntelUndiPkg/GigUndiDxe: move BRAND_STRUCT declaration
> > after type
> > definition
> > IntelUndiPkg/GigUndiDxe: use intermediate UINTN casts
> > for pointers
> > IntelUndiPkg/GigUndiDxe: create GCC alternatives for
> > MSFT build
> > options
> > IntelUndiPkg/GigUndiDxe: add missing VOID** cast
> > IntelUndiPkg/GigUndiDxe: add missing UINT8* cast
> > IntelUndiPkg/GigUndiDxe: add missing braces to GUID
> > literals
> > IntelUndiPkg/GigUndiDxe: fix incorrect use of CPP
> > token pasting
> > IntelUndiPkg/GigUndiDxe: cast E1000MemCopy () args to
> > correct pointer
> > type
> > IntelUndiPkg/GigUndiDxe: don't take address of cast
> > expression
> > IntelUndiPkg/GigUndiDxe: redefine
> > UNREFERENCED_nPARAMETER macros for
> > GCC
> > IntelUndiPkg/GigUndiDxe: remove forward declaration of
> > non-existent
> > function
> > IntelUndiPkg/GigUndiDxe: fix incorrect indentation
> > IntelUndiPkg/GigUndiDxe: move MSFT warning overrides
> > to INF file
> > IntelUndiPkg/GigUndiDxe: add missing EFIAPI modifiers
> > IntelUndiPkg/GigUndiDxe: remove or reorganize unused
> > variables
> >
> > IntelUndiPkg/GigUndiDxe/AdapterInformation.c | 6 ++-
> > IntelUndiPkg/GigUndiDxe/AdapterInformation.h | 1 -
> > IntelUndiPkg/GigUndiDxe/Brand.c | 1 -
> > IntelUndiPkg/GigUndiDxe/ComponentName.c | 5 ++-
> > IntelUndiPkg/GigUndiDxe/ComponentName.h | 2 +-
> > IntelUndiPkg/GigUndiDxe/Decode.c | 5 +--
> > IntelUndiPkg/GigUndiDxe/Decode.h | 1 -
> > IntelUndiPkg/GigUndiDxe/DeviceSupport.c | 1 -
> > IntelUndiPkg/GigUndiDxe/DeviceSupport.h | 9 ++--
> > -
> > IntelUndiPkg/GigUndiDxe/Dma.c | 11 +++-
> > --
> > IntelUndiPkg/GigUndiDxe/Dma.h | 1 -
> > IntelUndiPkg/GigUndiDxe/DriverConfiguration.c | 6 ++-
> > IntelUndiPkg/GigUndiDxe/DriverConfiguration.h | 1 -
> > IntelUndiPkg/GigUndiDxe/DriverDiagnostics.c | 12 +++-
> > --
> > IntelUndiPkg/GigUndiDxe/DriverDiagnostics.h | 1 -
> > IntelUndiPkg/GigUndiDxe/DriverHealth.c| 5 ++-
> > IntelUndiPkg/GigUndiDxe/EepromConfig.c| 1 -
> > IntelUndiPkg/GigUndiDxe/EepromConfig.h| 3 +-
> > IntelUndiPkg/GigUndiDxe/GigUndiDxe.inf| 39
> > +--
> > IntelUndiPkg/GigUndiDxe/Hii.c | 11 +++-
> > --
> > IntelUndiPkg/GigUndiDxe/Hii.h | 1 -
> > IntelUndiPkg/GigUndiDxe/HiiInternalLib.c | 3 --
> > IntelUndiPkg/GigUndiDxe/HiiInternalLib.h | 1 -
> > IntelUndiPkg/GigUndiDxe/Init.c| 11 +++-
> > --
> > IntelUndiPkg/GigUndiDxe/Init.h| 1 -
> > IntelUndiPkg/GigUndiDxe/Inventory.vfr | 1 -
> > IntelUndiPkg/GigUndiDxe/NVDataStruc.h | 7 ++--
> > IntelUndiPkg/GigUndiDxe/StartStop.c | 5 ++-
> > IntelUndiPkg/GigUndiDxe/StartStop.h | 7 ++--
> > IntelUndiPkg/GigUndiDxe/Version.h | 1 -
> > IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} | 37
> > --
> > IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} | 5 +--
> > IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.c | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.h | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_82571.c | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_82571.h | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_82575.c | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_82575.h | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_api.c | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_api.h | 1 -
> > IntelUndiPkg/GigUndiDxe/e1000_defines.
Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
On 6 November 2018 at 23:10, Kinney, Michael D wrote: > Hi Ard, > > Can you please add CC lines to the commit message > for the developers that have contributed to the > edk2-staging/Intel_UNDI branch? > > This would include: > > Cc: Maciej Rabeda Maciej was already on cc. I hope he can forward the emails to his colleagues, if they can't find them in the archives. ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
Hi Ard,
Can you please add CC lines to the commit message
for the developers that have contributed to the
edk2-staging/Intel_UNDI branch?
This would include:
Cc: Maciej Rabeda
Cc: Kamil Kacperski
Cc: Pawel Orlowski
Thanks,
Mike
> -Original Message-
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> Sent: Tuesday, November 6, 2018 9:58 AM
> To: edk2-devel@lists.01.org
> Cc: Rabeda, Maciej ; Kinney,
> Michael D ; Jin, Eric
> ; leif.lindh...@linaro.org; Ard
> Biesheuvel
> Subject: [PATCH edk2-staging 00/19]
> IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
>
> This series fixes the GigUndiDxe in the edk2-
> staging/Intel_UNDI branch
> at github.com/tianocore so it can be built with GCC on
> Linux for ARM
> and AARCH64 (as well as X64)
>
> Ard Biesheuvel (19):
> IntelOpenSourceUndiPkg.dsc: add AARCH64 and ARM to
> supported
> architectures
> IntelUndiPkg: remove EOF markers
> IntelUndiPkg/GigUndiDxe: consistently use lowercase
> for e1000 in
> filenames
> IntelUndiPkg/GigUndiDxe: consistently use forward
> slashes as path
> separators
> IntelUndiPkg/GigUndiDxe: move BRAND_STRUCT declaration
> after type
> definition
> IntelUndiPkg/GigUndiDxe: use intermediate UINTN casts
> for pointers
> IntelUndiPkg/GigUndiDxe: create GCC alternatives for
> MSFT build
> options
> IntelUndiPkg/GigUndiDxe: add missing VOID** cast
> IntelUndiPkg/GigUndiDxe: add missing UINT8* cast
> IntelUndiPkg/GigUndiDxe: add missing braces to GUID
> literals
> IntelUndiPkg/GigUndiDxe: fix incorrect use of CPP
> token pasting
> IntelUndiPkg/GigUndiDxe: cast E1000MemCopy () args to
> correct pointer
> type
> IntelUndiPkg/GigUndiDxe: don't take address of cast
> expression
> IntelUndiPkg/GigUndiDxe: redefine
> UNREFERENCED_nPARAMETER macros for
> GCC
> IntelUndiPkg/GigUndiDxe: remove forward declaration of
> non-existent
> function
> IntelUndiPkg/GigUndiDxe: fix incorrect indentation
> IntelUndiPkg/GigUndiDxe: move MSFT warning overrides
> to INF file
> IntelUndiPkg/GigUndiDxe: add missing EFIAPI modifiers
> IntelUndiPkg/GigUndiDxe: remove or reorganize unused
> variables
>
> IntelUndiPkg/GigUndiDxe/AdapterInformation.c | 6 ++-
> IntelUndiPkg/GigUndiDxe/AdapterInformation.h | 1 -
> IntelUndiPkg/GigUndiDxe/Brand.c | 1 -
> IntelUndiPkg/GigUndiDxe/ComponentName.c | 5 ++-
> IntelUndiPkg/GigUndiDxe/ComponentName.h | 2 +-
> IntelUndiPkg/GigUndiDxe/Decode.c | 5 +--
> IntelUndiPkg/GigUndiDxe/Decode.h | 1 -
> IntelUndiPkg/GigUndiDxe/DeviceSupport.c | 1 -
> IntelUndiPkg/GigUndiDxe/DeviceSupport.h | 9 ++--
> -
> IntelUndiPkg/GigUndiDxe/Dma.c | 11 +++-
> --
> IntelUndiPkg/GigUndiDxe/Dma.h | 1 -
> IntelUndiPkg/GigUndiDxe/DriverConfiguration.c | 6 ++-
> IntelUndiPkg/GigUndiDxe/DriverConfiguration.h | 1 -
> IntelUndiPkg/GigUndiDxe/DriverDiagnostics.c | 12 +++-
> --
> IntelUndiPkg/GigUndiDxe/DriverDiagnostics.h | 1 -
> IntelUndiPkg/GigUndiDxe/DriverHealth.c| 5 ++-
> IntelUndiPkg/GigUndiDxe/EepromConfig.c| 1 -
> IntelUndiPkg/GigUndiDxe/EepromConfig.h| 3 +-
> IntelUndiPkg/GigUndiDxe/GigUndiDxe.inf| 39
> +--
> IntelUndiPkg/GigUndiDxe/Hii.c | 11 +++-
> --
> IntelUndiPkg/GigUndiDxe/Hii.h | 1 -
> IntelUndiPkg/GigUndiDxe/HiiInternalLib.c | 3 --
> IntelUndiPkg/GigUndiDxe/HiiInternalLib.h | 1 -
> IntelUndiPkg/GigUndiDxe/Init.c| 11 +++-
> --
> IntelUndiPkg/GigUndiDxe/Init.h| 1 -
> IntelUndiPkg/GigUndiDxe/Inventory.vfr | 1 -
> IntelUndiPkg/GigUndiDxe/NVDataStruc.h | 7 ++--
> IntelUndiPkg/GigUndiDxe/StartStop.c | 5 ++-
> IntelUndiPkg/GigUndiDxe/StartStop.h | 7 ++--
> IntelUndiPkg/GigUndiDxe/Version.h | 1 -
> IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} | 37
> --
> IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} | 5 +--
> IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_82571.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_82571.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_82575.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_82575.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_api.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_api.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_defines.h | 10
> -
> IntelUndiPkg/GigUndiDxe/e1000_hw.h| 1 -
> IntelUndiPkg/GigUndiDxe/e1000_i210.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_i210.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_ich8lan.c | 18
> -
> IntelUndiPkg/GigUndiDxe/e1000_ich8lan.h | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_mac.c | 1 -
> IntelUndiPkg/GigUndiDxe/e1000_mac.h | 1 -
> IntelUndiP
[edk2] [PATCH edk2-staging 00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC
This series fixes the GigUndiDxe in the edk2-staging/Intel_UNDI branch
at github.com/tianocore so it can be built with GCC on Linux for ARM
and AARCH64 (as well as X64)
Ard Biesheuvel (19):
IntelOpenSourceUndiPkg.dsc: add AARCH64 and ARM to supported
architectures
IntelUndiPkg: remove EOF markers
IntelUndiPkg/GigUndiDxe: consistently use lowercase for e1000 in
filenames
IntelUndiPkg/GigUndiDxe: consistently use forward slashes as path
separators
IntelUndiPkg/GigUndiDxe: move BRAND_STRUCT declaration after type
definition
IntelUndiPkg/GigUndiDxe: use intermediate UINTN casts for pointers
IntelUndiPkg/GigUndiDxe: create GCC alternatives for MSFT build
options
IntelUndiPkg/GigUndiDxe: add missing VOID** cast
IntelUndiPkg/GigUndiDxe: add missing UINT8* cast
IntelUndiPkg/GigUndiDxe: add missing braces to GUID literals
IntelUndiPkg/GigUndiDxe: fix incorrect use of CPP token pasting
IntelUndiPkg/GigUndiDxe: cast E1000MemCopy () args to correct pointer
type
IntelUndiPkg/GigUndiDxe: don't take address of cast expression
IntelUndiPkg/GigUndiDxe: redefine UNREFERENCED_nPARAMETER macros for
GCC
IntelUndiPkg/GigUndiDxe: remove forward declaration of non-existent
function
IntelUndiPkg/GigUndiDxe: fix incorrect indentation
IntelUndiPkg/GigUndiDxe: move MSFT warning overrides to INF file
IntelUndiPkg/GigUndiDxe: add missing EFIAPI modifiers
IntelUndiPkg/GigUndiDxe: remove or reorganize unused variables
IntelUndiPkg/GigUndiDxe/AdapterInformation.c | 6 ++-
IntelUndiPkg/GigUndiDxe/AdapterInformation.h | 1 -
IntelUndiPkg/GigUndiDxe/Brand.c | 1 -
IntelUndiPkg/GigUndiDxe/ComponentName.c | 5 ++-
IntelUndiPkg/GigUndiDxe/ComponentName.h | 2 +-
IntelUndiPkg/GigUndiDxe/Decode.c | 5 +--
IntelUndiPkg/GigUndiDxe/Decode.h | 1 -
IntelUndiPkg/GigUndiDxe/DeviceSupport.c | 1 -
IntelUndiPkg/GigUndiDxe/DeviceSupport.h | 9 ++---
IntelUndiPkg/GigUndiDxe/Dma.c | 11 +++---
IntelUndiPkg/GigUndiDxe/Dma.h | 1 -
IntelUndiPkg/GigUndiDxe/DriverConfiguration.c | 6 ++-
IntelUndiPkg/GigUndiDxe/DriverConfiguration.h | 1 -
IntelUndiPkg/GigUndiDxe/DriverDiagnostics.c | 12 +++---
IntelUndiPkg/GigUndiDxe/DriverDiagnostics.h | 1 -
IntelUndiPkg/GigUndiDxe/DriverHealth.c| 5 ++-
IntelUndiPkg/GigUndiDxe/EepromConfig.c| 1 -
IntelUndiPkg/GigUndiDxe/EepromConfig.h| 3 +-
IntelUndiPkg/GigUndiDxe/GigUndiDxe.inf| 39 +--
IntelUndiPkg/GigUndiDxe/Hii.c | 11 +++---
IntelUndiPkg/GigUndiDxe/Hii.h | 1 -
IntelUndiPkg/GigUndiDxe/HiiInternalLib.c | 3 --
IntelUndiPkg/GigUndiDxe/HiiInternalLib.h | 1 -
IntelUndiPkg/GigUndiDxe/Init.c| 11 +++---
IntelUndiPkg/GigUndiDxe/Init.h| 1 -
IntelUndiPkg/GigUndiDxe/Inventory.vfr | 1 -
IntelUndiPkg/GigUndiDxe/NVDataStruc.h | 7 ++--
IntelUndiPkg/GigUndiDxe/StartStop.c | 5 ++-
IntelUndiPkg/GigUndiDxe/StartStop.h | 7 ++--
IntelUndiPkg/GigUndiDxe/Version.h | 1 -
IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} | 37 --
IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} | 5 +--
IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.c | 1 -
IntelUndiPkg/GigUndiDxe/e1000_80003es2lan.h | 1 -
IntelUndiPkg/GigUndiDxe/e1000_82571.c | 1 -
IntelUndiPkg/GigUndiDxe/e1000_82571.h | 1 -
IntelUndiPkg/GigUndiDxe/e1000_82575.c | 1 -
IntelUndiPkg/GigUndiDxe/e1000_82575.h | 1 -
IntelUndiPkg/GigUndiDxe/e1000_api.c | 1 -
IntelUndiPkg/GigUndiDxe/e1000_api.h | 1 -
IntelUndiPkg/GigUndiDxe/e1000_defines.h | 10 -
IntelUndiPkg/GigUndiDxe/e1000_hw.h| 1 -
IntelUndiPkg/GigUndiDxe/e1000_i210.c | 1 -
IntelUndiPkg/GigUndiDxe/e1000_i210.h | 1 -
IntelUndiPkg/GigUndiDxe/e1000_ich8lan.c | 18 -
IntelUndiPkg/GigUndiDxe/e1000_ich8lan.h | 1 -
IntelUndiPkg/GigUndiDxe/e1000_mac.c | 1 -
IntelUndiPkg/GigUndiDxe/e1000_mac.h | 1 -
IntelUndiPkg/GigUndiDxe/e1000_manage.c| 1 -
IntelUndiPkg/GigUndiDxe/e1000_manage.h| 1 -
IntelUndiPkg/GigUndiDxe/e1000_nvm.c | 1 -
IntelUndiPkg/GigUndiDxe/e1000_nvm.h | 1 -
.../{E1000_osdep.c => e1000_osdep.c} | 3 +-
.../{E1000_osdep.h => e1000_osdep.h} | 8 +---
IntelUndiPkg/GigUndiDxe/e1000_phy.c | 1 -
IntelUndiPkg/GigUndiDxe/e1000_phy.h | 1 -
IntelUndiPkg/GigUndiDxe/e1000_regs.h | 1 -
IntelUndiPkg/GigUndiDxe/wol/wol.c | 1 -
IntelUndiPkg/GigUndiDxe/wol/wol.h | 1 -
IntelUndiPkg/GigUndiDxe/wol/wol_1G.c | 1 -
IntelUndiPkg/GigUndiDxe/wol/wolfamily.c | 1 -
IntelUndiPkg/GigUndiDxe/wol/wolimpl.c | 1 -
IntelUndiPkg/GigUndiDxe/wol/wolimpl.h | 3 +-
IntelUndiPkg/Gi
