On Fri,  9 Jan 2026 18:14:35 +0530
Neeraj Kumar <[email protected]> wrote:

> Using these attributes region label is added/deleted into LSA. These
> attributes are called from userspace (ndctl) after region creation.
> 
> Signed-off-by: Neeraj Kumar <[email protected]>
One wrong field name.

With that and the version number updated as Dave pointed out
Reviewed-by: Jonathan Cameron <[email protected]>


> diff --git a/drivers/cxl/cxl.h b/drivers/cxl/cxl.h
> index 6ac3b40cb5ff..8c76c4a981bf 100644
> --- a/drivers/cxl/cxl.h
> +++ b/drivers/cxl/cxl.h

>  /**
>   * struct cxl_region_params - region settings
>   * @state: allow the driver to lockdown further parameter changes
> + * @state: region label state

wrong name.

Run scripts/kernel-doc over files you add documentation to and it'll
tell you when you get anything like this wrong.

>   * @uuid: unique id for persistent regions
>   * @interleave_ways: number of endpoints in the region
>   * @interleave_granularity: capacity each endpoint contributes to a stripe
> @@ -488,6 +494,7 @@ enum cxl_config_state {
>   */
>  struct cxl_region_params {
>       enum cxl_config_state state;
> +     enum region_label_state state_region_label;
>       uuid_t uuid;
>       int interleave_ways;
>       int interleave_granularity;


Reply via email to