Subject: Re: [PATCH v19 07/22] cxl: allow Type2 drivers to map cxl component regs

From: Jonathan Cameron <[email-masked]>
Date: Tue, 7 Oct 2025 14:18:22 +0100

On Mon, 6 Oct 2025 11:01:15 +0100
[email-masked] wrote:

> From: Alejandro Lucero <[email-masked]>
> 
I'd amend the patch title to 
	cxl/sfc: Map CXL component regs.

And talk about exports in the description.

Other options are fine but the patch title should indicate
this is being used by the sfc driver.

> Export cxl core functions for a Type2 driver being able to discover and
> map the device component registers.
> 
> Use it in sfc driver cxl initialization.
> 
> Signed-off-by: Alejandro Lucero <[email-masked]>
> Reviewed-by: Dan Williams <[email-masked]>
> Reviewed-by: Jonathan Cameron <[email-masked]>
> Reviewed-by: Dave Jiang <[email-masked]>
> Reviewed-by: Ben Cheatham <[email-masked]>
> ---
>  drivers/cxl/core/pci.c             |  1 +
>  drivers/cxl/core/port.c            |  1 +
>  drivers/cxl/core/regs.c            |  1 +
>  drivers/cxl/cxl.h                  |  7 ------
>  drivers/cxl/cxlpci.h               | 12 ----------
>  drivers/cxl/pci.c                  |  1 +
>  drivers/net/ethernet/sfc/efx_cxl.c | 35 ++++++++++++++++++++++++++++++
>  include/cxl/cxl.h                  | 19 ++++++++++++++++
>  include/cxl/pci.h                  | 21 ++++++++++++++++++
>  9 files changed, 79 insertions(+), 19 deletions(-)
>  create mode 100644 include/cxl/pci.h