Re: [Xen-devel] [PATCH v2 14/15] xen/arm: psci: Consolidate PSCI version print

[Volodymyr Babchuk <volodymyr_babchuk@xxxxxxxx>]

On 08.02.18 21:22, Julien Grall wrote:
> Xen is printing the same way the PSCI version for 0.1, 0.2 and later.
> The only different is the former is hardcoded.
>
> Furthermore PSCI is now used for other things than SMP bring up. So only
> print the PSCI version in psci_init.
>
> Signed-off-by: Julien Grall <julien.grall@xxxxxxx>
Reviewed-by: Volodymyr Babchuk <volodymyr_babchuk@xxxxxxxx>

> ---
>      Changes in v2:
>          - Patch added
> ---
>   xen/arch/arm/psci.c | 8 +++-----
>   1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/xen/arch/arm/psci.c b/xen/arch/arm/psci.c
> index bc7b2260e8..7a8cf54e6d 100644
> --- a/xen/arch/arm/psci.c
> +++ b/xen/arch/arm/psci.c
> @@ -136,8 +136,6 @@ int __init psci_init_0_1(void)
>   
>       psci_ver = PSCI_VERSION(0, 1);
>   
> -    printk(XENLOG_INFO "Using PSCI-0.1 for SMP bringup\n");
> -
>       return 0;
>   }
>   
> @@ -183,9 +181,6 @@ int __init psci_init_0_2(void)
>   
>       psci_cpu_on_nr = PSCI_0_2_FN_NATIVE(CPU_ON);
>   
> -    printk(XENLOG_INFO "Using PSCI-%u.%u for SMP bringup\n",
> -           PSCI_VERSION_MAJOR(psci_ver), PSCI_VERSION_MINOR(psci_ver));
> -
>       return 0;
>   }
>   
> @@ -205,6 +200,9 @@ int __init psci_init(void)
>   
>       psci_init_smccc();
>   
> +    printk(XENLOG_INFO "Using PSCI v%u.%u\n",
> +           PSCI_VERSION_MAJOR(psci_ver), PSCI_VERSION_MINOR(psci_ver));
> +
>       return 0;
>   }
>   
--
Volodymyr Babchuk

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel