On 18.03.2022 16:25, Luca Fancellu wrote:
> --- a/xen/include/xen/sched.h
> +++ b/xen/include/xen/sched.h
> @@ -1182,6 +1182,7 @@ unsigned int btcpupools_get_cpupool_id(unsigned int
> cpu);
>
> #ifdef CONFIG_HAS_DEVICE_TREE
> void btcpupools_dtb_parse(void);
> +int btcpupools_get_domain_pool_id(const struct dt_device_node *node);
> #else
> static inline void btcpupools_dtb_parse(void) {}
> #endif
> @@ -1193,6 +1194,14 @@ static inline unsigned int
> btcpupools_get_cpupool_id(unsigned int cpu)
> {
> return 0;
> }
> +#ifdef CONFIG_HAS_DEVICE_TREE
> +static inline int
> +btcpupools_get_domain_pool_id(const struct dt_device_node *node)
> +{
> + return 0;
> +}
> +#endif
Was this perhaps meant to go inside the #else visible in the context of
the earlier hunk? It's odd in any event that you have #ifdef twice, not
once #ifdef and once #ifndef.
Jan
