Hi Pierre, On Thu, Jan 21, 2021 at 01:51 AM, PierreGondois wrote:
> > -#ifndef __ARM_CORTEX_A5x_H__ > -#define __ARM_CORTEX_A5x_H__ > +#ifndef ARM_CORTEX_A5X_H__ > +#define ARM_CORTEX_A5X_H__ There should be a single trailing underscore for #include guards. See https://edk2-docs.gitbook.io/edk-ii-c-coding-standards-specification/5_source_files/53_include_files#5-3-5-all-include-file-contents-must-be-protected-by-a-include-guard With that changed: Reviewed-by: Sami Mujawar <sami.muja...@arm.com> Regards, Sami Mujawar -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#71357): https://edk2.groups.io/g/devel/message/71357 Mute This Topic: https://groups.io/mt/80000379/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-