Re: [edk2-devel] [PATCH v2 22/34] MdePkg/Include: LoongArch definitions.

2022-09-26 Thread Chao Li
quot;devel@edk2.groups.io\ (mailto:devel@edk2.groups.io/)"" > mailto:devel@edk2.groups.io)>; Gao, Liming > mailto:gaolim...@byosoft.com.cn)>; Liu, Zhiguang > mailto:zhiguang@intel.com)> > 主题: Re: [edk2-devel] [PATCH v2 22/34] MdePkg/Inclu

Re: [edk2-devel] [PATCH v2 22/34] MdePkg/Include: LoongArch definitions.

2022-09-23 Thread Chao Li
o, Liming > ; Liu, Zhiguang > Subject: Re: [edk2-devel] [PATCH v2 22/34] MdePkg/Include: LoongArch > definitions. > > > > > > Hi Mkie, > I responded to your comment below. > > > > > Thanks, > Chao > > > > On 9月

Re: [edk2-devel] [PATCH v2 22/34] MdePkg/Include: LoongArch definitions.

2022-09-23 Thread Michael D Kinney
If it is in spec, then the comment about “coding convenience” is not required. Mike From: chao li Sent: Friday, September 23, 2022 7:17 PM To: Kinney, Michael D Cc: "devel@edk2.groups.io" ; Gao, Liming ; Liu, Zhiguang Subject: Re: [edk2-devel] [PATCH v2 22/34] MdePkg/Include:

Re: [edk2-devel] [PATCH v2 22/34] MdePkg/Include: LoongArch definitions.

2022-09-23 Thread Chao Li
Hi Mkie, I responded to your comment below. Thanks, Chao On 9月 23 2022, at 11:46 晚上, "Kinney, Michael D" wrote: > One comment below. > > Mike > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of Chao Li > > Sent: Wednesday, September 14, 2022 2:41 AM > > To: deve

Re: [edk2-devel] [PATCH v2 22/34] MdePkg/Include: LoongArch definitions.

2022-09-23 Thread Michael D Kinney
One comment below. Mike > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chao Li > Sent: Wednesday, September 14, 2022 2:41 AM > To: devel@edk2.groups.io > Cc: Kinney, Michael D ; Gao, Liming > ; Liu, Zhiguang > Subject: [edk2-devel] [PATCH v2 22/34] MdePkg/Include: Loong

Re: [edk2-devel] [PATCH v2 22/34] MdePkg/Include: LoongArch definitions.

2022-09-22 Thread Chao Li
Hi Mike, In the V2, I added IANA, Microsoft and UEFI specification links in every patches that uses them. Associated patches: 0008-MdePkg-Include-Add-LOONGARCH-related-definitions-.patch 0015-BaseTools-BaseTools-changes-for-LoongArch-platfor.patch 0022-MdePkg-Include-LoongArch-definitions.patch(T