> On 15 Feb 2024, at 14:39, Michal Orzel <michal.or...@amd.com> wrote: > > At the moment, the hardware domain vUART is always compiled in. In the > spirit of fine granular configuration, make it optional so that the > feature can be disabled if not needed. This UART is not exposed (e.g. > via device tree) to a domain and is mostly used to support special use > cases like Linux early printk, prints from the decompressor code, etc. > > Introduce Kconfig option CONFIG_HWDOM_VUART, enabled by default (to keep > the current behavior) and use it to protect the vUART related code. > Provide stubs for domain_vuart_{init,free}() in case the feature is > disabled. Take the opportunity to add a struct domain forward declaration > to vuart.h, so that the header is self contained. > > Signed-off-by: Michal Orzel <michal.or...@amd.com> > ---
Hi Michal, Looks good to me, I’ve also tested with and without vuart on qemu arm64. Reviewed-by: Luca Fancellu <luca.fance...@arm.com> Tested-by: Luca Fancellu <luca.fance...@arm.com>