Re: [PATCH v7] ufs: core: Add WB buffer resize support

[Bart Van Assche]

On 4/1/25 9:45 PM, Huan Tang wrote:
> diff --git a/Documentation/ABI/testing/sysfs-driver-ufs b/Documentation/ABI/testing/sysfs-driver-ufs
> index ae0191295d29..efa1e2df292c 100644
> --- a/Documentation/ABI/testing/sysfs-driver-ufs
> +++ b/Documentation/ABI/testing/sysfs-driver-ufs
> @@ -1604,3 +1604,55 @@ Description:
>   		prevent the UFS from frequently performing clock gating/ungating.
>   
>   		The attribute is read/write.
> +
> +What:		/sys/bus/platform/drivers/ufshcd/*/wb_resize_enable
> +What:		/sys/bus/platform/devices/*.ufs/wb_resize_enable
> +Date:		April 2025
> +Contact:	Huan Tang <tanghuan@xxxxxxxx>
> +Description:
> +		The host can decrease or increase the WriteBooster Buffer size by setting
> +		this attribute.
> +
> +		========  ======================================
> +		IDLE      There is no resize operation
> +		DECREASE  Decrease WriteBooster buffer size
> +		INCREASE  Increase WriteBooster buffer size
> +		Others    Reserved
> +		========  ======================================
> +
> +		The attribute is write only.

The description of the attribute contradicts the name of the attribute.
Wouldn't wb_resize_enable be a better name for this attribute?
Additionally, that name will be easier to recognize for anyone who is
familiar with the UFS 4.1 standard.

To be consistent with other sysfs attributes, the names of the verbs
should be changed from upper case to lower case and "Others - Reserved"
should be left out. This comment also applies to the other attributes.

> +static const char *ufs_wb_resize_hint_to_string(enum wb_resize_hint hint)
> +{
> +	switch (hint) {
> +	case WB_RESIZE_HINT_KEEP:	return "KEEP";
> +	case WB_RESIZE_HINT_DECREASE:	return "DECREASE";
> +	case WB_RESIZE_HINT_INCREASE:	return "INCREASE";
> +	default:	return "UNKNOWN";
> +	}
> +}

The formatting of the above switch/case statement is not compliant with
the Linux kernel coding style.

Thanks,

Bart.