Re: [patch 19/53] x86/mpparse: Remove the physid_t bitmap wrapper

[Andy Shevchenko]

On Mon, Aug 7, 2023 at 4:53 PM Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
>
> physid_t is a wrapper around bitmap. Just remove the onion layer and use
> bitmap functionality directly.

...

>  #include <linux/clockchips.h>
>  #include <linux/interrupt.h>
>  #include <linux/memblock.h>

> +#include <linux/bitmap.h>

Hmm... What is the ordering here? Reversed xmas tree?
Otherwise wouldn't it be better to put this before clockchips.h?

>  #include <linux/ftrace.h>
>  #include <linux/ioport.h>
>  #include <linux/export.h>

-- 
With Best Regards,
Andy Shevchenko