Re: [PATCH v3 01/11] cxl/region: Store root decoder in struct cxl_region

[Gregory Price]

On Fri, Sep 12, 2025 at 04:45:03PM +0200, Robert Richter wrote:
> A region is always bound to a root decoder. The region's associated
> root decoder is often needed. Add it to struct cxl_region.
> 
> This simplifies code by removing dynamic lookups and removing the root
> decoder argument from the function argument list where possible.
> 
> Patch is a prerequisite to implement address translation which uses
> struct cxl_region to store all relevant region and interleaving
> parameters.
> 
> Signed-off-by: Robert Richter <rrichter@xxxxxxx>

Nice

Reviewed-by: Gregory Price <gourry@xxxxxxxxxx>