Reviewed-by: Giri P Mudusuru <giri.p.mudus...@intel.com> 

> -----Original Message-----
> From: Gao, Liming
> Sent: Friday, July 8, 2016 7:49 AM
> To: edk2-devel@lists.01.org
> Cc: Yao, Jiewen <jiewen....@intel.com>; Mudusuru, Giri P
> <giri.p.mudus...@intel.com>
> Subject: [Patch 5/5] IntelFsp2Pkg SecFspSecPlatformLibNull: Remove
> MASM/GAS files
> 
> Keep NASM file only for new added modules.
> 
> Cc: Jiewen Yao <jiewen....@intel.com>
> Cc: Giri Mudusuru <giri.p.mudus...@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Liming Gao <liming....@intel.com>
> ---
>  .../SecFspSecPlatformLibNull/Ia32/Flat32.asm       | 131 
> ---------------------
>  .../Library/SecFspSecPlatformLibNull/Ia32/Flat32.s | 110 -----------------
>  .../SecFspSecPlatformLibNull/Ia32/SecCarInit.asm   |  51 --------
>  .../SecFspSecPlatformLibNull/Ia32/SecCarInit.s     |  37 ------
>  .../SecFspSecPlatformLibNull.inf                   |   6 +-
>  5 files changed, 1 insertion(+), 334 deletions(-)
>  delete mode 100644
> IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.asm
>  delete mode 100644
> IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.s
>  delete mode 100644
> IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/SecCarInit.asm
>  delete mode 100644
> IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/SecCarInit.s
> 
> diff --git a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.asm
> b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.asm
> deleted file mode 100644
> index 9f144fc..0000000
> --- a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.asm
> +++ /dev/null
> @@ -1,131 +0,0 @@
> -;; @file
> -;  This is the code that goes from real-mode to protected mode.
> -;  It consumes the reset vector, configures the stack.
> -;
> -; Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.<BR>
> -; This program and the accompanying materials
> -; are licensed and made available under the terms and conditions of the BSD
> License
> -; which accompanies this distribution.  The full text of the license may be 
> found
> at
> -; http://opensource.org/licenses/bsd-license.php.
> -;
> -; THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> -; WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> -;;
> -
> -;
> -; Define assembler characteristics
> -;
> -.586p
> -.xmm
> -.model flat, c
> -
> -EXTRN   TempRamInitApi:NEAR
> -
> -.code
> -
> -RET_ESI  MACRO
> -
> -  movd    esi, mm7                      ; restore ESP from MM7
> -  jmp     esi
> -
> -ENDM
> -
> -;
> -; Perform early platform initialization
> -;
> -SecPlatformInit    PROC    NEAR    PUBLIC
> -
> -  RET_ESI
> -
> -SecPlatformInit    ENDP
> -
> -;
> -; Protected mode portion initializes stack, configures cache, and calls C 
> entry
> point
> -;
> -
> -;----------------------------------------------------------------------------
> -;
> -; Procedure:    ProtectedModeEntryPoint
> -;
> -; Input:        Executing in 32 Bit Protected (flat) mode
> -;               cs: 0-4GB
> -;               ds: 0-4GB
> -;               es: 0-4GB
> -;               fs: 0-4GB
> -;               gs: 0-4GB
> -;               ss: 0-4GB
> -;
> -; Output:       This function never returns
> -;
> -; Destroys:
> -;               ecx
> -;               edi
> -;               esi
> -;               esp
> -;
> -; Description:
> -;               Perform any essential early platform initilaisation
> -;               Setup a stack
> -;
> -;----------------------------------------------------------------------------
> -
> -ProtectedModeEntryPoint PROC NEAR C PUBLIC
> -  ;
> -  ; Dummy function. Consume 2 API to make sure they can be linked.
> -  ;
> -  mov  eax, TempRamInitApi
> -
> -  ; Should never return
> -  jmp  $
> -
> -ProtectedModeEntryPoint ENDP
> -
> -;
> -; ROM-based Global-Descriptor Table for the PEI Phase
> -;
> -align 16
> -PUBLIC  BootGdtTable
> -
> -;
> -; GDT[0]: 0x00: Null entry, never used.
> -;
> -NULL_SEL        equ     $ - GDT_BASE        ; Selector [0]
> -GDT_BASE:
> -BootGdtTable    DD      0
> -                DD      0
> -;
> -; Linear code segment descriptor
> -;
> -LINEAR_CODE_SEL equ     $ - GDT_BASE        ; Selector [0x8]
> -        DW      0FFFFh                      ; limit 0xFFFF
> -        DW      0                           ; base 0
> -        DB      0
> -        DB      09Bh                        ; present, ring 0, data, 
> expand-up, not-writable
> -        DB      0CFh                        ; page-granular, 32-bit
> -        DB      0
> -;
> -; System data segment descriptor
> -;
> -SYS_DATA_SEL    equ     $ - GDT_BASE        ; Selector [0x10]
> -        DW      0FFFFh                      ; limit 0xFFFF
> -        DW      0                           ; base 0
> -        DB      0
> -        DB      093h                        ; present, ring 0, data, 
> expand-up, not-writable
> -        DB      0CFh                        ; page-granular, 32-bit
> -        DB      0
> -
> -GDT_SIZE        EQU     $ - BootGDTtable    ; Size, in bytes
> -
> -;
> -; GDT Descriptor
> -;
> -GdtDesc:                                    ; GDT descriptor
> -        DW      GDT_SIZE - 1                ; GDT limit
> -        DD      OFFSET BootGdtTable         ; GDT base address
> -
> -ProtectedModeEntryLinearAddress   LABEL   FWORD
> -ProtectedModeEntryLinearOffset    LABEL   DWORD
> -  DD      OFFSET ProtectedModeEntryPoint  ; Offset of our 32 bit code
> -  DW      LINEAR_CODE_SEL
> -
> -END
> diff --git a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.s
> b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.s
> deleted file mode 100644
> index d46d792..0000000
> --- a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/Flat32.s
> +++ /dev/null
> @@ -1,110 +0,0 @@
> -#------------------------------------------------------------------------------
> -#
> -# Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved.<BR>
> -# This program and the accompanying materials
> -# are licensed and made available under the terms and conditions of the BSD
> License
> -# which accompanies this distribution.  The full text of the license may be 
> found
> at
> -# http://opensource.org/licenses/bsd-license.php.
> -#
> -# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> -# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> -#
> -# Abstract:
> -#
> -#  This is the code that goes from real-mode to protected mode.
> -#  It consumes the reset vector, configures the stack.
> -#
> -#------------------------------------------------------------------------------
> -
> -
> -#
> -# Contrary to the name, this file contains 16 bit code as well.
> -#
> -.text
> -
> -ASM_GLOBAL ASM_PFX(SecPlatformInit)
> -ASM_PFX(SecPlatformInit):
> -  movd    %mm7, %esi                      # restore ESP from MM7
> -  jmp     *%esi
> -
> -#----------------------------------------------------------------------------
> -#
> -# Procedure:    ProtectedModeEntryPoint
> -#
> -# Input:        Executing in 32 Bit Protected (flat) mode
> -#               cs: 0-4GB
> -#               ds: 0-4GB
> -#               es: 0-4GB
> -#               fs: 0-4GB
> -#               gs: 0-4GB
> -#               ss: 0-4GB
> -#
> -# Output:       This function never returns
> -#
> -# Destroys:
> -#               ecx
> -#               edi
> -#               esi
> -#               esp
> -#
> -# Description:
> -#               Perform any essential early platform initilaisation
> -#               Setup a stack
> -#
> -#----------------------------------------------------------------------------
> -ProtectedModeEntryPoint:
> -  #
> -  # Dummy function. Consume 2 API to make sure they can be linked.
> -  #
> -  movl   ASM_PFX(TempRamInitApi), %eax
> -  #
> -  # Should never return
> -  #
> -  jmp     . #'$'
> -
> -#
> -# ROM-based Global-Descriptor Table for the PEI Phase
> -#
> -.align 16
> -#
> -# GDT[0]: 000h: Null entry, never used.
> -#
> -.equ   NULL_SEL, . - GDT_BASE         # Selector [0]
> -GDT_BASE:
> -BootGdtTable:
> -        .long   0
> -        .long   0
> -#
> -# Linear code segment descriptor
> -#
> -.equ     LINEAR_CODE_SEL, . - GDT_BASE         # Selector [08h]
> -        .word   0xFFFF                      # limit 0FFFFh
> -        .word   0                           # base 0
> -        .byte   0
> -        .byte   0x9B                        # present, ring 0, data, 
> expand-up, not-writable
> -        .byte   0xCF                        # page-granular, 32-bit
> -        .byte   0
> -#
> -# System data segment descriptor
> -#
> -.equ    SYS_DATA_SEL, . - GDT_BASE         # Selector [010h]
> -        .word   0xFFFF                      # limit 0FFFFh
> -        .word   0                           # base 0
> -        .byte   0
> -        .byte   0x93                        # present, ring 0, data, 
> expand-up, not-writable
> -        .byte   0xCF                        # page-granular, 32-bit
> -        .byte   0
> -
> -.equ            GDT_SIZE, . - BootGdtTable  # Size, in bytes
> -
> -#
> -# GDT Descriptor
> -#
> -GdtDesc:                                     # GDT descriptor
> -       .word    GDT_SIZE - 1
> -       .long    BootGdtTable
> -
> -ProtectedModeEntryLinearAddress:
> -ProtectedModeEntryLinearOffset:
> -       .long    ProtectedModeEntryPoint
> -       .word    LINEAR_CODE_SEL
> diff --git a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/SecCarInit.asm
> b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/SecCarInit.asm
> deleted file mode 100644
> index 31296e0..0000000
> --- a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/SecCarInit.asm
> +++ /dev/null
> @@ -1,51 +0,0 @@
> -;; @file
> -;  SEC CAR function
> -;
> -; Copyright (c) 2015, Intel Corporation. All rights reserved.<BR>
> -; This program and the accompanying materials
> -; are licensed and made available under the terms and conditions of the BSD
> License
> -; which accompanies this distribution.  The full text of the license may be 
> found
> at
> -; http://opensource.org/licenses/bsd-license.php.
> -;
> -; THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> -; WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> -;;
> -
> -;
> -; Define assembler characteristics
> -;
> -.586p
> -.xmm
> -.model flat, c
> -
> -RET_ESI  MACRO
> -
> -  movd    esi, mm7                      ; move ReturnAddress from MM7 to ESI
> -  jmp     esi
> -
> -ENDM
> -
> -.code
> -
> -;-----------------------------------------------------------------------------
> -;
> -;  Section:     SecCarInit
> -;
> -;  Description: This function initializes the Cache for Data, Stack, and Code
> -;
> -;-----------------------------------------------------------------------------
> -SecCarInit    PROC    NEAR    PUBLIC
> -
> -  ;
> -  ; Set up CAR
> -  ;
> -
> -  xor    eax, eax
> -
> -SecCarInitExit:
> -
> -  RET_ESI
> -
> -SecCarInit    ENDP
> -
> -END
> diff --git a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/SecCarInit.s
> b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/SecCarInit.s
> deleted file mode 100644
> index 7bd40df..0000000
> --- a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/Ia32/SecCarInit.s
> +++ /dev/null
> @@ -1,37 +0,0 @@
> -#------------------------------------------------------------------------------
> -#
> -# Copyright (c) 2015, Intel Corporation. All rights reserved.<BR>
> -# This program and the accompanying materials
> -# are licensed and made available under the terms and conditions of the BSD
> License
> -# which accompanies this distribution.  The full text of the license may be 
> found
> at
> -# http://opensource.org/licenses/bsd-license.php.
> -#
> -# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> -# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> -#
> -# Abstract:
> -#
> -#  SEC CAR function
> -#
> -#------------------------------------------------------------------------------
> -
> -#-----------------------------------------------------------------------------
> -#
> -#  Section:     SecCarInit
> -#
> -#  Description: This function initializes the Cache for Data, Stack, and Code
> -#
> -#-----------------------------------------------------------------------------
> -ASM_GLOBAL ASM_PFX(SecCarInit)
> -ASM_PFX(SecCarInit):
> -
> -  #
> -  # Set up CAR
> -  #
> -
> -  xor     %eax, %eax
> -
> -SecCarInitExit:
> -
> -  movd       %mm7, %esi                      #RET_ESI
> -  jmp        *%esi
> diff --git
> a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/SecFspSecPlatformLibNull.inf
> b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/SecFspSecPlatformLibNull.inf
> index 2bc638c..e1cab05 100644
> ---
> a/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/SecFspSecPlatformLibNull.inf
> +++
> b/IntelFsp2Pkg/Library/SecFspSecPlatformLibNull/SecFspSecPlatformLibNull.inf
> @@ -1,9 +1,9 @@
>  ## @file
>  #  NULL instance of Platform Sec Lib.
>  #
> -#  Copyright (c) 2014 - 2015, Intel Corporation. All rights reserved.<BR>
> +#  Copyright (c) 2014 - 2016, Intel Corporation. All rights reserved.<BR>
>  #
>  #  This program and the accompanying materials
>  #  are licensed and made available under the terms and conditions of the BSD
> License
>  #  which accompanies this distribution. The full text of the license may be 
> found
> at
>  #  http://opensource.org/licenses/bsd-license.php.
> @@ -39,15 +39,11 @@
> 
>  [Sources]
>    PlatformSecLibNull.c
> 
>  [Sources.IA32]
> -  Ia32/Flat32.asm
>    Ia32/Flat32.nasm
> -  Ia32/Flat32.s
> -  Ia32/SecCarInit.asm
> -  Ia32/SecCarInit.s
>    Ia32/SecCarInit.nasm
> 
> 
> #################################################################
> ###############
>  #
>  # Package Dependency Section - list of Package files that are required for
> --
> 1.9.5.msysgit.0

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

Reply via email to