mbox series

[edk2,edk2-staging,00/19] IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC

Message ID 20181106175833.26964-1-ard.biesheuvel@linaro.org
Headers show
Series IntelUndiPkg/GigUndiDxe: build fixes for AARCH64/ARM/GCC | expand

Message

Ard Biesheuvel Nov. 6, 2018, 5:58 p.m. UTC
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/GigUndiDxe/wol/wolinfo.c         |  1 -
 IntelUndiPkg/I40eUndiDxe/AdapterInformation.c |  1 -
 IntelUndiPkg/I40eUndiDxe/AdapterInformation.h |  1 -
 IntelUndiPkg/I40eUndiDxe/Brand.c              |  1 -
 IntelUndiPkg/I40eUndiDxe/ComponentName.c      |  1 -
 IntelUndiPkg/I40eUndiDxe/ComponentName.h      |  1 -
 IntelUndiPkg/I40eUndiDxe/Decode.c             |  1 -
 IntelUndiPkg/I40eUndiDxe/Decode.h             |  1 -
 IntelUndiPkg/I40eUndiDxe/DeviceSupport.c      |  1 -
 IntelUndiPkg/I40eUndiDxe/DeviceSupport.h      |  1 -
 IntelUndiPkg/I40eUndiDxe/Dma.c                |  1 -
 IntelUndiPkg/I40eUndiDxe/Dma.h                |  1 -
 IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.c  |  1 -
 IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.h  |  1 -
 IntelUndiPkg/I40eUndiDxe/DriverHealth.c       |  1 -
 IntelUndiPkg/I40eUndiDxe/DriverHealthCommon.h |  1 -
 IntelUndiPkg/I40eUndiDxe/EepromConfig.c       |  1 -
 IntelUndiPkg/I40eUndiDxe/EepromConfig.h       |  1 -
 IntelUndiPkg/I40eUndiDxe/Hii.c                |  1 -
 IntelUndiPkg/I40eUndiDxe/Hii.h                |  1 -
 IntelUndiPkg/I40eUndiDxe/HiiInternalLib.c     |  1 -
 IntelUndiPkg/I40eUndiDxe/HiiInternalLib.h     |  1 -
 IntelUndiPkg/I40eUndiDxe/I40e.c               |  1 -
 IntelUndiPkg/I40eUndiDxe/I40e.h               |  1 -
 IntelUndiPkg/I40eUndiDxe/I40eUndiDxe.inf      |  1 -
 IntelUndiPkg/I40eUndiDxe/I40e_osdep.h         |  1 -
 IntelUndiPkg/I40eUndiDxe/Init.c               |  1 -
 IntelUndiPkg/I40eUndiDxe/Init.h               |  1 -
 IntelUndiPkg/I40eUndiDxe/Inventory.vfr        |  1 -
 IntelUndiPkg/I40eUndiDxe/NVDataStruc.h        |  1 -
 IntelUndiPkg/I40eUndiDxe/StartStop.c          |  1 -
 IntelUndiPkg/I40eUndiDxe/StartStop.h          |  1 -
 IntelUndiPkg/I40eUndiDxe/Version.h            |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_adminq.c        |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_adminq.h        |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_adminq_cmd.h    |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_alloc.h         |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_common.c        |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_devids.h        |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_diag.c          |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_diag.h          |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_hmc.c           |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_hmc.h           |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.c       |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.h       |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_nvm.c           |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_prototype.h     |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_register.h      |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_status.h        |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_type.h          |  1 -
 IntelUndiPkg/I40eUndiDxe/i40e_virtchnl.h      |  1 -
 IntelUndiPkg/IntelOpenSourceUndiPkg.dsc       | 13 ++++++-
 IntelUndiPkg/XGigUndiDxe/AdapterInformation.c |  1 -
 IntelUndiPkg/XGigUndiDxe/AdapterInformation.h |  1 -
 IntelUndiPkg/XGigUndiDxe/Brand.c              |  1 -
 IntelUndiPkg/XGigUndiDxe/ComponentName.c      |  1 -
 IntelUndiPkg/XGigUndiDxe/ComponentName.h      |  1 -
 IntelUndiPkg/XGigUndiDxe/Decode.c             |  1 -
 IntelUndiPkg/XGigUndiDxe/Decode.h             |  1 -
 IntelUndiPkg/XGigUndiDxe/DeviceSupport.c      |  1 -
 IntelUndiPkg/XGigUndiDxe/DeviceSupport.h      |  1 -
 IntelUndiPkg/XGigUndiDxe/Dma.c                |  1 -
 IntelUndiPkg/XGigUndiDxe/Dma.h                |  1 -
 IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.c  |  1 -
 IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.h  |  1 -
 IntelUndiPkg/XGigUndiDxe/DriverHealth.c       |  1 -
 IntelUndiPkg/XGigUndiDxe/DriverHealthCommon.h |  1 -
 IntelUndiPkg/XGigUndiDxe/EepromConfig.c       |  1 -
 IntelUndiPkg/XGigUndiDxe/EepromConfig.h       |  1 -
 IntelUndiPkg/XGigUndiDxe/Hii.c                |  1 -
 IntelUndiPkg/XGigUndiDxe/Hii.h                |  1 -
 IntelUndiPkg/XGigUndiDxe/HiiInternalLib.c     |  1 -
 IntelUndiPkg/XGigUndiDxe/HiiInternalLib.h     |  1 -
 IntelUndiPkg/XGigUndiDxe/Init.c               |  1 -
 IntelUndiPkg/XGigUndiDxe/Init.h               |  1 -
 IntelUndiPkg/XGigUndiDxe/Inventory.vfr        |  1 -
 IntelUndiPkg/XGigUndiDxe/NVDataStruc.h        |  1 -
 IntelUndiPkg/XGigUndiDxe/StartStop.c          |  1 -
 IntelUndiPkg/XGigUndiDxe/StartStop.h          |  1 -
 IntelUndiPkg/XGigUndiDxe/Version.h            |  1 -
 IntelUndiPkg/XGigUndiDxe/XGigUndiDxe.inf      |  1 -
 IntelUndiPkg/XGigUndiDxe/Xgbe.c               |  1 -
 IntelUndiPkg/XGigUndiDxe/Xgbe.h               |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_82598.c        |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_82598.h        |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_82599.c        |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_82599.h        |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_api.c          |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_api.h          |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_common.c       |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_common.h       |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_dcb.h          |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_dcb_82599.h    |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_osdep.h        |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_phy.c          |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_phy.h          |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_type.h         |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_x540.c         |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_x540.h         |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_x550.c         |  1 -
 IntelUndiPkg/XGigUndiDxe/ixgbe_x550.h         |  1 -
 IntelUndiPkg/XGigUndiDxe/wol/wol.c            |  1 -
 IntelUndiPkg/XGigUndiDxe/wol/wol.h            |  1 -
 IntelUndiPkg/XGigUndiDxe/wol/wol_10G.c        |  1 -
 IntelUndiPkg/XGigUndiDxe/wol/wolfamily.c      |  1 -
 IntelUndiPkg/XGigUndiDxe/wol/wolimpl.c        |  1 -
 IntelUndiPkg/XGigUndiDxe/wol/wolimpl.h        |  1 -
 IntelUndiPkg/XGigUndiDxe/wol/wolinfo.c        |  1 -
 171 files changed, 120 insertions(+), 270 deletions(-)
 rename IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} (99%)
 rename IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} (99%)
 rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.c => e1000_osdep.c} (99%)
 rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.h => e1000_osdep.h} (99%)

-- 
2.19.1

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Comments

Kinney, Michael D Nov. 6, 2018, 10:10 p.m. UTC | #1
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.rabeda@intel.com>
Cc: Kamil Kacperski <kamil.kacperski@intel.com>
Cc: Pawel Orlowski <pawel.orlowski@intel.com>

Thanks,

Mike


> -----Original Message-----

> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]

> Sent: Tuesday, November 6, 2018 9:58 AM

> To: edk2-devel@lists.01.org

> Cc: Rabeda, Maciej <maciej.rabeda@intel.com>; Kinney,

> Michael D <michael.d.kinney@intel.com>; Jin, Eric

> <eric.jin@intel.com>; leif.lindholm@linaro.org; Ard

> Biesheuvel <ard.biesheuvel@linaro.org>

> 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 -

>  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/GigUndiDxe/wol/wolinfo.c         |  1 -

>  IntelUndiPkg/I40eUndiDxe/AdapterInformation.c |  1 -

>  IntelUndiPkg/I40eUndiDxe/AdapterInformation.h |  1 -

>  IntelUndiPkg/I40eUndiDxe/Brand.c              |  1 -

>  IntelUndiPkg/I40eUndiDxe/ComponentName.c      |  1 -

>  IntelUndiPkg/I40eUndiDxe/ComponentName.h      |  1 -

>  IntelUndiPkg/I40eUndiDxe/Decode.c             |  1 -

>  IntelUndiPkg/I40eUndiDxe/Decode.h             |  1 -

>  IntelUndiPkg/I40eUndiDxe/DeviceSupport.c      |  1 -

>  IntelUndiPkg/I40eUndiDxe/DeviceSupport.h      |  1 -

>  IntelUndiPkg/I40eUndiDxe/Dma.c                |  1 -

>  IntelUndiPkg/I40eUndiDxe/Dma.h                |  1 -

>  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.c  |  1 -

>  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.h  |  1 -

>  IntelUndiPkg/I40eUndiDxe/DriverHealth.c       |  1 -

>  IntelUndiPkg/I40eUndiDxe/DriverHealthCommon.h |  1 -

>  IntelUndiPkg/I40eUndiDxe/EepromConfig.c       |  1 -

>  IntelUndiPkg/I40eUndiDxe/EepromConfig.h       |  1 -

>  IntelUndiPkg/I40eUndiDxe/Hii.c                |  1 -

>  IntelUndiPkg/I40eUndiDxe/Hii.h                |  1 -

>  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.c     |  1 -

>  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.h     |  1 -

>  IntelUndiPkg/I40eUndiDxe/I40e.c               |  1 -

>  IntelUndiPkg/I40eUndiDxe/I40e.h               |  1 -

>  IntelUndiPkg/I40eUndiDxe/I40eUndiDxe.inf      |  1 -

>  IntelUndiPkg/I40eUndiDxe/I40e_osdep.h         |  1 -

>  IntelUndiPkg/I40eUndiDxe/Init.c               |  1 -

>  IntelUndiPkg/I40eUndiDxe/Init.h               |  1 -

>  IntelUndiPkg/I40eUndiDxe/Inventory.vfr        |  1 -

>  IntelUndiPkg/I40eUndiDxe/NVDataStruc.h        |  1 -

>  IntelUndiPkg/I40eUndiDxe/StartStop.c          |  1 -

>  IntelUndiPkg/I40eUndiDxe/StartStop.h          |  1 -

>  IntelUndiPkg/I40eUndiDxe/Version.h            |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_adminq.c        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_adminq.h        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_adminq_cmd.h    |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_alloc.h         |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_common.c        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_devids.h        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_diag.c          |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_diag.h          |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_hmc.c           |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_hmc.h           |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.c       |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.h       |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_nvm.c           |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_prototype.h     |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_register.h      |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_status.h        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_type.h          |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_virtchnl.h      |  1 -

>  IntelUndiPkg/IntelOpenSourceUndiPkg.dsc       | 13

> ++++++-

>  IntelUndiPkg/XGigUndiDxe/AdapterInformation.c |  1 -

>  IntelUndiPkg/XGigUndiDxe/AdapterInformation.h |  1 -

>  IntelUndiPkg/XGigUndiDxe/Brand.c              |  1 -

>  IntelUndiPkg/XGigUndiDxe/ComponentName.c      |  1 -

>  IntelUndiPkg/XGigUndiDxe/ComponentName.h      |  1 -

>  IntelUndiPkg/XGigUndiDxe/Decode.c             |  1 -

>  IntelUndiPkg/XGigUndiDxe/Decode.h             |  1 -

>  IntelUndiPkg/XGigUndiDxe/DeviceSupport.c      |  1 -

>  IntelUndiPkg/XGigUndiDxe/DeviceSupport.h      |  1 -

>  IntelUndiPkg/XGigUndiDxe/Dma.c                |  1 -

>  IntelUndiPkg/XGigUndiDxe/Dma.h                |  1 -

>  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.c  |  1 -

>  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.h  |  1 -

>  IntelUndiPkg/XGigUndiDxe/DriverHealth.c       |  1 -

>  IntelUndiPkg/XGigUndiDxe/DriverHealthCommon.h |  1 -

>  IntelUndiPkg/XGigUndiDxe/EepromConfig.c       |  1 -

>  IntelUndiPkg/XGigUndiDxe/EepromConfig.h       |  1 -

>  IntelUndiPkg/XGigUndiDxe/Hii.c                |  1 -

>  IntelUndiPkg/XGigUndiDxe/Hii.h                |  1 -

>  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.c     |  1 -

>  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.h     |  1 -

>  IntelUndiPkg/XGigUndiDxe/Init.c               |  1 -

>  IntelUndiPkg/XGigUndiDxe/Init.h               |  1 -

>  IntelUndiPkg/XGigUndiDxe/Inventory.vfr        |  1 -

>  IntelUndiPkg/XGigUndiDxe/NVDataStruc.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/StartStop.c          |  1 -

>  IntelUndiPkg/XGigUndiDxe/StartStop.h          |  1 -

>  IntelUndiPkg/XGigUndiDxe/Version.h            |  1 -

>  IntelUndiPkg/XGigUndiDxe/XGigUndiDxe.inf      |  1 -

>  IntelUndiPkg/XGigUndiDxe/Xgbe.c               |  1 -

>  IntelUndiPkg/XGigUndiDxe/Xgbe.h               |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.c        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.c        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_api.c          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_api.h          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_common.c       |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_common.h       |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb.h          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb_82599.h    |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_osdep.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.c          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.h          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_type.h         |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.c         |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.h         |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.c         |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.h         |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wol.c            |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wol.h            |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wol_10G.c        |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wolfamily.c      |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.c        |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wolinfo.c        |  1 -

>  171 files changed, 120 insertions(+), 270 deletions(-)

>  rename IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c}

> (99%)

>  rename IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h}

> (99%)

>  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.c =>

> e1000_osdep.c} (99%)

>  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.h =>

> e1000_osdep.h} (99%)

> 

> --

> 2.19.1


_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ard Biesheuvel Nov. 6, 2018, 11:03 p.m. UTC | #2
On 6 November 2018 at 23:10, Kinney, Michael D
<michael.d.kinney@intel.com> 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.rabeda@intel.com>


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
Leif Lindholm Nov. 7, 2018, 2:56 p.m. UTC | #3
On Tue, Nov 06, 2018 at 10:10:16PM +0000, 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.rabeda@intel.com>

> Cc: Kamil Kacperski <kamil.kacperski@intel.com>


Could these two be added to the top-level Readme.MD?

(Pawel is already listed as the branch owner.)

Regards,

Leif

> Cc: Pawel Orlowski <pawel.orlowski@intel.com>

> 

> Thanks,

> 

> Mike

> 

> 

> > -----Original Message-----

> > From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]

> > Sent: Tuesday, November 6, 2018 9:58 AM

> > To: edk2-devel@lists.01.org

> > Cc: Rabeda, Maciej <maciej.rabeda@intel.com>; Kinney,

> > Michael D <michael.d.kinney@intel.com>; Jin, Eric

> > <eric.jin@intel.com>; leif.lindholm@linaro.org; Ard

> > Biesheuvel <ard.biesheuvel@linaro.org>

> > 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 -

> >  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/GigUndiDxe/wol/wolinfo.c         |  1 -

> >  IntelUndiPkg/I40eUndiDxe/AdapterInformation.c |  1 -

> >  IntelUndiPkg/I40eUndiDxe/AdapterInformation.h |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Brand.c              |  1 -

> >  IntelUndiPkg/I40eUndiDxe/ComponentName.c      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/ComponentName.h      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Decode.c             |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Decode.h             |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DeviceSupport.c      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DeviceSupport.h      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Dma.c                |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Dma.h                |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.c  |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.h  |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DriverHealth.c       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DriverHealthCommon.h |  1 -

> >  IntelUndiPkg/I40eUndiDxe/EepromConfig.c       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/EepromConfig.h       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Hii.c                |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Hii.h                |  1 -

> >  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.c     |  1 -

> >  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.h     |  1 -

> >  IntelUndiPkg/I40eUndiDxe/I40e.c               |  1 -

> >  IntelUndiPkg/I40eUndiDxe/I40e.h               |  1 -

> >  IntelUndiPkg/I40eUndiDxe/I40eUndiDxe.inf      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/I40e_osdep.h         |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Init.c               |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Init.h               |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Inventory.vfr        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/NVDataStruc.h        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/StartStop.c          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/StartStop.h          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Version.h            |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_adminq.c        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_adminq.h        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_adminq_cmd.h    |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_alloc.h         |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_common.c        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_devids.h        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_diag.c          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_diag.h          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_hmc.c           |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_hmc.h           |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.c       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.h       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_nvm.c           |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_prototype.h     |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_register.h      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_status.h        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_type.h          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_virtchnl.h      |  1 -

> >  IntelUndiPkg/IntelOpenSourceUndiPkg.dsc       | 13

> > ++++++-

> >  IntelUndiPkg/XGigUndiDxe/AdapterInformation.c |  1 -

> >  IntelUndiPkg/XGigUndiDxe/AdapterInformation.h |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Brand.c              |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ComponentName.c      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ComponentName.h      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Decode.c             |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Decode.h             |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DeviceSupport.c      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DeviceSupport.h      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Dma.c                |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Dma.h                |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.c  |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.h  |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DriverHealth.c       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DriverHealthCommon.h |  1 -

> >  IntelUndiPkg/XGigUndiDxe/EepromConfig.c       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/EepromConfig.h       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Hii.c                |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Hii.h                |  1 -

> >  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.c     |  1 -

> >  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.h     |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Init.c               |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Init.h               |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Inventory.vfr        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/NVDataStruc.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/StartStop.c          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/StartStop.h          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Version.h            |  1 -

> >  IntelUndiPkg/XGigUndiDxe/XGigUndiDxe.inf      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Xgbe.c               |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Xgbe.h               |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.c        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.c        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_api.c          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_api.h          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_common.c       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_common.h       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb.h          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb_82599.h    |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_osdep.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.c          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.h          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_type.h         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.c         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.h         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.c         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.h         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wol.c            |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wol.h            |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wol_10G.c        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wolfamily.c      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.c        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wolinfo.c        |  1 -

> >  171 files changed, 120 insertions(+), 270 deletions(-)

> >  rename IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c}

> > (99%)

> >  rename IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h}

> > (99%)

> >  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.c =>

> > e1000_osdep.c} (99%)

> >  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.h =>

> > e1000_osdep.h} (99%)

> > 

> > --

> > 2.19.1

> 

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Knop, Ryszard Nov. 9, 2018, 5:54 p.m. UTC | #4
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-bounces@lists.01.org] On Behalf Of Ard Biesheuvel

Sent: Wednesday, November 7, 2018 0:04
To: Kinney, Michael D <michael.d.kinney@intel.com>
Cc: Jin, Eric <eric.jin@intel.com>; edk2-devel@lists.01.org; Kacperski, Kamil <kamil.kacperski@intel.com>; Orlowski, Pawel <pawel.orlowski@intel.com>
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 <michael.d.kinney@intel.com> 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.rabeda@intel.com>


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
Ard Biesheuvel Nov. 9, 2018, 6 p.m. UTC | #5
On 9 November 2018 at 18:54, Knop, Ryszard <ryszard.knop@intel.com> 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-bounces@lists.01.org] On Behalf Of Ard Biesheuvel

> Sent: Wednesday, November 7, 2018 0:04

> To: Kinney, Michael D <michael.d.kinney@intel.com>

> Cc: Jin, Eric <eric.jin@intel.com>; edk2-devel@lists.01.org; Kacperski, Kamil <kamil.kacperski@intel.com>; Orlowski, Pawel <pawel.orlowski@intel.com>

> 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 <michael.d.kinney@intel.com> 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.rabeda@intel.com>

>

> 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
Ryszard Knop Jan. 29, 2019, 1:13 p.m. UTC | #6
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        |  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/GigUndiDxe/wol/wolinfo.c         |  1 -

>  IntelUndiPkg/I40eUndiDxe/AdapterInformation.c |  1 -

>  IntelUndiPkg/I40eUndiDxe/AdapterInformation.h |  1 -

>  IntelUndiPkg/I40eUndiDxe/Brand.c              |  1 -

>  IntelUndiPkg/I40eUndiDxe/ComponentName.c      |  1 -

>  IntelUndiPkg/I40eUndiDxe/ComponentName.h      |  1 -

>  IntelUndiPkg/I40eUndiDxe/Decode.c             |  1 -

>  IntelUndiPkg/I40eUndiDxe/Decode.h             |  1 -

>  IntelUndiPkg/I40eUndiDxe/DeviceSupport.c      |  1 -

>  IntelUndiPkg/I40eUndiDxe/DeviceSupport.h      |  1 -

>  IntelUndiPkg/I40eUndiDxe/Dma.c                |  1 -

>  IntelUndiPkg/I40eUndiDxe/Dma.h                |  1 -

>  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.c  |  1 -

>  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.h  |  1 -

>  IntelUndiPkg/I40eUndiDxe/DriverHealth.c       |  1 -

>  IntelUndiPkg/I40eUndiDxe/DriverHealthCommon.h |  1 -

>  IntelUndiPkg/I40eUndiDxe/EepromConfig.c       |  1 -

>  IntelUndiPkg/I40eUndiDxe/EepromConfig.h       |  1 -

>  IntelUndiPkg/I40eUndiDxe/Hii.c                |  1 -

>  IntelUndiPkg/I40eUndiDxe/Hii.h                |  1 -

>  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.c     |  1 -

>  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.h     |  1 -

>  IntelUndiPkg/I40eUndiDxe/I40e.c               |  1 -

>  IntelUndiPkg/I40eUndiDxe/I40e.h               |  1 -

>  IntelUndiPkg/I40eUndiDxe/I40eUndiDxe.inf      |  1 -

>  IntelUndiPkg/I40eUndiDxe/I40e_osdep.h         |  1 -

>  IntelUndiPkg/I40eUndiDxe/Init.c               |  1 -

>  IntelUndiPkg/I40eUndiDxe/Init.h               |  1 -

>  IntelUndiPkg/I40eUndiDxe/Inventory.vfr        |  1 -

>  IntelUndiPkg/I40eUndiDxe/NVDataStruc.h        |  1 -

>  IntelUndiPkg/I40eUndiDxe/StartStop.c          |  1 -

>  IntelUndiPkg/I40eUndiDxe/StartStop.h          |  1 -

>  IntelUndiPkg/I40eUndiDxe/Version.h            |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_adminq.c        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_adminq.h        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_adminq_cmd.h    |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_alloc.h         |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_common.c        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_devids.h        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_diag.c          |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_diag.h          |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_hmc.c           |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_hmc.h           |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.c       |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.h       |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_nvm.c           |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_prototype.h     |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_register.h      |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_status.h        |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_type.h          |  1 -

>  IntelUndiPkg/I40eUndiDxe/i40e_virtchnl.h      |  1 -

>  IntelUndiPkg/IntelOpenSourceUndiPkg.dsc       | 13 ++++++-

>  IntelUndiPkg/XGigUndiDxe/AdapterInformation.c |  1 -

>  IntelUndiPkg/XGigUndiDxe/AdapterInformation.h |  1 -

>  IntelUndiPkg/XGigUndiDxe/Brand.c              |  1 -

>  IntelUndiPkg/XGigUndiDxe/ComponentName.c      |  1 -

>  IntelUndiPkg/XGigUndiDxe/ComponentName.h      |  1 -

>  IntelUndiPkg/XGigUndiDxe/Decode.c             |  1 -

>  IntelUndiPkg/XGigUndiDxe/Decode.h             |  1 -

>  IntelUndiPkg/XGigUndiDxe/DeviceSupport.c      |  1 -

>  IntelUndiPkg/XGigUndiDxe/DeviceSupport.h      |  1 -

>  IntelUndiPkg/XGigUndiDxe/Dma.c                |  1 -

>  IntelUndiPkg/XGigUndiDxe/Dma.h                |  1 -

>  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.c  |  1 -

>  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.h  |  1 -

>  IntelUndiPkg/XGigUndiDxe/DriverHealth.c       |  1 -

>  IntelUndiPkg/XGigUndiDxe/DriverHealthCommon.h |  1 -

>  IntelUndiPkg/XGigUndiDxe/EepromConfig.c       |  1 -

>  IntelUndiPkg/XGigUndiDxe/EepromConfig.h       |  1 -

>  IntelUndiPkg/XGigUndiDxe/Hii.c                |  1 -

>  IntelUndiPkg/XGigUndiDxe/Hii.h                |  1 -

>  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.c     |  1 -

>  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.h     |  1 -

>  IntelUndiPkg/XGigUndiDxe/Init.c               |  1 -

>  IntelUndiPkg/XGigUndiDxe/Init.h               |  1 -

>  IntelUndiPkg/XGigUndiDxe/Inventory.vfr        |  1 -

>  IntelUndiPkg/XGigUndiDxe/NVDataStruc.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/StartStop.c          |  1 -

>  IntelUndiPkg/XGigUndiDxe/StartStop.h          |  1 -

>  IntelUndiPkg/XGigUndiDxe/Version.h            |  1 -

>  IntelUndiPkg/XGigUndiDxe/XGigUndiDxe.inf      |  1 -

>  IntelUndiPkg/XGigUndiDxe/Xgbe.c               |  1 -

>  IntelUndiPkg/XGigUndiDxe/Xgbe.h               |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.c        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.c        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_api.c          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_api.h          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_common.c       |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_common.h       |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb.h          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb_82599.h    |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_osdep.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.c          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.h          |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_type.h         |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.c         |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.h         |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.c         |  1 -

>  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.h         |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wol.c            |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wol.h            |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wol_10G.c        |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wolfamily.c      |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.c        |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.h        |  1 -

>  IntelUndiPkg/XGigUndiDxe/wol/wolinfo.c        |  1 -

>  171 files changed, 120 insertions(+), 270 deletions(-)

>  rename IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} (99%)

>  rename IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} (99%)

>  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.c => e1000_osdep.c}

> (99%)

>  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.h => e1000_osdep.h}

> (99%)

> 


_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ryszard Knop Jan. 29, 2019, 1:55 p.m. UTC | #7
+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 ++++-----

> >  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/GigUndiDxe/wol/wolinfo.c         |  1 -

> >  IntelUndiPkg/I40eUndiDxe/AdapterInformation.c |  1 -

> >  IntelUndiPkg/I40eUndiDxe/AdapterInformation.h |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Brand.c              |  1 -

> >  IntelUndiPkg/I40eUndiDxe/ComponentName.c      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/ComponentName.h      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Decode.c             |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Decode.h             |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DeviceSupport.c      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DeviceSupport.h      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Dma.c                |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Dma.h                |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.c  |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.h  |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DriverHealth.c       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/DriverHealthCommon.h |  1 -

> >  IntelUndiPkg/I40eUndiDxe/EepromConfig.c       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/EepromConfig.h       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Hii.c                |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Hii.h                |  1 -

> >  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.c     |  1 -

> >  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.h     |  1 -

> >  IntelUndiPkg/I40eUndiDxe/I40e.c               |  1 -

> >  IntelUndiPkg/I40eUndiDxe/I40e.h               |  1 -

> >  IntelUndiPkg/I40eUndiDxe/I40eUndiDxe.inf      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/I40e_osdep.h         |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Init.c               |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Init.h               |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Inventory.vfr        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/NVDataStruc.h        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/StartStop.c          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/StartStop.h          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/Version.h            |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_adminq.c        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_adminq.h        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_adminq_cmd.h    |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_alloc.h         |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_common.c        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_devids.h        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_diag.c          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_diag.h          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_hmc.c           |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_hmc.h           |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.c       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.h       |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_nvm.c           |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_prototype.h     |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_register.h      |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_status.h        |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_type.h          |  1 -

> >  IntelUndiPkg/I40eUndiDxe/i40e_virtchnl.h      |  1 -

> >  IntelUndiPkg/IntelOpenSourceUndiPkg.dsc       | 13 ++++++-

> >  IntelUndiPkg/XGigUndiDxe/AdapterInformation.c |  1 -

> >  IntelUndiPkg/XGigUndiDxe/AdapterInformation.h |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Brand.c              |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ComponentName.c      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ComponentName.h      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Decode.c             |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Decode.h             |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DeviceSupport.c      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DeviceSupport.h      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Dma.c                |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Dma.h                |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.c  |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.h  |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DriverHealth.c       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/DriverHealthCommon.h |  1 -

> >  IntelUndiPkg/XGigUndiDxe/EepromConfig.c       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/EepromConfig.h       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Hii.c                |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Hii.h                |  1 -

> >  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.c     |  1 -

> >  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.h     |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Init.c               |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Init.h               |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Inventory.vfr        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/NVDataStruc.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/StartStop.c          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/StartStop.h          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Version.h            |  1 -

> >  IntelUndiPkg/XGigUndiDxe/XGigUndiDxe.inf      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Xgbe.c               |  1 -

> >  IntelUndiPkg/XGigUndiDxe/Xgbe.h               |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.c        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.c        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_api.c          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_api.h          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_common.c       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_common.h       |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb.h          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb_82599.h    |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_osdep.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.c          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.h          |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_type.h         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.c         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.h         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.c         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.h         |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wol.c            |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wol.h            |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wol_10G.c        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wolfamily.c      |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.c        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.h        |  1 -

> >  IntelUndiPkg/XGigUndiDxe/wol/wolinfo.c        |  1 -

> >  171 files changed, 120 insertions(+), 270 deletions(-)

> >  rename IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} (99%)

> >  rename IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} (99%)

> >  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.c => e1000_osdep.c}

> > (99%)

> >  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.h => e1000_osdep.h}

> > (99%)

> > 

> 

> _______________________________________________

> edk2-devel mailing list

> edk2-devel@lists.01.org

> https://lists.01.org/mailman/listinfo/edk2-devel


_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ard Biesheuvel March 27, 2019, 3:32 p.m. UTC | #8
On Tue, 29 Jan 2019 at 14:55, Ryszard Knop <ryszard.knop@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.





> 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 ++++-----

> > >  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/GigUndiDxe/wol/wolinfo.c         |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/AdapterInformation.c |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/AdapterInformation.h |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Brand.c              |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/ComponentName.c      |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/ComponentName.h      |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Decode.c             |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Decode.h             |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/DeviceSupport.c      |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/DeviceSupport.h      |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Dma.c                |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Dma.h                |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.c  |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.h  |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/DriverHealth.c       |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/DriverHealthCommon.h |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/EepromConfig.c       |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/EepromConfig.h       |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Hii.c                |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Hii.h                |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.c     |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.h     |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/I40e.c               |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/I40e.h               |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/I40eUndiDxe.inf      |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/I40e_osdep.h         |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Init.c               |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Init.h               |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Inventory.vfr        |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/NVDataStruc.h        |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/StartStop.c          |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/StartStop.h          |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/Version.h            |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_adminq.c        |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_adminq.h        |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_adminq_cmd.h    |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_alloc.h         |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_common.c        |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_devids.h        |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_diag.c          |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_diag.h          |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_hmc.c           |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_hmc.h           |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.c       |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.h       |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_nvm.c           |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_prototype.h     |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_register.h      |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_status.h        |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_type.h          |  1 -

> > >  IntelUndiPkg/I40eUndiDxe/i40e_virtchnl.h      |  1 -

> > >  IntelUndiPkg/IntelOpenSourceUndiPkg.dsc       | 13 ++++++-

> > >  IntelUndiPkg/XGigUndiDxe/AdapterInformation.c |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/AdapterInformation.h |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Brand.c              |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ComponentName.c      |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ComponentName.h      |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Decode.c             |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Decode.h             |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/DeviceSupport.c      |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/DeviceSupport.h      |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Dma.c                |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Dma.h                |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.c  |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.h  |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/DriverHealth.c       |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/DriverHealthCommon.h |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/EepromConfig.c       |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/EepromConfig.h       |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Hii.c                |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Hii.h                |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.c     |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.h     |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Init.c               |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Init.h               |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Inventory.vfr        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/NVDataStruc.h        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/StartStop.c          |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/StartStop.h          |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Version.h            |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/XGigUndiDxe.inf      |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Xgbe.c               |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/Xgbe.h               |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.c        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.h        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.c        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.h        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_api.c          |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_api.h          |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_common.c       |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_common.h       |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb.h          |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb_82599.h    |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_osdep.h        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.c          |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.h          |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_type.h         |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.c         |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.h         |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.c         |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.h         |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/wol/wol.c            |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/wol/wol.h            |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/wol/wol_10G.c        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/wol/wolfamily.c      |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.c        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.h        |  1 -

> > >  IntelUndiPkg/XGigUndiDxe/wol/wolinfo.c        |  1 -

> > >  171 files changed, 120 insertions(+), 270 deletions(-)

> > >  rename IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} (99%)

> > >  rename IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} (99%)

> > >  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.c => e1000_osdep.c}

> > > (99%)

> > >  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.h => e1000_osdep.h}

> > > (99%)

> > >

> >

> > _______________________________________________

> > edk2-devel mailing list

> > edk2-devel@lists.01.org

> > https://lists.01.org/mailman/listinfo/edk2-devel

>

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ryszard Knop March 28, 2019, 10:46 a.m. UTC | #9
On Wed, 2019-03-27 at 16:32 +0100, Ard Biesheuvel wrote:
> On Tue, 29 Jan 2019 at 14:55, Ryszard Knop <

> ryszard.knop@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 -

> > > >  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/GigUndiDxe/wol/wolinfo.c         |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/AdapterInformation.c |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/AdapterInformation.h |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Brand.c              |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/ComponentName.c      |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/ComponentName.h      |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Decode.c             |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Decode.h             |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/DeviceSupport.c      |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/DeviceSupport.h      |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Dma.c                |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Dma.h                |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.c  |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/DriverDiagnostics.h  |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/DriverHealth.c       |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/DriverHealthCommon.h |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/EepromConfig.c       |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/EepromConfig.h       |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Hii.c                |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Hii.h                |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.c     |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/HiiInternalLib.h     |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/I40e.c               |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/I40e.h               |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/I40eUndiDxe.inf      |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/I40e_osdep.h         |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Init.c               |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Init.h               |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Inventory.vfr        |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/NVDataStruc.h        |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/StartStop.c          |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/StartStop.h          |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/Version.h            |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_adminq.c        |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_adminq.h        |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_adminq_cmd.h    |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_alloc.h         |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_common.c        |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_devids.h        |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_diag.c          |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_diag.h          |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_hmc.c           |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_hmc.h           |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.c       |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_lan_hmc.h       |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_nvm.c           |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_prototype.h     |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_register.h      |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_status.h        |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_type.h          |  1 -

> > > >  IntelUndiPkg/I40eUndiDxe/i40e_virtchnl.h      |  1 -

> > > >  IntelUndiPkg/IntelOpenSourceUndiPkg.dsc       | 13 ++++++-

> > > >  IntelUndiPkg/XGigUndiDxe/AdapterInformation.c |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/AdapterInformation.h |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Brand.c              |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ComponentName.c      |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ComponentName.h      |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Decode.c             |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Decode.h             |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/DeviceSupport.c      |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/DeviceSupport.h      |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Dma.c                |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Dma.h                |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.c  |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/DriverDiagnostics.h  |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/DriverHealth.c       |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/DriverHealthCommon.h |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/EepromConfig.c       |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/EepromConfig.h       |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Hii.c                |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Hii.h                |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.c     |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/HiiInternalLib.h     |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Init.c               |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Init.h               |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Inventory.vfr        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/NVDataStruc.h        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/StartStop.c          |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/StartStop.h          |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Version.h            |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/XGigUndiDxe.inf      |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Xgbe.c               |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/Xgbe.h               |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.c        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_82598.h        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.c        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_82599.h        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_api.c          |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_api.h          |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_common.c       |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_common.h       |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb.h          |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_dcb_82599.h    |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_osdep.h        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.c          |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_phy.h          |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_type.h         |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.c         |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_x540.h         |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.c         |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/ixgbe_x550.h         |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/wol/wol.c            |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/wol/wol.h            |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/wol/wol_10G.c        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/wol/wolfamily.c      |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.c        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/wol/wolimpl.h        |  1 -

> > > >  IntelUndiPkg/XGigUndiDxe/wol/wolinfo.c        |  1 -

> > > >  171 files changed, 120 insertions(+), 270 deletions(-)

> > > >  rename IntelUndiPkg/GigUndiDxe/{E1000.c => e1000.c} (99%)

> > > >  rename IntelUndiPkg/GigUndiDxe/{E1000.h => e1000.h} (99%)

> > > >  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.c =>

> > > > e1000_osdep.c}

> > > > (99%)

> > > >  rename IntelUndiPkg/GigUndiDxe/{E1000_osdep.h =>

> > > > e1000_osdep.h}

> > > > (99%)

> > > > 

> > > 

> > > _______________________________________________

> > > edk2-devel mailing list

> > > edk2-devel@lists.01.org

> > > https://lists.01.org/mailman/listinfo/edk2-devel


_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ard Biesheuvel March 28, 2019, 11:38 a.m. UTC | #10
On Thu, 28 Mar 2019 at 11:46, Ryszard Knop <ryszard.knop@linux.intel.com> wrote:
>

> On Wed, 2019-03-27 at 16:32 +0100, Ard Biesheuvel wrote:

> > On Tue, 29 Jan 2019 at 14:55, Ryszard Knop <

> > ryszard.knop@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