On Sat May 31, 2025 at 2:46 AM CEST, Stefano Stabellini wrote:
> On Fri, 30 May 2025, Alejandro Vallejo wrote:
>> And remove the ifdef guard, as it's inconsequential.
>>
>> Signed-off-by: Alejandro Vallejo <agarc...@amd.com>
>> ---
>> xen/include/xen/sched.h | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/xen/include/xen/sched.h b/xen/include/xen/sched.h
>> index 559d201e0c..b5a6a22c7f 100644
>> --- a/xen/include/xen/sched.h
>> +++ b/xen/include/xen/sched.h
>> @@ -1291,15 +1291,15 @@ static inline unsigned int
>> btcpupools_get_cpupool_id(unsigned int cpu)
>> {
>> return 0;
>> }
>> -#ifdef CONFIG_HAS_DEVICE_TREE
>> +
>> +struct dt_device_node;
>> +
>> static inline int
>> btcpupools_get_domain_pool_id(const struct dt_device_node *node)
>> {
>> return 0;
>> }
>> -#endif
>> -
>> -#endif
>> +#endif /* !CONFIG_BOOT_TIME_CPUPOOLS */
>
> Usually the commend would just be /* CONFIG_BOOT_TIME_CPUPOOLS */
> without the !
True, my bad. I think I've seen both styles around (e.g: arm/include/asm/pci.h),
but it's true this one is more prevalent.
>
> Other than that:
>
> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>
Thanks!
Cheers,
Alejandro
