Hi Liming,
        I am about to resubmit the patch with the changes in the files 
SpcrGenerator.c and Spcr.aslc which are consuming the SPCR structure from 
SerialPortConsoleRedirectionTable.h file and these files seemingly initializes 
the SPCR structure members. So please let me know about your opinion on this 
and I think you mean the incompatible change as these files are affected with 
the initial patch.

Thanks,
Praveen Sankar N

-----Original Message-----
From: gaoliming <gaolim...@byosoft.com.cn>
Sent: Tuesday, April 23, 2024 6:47 PM
To: Praveen Sankar N <praveensank...@ami.com>; devel@edk2.groups.io
Cc: michael.d.kin...@intel.com; zhiguang....@intel.com; Felix Polyudov 
<fel...@ami.com>; Srinivasan Mani <srinivas...@ami.com>; Sundaresan S 
<sundares...@ami.com>
Subject: [EXTERNAL] 回复: [PATCH v2 1/1] Subject: [PATCH] MdePkg:Added new SPCR 
table stucture members as in Rev4.


**CAUTION: The e-mail below is from an external source. Please exercise caution 
before opening attachments, clicking links, or following guidance.**

Seemly, this is an incompatible change. What's impact with this change?

Thanks
Liming
> -----邮件原件-----
> 发件人: Praveen Sankar N <praveensank...@ami.com>
> 发送时间: 2024年4月22日 19:26
> 收件人: devel@edk2.groups.io
> 抄送: michael.d.kin...@intel.com; gaolim...@byosoft.com.cn;
> zhiguang....@intel.com; Felix Polyudov <fel...@ami.com>; Srinivasan
> Mani <srinivas...@ami.com>; Sundaresan S <sundares...@ami.com>
> 主题: [PATCH v2 1/1] Subject: [PATCH] MdePkg:Added new SPCR table
> stucture members as in Rev4.
>
> In SPCR table, 4 structure members have been added newly as per SPCR
> table Rev4, which has to be added in
> MdePkg/SerialPortConsoleRedirectionTable.h file.
>
> Signed-off-by: Praveen Sankar N praveensank...@ami.com
> Cc: michael.d.kin...@intel.com
> Cc: gaolim...@byosoft.com.cn
> Cc: zhiguang....@intel.com
> Cc: fel...@ami.com
> Cc: srinivas...@ami.com
> Cc: sundares...@ami.com
> ---
>  .../IndustryStandard/SerialPortConsoleRedirectionTable.h    | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git
> a/MdePkg/Include/IndustryStandard/SerialPortConsoleRedirectionTable.h
> b/MdePkg/Include/IndustryStandard/SerialPortConsoleRedirectionTable.h
> index eb5ae28390..f0001ab137 100644
> ---
> a/MdePkg/Include/IndustryStandard/SerialPortConsoleRedirectionTable.h
> +++
> b/MdePkg/Include/IndustryStandard/SerialPortConsoleRedirectionTable.h
> @@ -47,7 +47,11 @@ typedef struct {
>    UINT8                                     PciFunctionNumber;
>
>    UINT32                                    PciFlags;
>
>    UINT8                                     PciSegment;
>
> -  UINT32                                    Reserved3;
>
> +  UINT32                                    UartClockFrequency;
>
> +  UINT32                                    PreciseBaudRate;
>
> +  UINT16
> NameSpaceStrLength;
>
> +  UINT16
> NameSpaceStrOffset;
>
> +  CHAR8
> NameSpaceString[0];
>
>  } EFI_ACPI_SERIAL_PORT_CONSOLE_REDIRECTION_TABLE;
>
>
>
>  #pragma pack()
>
> --
> 2.38.1.windows.1
> -The information contained in this message may be confidential and
> proprietary to American Megatrends (AMI). This communication is
> intended
to
> be read only by the individual or entity to whom it is addressed or by
their
> designee. If the reader of this message is not the intended recipient,
> you
are
> on notice that any distribution of this message, in any form, is
> strictly prohibited. Please promptly notify the sender by reply e-mail
> or by
telephone
> at 770-246-8600, and then delete or destroy all copies of the
transmission=


-The information contained in this message may be confidential and proprietary 
to American Megatrends (AMI). This communication is intended to be read only by 
the individual or entity to whom it is addressed or by their designee. If the 
reader of this message is not the intended recipient, you are on notice that 
any distribution of this message, in any form, is strictly prohibited. Please 
promptly notify the sender by reply e-mail or by telephone at 770-246-8600, and 
then delete or destroy all copies of the transmission.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#118351): https://edk2.groups.io/g/devel/message/118351
Mute This Topic: https://groups.io/mt/105689097/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to