On 22.04.2025 17:12, Oleksii Kurochko wrote: > On 4/17/25 4:36 PM, Jan Beulich wrote: >> On 14.04.2025 17:56, Oleksii Kurochko wrote: >>> --- /dev/null >>> +++ b/xen/include/xen/fdt-domain-build.h >>> @@ -0,0 +1,46 @@ >>> +/* SPDX-License-Identifier: GPL-2.0-only */ >>> +#ifndef __XEN_FDT_DOMAIN_BUILD_H__ >>> +#define __XEN_FDT_DOMAIN_BUILD_H__ >>> + >>> +#include <xen/bootfdt.h> >>> +#include <xen/device_tree.h> >>> +#include <xen/fdt-kernel.h> >>> +#include <xen/types.h> >>> + >>> +#if __has_include(<asm/domain_build.h>) >>> +# include <asm/domain_build.h> >>> +#endif >> Why is this conditional include needed? There's ... > > To use everywhere just the generic header xen/fdt-domain-build.h instead of > both > xen/fdt-domain-build.h and <asm/domain_build.h>.
But elsewhere we're actively trying to remove dependencies between headers. Jan
