>
> +/**
> + * struct cxl_root_decoder - Static platform CXL address decoder
> + * @res: host / parent resource for region allocations
> + * @cxlsd: base cxl switch decoder
> + */
> +struct cxl_root_decoder {
> + struct resource *res;
> + struct cxl_switch_decoder cxlsd;
Ordering in these inheriting structures is inconsistent. I'd put
the cxlsd entry first here. Doesn't matter hugely but seems a bit
odd when looking at next patch
> +};
> +
> /**
