Re: [PATCH v3 1/2] soc/aspeed: Add host side BMC device driver

From: Randy Dunlap
Date: Tue Sep 12 2023 - 18:15:57 EST


Hi,

diff --git a/drivers/soc/aspeed/Kconfig b/drivers/soc/aspeed/Kconfig
index f579ee0b5afa..8a4f096d84cd 100644
--- a/drivers/soc/aspeed/Kconfig
+++ b/drivers/soc/aspeed/Kconfig
@@ -52,6 +52,17 @@ config ASPEED_SOCINFO
help
Say yes to support decoding of ASPEED BMC information.

+config ASPEED_HOST_BMC_DEV
+ bool "ASPEED SoC Host BMC device driver"
+ select SOC_BUS
+ default ARCH_ASPEED
+ depends on PCI
+ help
+ Say yes to support PCIe based VUARTs from the host side. The end to
+ end VUART communication between host and BMC requires BMC side driver
+ to activate VUART functionality. This is a driver for the BMC device
+ on host.

~~~
Please clean up that indentation. Consistently use one tab + 2 spaces,
as documented in coding-style.rst.

Also, can this be tristate instead of bool? If not, why not?

Thanks.
--
~Randy