Attention is currently required from: Subrata Banik.
Angel Pons has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/83510?usp=email )
Change subject: device/pci_ids: Add new Intel PTL device IDs for UARTx ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File src/include/device/pci_ids.h:
https://review.coreboot.org/c/coreboot/+/83510/comment/f327a6a4_0515b01b?usp... : PS1, Line 3999: #define PCI_DID_INTEL_PTL_H_UART0 0xe425 : #define PCI_DID_INTEL_PTL_H_UART1 0xe426 : #define PCI_DID_INTEL_PTL_H_UART2 0xe452 : #define PCI_DID_INTEL_PTL_U_H_UART0 0xe325 : #define PCI_DID_INTEL_PTL_U_H_UART1 0xe326 : #define PCI_DID_INTEL_PTL_U_H_UART2 0xe352 nit: the alignment ðŸ˜
```suggestion #define PCI_DID_INTEL_PTL_H_UART0 0xe425 #define PCI_DID_INTEL_PTL_H_UART1 0xe426 #define PCI_DID_INTEL_PTL_H_UART2 0xe452 #define PCI_DID_INTEL_PTL_U_H_UART0 0xe325 #define PCI_DID_INTEL_PTL_U_H_UART1 0xe326 #define PCI_DID_INTEL_PTL_U_H_UART2 0xe352 ```