[PATCH v2 08/11] include: sbi: Add domains_init() platform operation

[Anup Patel]

> -----Original Message-----
> From: Anup Patel <Anup.Patel at wdc.com>
> Sent: 04 December 2020 21:22
> To: Atish Patra <Atish.Patra at wdc.com>; Alistair Francis
> <Alistair.Francis at wdc.com>
> Cc: Anup Patel <anup at brainfault.org>; opensbi at lists.infradead.org; Anup
> Patel <Anup.Patel at wdc.com>; Alistair Francis <Alistair.Francis at wdc.com>
> Subject: [PATCH v2 08/11] include: sbi: Add domains_init() platform
> operation
> 
> We introduce domains_init() platform operation which can be used by
> platform support to initialize/populate domains in the coldboot path.
> 
> The domains_init() is called late in the coldboot sequence from the
> sbi_domain_finalize() so sbi_printf() can be used by platform support to print
> errors/warnings at time of populating domains.
> 
> Signed-off-by: Anup Patel <anup.patel at wdc.com>
> Reviewed-by: Alistair Francis <alistair.francis at wdc.com>
> Reviewed-by: Atish Patra <atish.patra at wdc.com>
> ---
>  include/sbi/sbi_platform.h | 16 ++++++++++++++++
>  lib/sbi/sbi_domain.c       |  8 ++++++++
>  2 files changed, 24 insertions(+)
> 
> diff --git a/include/sbi/sbi_platform.h b/include/sbi/sbi_platform.h index
> ee72323..3681a78 100644
> --- a/include/sbi/sbi_platform.h
> +++ b/include/sbi/sbi_platform.h
> @@ -91,6 +91,8 @@ struct sbi_platform_operations {
>  	 */
>  	int (*misa_get_xlen)(void);
> 
> +	/** Initialize (or populate) domains for the platform */
> +	int (*domains_init)(void);
>  	/** Get domain pointer for given HART id */
>  	struct sbi_domain *(*domain_get)(u32 hartid);
> 
> @@ -451,6 +453,20 @@ static inline int sbi_platform_misa_xlen(const struct
> sbi_platform *plat)
>  	return -1;
>  }
> 
> +/**
> + * Initialize (or populate) domains for the platform
> + *
> + * @param plat pointer to struct sbi_platform
> + *
> + * @return 0 on success and negative error code on failure  */ static
> +inline int sbi_platform_domains_init(const struct sbi_platform *plat) {
> +	if (plat && sbi_platform_ops(plat)->domains_init)
> +		return sbi_platform_ops(plat)->domains_init();
> +	return 0;
> +}
> +
>  /**
>   * Get domain pointer for given HART
>   *
> diff --git a/lib/sbi/sbi_domain.c b/lib/sbi/sbi_domain.c index
> 639e016..1a3651b 100644
> --- a/lib/sbi/sbi_domain.c
> +++ b/lib/sbi/sbi_domain.c
> @@ -375,6 +375,14 @@ int sbi_domain_finalize(struct sbi_scratch *scratch,
> u32 cold_hartid)
>  	struct sbi_domain *dom, *tdom;
>  	const struct sbi_platform *plat = sbi_platform_ptr(scratch);
> 
> +	/* Initialize domains for the platform */
> +	rc = sbi_platform_domains_init(plat);
> +	if (rc) {
> +		sbi_printf("%s: platform domains_init() failed (error %d)\n",
> +			   __func__, rc);
> +		return rc;
> +	}
> +
>  	/* Discover domains */
>  	for (i = 0; i < SBI_HARTMASK_MAX_BITS; i++) {
>  		/* Ignore invalid HART */
> --
> 2.25.1

Applied this patch to the riscv/opensbi repo.

Regards,
Anup