Re: [PATCH v4 01/27] cxl: Fix resource.c include path and export cxl_restore_hdm_after_pci_reset

[Jonathan Cameron]

On Thu, 13 Aug 2026 15:06:05 +0530
<mhonap@xxxxxxxxxx> wrote:

> From: Manish Honap <mhonap@xxxxxxxxxx>
> 
> The vfio-cxl Type-2 driver restores the endpoint HDM decoder after a
> D3hot->D0 soft reset by calling cxl_restore_hdm_after_pci_reset(), so
> export it to the CXL namespace. While here, correct the resource.c
> include of the CXL PCI definitions from <cxlpci.h> to <cxl/pci.h>.

Why?  + if this makes sense (I haven't checked) please make that
a separate patch (for now) + make sure to call it out in review of
Srirangan Madhavan series where this file comes from and which hasn't
merged yet. I think it is in patch 5.

Jonathan

> 
> Signed-off-by: Manish Honap <mhonap@xxxxxxxxxx>
> ---
>  drivers/cxl/core/resource.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/cxl/core/resource.c b/drivers/cxl/core/resource.c
> index daa5e7fa2509..492ffe8e3576 100644
> --- a/drivers/cxl/core/resource.c
> +++ b/drivers/cxl/core/resource.c
> @@ -16,7 +16,7 @@
>  #include <linux/pci.h>
>  #include <linux/slab.h>
>  
> -#include <cxlpci.h>
> +#include <cxl/pci.h>
>  
>  #include "cxl.h"
>  #include "core.h"
> @@ -850,6 +850,7 @@ int cxl_restore_hdm_after_pci_reset(struct pci_dev *pdev)
>  
>  	return cxl_reset_save_restored_state(pdev, command);
>  }
> +EXPORT_SYMBOL_NS_GPL(cxl_restore_hdm_after_pci_reset, "CXL");
>  
>  static void cxl_hdm_range_context_init(struct cxl_hdm_range_context *ctx)
>  {