Re: [PATCH v1 3/3] xen/arm: check for psci-0.2 dt node

[Julien Grall <julien.grall@linaro.org>]

Hi Vijay,

On 16/02/15 09:06, vijay.kilari@gmail.com wrote:
> From: Vijaya Kumar K <Vijaya.Kumar@caviumnetworks.com>
> 
> Add psci-0.2 compatibility check while parsing psci dt node

Reading the commit title/description gives the impression that you are
adding support with PSCI 0.2.

While it has been added in Xen 4.5, this patch avoids to copy the PSCI
0.2 node from the host DT to DOM0 DT (because we generate it).

Please update the commit title/message according the behavior of the code.

Regards,

> Signed-off-by: Vijaya Kumar K <Vijaya.Kumar@caviumnetworks.com>
> ---
>  xen/arch/arm/domain_build.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c
> index c2dcb49..0be639b 100644
> --- a/xen/arch/arm/domain_build.c
> +++ b/xen/arch/arm/domain_build.c
> @@ -1029,6 +1029,7 @@ static int handle_node(struct domain *d, struct kernel_info *kinfo,
>          DT_MATCH_COMPATIBLE("xen,multiboot-module"),
>          DT_MATCH_COMPATIBLE("multiboot,module"),
>          DT_MATCH_COMPATIBLE("arm,psci"),
> +        DT_MATCH_COMPATIBLE("arm,psci-0.2"),
>          DT_MATCH_PATH("/cpus"),
>          DT_MATCH_TYPE("memory"),
>          /* The memory mapped timer is not supported by Xen. */
> 


-- 
Julien Grall