2023-09-14 18:01:29

by Nitin Rawat

[permalink] [raw]
Subject: Re: [PATCH 6/6] scsi: ufs: ufs-sysfs: Introduce UFS power info sysfs nodes



On 9/11/2023 11:29 AM, Can Guo wrote:
> Having UFS power info available in sysfs makes it easier to tell the state
> of the link during runtime considering we have a bounch of power saving
> features and various combinations for backward compatiblity.

Please fix spelling mistake *bounch -> bunch
>
> Signed-off-by: Can Guo <[email protected]>
> ---
> Documentation/ABI/testing/sysfs-driver-ufs | 48 ++++++++++++++++++++++++++----
> 1 file changed, 43 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/ABI/testing/sysfs-driver-ufs b/Documentation/ABI/testing/sysfs-driver-ufs
> index 0c7efaf..25169f2 100644
> --- a/Documentation/ABI/testing/sysfs-driver-ufs
> +++ b/Documentation/ABI/testing/sysfs-driver-ufs
> @@ -1214,12 +1214,50 @@ Description: This file shows the average latency (in micro seconds) of write
>
> The file is read only.
>
> -What: /sys/bus/platform/drivers/ufshcd/*/monitor/write_req_latency_sum
> -What: /sys/bus/platform/devices/*.ufs/monitor/write_req_latency_sum
> -Date: January 2021
> +What: /sys/bus/platform/drivers/ufshcd/*/power_info/gear
> +What: /sys/bus/platform/devices/*.ufs/power_info/gear
> +Date: September 2023
> Contact: Can Guo <[email protected]>
> -Description: This file shows the total latency (in micro seconds) of write
> - requests after monitor gets started.
> +Description: This file shows the gear of UFS link.
> +
> + The file is read only.
> +
> +What: /sys/bus/platform/drivers/ufshcd/*/power_info/lane
> +What: /sys/bus/platform/devices/*.ufs/power_info/lane
> +Date: September 2023
> +Contact: Can Guo <[email protected]>
> +Description: This file shows how many lanes are enabled on the UFS link.
> +
> + The file is read only.
> +
> +What: /sys/bus/platform/drivers/ufshcd/*/power_info/mode
> +What: /sys/bus/platform/devices/*.ufs/power_info/mode
> +Date: September 2023
> +Contact: Can Guo <[email protected]>
> +Description: This file shows the power mode of UFS link.
> +
> + The file is read only.
> +
> +What: /sys/bus/platform/drivers/ufshcd/*/power_info/rate
> +What: /sys/bus/platform/devices/*.ufs/power_info/rate
> +Date: September 2023
> +Contact: Can Guo <[email protected]>
> +Description: This file shows the high speed rate of UFS link.
> +
> + The file is read only.
> +
> +What: /sys/bus/platform/drivers/ufshcd/*/power_info/dev_pm
> +What: /sys/bus/platform/devices/*.ufs/power_info/dev_pm
> +Date: September 2023
> +Contact: Can Guo <[email protected]>
> +Description: This file shows the UFS device power mode, i.e., the power mode
> + set to UFS device via the Start Stop Unit command.
> +
> +What: /sys/bus/platform/drivers/ufshcd/*/power_info/link_state
> +What: /sys/bus/platform/devices/*.ufs/power_info/link_state
> +Date: September 2023
> +Contact: Can Guo <[email protected]>
> +Description: This file shows the the state of the UFS link.
>
> The file is read only.
>

Thanks,
nitin


2023-09-15 06:02:13

by Can Guo

[permalink] [raw]
Subject: Re: [PATCH 6/6] scsi: ufs: ufs-sysfs: Introduce UFS power info sysfs nodes

Hi Nitin,

On 9/14/2023 7:35 PM, Nitin Rawat wrote:
>
>
> On 9/11/2023 11:29 AM, Can Guo wrote:
>> Having UFS power info available in sysfs makes it easier to tell the
>> state
>> of the link during runtime considering we have a bounch of power saving
>> features and various combinations for backward compatiblity.
>
> Please fix spelling mistake *bounch -> bunch

done


Thanks,

Can Guo