From: "Wang, Jian J" <jian.j.wang@intel.com>
To: "Bi, Dandan" <dandan.bi@intel.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Zeng, Star" <star.zeng@intel.com>, "Dong, Eric" <eric.dong@intel.com>
Subject: Re: [PATCH 1/4] MdePkg/BaseLib.h: Coding style clean-up
Date: Mon, 25 Dec 2017 02:23:33 +0000 [thread overview]
Message-ID: <D827630B58408649ACB04F44C510003624CC2FA6@SHSMSX103.ccr.corp.intel.com> (raw)
In-Reply-To: <3C0D5C461C9E904E8F62152F6274C0BB3B9F0502@SHSMSX103.ccr.corp.intel.com>
Ok. Thanks for catching it.
Regards,
Jian
> -----Original Message-----
> From: Bi, Dandan
> Sent: Monday, December 25, 2017 9:51 AM
> To: Wang, Jian J <jian.j.wang@intel.com>; edk2-devel@lists.01.org
> Cc: Zeng, Star <star.zeng@intel.com>; Dong, Eric <eric.dong@intel.com>
> Subject: RE: [PATCH 1/4] MdePkg/BaseLib.h: Coding style clean-up
>
> Hi Jian,
>
> This patch is fine.
> Could you help to add a period(.) for following function description when you
> commit the patch? Thanks!
> /**
> Load given selector into TR register // Add a period(.) here, or ECC tool will
> report an error here.
>
> @param[in] Selector Task segment selector
> **/
> VOID
> EFIAPI
> AsmWriteTr (
> IN UINT16 Selector
> );
>
> With this update, Reviewed-by: Dandan Bi <dandan.bi@intel.com>
>
> Regards,
> Dandan
> -----Original Message-----
> From: Wang, Jian J
> Sent: Monday, December 25, 2017 9:07 AM
> To: edk2-devel@lists.01.org
> Cc: Bi, Dandan <dandan.bi@intel.com>; Zeng, Star <star.zeng@intel.com>;
> Dong, Eric <eric.dong@intel.com>
> Subject: [PATCH 1/4] MdePkg/BaseLib.h: Coding style clean-up
>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Jian J Wang <jian.j.wang@intel.com>
> ---
> MdePkg/Include/Library/BaseLib.h | 72 ++++++++++++++++++++------------------
> --
> 1 file changed, 36 insertions(+), 36 deletions(-)
>
> diff --git a/MdePkg/Include/Library/BaseLib.h
> b/MdePkg/Include/Library/BaseLib.h
> index 2b98af4cd1..f51079aeb1 100644
> --- a/MdePkg/Include/Library/BaseLib.h
> +++ b/MdePkg/Include/Library/BaseLib.h
> @@ -6672,41 +6672,41 @@ typedef union {
> typedef struct {
> UINT16 PreviousTaskLink;
> UINT16 Reserved_2;
> - UINT32 ESP0;
> - UINT16 SS0;
> + UINT32 Esp0;
> + UINT16 Ss0;
> UINT16 Reserved_10;
> - UINT32 ESP1;
> - UINT16 SS1;
> + UINT32 Esp1;
> + UINT16 Ss1;
> UINT16 Reserved_18;
> - UINT32 ESP2;
> - UINT16 SS2;
> + UINT32 Esp2;
> + UINT16 Ss2;
> UINT16 Reserved_26;
> - UINT32 CR3;
> - UINT32 EIP;
> - UINT32 EFLAGS;
> - UINT32 EAX;
> - UINT32 ECX;
> - UINT32 EDX;
> - UINT32 EBX;
> - UINT32 ESP;
> - UINT32 EBP;
> - UINT32 ESI;
> - UINT32 EDI;
> - UINT16 ES;
> + UINT32 Cr3;
> + UINT32 Eip;
> + UINT32 Eflags;
> + UINT32 Eax;
> + UINT32 Ecx;
> + UINT32 Edx;
> + UINT32 Ebx;
> + UINT32 Esp;
> + UINT32 Ebp;
> + UINT32 Esi;
> + UINT32 Edi;
> + UINT16 Es;
> UINT16 Reserved_74;
> - UINT16 CS;
> + UINT16 Cs;
> UINT16 Reserved_78;
> - UINT16 SS;
> + UINT16 Ss;
> UINT16 Reserved_82;
> - UINT16 DS;
> + UINT16 Ds;
> UINT16 Reserved_86;
> - UINT16 FS;
> + UINT16 Fs;
> UINT16 Reserved_90;
> - UINT16 GS;
> + UINT16 Gs;
> UINT16 Reserved_94;
> UINT16 LDTSegmentSelector;
> UINT16 Reserved_98;
> - UINT16 T;
> + UINT16 Tflag;
> UINT16 IOMapBaseAddress;
> } IA32_TASK_STATE_SEGMENT;
>
> @@ -6717,12 +6717,12 @@ typedef union {
> UINT32 BaseMid:8; ///< Base Address 23..16
> UINT32 Type:4; ///< Type (1 0 B 1)
> UINT32 Reserved_43:1; ///< 0
> - UINT32 DPL:2; ///< Descriptor Privilege Level
> - UINT32 P:1; ///< Segment Present
> + UINT32 Dpl:2; ///< Descriptor Privilege Level
> + UINT32 Present:1; ///< Segment Present
> UINT32 LimitHigh:4; ///< Segment Limit 19..16
> - UINT32 AVL:1; ///< Available for use by system software
> + UINT32 Avl:1; ///< Available for use by system software
> UINT32 Reserved_52:2; ///< 0 0
> - UINT32 G:1; ///< Granularity
> + UINT32 Granularity:1; ///< Granularity
> UINT32 BaseHigh:8; ///< Base Address 31..24
> } Bits;
> UINT64 Uint64;
> @@ -6757,11 +6757,11 @@ typedef union {
> //
> typedef struct {
> UINT32 Reserved_0;
> - UINT64 RSP0;
> - UINT64 RSP1;
> - UINT64 RSP2;
> + UINT64 Rsp0;
> + UINT64 Rsp1;
> + UINT64 Rsp2;
> UINT64 Reserved_28;
> - UINT64 IST[7];
> + UINT64 Ist[7];
> UINT64 Reserved_92;
> UINT16 Reserved_100;
> UINT16 IOMapBaseAddress;
> @@ -6774,12 +6774,12 @@ typedef union {
> UINT32 BaseMidl:8; ///< Base Address 23..16
> UINT32 Type:4; ///< Type (1 0 B 1)
> UINT32 Reserved_43:1; ///< 0
> - UINT32 DPL:2; ///< Descriptor Privilege Level
> - UINT32 P:1; ///< Segment Present
> + UINT32 Dpl:2; ///< Descriptor Privilege Level
> + UINT32 Present:1; ///< Segment Present
> UINT32 LimitHigh:4; ///< Segment Limit 19..16
> - UINT32 AVL:1; ///< Available for use by system software
> + UINT32 Avl:1; ///< Available for use by system software
> UINT32 Reserved_52:2; ///< 0 0
> - UINT32 G:1; ///< Granularity
> + UINT32 Granularity:1; ///< Granularity
> UINT32 BaseMidh:8; ///< Base Address 31..24
> UINT32 BaseHigh:32; ///< Base Address 63..32
> UINT32 Reserved_96:32; ///< Reserved
> --
> 2.15.1.windows.2
next prev parent reply other threads:[~2017-12-25 2:18 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-25 1:06 [PATCH 0/4] Coding style clean-up Jian J Wang
2017-12-25 1:06 ` [PATCH 1/4] MdePkg/BaseLib.h: " Jian J Wang
2017-12-25 1:51 ` Bi, Dandan
2017-12-25 2:23 ` Wang, Jian J [this message]
2017-12-25 1:06 ` [PATCH 2/4] MdeModulePkg/Core: " Jian J Wang
2017-12-25 1:19 ` Zeng, Star
2017-12-25 2:01 ` Bi, Dandan
2017-12-25 1:06 ` [PATCH 3/4] UefiCpuPkg/UefiCpuPkg.uni: Add missing string definition for new PCDs Jian J Wang
2017-12-25 1:58 ` Bi, Dandan
2017-12-25 2:28 ` Wang, Jian J
2017-12-25 1:06 ` [PATCH 4/4] UefiCpuPkg: Update code to use new structure field names Jian J Wang
2017-12-25 2:01 ` Bi, Dandan
2017-12-26 16:47 ` [PATCH 0/4] Coding style clean-up Kinney, Michael D
2017-12-27 0:40 ` Wang, Jian J
2017-12-27 9:29 ` Gao, Liming
2017-12-27 16:35 ` Kinney, Michael D
2017-12-28 0:27 ` Wang, Jian J
2017-12-28 0:37 ` Kinney, Michael D
2017-12-28 0:53 ` Wang, Jian J
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=D827630B58408649ACB04F44C510003624CC2FA6@SHSMSX103.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox