> 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>


Reply via email to