[edk2-devel] [PATCH v1 0/7] DynamicTablesPkg: Fix assert ordering and ECC reported issues

[Sami Mujawar]

This patch series fixes an assert ordering issue and the following
issues reported by the ECC tool.
  - [9002]  The function headers should follow Doxygen special
            documentation blocks in section 2.3.5 in Comment,
              - <@param [in]> does NOT have [in, out]
              - does NOT have tail **/
  - [1008]  File has invalid Non-ACSII char.
  - [5007]  There should be no initialization of a variable as part of
              its declaration Variable Name.
  - Changes the AML_DEBUG_STR() macro to AML_OPCODE_DEF().
 
The changes can be seen at:
https://github.com/samimujawar/edk2/tree/1423_fix_assert_ordering_v1

Sami Mujawar (7):
  DynamicTablesPkg: Fix order of assert checks
  DynamicTablesPkg: SsdtSerialPortLibArm fix ECC error
  DynamicTablesPkg: AcpiSratLibArm fix ECC error
  DynamicTablesPkg: AmlLib fix ECC errors
  DynamicTablesPkg: AmlLib\AmlDbgPrint fix ECC error
  DynamicTablesPkg: AmlEncoding change debug macro
  DynamicTablesPkg: SsdtSerialPortFixupLib fix ECC errors

 DynamicTablesPkg/DynamicTablesPkg.ci.yaml                                      
      |   1 +
 DynamicTablesPkg/Library/Acpi/Arm/AcpiSratLibArm/SratGenerator.c               
      |   4 +-
 
DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtSerialPortLibArm/SsdtSerialPortGenerator.c
 |  10 +-
 DynamicTablesPkg/Library/Common/AmlLib/AmlDbgPrint/AmlDbgPrint.c               
      |  10 +-
 DynamicTablesPkg/Library/Common/AmlLib/AmlDbgPrint/AmlDbgPrint.h               
      |  40 ++-
 DynamicTablesPkg/Library/Common/AmlLib/AmlEncoding/Aml.c                       
      | 304 ++++++++++----------
 DynamicTablesPkg/Library/Common/AmlLib/AmlEncoding/Aml.h                       
      |   4 +-
 DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c                    
      |   4 +-
 DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlResourceDataCodeGen.c        
      |   2 +-
 DynamicTablesPkg/Library/Common/AmlLib/NameSpace/AmlNameSpace.c                
      |   6 +-
 DynamicTablesPkg/Library/Common/AmlLib/Parser/AmlFieldListParser.c             
      |   8 +-
 DynamicTablesPkg/Library/Common/AmlLib/Parser/AmlMethodParser.c                
      |  16 +-
 DynamicTablesPkg/Library/Common/AmlLib/Parser/AmlParser.c                      
      |  20 +-
 DynamicTablesPkg/Library/Common/AmlLib/Parser/AmlResourceDataParser.c          
      |   4 +-
 DynamicTablesPkg/Library/Common/AmlLib/Stream/AmlStream.c                      
      |   4 +-
 DynamicTablesPkg/Library/Common/AmlLib/Tree/AmlNode.c                          
      |   6 +-
 DynamicTablesPkg/Library/Common/AmlLib/Tree/AmlTree.c                          
      |   2 +-
 
DynamicTablesPkg/Library/Common/SsdtSerialPortFixupLib/SsdtSerialPortFixupLib.c 
     |  18 +-
 18 files changed, 246 insertions(+), 217 deletions(-)

-- 
'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'



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