Re: [PATCH v3 16/18] cxl/region: Add a dev_err() on missing target list entries

Jonathan Cameron
2025-02-14T16:36:43Z

On Tue, 11 Feb 2025 10:53:46 +0100
Robert Richter <rrichter@amd.com> wrote:

> Broken target lists are hard to discover as the driver fails at a
> later initialization stage. Add an error message for this.
>
> Example log messages:
>
> cxl_mem mem1: failed to find endpoint6:0000:e0:01.3 in target list of decoder1.1
> cxl_port endpoint6: failed to register decoder6.0: -6
> cxl_port endpoint6: probe: 0
>
> Signed-off-by: Robert Richter <rrichter@amd.com>
> Reviewed-by: Gregory Price <gourry@gourry.net>
> Tested-by: Gregory Price <gourry@gourry.net>
Seems reasonable to me as I've also run into fun problems with these
due to setup script bugs...
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>