Re: [PATCH v4 4/7] serial: 8250_aspeed_vuart: add VUART over PCI

Hi Jiri,

On Thu, 9 Jul 2026 at 07:17, Jiri Slaby [off-list ref] wrote:

On 08. 07. 26, 17:35, Grégoire Layet wrote:
...

quoted

--- a/drivers/tty/serial/8250/8250_aspeed_vuart.c
+++ b/drivers/tty/serial/8250/8250_aspeed_vuart.c

@@ -32,6 +32,26 @@
  #define ASPEED_VUART_DEFAULT_SIRQ   4
  #define ASPEED_VUART_DEFAULT_SIRQ_POLARITY  IRQ_TYPE_LEVEL_LOW

+#define ASPEED_SCU_SILICON_REVISION_ID                       0x04
+#define AST2600A3_REVISION_ID                                0x05030303
+
+#define ASPEED_SCUC24                        0xC24
+#define  ASPEED_SCUC24_MSI_ROUTING_MASK                      GENMASK(11, 10)
+#define  ASPEED_SCUC24_MSI_ROUTING_PCIE2LPC_PCIDEV1          (0x2 << 10)

So is this
FIELD_PREP(ASPEED_SCUC24_MSI_ROUTING_MASK, 2)
?

Yes, replaced

quoted

+#define  ASPEED_SCUC24_PCIDEV1_INTX_MSI_HOST2BMC_EN          BIT(18)
+#define  ASPEED_SCUC24_PCIDEV1_INTX_MSI_SCU560_EN                    BIT(17)

Perhaps switch the two (to be in asc order)? And define 14 as _RESERVED
as well?

Acknowledged. I have also removed the comment afterwards.

quoted

+#define ASPEED_SCU_PCIE_CONF_CTRL    0xC20

Hmm, should these go before 0xC24?

Yes

Thanks for the feedback, I have made the changes for the next revision.

Regards,
Grégoire