Re: [RFC 07/11] Documentation/hwmon: Revise sysfs interface specification

From: Bagas Sanjaya
Date: Fri May 05 2023 - 00:51:16 EST


On Thu, May 04, 2023 at 12:57:48AM -0700, James Seo wrote:
> +**RO** Read-only value.
> +**WO** Write-only value.
> +**RW** Read/write value.

Why are the permissions be emphasized?

> +| | ``pwm[1-*]_auto_point[1-*]_pwm`` | **RW** | PWM temperature curve definition |
> +| | ``pwm[1-*]_auto_point[1-*]_temp`` | | for chips that associate trip |
> +| | ``pwm[1-*]_auto_point[1-*]_temp_hyst`` | | points with PWM output channels. |
> ++-------------------------------------------+--------+-------------------------------------+
> +| | ``temp[1-*]_auto_point[1-*]_pwm`` | **RW** | PWM temperature curve definition |
> +| | ``temp[1-*]_auto_point[1-*]_temp`` | | for chips that associate trip |
> +| | ``temp[1-*]_auto_point[1-*]_temp_hyst`` | | points with temperature channels. |

Dunno if replacing line blocks with bullet lists should improve the semantic
of three sysfs files having the same description.

Thanks.

--
An old man doll... just what I always wanted! - Clara

Attachment: signature.asc
Description: PGP signature