[AMD Official Use Only - General] Thanks! I will send out the V3 for the update. Will work on the spec publication with Mike.
Abner From: Ni, Ray <ray...@intel.com> Sent: Wednesday, October 19, 2022 11:34 AM To: Chang, Abner <abner.ch...@amd.com>; devel@edk2.groups.io Subject: RE: [edk2-devel] [tianocore-docs][PATCH V2 2/2] edk II C Coding Standard: Updates 4.2 and 4.3 sections Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Oh I see! Reviewed-by: Ray Ni <ray...@intrel.com<mailto:ray...@intrel.com>> From: Chang, Abner <abner.ch...@amd.com<mailto:abner.ch...@amd.com>> Sent: Tuesday, October 18, 2022 5:50 PM To: Ni, Ray <ray...@intel.com<mailto:ray...@intel.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io> Subject: RE: [edk2-devel] [tianocore-docs][PATCH V2 2/2] edk II C Coding Standard: Updates 4.2 and 4.3 sections [AMD Official Use Only - General] Hi Ray, <CpuArch>][<Vendor>]<FileName>.* The simple <FileName> naming rule was already defined in the 4.3.1 to 4.3.4 sections. We are not intending for defining the file naming style. This section is mainly for the format of attaching <CpuArch> and <Vendor> to the filename. For example, how do we add AMD to SmmCpuFeaturesLib? Is "Amd"SmmCpuFeaturesLib or SmmCpuFeaturesLib"Amd"? This is what we would like to define for the processor archs and vendors. My opinion is we can just leave the file format naming as it was defined in this spec. Or how about if we say "Refer to 4.3.1 to 4.3.4 sections for the file naming format" for <FileNmae> in below? [<CpuArch>][<Vendor>]<FileName>.* <CpuArch> OPTIONAL The <CpuArch> is represented with a BNF, <arch> ::='Ia32' | 'X64' | 'Arm' | 'AArch64' | 'RiscV64' | 'LoongArch64' | 'Ebc' <CpuArch> ::= <arch>[<arch>]* Example: Ia32X64Arm or RiscV64LoongArch64 <Vendor> OPTIONAL * <FileName> REQUIRED Refer to 4.3.1 to 4.3.4 sections for the file Naming format. Thanks Abner From: Ni, Ray <ray...@intel.com<mailto:ray...@intel.com>> Sent: Tuesday, October 18, 2022 2:02 PM To: Chang; Chang, Abner <abner.ch...@amd.com<mailto:abner.ch...@amd.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io> Subject: Re: [edk2-devel] [tianocore-docs][PATCH V2 2/2] edk II C Coding Standard: Updates 4.2 and 4.3 sections Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. All look good to me. Thanks for addressing my comments regarding simplifying the rules. Except for one minor comment: I still don't think we need to define rules for source file names (4.3.5.4 EDK II source files within a Library/Module instance). And the rule "[][].*" doesn't specify what could be. That leads to allowing any style of the file name. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#95359): https://edk2.groups.io/g/devel/message/95359 Mute This Topic: https://groups.io/mt/94344815/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
