On 16.12.2022 12:48, Julien Grall wrote:
> From: Julien Grall <jgr...@amazon.com>
> 
> Order the includes with the xen headers first, then asm headers and
> last public headers. Within each category, they are sorted alphabetically.
> 
> Note that the includes in protected by CONFIG_X86 hasn't been sorted
> to avoid adding multiple #ifdef.
> 
> Signed-off-by: Julien Grall <jgr...@amazon.com>

Acked-by: Jan Beulich <jbeul...@suse.com>

> ----
> 
>     I am open to add sort the includes protected by CONFIG_X86
>     and add multiple #ifdef if this is preferred.

I, for one, prefer it the way you've done it.

Jan

Reply via email to