From 3e52d7955f8741e4645d183e20f648d4523eb4c6 Mon Sep 17 00:00:00 2001 From: Naresh Solanki Date: Mon, 10 Jul 2023 12:43:26 +0200 Subject: [PATCH] mb/ibm/sbp1: Disable SIO Uarts Avoid enabling SIO UART to prevent conflicts with BMC console; utilize VUART0 instead. TEST=Build for sbp1 & make sure coreboot logs do not spill into BMC console. Also made sure coreboot logs are accessible via VUART. Change-Id: I2d4bbd74bb7d37b74378650dd569bca7fa13c29b Signed-off-by: Naresh Solanki Reviewed-on: https://review.coreboot.org/c/coreboot/+/76396 Reviewed-by: Paul Menzel Reviewed-by: Eric Lai Tested-by: build bot (Jenkins) Reviewed-by: Lean Sheng Tan Reviewed-by: Patrick Rudolph --- src/mainboard/ibm/sbp1/bootblock.c | 3 --- src/mainboard/ibm/sbp1/devicetree.cb | 10 ++-------- 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/src/mainboard/ibm/sbp1/bootblock.c b/src/mainboard/ibm/sbp1/bootblock.c index 9aca04e1ef..e04d8ef6be 100644 --- a/src/mainboard/ibm/sbp1/bootblock.c +++ b/src/mainboard/ibm/sbp1/bootblock.c @@ -28,7 +28,4 @@ void bootblock_mainboard_early_init(void) /* Enable com1 (0x3f8) and superio (0x2e) */ pci_write_config16(PCH_DEV_LPC, LPC_IO_DECODE, lpciod); pci_write_config16(PCH_DEV_LPC, LPC_IO_ENABLES, lpcioe); - - const pnp_devfn_t serial_dev = PNP_DEV(ASPEED_SIO_PORT, AST2400_SUART1); - aspeed_enable_serial(serial_dev, CONFIG_TTYS0_BASE); } diff --git a/src/mainboard/ibm/sbp1/devicetree.cb b/src/mainboard/ibm/sbp1/devicetree.cb index 098fd9a8eb..99c14b17f5 100644 --- a/src/mainboard/ibm/sbp1/devicetree.cb +++ b/src/mainboard/ibm/sbp1/devicetree.cb @@ -10,14 +10,8 @@ chip soc/intel/xeon_sp/spr device pnp 2e.0 on chip superio/aspeed/ast2400 register "use_espi" = "1" - device pnp 2e.2 on # SUART1 - io 0x60 = 0x3f8 - irq 0x70 = 4 - end - device pnp 2e.3 on # SUART2 - io 0x60 = 0x2f8 - irq 0x70 = 3 - end + device pnp 2e.2 off end # SUART1 + device pnp 2e.3 off end # SUART2 end end end