Currently in the Dynamic Tables Framework, the interrupt node for the AML description of the serial-ports is populated using the template and so is mandatorily added even if the serial-port is enumerated as a DBG2 port in the platform's configuration manager where the interrupt is not mandatory. The proposed implementation adds the interrupt node only if the interrupt defined for the serial-port is a valid SPI or a valid extended SPI. So, in case of DBG2 ports, he platforms with interrupt defined as SPI (like Morello) can have the interrupt node added to the description and the platforms where it is not defined (like N1SDP) can ignore the addition of the interrupt node.
The changes include adding the SPI range macros in ArmGicArchLib (ArmPkg) which can be used by the SSDTSerialPortFixupLib (DynamicTablesPkg) to put a check for generating the interrupt node using AML Codegen API. Link to branch with the patches in this series - https://github.com/himsha01/edk2/tree/ssdt_serial_port_interrupt Himanshu Sharma (2): ArmPkg/ArmGicArchLib: Add macros for SPI and extended SPI ranges DynamicTablesPkg/SsdtSerialPortFixupLib: Add Interrupt node for SPIs only DynamicTablesPkg/Library/Common/SsdtSerialPortFixupLib/SsdtSerialPortFixupLib.inf | 3 +- ArmPkg/Include/Library/ArmGicArchLib.h | 9 +++++ DynamicTablesPkg/Library/Common/SsdtSerialPortFixupLib/SsdtSerialPortFixupLib.c | 38 ++++++++++++-------- DynamicTablesPkg/Library/Common/SsdtSerialPortFixupLib/SsdtSerialPortTemplate.asl | 29 ++++++++------- 4 files changed, 51 insertions(+), 28 deletions(-) -- 2.25.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#112116): https://edk2.groups.io/g/devel/message/112116 Mute This Topic: https://groups.io/mt/103010239/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-