[PATCH v3 2/3] platform: generic: Allow platform_override to select cold boot HART

[Anup Patel]

On Mon, Dec 12, 2022 at 11:10 AM Wei Liang Lim
<weiliang.lim at starfivetech.com> wrote:
>
> From: Anup Patel <apatel at ventanamicro.com>
>
> We add a generic platform override callback to allow platform specific
> selection of cold boot HART.
>
> Signed-off-by: Anup Patel <apatel at ventanamicro.com>
> ---
>  platform/generic/include/platform_override.h | 1 +
>  platform/generic/platform.c                  | 9 +++++++++
>  2 files changed, 10 insertions(+)
>
> diff --git a/platform/generic/include/platform_override.h b/platform/generic/include/platform_override.h
> index 7f1558d..21354e9 100644
> --- a/platform/generic/include/platform_override.h
> +++ b/platform/generic/include/platform_override.h
> @@ -18,6 +18,7 @@ struct platform_override {
>         const struct fdt_match *match_table;
>         u64 (*features)(const struct fdt_match *match);
>         u64 (*tlbr_flush_limit)(const struct fdt_match *match);
> +       bool (*cold_boot_allowed)(u32 hartid, const struct fdt_match *match);
>         int (*early_init)(bool cold_boot, const struct fdt_match *match);
>         int (*final_init)(bool cold_boot, const struct fdt_match *match);
>         void (*early_exit)(const struct fdt_match *match);
> diff --git a/platform/generic/platform.c b/platform/generic/platform.c
> index bfe15f0..80961f0 100644
> --- a/platform/generic/platform.c
> +++ b/platform/generic/platform.c
> @@ -122,6 +122,14 @@ fail:
>                 wfi();
>  }
>
> +static bool generic_cold_boot_allowed(u32 hartid)
> +{
> +       if (generic_plat && generic_plat->cold_boot_allowed)
> +               return generic_plat && generic_plat->cold_boot_allowed(
> +                                               hartid, generic_plat_match);

There is a typo here. This line should be:
"return generic_plat->cold_boot_allowed(hartid, generic_plat_match);"

> +       return TRUE;
> +}
> +
>  static int generic_nascent_init(void)
>  {
>         if (platform_has_mlevel_imsic)
> @@ -261,6 +269,7 @@ static int generic_console_init(void)
>  }
>
>  const struct sbi_platform_operations platform_ops = {
> +       .cold_boot_allowed      = generic_cold_boot_allowed,
>         .nascent_init           = generic_nascent_init,
>         .early_init             = generic_early_init,
>         .final_init             = generic_final_init,
> --
> 2.25.1
>

Regards,
Anup