Re: [PATCH v4 1/4] cxl: Change 'struct cxl_memdev_state' *_perf_list to single 'struct cxl_dpa_perf'

[Jonathan Cameron <Jonathan.Cameron@Huawei.com>]

> diff --git a/drivers/cxl/cxlmem.h b/drivers/cxl/cxlmem.h
> index 5303d6942b88..20fb3b35e89e 100644
> --- a/drivers/cxl/cxlmem.h
> +++ b/drivers/cxl/cxlmem.h
> @@ -395,13 +395,11 @@ enum cxl_devtype {
>  
>  /**
>   * struct cxl_dpa_perf - DPA performance property entry
> - * @list - list entry
>   * @dpa_range - range for DPA address
>   * @coord - QoS performance data (i.e. latency, bandwidth)
>   * @qos_class - QoS Class cookies
>   */
>  struct cxl_dpa_perf {
> -	struct list_head list;
>  	struct range dpa_range;
>  	struct access_coordinate coord;
>  	int qos_class;
> @@ -471,8 +469,8 @@ struct cxl_dev_state {
>   * @security: security driver state info
>   * @fw: firmware upload / activation state
>   * @mbox_send: @dev specific transport for transmitting mailbox commands
> - * @ram_perf_list: performance data entries matched to RAM
> - * @pmem_perf_list: performance data entries matched to PMEM
> + * @ram_perf: performance data entry matched to RAM partition
> + * @pmem_perf: performance data entry matched to PMEM partition

Just noticed in review of DCD but these are in wrong place in docs.
Intentional or accident?

>   *
>   * See CXL 3.0 8.2.9.8.2 Capacity Configuration and Label Storage for
>   * details on capacity parameters.
> @@ -494,8 +492,8 @@ struct cxl_memdev_state {
>  	u64 next_volatile_bytes;
>  	u64 next_persistent_bytes;
>  
> -	struct list_head ram_perf_list;
> -	struct list_head pmem_perf_list;
> +	struct cxl_dpa_perf ram_perf;
> +	struct cxl_dpa_perf pmem_perf;
>  
>  	struct cxl_event_state event;
>  	struct cxl_poison_state poison;