回复: [edk2-devel] [Patch V4 1/1] Tools\FitGen: Add extra parameter to input the Higher Flash Address

[gaoliming]

Chinni:
  Can you change subject and commit message with the top flash address
instead of higher flash address? 

Thanks
Liming
> -----邮件原件-----
> 发件人: bounce+27952+65758+4905953+8761045 at groups.io
> <bounce+27952+65758+4905953+8761045 at groups.io> 代表 cbduggap
> 发送时间: 2020年9月30日 13:49
> 收件人: devel at edk2.groups.io
> 抄送: cbduggap <chinni.b.duggapu at intel.com>; Bob Feng
> <bob.c.feng at intel.com>; Liming Gao <gaoliming at byosoft.com.cn>
> 主题: [edk2-devel] [Patch V4 1/1] Tools\FitGen: Add extra parameter to
input
> the Higher Flash Address
> 
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2981
> 
> Add extra parameter to the Fit Gen Tool to input the Higher Address.
> Default Address should be 4GB and if some one inputs new address,
> tool must consume that address instead of Default address (4GB).
> 
> Signed-off-by: cbduggap <chinni.b.duggapu at intel.com>
> Cc: Bob Feng <bob.c.feng at intel.com>
> Cc: Liming Gao <gaoliming at byosoft.com.cn>
> 
> Signed-off-by: cbduggap <chinni.b.duggapu at intel.com>
> ---
>  Silicon/Intel/Tools/FitGen/FitGen.c | 23 +++++++++++++++++++++--
>  1 file changed, 21 insertions(+), 2 deletions(-)
> 
> diff --git a/Silicon/Intel/Tools/FitGen/FitGen.c
> b/Silicon/Intel/Tools/FitGen/FitGen.c
> index c4006e69c8..cb60a3c324 100644
> --- a/Silicon/Intel/Tools/FitGen/FitGen.c
> +++ b/Silicon/Intel/Tools/FitGen/FitGen.c
> @@ -209,10 +209,12 @@ typedef struct {
>  #define DEFAULT_FIT_TABLE_POINTER_OFFSET  0x40
> 
>  #define DEFAULT_FIT_ENTRY_VERSION         0x0100
> 
> 
> 
> +#define TOP_FLASH_ADDRESS
> (gFitTableContext.TopFlashAddressRemapValue)
> 
> +
> 
>  #define MEMORY_TO_FLASH(FileBuffer, FvBuffer, FvSize)  \
> 
> -                 (UINTN)(0x100000000 - ((UINTN)(FvBuffer) +
> (UINTN)(FvSize) - (UINTN)(FileBuffer)))
> 
> +                 (UINTN)(TOP_FLASH_ADDRESS - ((UINTN)(FvBuffer) +
> (UINTN)(FvSize) - (UINTN)(FileBuffer)))
> 
>  #define FLASH_TO_MEMORY(Address, FvBuffer, FvSize)  \
> 
> -                 (VOID *)(UINTN)((UINTN)(FvBuffer) + (UINTN)(FvSize) -
> (0x100000000 - (UINTN)(Address)))
> 
> +                 (VOID *)(UINTN)((UINTN)(FvBuffer) + (UINTN)(FvSize) -
> (TOP_FLASH_ADDRESS - (UINTN)(Address)))
> 
> 
> 
>  #define FIT_TABLE_TYPE_HEADER                 0
> 
>  #define FIT_TABLE_TYPE_MICROCODE              1
> 
> @@ -268,6 +270,7 @@ typedef struct {
>    UINT32                     MicrocodeVersion;
> 
>    FIT_TABLE_CONTEXT_ENTRY
> OptionalModule[MAX_OPTIONAL_ENTRY];
> 
>    FIT_TABLE_CONTEXT_ENTRY    PortModule[MAX_PORT_ENTRY];
> 
> +  UINT64                     TopFlashAddressRemapValue;
> 
>  } FIT_TABLE_CONTEXT;
> 
> 
> 
>  FIT_TABLE_CONTEXT   gFitTableContext = {0};
> 
> @@ -330,6 +333,7 @@ Returns:
>            "\t[-F <FitTablePointerOffset>] [-F <FitTablePointerOffset>]
[-V
> <FitHeaderVersion>]\n"
> 
>            "\t[-NA]\n"
> 
>            "\t[-A <MicrocodeAlignment>]\n"
> 
> +           "\t[-REMAP <TopFlashAddress>\n"
> 
>            "\t[-CLEAR]\n"
> 
>            "\t[-L <MicrocodeSlotSize> <MicrocodeFfsGuid>]\n"
> 
>            "\t[-I <BiosInfoGuid>]\n"
> 
> @@ -986,6 +990,21 @@ Returns:
>      Index += 2;
> 
>    }
> 
> 
> 
> +  if ((Index >= argc) ||
> 
> +      ((strcmp (argv[Index], "-REMAP") == 0) ||
> 
> +       (strcmp (argv[Index], "-remap") == 0)) ) {
> 
> +    //
> 
> +    // by pass
> 
> +    //
> 
> +    gFitTableContext.TopFlashAddressRemapValue = xtoi (argv[Index + 1]);
> 
> +    Index += 2;
> 
> +  } else {
> 
> +    //
> 
> +    // no remapping
> 
> +    //
> 
> +    gFitTableContext.TopFlashAddressRemapValue = 0x100000000;
> 
> +  }
> 
> +  printf ("Top Flash Address Value : 0x%llx\n",
> gFitTableContext.TopFlashAddressRemapValue);
> 
>    //
> 
>    // 0.4 Clear FIT table related memory
> 
>    //
> 
> --
> 2.26.2.windows.1
> 
> 
> 
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#65758): https://edk2.groups.io/g/devel/message/65758
> Mute This Topic: https://groups.io/mt/77212511/4905953
> Group Owner: devel+owner at edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub
> [gaoliming at byosoft.com.cn]
> -=-=-=-=-=-=
> 





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#65759): https://edk2.groups.io/g/devel/message/65759
Mute This Topic: https://groups.io/mt/77212521/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-